feat: Bedrock AgentCore Runtime + Amplify Gen 2 ホスト型 Web UI (feature/agentcore-amplify-deployment)

[clouddev-code]

feat: Bedrock AgentCore Runtime + Amplify Gen 2 ホスト型 Web UI

Branch: feature/agentcore-amplify-deployment
Base: main
Commit: 5b35091 feat: Add Bedrock AgentCore runtime and Amplify-hosted Next.js web UI

概要

Slidev Agent を Bedrock AgentCore Runtime にデプロイ可能なマルチエージェント (Strands Graph) 構成 に拡張し、それを呼び出す AWS Amplify Gen 2 (Next.js 14 App Router) の Web UI を追加しました。インフラは AWS CDK で @aws-cdk/aws-bedrock-agentcore-alpha の L2 Construct を用いて IaC 化しています。

アーキテクチャ要約

[ Next.js (Amplify Hosting) ]
        │
        │  GraphQL mutation
        ▼
[ AppSync (Amplify Data) ] ──► [ DynamoDB SlideJob ]
                                       │ Streams
                                       ▼
                          [ Lambda: generate-slides ]
                                       │
                                       │  InvokeAgentRuntime (SSE)
                                       ▼
                       [ Bedrock AgentCore Runtime ]
                                       │
              planner → researcher → writer → validator
                                                │
                                                └─[needs_revision]→ writer
                                       │
                                       ▼
                                  [ S3 (slides.md) ]

主な変更内容

1. マルチエージェント Strands Graph (src/slidev_agent/agent.py)

• 単一エージェント (CLI) と並行して、4 ロールの Strands GraphBuilder 構成 を導入
  • planner … 構成案を作る (tool 不要)
  • researcher … web_search / web_extract で情報収集
  • writer … write_slidev_markdown で .md / S3 へ書き出し
  • validator … validate_slides_fit で枠内検証、approved / revision needed を出力
• validator → writer のフィードバックループ (_needs_revision で条件分岐)

2. AgentCore Runtime ハンドラ (src/slidev_agent/runtime.py)

• BedrockAgentCore Python SDK の @app.entrypoint でグラフを SSE ストリーミング起動
• multiagent_node_stream イベントから Lambda 側へノード進捗を中継
• セッション ID は AgentCore 規定の X-Amzn-Bedrock-AgentCore-Runtime-Session-Id を尊重

3. ツール拡張

• tools/writer.py … ローカル / S3 両対応のスライド書き出し
• tools/validator.py … スライドのオーバーフロー検証ロジック追加

4. CDK インフラ (infra/)

• slidev-agent-runtime-stack.ts … AgentCore Runtime, IAM Role, Secrets Manager 参照, S3 バケット, CloudWatch LogRetention
• @aws-cdk/aws-bedrock-agentcore-alpha L2 を使用
• Tavily API Key は Secrets Manager から runtime 内で取得

5. Amplify Gen 2 バックエンド (web/amplify/)

