Repository navigation
#435 を実装する(スプリント m10b) - #449
Merged
Merged
Conversation
bash と zsh の補完が、--context NAME を取るすべての道(container の配下・ env exec・env token・入れ子の profile を含む)で --context を出す。値(docker context の名前)は補完しない。一致テストが補完の --context を列挙の正本と比べる。 Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
takemi-ohama
marked this pull request as ready for review
October 8, 2026 08:45
This was referenced Oct 8, 2026
Merged
Merged
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
devbase の補完(zsh の
etc/_devbaseと bash のetc/devbase-completion.bash)で、--contextを受け付けるすべてのサブコマンドが--contextを候補に出すようにしました。あわせて、一致テストが補完の--contextも比べることを、確定仕様と CHANGELOG に書きました。スプリント m10b のブランチへ集める変更です(#435)。利用者向けの変化
--contextを受け付けるすべてのサブコマンドで、zsh・bash の補完が--contextを候補に出します移行の手順
変更の要点
etc/_devbase(zsh)とetc/devbase-completion.bash(bash)で、--contextを取るサブコマンドすべてに--contextの補完を加えたtests/cli/test_name_context_consistency.pyの一致テストで、補完が出す--contextも比べるようにしたtests/cli/test_completion.pyを補完の変更に合わせて直したdocs/specifications/cli-argument-resolution.mdとCHANGELOG.mdに、一致テストが補完の--contextを比べることを書いたテスト
モード: standard / 通した工程: 実装 → 完了判定 → Pull Request
🤖 Generated with Claude Code
外すと落ちることの確認(受け入れ条件 5 の手動確認)
bash から
env execの--contextを、zsh からenv tokenの--contextを一時的に外し、一致テストを打った。確認の後に元へ戻した。課題と違えたところ: 一致テストは課題の原文が挙げる
test_completion.pyではなく、#214 の一致テストのあるtest_name_context_consistency.pyに足した(要求の前提 1)。トップレベルpsの zsh 補完は、打った語で分岐する形からproject psと同じ_argumentsの形に揃えた。固有の語の検査は手元では語の一覧が無く飛ばされた(検査していない)。CI の Proper term check の結果で確かめる。