kuju63 / kuju63/code-review-agents
評価コンテナのA2Aサーバー readiness契約が不一致(ポート・/healthマウント)
- Dominant language
- HTML
- Stars
- 0
- Forks
- 0
- Avg merge
- 1d 7h
- Merged PRs (30d)
- 96
Description
## 問題
評価パイプラインのコンテナ起動スクリプトとTS版A2Aサーバーのreadiness契約が食い違っている。
- `.claude/skills/run-evaluation/scripts/start_a2a_container.sh` はコンテナ起動後 `http://localhost:8000/health` をポーリングして起動完了を待つ。
- 現行の `packages/a2a-server/src/index.ts` はport `3000` でリッスンしており、`modules/health/` の `/health` ルートは `index.ts` にマウントされていない。
- コンテナビルドで使う `Dockerfile` の既定 `CMD` はA2Aサーバー起動ではなく `node -e "console.log(...)"` というプレースホルダのログ出力になっている。
この状態では、コンテナを起動しても評価スクリプトのヘルスチェックが成立するかどうか、本ドキュメント作成時点では実機検証できていない([docs/a2a-api-design.md](https://github.com/kuju63/code-review-agents/blob/main/docs/a2a-api-design.md) §1「既知の未接続箇所」、[docs/eval-a2a-container-runtime-spec.md](https://github.com/kuju63/code-review-agents/blob/main/docs/eval-a2a-container-runtime-spec.md)、[docs/plan/eval-sharded-execution-spec.md](https://github.com/kuju63/code-review-agents/blob/main/docs/plan/eval-sharded-execution-spec.md) に記録済み)。
## 受け入れ条件
1. `Dockerfile` の既定 `CMD` をA2Aサーバー起動に変更する。
2. `index.ts` のリスニングポートと `start_a2a_container.sh`/`stop_a2a_container.sh` が待ち受ける/操作するポートを一致させる。
3. `modules/health/` の `/health` ルートを `index.ts` にマウントする。
4. 実際にコンテナを起動して `GET /health` が200を返すことを検証する自動テストを追加する。
## 経緯
PR #360 のレビューでCodeRabbitから指摘されたが、そのPRはドキュメントとソースコードの整合性を取る範囲(docs-onlyブランチ)であり、インフラ/コード変更はスコープ外と判断されたため、フォローアップIssueとして切り出した。
Co-Authored-By: Claude Sonnet 5
Contributor guide
Research direction
Start with Dockerfile, packages/a2a-server/src/index.ts, and .claude/skills/run-evaluation/scripts/start_a2a_container.sh and stop_a2a_container.sh; compare the configured command, port, and health route. Review the referenced runtime specifications, then add the container-level test and verify that starting the container makes GET /health return 200 on the port used by the scripts.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- docker, dockerfile, shell, typescript
- Domain
- api, backend, devops, testing-qa
- Issue type
- Bug
- Difficulty
- 4/5
- Estimated time
- 3-5 days
- Activity status
- Active
- Clarity
- Clearly specified
- Newbie friendliness
- 68/100