設計(PLAN67): base イメージに shellcheck を入れる (#249) - #250
Conversation
Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
takemi-ohama
left a comment
takemi-ohama
left a comment
There was a problem hiding this comment.
🤖 cross-review | round 1 | agy | REQUEST_CHANGES
docs/user/container-operations.md への配置見直し（既存の「文字の描画と、文書を扱う道具」節および base-image-rendering.md との分離、およびイメージ一覧表の更新）と、test_base_dockerfile_shellcheck.py で 1 回目の apt-get install を厳密に識別して固定するための補助関数の見直しを提案します。
takemi-ohama
left a comment
takemi-ohama
left a comment
There was a problem hiding this comment.
🤖 cross-review | round 1 | kiro | REQUEST_CHANGES
PLAN67 の要求・設計は Dockerfile / CI / 既存テストの実態と整合し、内部の相互参照も解決する。1 点、受け入れ条件 6 の「新しく入るのは 2 パッケージ」という断定が、そこに列挙された検証手順では反証できない（第三の依存が混ざっても検出されない）。検証手順を new-package 集合の列挙へ寄せることを提案する。
- container-operations.md は独立した小節「Bash の静的検査（base 以降）」を立て、 イメージの詳細の表の base の「主な内容」にも shellcheck を足す - 新しいテストの補助関数に _first_apt_install を加える - 受け入れ条件 6 は apt-get install -s の Inst 行で新規の集合そのものを列挙して確かめる Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
🔧 /ndf:fix サマリ | round 1 | commit 4cbb1b8対応件数: critical=0 / major=0 / minor=3（合計 3 件） |
takemi-ohama
left a comment
takemi-ohama
left a comment
There was a problem hiding this comment.
🤖 cross-review | round 2 | codex | APPROVE
修正を求める指摘はありません。
takemi-ohama
left a comment
takemi-ohama
left a comment
There was a problem hiding this comment.
🤖 cross-review | round 2 | kiro | APPROVE
PLAN67 設計/計画文書の追記位置・テスト補助関数・検証手順の記述を精緻化する docs-only PR。行番号（base 行=366、節=375〜419）・節名・参照する既存テスト補助関数 _first_apt_install の意味論はいずれも現行ファイルと一致しており、修正を要する不整合は見つからなかった。
🔧 /ndf:fix サマリ | commit (なし)対応件数: critical=0 / major=0 / minor=0（合計 0 件） |
1 participant
Summary
shellcheckを入れる）の要求と受け入れ条件、および設計だけを載せた Pull Request です。実装は含みませんstandard。この Pull Request をマージしてから実装へ進みます決めたこと
issues/PLAN67_base-shellcheck-design.mdRUNの 1 回目のapt-get installの一覧へ足すRUNへshellcheck --versionを足して止めるcontainers/lfmには入れないTest plan
devbase-base:latest（arm64 / Ubuntu 26.04.1）から作った一時コンテナでapt-get install -y --no-install-recommends shellcheckが install_exit=0、新しく入るのはshellcheck/libnuma1の 2 つshellcheck --versionが exit=0（version: 0.11.0）、echo $fooの 1 行でSC2086を出し exit=1containers/*/DockerfileのFROMを確認し、base を継ぐ派生イメージが 7 つ、lfm / snapshot は継がないことを確認