Skip to content

chore: deprecated だった V1 Client を削除#116

Merged
t-kizawa-digima merged 3 commits into
mainfrom
chore/remove-deprecated-v1-client
Jun 29, 2026
Merged

chore: deprecated だった V1 Client を削除#116
t-kizawa-digima merged 3 commits into
mainfrom
chore/remove-deprecated-v1-client

Conversation

@t-kizawa-digima

@t-kizawa-digima t-kizawa-digima commented Jun 19, 2026

Copy link
Copy Markdown
Contributor

Summary

Deprecated だった V1 Client (jquantsapi.Client) およびその関連コードを削除します。J-Quants Backend API V1 が既に廃止済みのため Client は呼んでも応答しない状態でしたが、ライブラリ側のコードを整理として除去します。

Changes

  • jquantsapi/client.py (V1 Client 本体, 1391行) を削除
  • jquantsapi/apis/v1/ ディレクトリ (V1 専用 API ラッパー 6 ファイル) を削除
  • tests/test_client.py を削除
  • V1 Client を使用していたサンプル notebook 7 本 (examples/20220825-00020230126-006) を削除
  • jquantsapi/__init__.py から Client の export を削除
  • jquantsapi/constants.py から V1 専用カラム定義 (SECTOR_33_COLUMNS 等) を削除 (V2 用 _V2 サフィックス付き定数は保持)
  • README.md / examples/README.md から V1 セクションを削除し、認証説明を API キー方式に統一
  • jquantsapi/apis/__init__.py / apis/base.py の docstring から v1 への言及を削除
  • .gitignore.claude/review-state/.claude/settings.local.json を追加

ClientV2 側への影響はなく、test_client_v2.py の 50 件のテストはすべて pass しています。

Test plan

  • `make lint` が pass する
  • `make test` で `tests/test_client_v2.py` (50件) が pass する
  • `python -c "import jquantsapi; print(jquantsapi.ClientV2)"` でパッケージが正常に import できる
  • Sprint review でデモする

V1Client (jquantsapi.Client) は deprecated だったため削除。

削除対象:
- jquantsapi/client.py (V1Client 本体)
- jquantsapi/apis/v1/ (V1 専用 API ラッパー群)
- tests/test_client.py
- V1 を使用していたサンプル notebook 7 本 (examples/20220825-000〜20230126-006)

修正:
- jquantsapi/__init__.py から Client の export を削除
- jquantsapi/constants.py から V1 専用カラム定義を削除
- README.md / examples/README.md から V1 セクションを削除

ClientV2 への影響なし (test_client_v2.py 50 件すべて通過)。
- README: 認証説明をリフレッシュトークン/メアド+パスワードから API キーに統一
- apis/__init__.py: docstring から v1 への言及を削除
- apis/base.py: docstring を ClientV2 ベースの記述に更新

Cursor #1, #3 を accept。#2/#4/#5/#6/#7/#8 はユーザー判断でスコープ外 reject。
- .claude/review-state/ (review-cycle のラウンド状態・セッションファイル)
- .claude/settings.local.json (Claude Code のローカル設定)

@shotaroshimizu shotaroshimizu left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

@t-kizawa-digima
t-kizawa-digima added this pull request to the merge queue Jun 29, 2026
Merged via the queue into main with commit 7db90f6 Jun 29, 2026
5 checks passed
@t-kizawa-digima
t-kizawa-digima deleted the chore/remove-deprecated-v1-client branch June 29, 2026 06:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants