Skip to content

派生イメージ: コンテナに入っても Python・TypeScript・PHP の言語サーバが無く、Claude Code の診断が届かない → general・php・php85 の派生イメージに言語サーバを入れ、入った時点で診断が使える(#236) - #477

Merged
takemi-ohama merged 2 commits into
mainfrom
design/issue-236
Oct 12, 2026

Conversation

@takemi-ohama

Copy link
Copy Markdown
Contributor

#236 の要求を派生イメージの形で書き直し、設計文書を加えました。設計では、言語サーバを general・php・php85 の派生イメージに入れる方針を書いています。あわせて用語集に語を足しています。文書だけの変更で、コードやイメージには手を入れていません。

利用者向けの変化

  • 無し

移行の手順

  • 無し

変更の要点

テスト

ステップ exit 結果
glossary 0 宣言と用語集が揃っている
requirements-check 2 issues/issue-236-requirements.md を読めずに停止した([Errno 2] No such file or ...)
  • requirements-check は要求の文書を読めず、検査が行われていません。このファイルはこの PR の変更に含まれているので、マージ前に検査をやり直して結果を確かめる必要があります。

モード: standard / 通した工程: 設計 → 要求と受け入れ条件 → ドキュメントレビュー

🤖 Generated with Claude Code

決めたこと

issues/issue-236-design.md

  • 決定 1: どのプロジェクトのコードでも同じ診断が使えるよう、general・php・php85 の 3 つに 3 つの言語をすべて入れる
  • 決定 2: base の tsc と利用者の npm のグローバル領域を変えないため、言語サーバを専用の置き場 /opt/language-servers に入れ、3 つのコマンドだけを PATH に出す
  • 決定 3: 版の固定を利用者の更新で崩さないため、置き場は root の RUN で作り、利用者は書けない形にする
  • 決定 4: イメージと ai-plugins の版の固定を結び付けないため、Serena の uv のキャッシュは温めず (b) を採る。Serena の初回の起動にはネットワークが要る
  • 決定 5: 3 つの Dockerfile の言語サーバの組を揃えるため、同じ RUN を各 Dockerfile に書き、検査で同じ組であることを縛る
  • 決定 6: 受け入れ条件 2・3 の手動確認は、entrypoint を bash に替えて打つ

takemi-ohama and others added 2 commits October 13, 2026 02:05
言語サーバは base ではなく派生イメージで入れるという利用者の決定(2026-10-13)に合わせ、
受け入れ条件を派生イメージの形へ書き換えた。用語集に「言語サーバ」「uv のキャッシュ」を足す。

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
言語サーバの組(pyright・typescript-language-server・typescript 5・intelephense)を
/opt/language-servers へ root で入れ、3 つのコマンドだけを PATH に出す。base の tsc(7.0.2)と
npm のグローバル領域は変えない。Serena は PATH の言語サーバを使わず自分で取得するため、
uv のキャッシュは温めない(案 (b))。用語集に「言語サーバの置き場」「言語サーバの組」を足す。

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>

@takemi-ohama takemi-ohama left a comment

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🤖 cross-review | round 1 | codex | APPROVE

対象のドメインモデル節と用語集差分に、critical / major の指摘はありません。判断の根拠: Mission / Vision / Value 1・3(実装の食い違い、再現性、プロジェクト境界への影響を確認)。

@takemi-ohama takemi-ohama left a comment

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🤖 cross-review | round 1 | kiro | APPROVE

ドメインモデルの節と用語集の差分を見た。コンテキストマップは image-lineage 内の変更として ai-plugins への順応者関係を宣言済み、集約の持ち主は 3 つの Dockerfile を同一値のインスタンスとして I3 で縛る明示があり、不変条件は 1 条件ずつで I2(typescript 5.x)と I4(PATH の tsc は 7.0.2 のまま別領域)に矛盾は無く、ドメインイベントは E1〜E7 すべてに受け手があり E2 を欠番として要求の番号を引き継ぎ、用語表の 4 語は glossary.json / glossary.md と内容が一致していた。critical / major の基準に当たる修正アクションは見つからなかった。

@takemi-ohama takemi-ohama left a comment

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🤖 cross-review | round 2 | claude | APPROVE

要求・設計・用語集の差分を、対象の Dockerfile(general・php・php85)と base の PATH・npm のグローバル領域の実際の形に照らして確かめた。critical / major の基準に当たる修正アクションは無い。根拠: Mission / Value 1 / Value 3 / P1 / P3(MVV 版 1)。

@takemi-ohama takemi-ohama left a comment

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🤖 cross-review | round 2 | kiro | APPROVE

critical / major の指摘は無い。設計は P1(base を変えない)を守り、差分の containers/base/ は空のまま。glossary.json と glossary.md の 4 語は一致し、I1〜I8 は受け入れ条件とテスト設計の行へ一対一で対応していて矛盾は見つからなかった。実装者がよく踏む npm install --prefix の bin の置き場(node_modules/.bin ではなく手で張る /opt/language-servers/bin)も設計本文が明示しており、違うものを作る食い違いには当たらない。

@takemi-ohama

Copy link
Copy Markdown
Contributor Author

🔧 /ndf:fix サマリ | commit (なし)

対応件数: critical=0 / major=0 / minor=0(合計 0 件)
決着: 0 件 / 見送り: 0 件 / 却下: 0 件
CI: NONE

@takemi-ohama
takemi-ohama marked this pull request as ready for review October 12, 2026 17:21
@takemi-ohama takemi-ohama added the design-approved ドキュメントレビューを人間が承認した label Oct 12, 2026
@takemi-ohama
takemi-ohama merged commit 1cad4b3 into main Oct 12, 2026
10 checks passed
@takemi-ohama takemi-ohama mentioned this pull request Oct 12, 2026
2 tasks done
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

design-approved ドキュメントレビューを人間が承認した

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant