docs(history): PLAN50 の振り返りを残す - #153
Merged
Merged
Conversation
進め方で分かったこと: - 環境変数の値を 2 度取り違えた。同名の変数がホストのシェル / 共通機密 / プロジェクトの env / 生成 compose に同時に存在するため、存在だけを見て 出所を確かめないと読み違える - 設計レビュー round 1 の指摘 (GOOGLE_CLOUD_PROJECT で認証方式を推論するのは category error) は、必要条件を十分条件として扱った誤りだった - 設計を差し替えたあと、同じ文書の他の節と PR 本文が古いまま残り、round 2・3 を 費やした。文書の外にある PR 本文はファイルを直しても追従しない - 仕様の「対象範囲 (含まない)」へ書いただけで起票したつもりになっていた。 振り返りで拾い #152 として起票した 失敗するテストを目で確認する工程は変えない。空振りで通るテストを実装前に 捕まえられた。 Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01V9tinzTAbF1LVEYvfUgzKx
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.
概要
PLAN50(
geminiの Vertex AI 強制をやめる)の振り返りをdocs/development-history/02-2026-09-03.mdとして残します。対象は進め方であり、成果物の良し悪しはレビューの工程が扱っています。
関連 Issue
変更点
docs/development-history/02-2026-09-03.mdを追加記録した主なこと
env/ 生成 compose に同時に存在するため、存在だけを見ると読み違えるclaudbのオプション重複を範囲に入れた(書き換えている行そのものが原因のため)「失敗するテストを目で確認する」工程は変えない、と結論しています。空振りで通るテスト 2 種類を実装前に捕まえられました。
動作確認
./bin/devbase --helpが正常に動作する(ドキュメントのみのためコード変更なし)リンク先(PR #148 / #149 / #151、issue #139 / #152、devbase-ext#20)が実在することを確認しています。