Skip to content

#454 を実装する(スプリント m8d) - #486

Merged
takemi-ohama merged 2 commits into
sprint/m8dfrom
feat/issue-454-m8d
Oct 14, 2026
Merged

takemi-ohama merged 2 commits into
sprint/m8dfrom
feat/issue-454-m8d

Conversation

@takemi-ohama

Copy link
Copy Markdown
Contributor

devbase env のシェル補完の候補に token と backend のサブコマンドを足しました(#454)。あわせて、補完のサブコマンドの一覧が一致するかを一致テストで比べるようにし、そのことを確定仕様と CHANGELOG に書きました。

利用者向けの変化

  • devbase env のシェル補完(bash・zsh)で、サブコマンドの候補に token と backend が出る

移行の手順

  • 無し

変更の要点

  • etc/devbase-completion.bash と etc/_devbase の env のサブコマンドの一覧に token と backend を足した
  • tests/cli/test_name_context_consistency.py の一致テストで、env のサブコマンドの一覧も比べるようにした
  • 一致テストが env のサブコマンドの一覧を比べることを docs/specifications/cli-argument-resolution.md と CHANGELOG.md に書いた
  • 変更は 5 ファイル、18 行の追加と 7 行の削除

テスト

ステップ exit 結果
test-limited 0 範囲テスト 1 本が通った(197.2 秒、戦略 local-full)
design-tests 3 停止。#454 に設計文書が無いため、3 つの確認は該当なし

設計と違う点

  • 該当なし(設計文書が無い)

変えたこと

  • etc/devbase-completion.bash と etc/_devbase の env_subcommands に token と backend を足した。zsh の説明は lib/devbase/cli.py の help に合わせた
  • 一致テスト(tests/cli/test_name_context_consistency.py)のサブコマンドの一覧の比較に env を足した。env は [name]・--context を取るかによらず、argparse の env の直下のサブコマンド(今は 18 個)の全部が bash・zsh の両方の一覧にあることを確かめる
  • 確定仕様 docs/specifications/cli-argument-resolution.md の一致テストの表と CHANGELOG.md の [Unreleased] に書いた

手で確かめたこと

  • 一覧から token を外すと一致テストが落ちる。bash と zsh の両方の一覧から token を外して -k lists_subcommands を走らせた結果:
    • AssertionError: etc/_devbase の env_subcommands に無い: ['token']
    • AssertionError: etc/devbase-completion.bash の env のサブコマンドの一覧に無い: ['token']
    • 2 failed, 4 passed。元に戻して通ることを確かめた
  • zsh -n etc/_devbase が通る
  • env -u DEVBASE_ROOT python3 -m pytest -q tests/cli が全部通る(1070 件)。シェル補完が --context を open と post-start でしか出さない #435 の --context の補完のテストも含む

残した事項

  • 課題の受け入れ条件 1 は「既存の 17 個」としているが、元の一覧は 16 個だった(init sync list set get delete edit project export import keygen exec encrypt decrypt rekey doctor)。16 個はすべて残している
  • devbase env backend <TAB> で status / use などの下のサブコマンドを出す補完は範囲外として足していない

モード: standard / 通した工程: 実装 → 完了判定 → Pull Request

🤖 Generated with Claude Code

@takemi-ohama
takemi-ohama marked this pull request as ready for review October 14, 2026 01:09
@takemi-ohama
takemi-ohama merged commit 53013e4 into sprint/m8d Oct 14, 2026
10 checks passed
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.

1 participant