kuju63 / kuju63/code-review-agents

評価コンテナのA2Aサーバー readiness契約が不一致(ポート・/healthマウント)

Open
#362 0 comments 0 reactions 0 assignees View on GitHub
bug
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

Open the contributing 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

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.