• auth/resource.ts … Cognito User Pool + Identity Pool
• data/resource.ts … AppSync GraphQL + DynamoDB SlideJob モデル (allow.owner)
• storage/resource.ts … S3 jobs/{entity_id}/* (Identity Pool 認可)
• functions/generate-slides/ … DynamoDB Streams トリガの Lambda
• backend.ts … Streams / AppSync / AgentCore InvokeAgentRuntime の IAM 接続

6. Next.js 14 App Router フロントエンド (web/app/, web/components/)

• signin / generate / dashboard / jobs/[id] ページ
• SlideForm, JobProgress, SlidevPreview, Nav コンポーネント
• observeQuery でジョブステータスをリアルタイム購読

7. デプロイ周辺

• Dockerfile (linux/arm64), .dockerignore
• amplify.yml (monorepo の web/ を Amplify Hosting でビルド)

変更ファイル統計

39 files changed, 2835 insertions(+), 269 deletions(-)

主要追加: infra/ (CDK), web/ (Amplify + Next.js), Dockerfile, amplify.yml
主要変更: src/slidev_agent/{agent,runtime,tools/writer,tools/validator}.py, pyproject.toml

既知の課題 (要対応)

3 名 (CDK / バックエンド / フロントエンド) 並列レビューで検出済み。詳細は docs/team-review-amplify-agentcore.md。

Critical (本番ブロッカー)

#	領域	内容
A	IaC / FE / BE	Storage 認可ルール jobs/{entity_id}/* (Cognito Identity ID) と Lambda が書く jobs/${ULID}/slides.md の不整合 — 正規ユーザーが自分のスライドをダウンロードしようとしても AccessDenied
B	Frontend	web/app/signin/page.tsx の Authenticator render-prop callback 内で useEffect を呼ぶ Rules of Hooks 違反
C	Backend	agent.py:_writer_invocation_state() の戻り値が未使用。Writer が invocation_state 経由で output_path (S3 URI) を受け取れていない
D	Backend / IaC	Bedrock cross-region inference profile ID us.anthropic.claude-opus-4-6-v1 に version suffix :0 が欠落。ValidationException で起動時に全失敗

High (リリース前対応)

• bedrock:InvokeModel / DDB Streams ポリシーの resources: ['*'] を ARN で絞り込み
• 同期 SSE 5〜15 分処理を 900s Lambda で受ける構造 → SQS / Step Functions Express への分解
• Amplify.configure() の 2 重実行 → ConfigureAmplifyClientSide パターンへ統一
• web/package.json のバックエンド依存 (@aws-sdk/*, aws-cdk-lib 等) を devDependencies に
• multiagent_node_stream で current_tool_use (toolUse / toolResult) を取りこぼし
• reset_on_revisit(True) で validator 指摘を writer が忘れる
• _needs_revision の文字列部分一致 → 日本語出力時に暗黙承認
• Dockerfile の uv sync フォールバックで venv パスがズレる

関連ドキュメント

• 仕様: docs/agentcore-amplify-deployment-spec.md
• 実装レポート: docs/IMPLEMENTATION_REPORT.md
• 3 観点並列レビュー: docs/team-review-amplify-agentcore.md

チェックリスト

• Critical-A: SlideJob に identityId 追加 + S3 キーを jobs/${identityId}/${job.id}/slides.md に
• Critical-B: signin/page.tsx の useEffect を別 Component に分離
• Critical-C: stream_async(invocation_state=...) を渡し、ツールを @tool(context=True) 化
• Critical-D: bedrockModelId に :0 suffix 付与
• High 各項目への対応
• Bedrock AgentCore Runtime への実デプロイ検証
• Amplify Hosting への実デプロイ検証

[clouddev-code]

Round 1 — レビュー結果 (review-loop)

対象: git diff origin/main...HEAD (39 files, +2835/-269)
focus: quality, cdk / max_rounds: 5
検出数: 🔴 critical=3 / 🟠 major=27 / 🟡 minor=12 / 🔵 nit=4 = 計 46 件

---

🔴 Critical

C1. IAM ポリシーが Bedrock InvokeModel に resources: ['*']

• file: infra/lib/slidev-agent-runtime-stack.ts:69
• issue: 全モデル・全リージョンの InvokeModel/InvokeModelWithResponseStream を許可。
• 修正案: arn:${partition}:bedrock:${region}::foundation-model/${modelId} と inference-profile ARN に絞る。

C2. DynamoDB Streams 読み取りが resources: ['*']

• file: web/amplify/backend.ts:28-37
• issue: 全アカウント・全テーブルの Streams DescribeStream/GetRecords を許可。
• 修正案: L2 の DynamoEventSource に置き換える (IAM が自動スコープされる)。

C3. Authenticator の render prop 内で useEffect 呼び出し (Rules of Hooks 違反)

• file: web/app/signin/page.tsx:13
• issue: render prop 内で Hook を呼ぶと呼び出し順序が変動し、本番でフック index ズレでクラッシュ。
• 修正案: ページ本体で useAuthenticator() を購読し、useEffect(() => { if (user) router.replace('/generate') }, [user, router]) に書き換える。

---

🟠 Major (27件 — 抜粋・分類)

IAM / セキュリティ

• M1 web/amplify/backend.ts:57-68 — Lambda → AgentCore Invoke が resources: ['*']。SSM 経由で runtime ARN に絞る。
• M2 web/amplify/functions/generate-slides/handler.ts:136 — sessionId が Math.random() で予測可能 → crypto.randomUUID()。
• M3 src/slidev_agent/tools/writer.py:57 — _write_local がパストラバーサル脆弱性 (../../etc/passwd 等)。is_relative_to(base) で正規化。
• M4 src/slidev_agent/runtime.py:98 — 未知 event を生 dict のままクライアントに転送 (内部情報露出リスク)。allowlist で絞る。
• M5 web/components/SlidevPreview.tsx:25 — 例外メッセージをそのまま UI に表示 (Cognito identity 等が漏れる可能性)。

Docker / IaC

• M6 Dockerfile:24-37 — root 実行・USER ディレクティブなし。非 root ユーザー追加。
• M7 Dockerfile:37 — HEALTHCHECK 未設定。
• M8 infra/lib/slidev-agent-runtime-stack.ts:42-60 — AgentCore が usingPublicNetwork()。VPC モード検討 (alpha 未対応なら TODO 明示)。
• M9 infra/lib/slidev-agent-runtime-stack.ts:90 — this.runtime.role! の non-null assertion はランタイム TypeError リスク。
• M10 infra/lib/slidev-agent-runtime-stack.ts:94-97 — LogRetention に未解決トークン agentRuntimeId を渡してデプロイ失敗リスク。
• M11 infra/cdk.json:15-19 — CDK feature flags が最小限。cdk init 最新版の context を取り込む。
• M12 web/amplify/backend.ts:41-51 — CfnEventSourceMapping で DLQ/bisect/retryAttempts 未設定 → サイレント失敗または無限リトライ。
• M13 web/amplify/functions/generate-slides/resource.ts:18 — AWS_REGION_AGENTCORE: 'us-east-1' ハードコード。secret 化。

Python / エージェント

• M14 src/slidev_agent/runtime.py:137 — entrypoint で except Exception でスタックトレース消失。logger.exception 必須。
• M15 src/slidev_agent/tools/validator.py:37 — except Exception: return None で S3 障害が全て「ファイルなし」扱い。ClientError 個別捕捉。
• M16 src/slidev_agent/tools/validator.py:460 — validate_slides_fit 最外層で例外握り潰し。再 raise + 構造化エラー化。
• M17 src/slidev_agent/tools/writer.py:101 — write 失敗時に全例外を {success: False} 化。ClientError 分岐 + ログ。
• M18 src/slidev_agent/agent.py:250 — _needs_revision / _is_approved が部分文字列マッチで脆弱 → validator JSON 構造化出力に変更。
• M19 src/slidev_agent/runtime.py:42 — payload を Pydantic で検証していない。InvokePayload(BaseModel) 導入。
• M20 src/slidev_agent/runtime.py:88 — クライアント切断時の CancelledError 未処理で graph がバックグラウンド継続 → コスト垂れ流し。
• M21 src/slidev_agent/agent.py:59 — BEDROCK_MODEL_ID デフォルト us.anthropic.claude-opus-4-6-v1 に :0 サフィックスがなく InvalidModelId リスク。
• M22 src/slidev_agent/runtime.py:117 — str(text)[:1000] のマジックナンバー (根拠コメントなし)。
• M23 pyproject.toml:6 — requires-python = \">=3.13\" が AgentCore ベースイメージ (python:3.12) 等で詰む可能性。

Web

• M24 web/app/jobs/[id]/page.tsx:9 — params の Next.js 14→15 互換性 + id フォーマット検証なし → 他人の job_id 推測リスク。
• M25 web/components/SlidevPreview.tsx:42 — frontmatter 除去正規表現が per-slide frontmatter を巻き込む可能性。
• M26 web/lib/amplify-client.ts:16 — モジュールトップレベルで configureAmplify() 呼び出し → SSR で amplify_outputs.json 必須化。
• M27 web/components/JobProgress.tsx:52 — 署名 URL expiresIn: 600 のリフレッシュなし。10 分後に 403。

---

🟡 Minor (12件 — 代表のみ)

• Dockerfile:21 — uv sync || uv pip install フォールバックが再現性を損なう。
• .dockerignore — research/, src/output/ 未除外。
• infra/lib/slidev-agent-runtime-stack.ts:100-105 — SSM パラメーター名固定で複数環境衝突。
• web/amplify/functions/generate-slides/handler.ts:207-209 — SSE catch ブロックでペイロード無検証ログ書込。
• web/amplify/backend.ts:19 — スタック境界の assert 漏れ。
• amplify.yml:6-10 — appRoot: web 設定下で cd web 重複の疑い。
• src/slidev_agent/agent.py:312 — set_max_node_executions(12) / set_execution_timeout(900) マジックナンバー。
• src/slidev_agent/tools/validator.py:192 — _estimate_rows 約100行で単一責任違反。
• tests/* — 新規モジュールの単体テスト欠如。
• web/components/Nav.tsx:17 — <a href=\"#\"> でサインアウト (a11y/UX)。
• web/components/SlideForm.tsx:63 — Number('')===0 で min=3 制約回避可能。

🔵 Nit (4件)

• infra/cdk.json — cdk.context.json を .gitignore 除外 → コミット推奨。
• infra/tsconfig.json:18 — strictPropertyInitialization: false の解除。
• infra/lib/slidev-agent-runtime-stack.ts:107-119 — Output に exportName の有無が混在。
• web/app/page.tsx:16 — インライン style と className 混在。

---

次の修正方針 (Round 2 案)

Critical 3 件 + Major のうち IAM/セキュリティ系 (M1–M5)・Docker (M6,M7)・CDK (M9,M10) を最優先で対応。
特に C1/C2/M1 の IAM 絞り込み と C3 の Hook ルール違反 は本番デプロイ前に必須。
M14–M21 の Python 例外/バリデーション系は次バッチで対応。
diff 規模が大きいため、ユーザー判断次第で対応スコープを「Critical + 一部 Major」に絞ることを推奨。

[clouddev-code]

Round 2 — 修正適用 (Critical のみ)

ユーザー指示により Critical 3 件のみを対応。Major/minor/nit は持ち越し (本ブランチでは未対応 → 別 PR / 別ループ推奨)。

diff stat

 infra/lib/slidev-agent-runtime-stack.ts | 13 ++++++++++++-
 web/amplify/backend.ts                  |  8 +++++++-
 web/app/signin/page.tsx                 | 20 +++++++++++---------
 3 files changed, 30 insertions(+), 11 deletions(-)

適用内容

C1: infra/lib/slidev-agent-runtime-stack.ts:69

• Bedrock InvokeModel の resources: ['*'] を以下に絞り込み:
  • arn:${partition}:bedrock:*::foundation-model/${strippedModelId} (us. / eu. / apac. プレフィックスを除去した foundation model ARN、cross-region inference 対応で region は *)
  • arn:${partition}:bedrock:${region}:${account}:inference-profile/${bedrockModelId} (システム inference profile ARN)

C2: web/amplify/backend.ts:35

• DynamoDB Streams ポリシーを 2 つに分割:
  • DescribeStream/GetRecords/GetShardIterator → slideJobTable.tableStreamArn! に限定
  • ListStreams のみリソースレベル非対応のため * を維持 (AWS 仕様)

C3: web/app/signin/page.tsx:13

• Authenticator の render prop 内 useEffect 呼び出しを削除
• RedirectAfterSignIn 子コンポーネントに切り出し、useAuthenticator((context) => [context.user]) で購読
• <Authenticator> の children として渡すことで、認証後のみマウントされる構造に変更
• useEffect の依存配列に router も追加 (lint 警告対策)

持ち越し

Severity	持ち越し件数	備考
🟠 Major	27	別途対応推奨 (M1〜M27)
🟡 Minor	12	同上
🔵 Nit	4	同上

検証

• 静的型チェック: Node toolchain がローカル未インストールのため未実行。CI (Amplify build / CDK synth) での検証を推奨。
• 修正は IAM 絞り込みと React 構造変更のみで、ランタイム動作の意味的変更なし。

Round 2 終了判定

ユーザー指示 (Critical のみ対応) のスコープ内では収束。Major/minor/nit が残るためループは終了せず本 Issue は open のまま とし、別途対応の起点とする。