Skip to content
This repository was archived by the owner on Jul 13, 2026. It is now read-only.

refactor: Use live fee rates from bitbank pairs API#334

Merged
tjackiet merged 2 commits into
mainfrom
claude/backtest-maker-fee-api-168ay3
Jul 6, 2026
Merged

refactor: Use live fee rates from bitbank pairs API#334
tjackiet merged 2 commits into
mainfrom
claude/backtest-maker-fee-api-168ay3

Conversation

@tjackiet

@tjackiet tjackiet commented Jul 6, 2026

Copy link
Copy Markdown
Owner

概要

バックテスト・シグナル探索・ペーパートレードの手数料計算を、ハードコードされた過去の数字(0.12%)から bitbank /spot/pairs API の現行レートに変更します。

変更内容

1. 手数料取得の動的化(cli/fees.ts

  • DEFAULT_TAKER_FEE_RATE のコメントを更新:過去の公称上限値であることを明記し、現行レートは常に API 取得値が優先されることを記載
  • resolveDryRunFee()getPairsWithCache() を呼び出し、対象ペアの taker_fee_rate_quote / maker_fee_rate_quote を取得
  • API 取得失敗時のみ既定フォールバック 0.12% を使用し、その旨を note に記載

2. ドキュメント更新

  • SKILL.md(backtest): 手数料取得方法を詳述
    • bitbank pairs --format=json --machine コマンド例
    • taker_fee_rate_quote / maker_fee_rate_quote の読み方
    • デフォルトは taker レート(次足始値での成行を想定)
    • maker は指値前提時のみ使用
    • 実行のたびに取得、失敗時のみフォールバック
    • 結果出力に「フォールバック値」と明記
  • strategy-patterns.md: 手数料率は SKILL.md Step 1 で取得した現行レートを使うことを明記
  • signal-explorer/SKILL.md / exploration-guide.md: 往復手数料を「bitbank pairs の taker レート × 2」に統一
  • README.md(paper trade): 手数料を「ライブ maker/taker レート(/spot/pairs 由来・24h キャッシュ)」に更新
  • customization-guide.md: 例示の手数料を「bitbank pairs の taker レート」に更新

3. テスト更新(cli/__tests__/fees.test.ts

  • テスト説明を「公称 taker 率」から「既定フォールバック率」に統一
  • 期待値の note 文言を同様に更新

背景

bitbank の手数料は時期・ペア・キャンペーンで改定される(例:BTC/JPY は 2026-02 時点で maker 0.00% / taker 0.10%)。バックテスト結果の信頼性を高めるため、実行時に常に現行レートを取得し、過去の数字を使い回さないようにします。

チェックリスト

  • 目的 / 変更概要を記載した
  • テストを追加・更新した(既存テストの説明文を更新)
  • npx vitest run が green
  • npx tsc --noEmit が green
  • npx biome check cli/ が green
  • chaos 規約・1 ファイル 100 行目安を満たす
  • シークレットを含めていない
  • ドキュメントを更新した

https://claude.ai/code/session_01V7tGPv2Kh8wvGs5dpPAVzM

claude added 2 commits July 6, 2026 01:17
旧 maker 0.12% は過去の数字(現行 maker はリベートで負)。ハードコードを
やめ、paper-trade と同様に /spot/pairs 由来の現行レート(デフォルト taker、
取得失敗時のみ 0.12% フォールバック + 明記)を使う手順に更新。

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01V7tGPv2Kh8wvGs5dpPAVzM
公式手数料ページ改定(BTC/JPY: maker 0.00% / taker 0.10%、マイナス手数料は
終了)を受け、リポジトリ内で旧レートを現行値として断定していた箇所を修正:

- backtest skill: 「現行 maker はリベートで負」の断定を撤回し、フォール
  バック 0.0012 は「保守値であり現行レートではない」と明記
- signal-explorer / customization-guide / README: ハードコードの 0.12% を
  bitbank pairs の取得値参照に統一
- cli/fees.ts: DEFAULT_TAKER_FEE_RATE のコメントと dry-run note の
  「公称 taker 率」を「既定フォールバック率」に変更(レート値は保守側の
  上限として据え置き)

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01V7tGPv2Kh8wvGs5dpPAVzM
@coderabbitai

coderabbitai Bot commented Jul 6, 2026

Copy link
Copy Markdown

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: ASSERTIVE

Plan: Pro

Run ID: e33f9094-b65d-4716-8092-b9ff2301d43a

📥 Commits

Reviewing files that changed from the base of the PR and between c01c35d and 35aa9d2.

📒 Files selected for processing (8)
  • README.md
  • cli/__tests__/fees.test.ts
  • cli/fees.ts
  • docs/customization-guide.md
  • skills/backtest/SKILL.md
  • skills/backtest/references/strategy-patterns.md
  • skills/signal-explorer/SKILL.md
  • skills/signal-explorer/references/exploration-guide.md
🔗 Linked repositories identified

CodeRabbit considers these linked repositories for cross-repo context during reviews:

  • bitbankinc/bitbank-api-docs (manual)

📝 Walkthrough

Summary

bitbank の /spot/pairs から取得した最新の maker/taker 手数料率を、バックテスト・signal exploration・ペーパートレードの手数料計算に反映するよう更新しました。取得失敗時のみ 0.12% を保守的なフォールバックとして使うようにし、関連ドキュメントとテストの文言もそれに合わせて整理しています。

Category

  • 🧪 Paper
  • 🤖 Skill
  • 📚 Docs / tests

Risk Level

  • 🟢 Low

Impact

  • cli/fees.ts
  • cli/__tests__/fees.test.ts
  • skills/backtest/SKILL.md
  • skills/backtest/references/strategy-patterns.md
  • skills/signal-explorer/SKILL.md
  • skills/signal-explorer/references/exploration-guide.md
  • README.md
  • docs/customization-guide.md

Walkthrough

指値発注時の手数料計算に関する説明を、固定の公称手数料率(0.12%)から /spot/pairs 由来のライブ maker/taker レート参照方式へ変更。関連するコードコメント、テストの期待文言、READMEおよび複数のスキルドキュメントを同趣旨で更新。取得失敗時のみ既定フォールバックを使用する旨を明記。

Changes

手数料説明のライブレート化

Layer / File(s) Summary
fees.tsのフォールバック文言とテスト更新
cli/fees.ts, cli/__tests__/fees.test.ts
フォールバック説明コメントを拡張し、resolveDryRunFee のnote文言を「公称taker率」から「既定フォールバック率」に変更、対応テストの期待文字列も更新。
README・customization-guideの手数料説明更新
README.md, docs/customization-guide.md
Paperの指値発注手数料およびRSI逆張り戦略の手数料説明を、固定値からbitbank pairsのライブレート参照へ更新。
backtestスキルの手数料取得手順更新
skills/backtest/SKILL.md, skills/backtest/references/strategy-patterns.md
バックテストにおける手数料取得方法を実行時取得方式に変更し、結果表示・Gotchas・P&L計算注記を更新。
signal-explorerスキルの手数料説明更新
skills/signal-explorer/SKILL.md, skills/signal-explorer/references/exploration-guide.md
コスト無視の注意書きと推奨backtestフローの手数料例を固定値からbitbank pairs参照表現へ置換。

Estimated code review effort: 1 (Trivial) | ~5 minutes

Suggested labels: skill

Poem

手数料の数字、固定じゃもう古い
pairsを覗いて、今のレートをつかむ
フォールバックはお守り、いざという時だけ
ぴょんぴょん跳ねて、ドキュメント直したよ 🐇📄
今日も安心、正しい計算で進もう

🚥 Pre-merge checks | ✅ 4
✅ Passed checks (4 passed)
Check name Status Explanation
Title check ✅ Passed PRタイトルは主要変更である bitbank pairs API によるライブ手数料取得を簡潔に表しており、内容と一致しています。
Description check ✅ Passed 概要、変更内容、背景、チェックリストが揃っており、テンプレートの必須情報をほぼ満たしています。
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
✨ Finishing Touches
📝 Generate docstrings
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch claude/backtest-maker-fee-api-168ay3

Comment @coderabbitai help to get the list of available commands.

@coderabbitai coderabbitai Bot added the skill label Jul 6, 2026
@tjackiet
tjackiet merged commit 7a1cb78 into main Jul 6, 2026
5 checks passed
@tjackiet
tjackiet deleted the claude/backtest-maker-fee-api-168ay3 branch July 6, 2026 01:51
@tjackiet tjackiet mentioned this pull request Jul 6, 2026
8 tasks
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants