Skip to content

docs(history): issue #139 の振り返りを残す - #147

Merged
takemi-ohama merged 1 commit into
mainfrom
docs/retrospective-issue139
Sep 2, 2026
Merged

takemi-ohama merged 1 commit into
mainfrom
docs/retrospective-issue139

Conversation

@takemi-ohama

Copy link
Copy Markdown
Contributor

概要

issue #139(devbase build <image> の修正)の振り返りを docs/development-history/01-2026-09-02.md として残します。このディレクトリの最初の記録です。

対象は進め方であり、成果物の良し悪しはレビューの工程が扱っています。

関連 Issue

変更点

  • docs/development-history/01-2026-09-02.md を追加(新規ディレクトリ)

記録した主なこと

観点 分かったこと
手戻り 設計 PR #143 は 1 round で通過。実装 PR #144 で 3 指摘(major 1 / minor 2)。設計前に「確認が要る」2 点を実測で潰したことが効いた
見落とし マージ前のテストは run_python / cmd_build をスタブ化する方式のため、dispatch より前の name 解決を素通りしていた。リリース後テストで実際に devbase build ../etc を打って #146 が見つかった
範囲 仕様で「前提」として範囲外に置いた領域は、レビューの視野からも外れる。実機が最後の網になった
工程 実機ビルドは 3 分 18 秒。「時間がかかる」を理由にマージ前の検証を落とした判断が、実測に見合っていなかった

設計 PR を実装より先に通す運びは変えない、と結論しています。理由も記録に書きました。

動作確認

  • 必要に応じてドキュメントを更新した(この PR 自体がドキュメント)
  • ローカルで ./bin/devbase --help が正常に動作する(ドキュメントのみのためコード変更なし)
  • CI が green である

リンク先(issue #139 / #141 / #142 / #146、PR #143 / #144 / #145)が実在することを確認しています。

docs/development-history/ を作り、最初の記録を置く。

進め方で分かったこと:
- マージ前に追加した wrapper のテストは、run_python/cmd_build をスタブへ
  差し替える方式のため dispatch より前の name 解決を素通りしていた。
  リリース後テストで実際にコマンドを打って初めて #146 が見つかった
- 仕様で「前提」として範囲外に置いた領域は、レビューの視野からも外れる
- 実機ビルドは 3 分 18 秒で、マージ前に避けた理由に見合っていなかった

設計 PR を実装より先に通す運びは変えない。shell / Python どちらへ寄せるかを
実装前に固定できた効果のほうが大きい。

Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01V9tinzTAbF1LVEYvfUgzKx
@takemi-ohama
takemi-ohama merged commit d938655 into main Sep 2, 2026
5 checks passed
@takemi-ohama
takemi-ohama deleted the docs/retrospective-issue139 branch September 2, 2026 06:57
@takemi-ohama takemi-ohama mentioned this pull request Sep 3, 2026
3 tasks
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