Skip to content

Conversation

@maekawataiki
Copy link
Collaborator

@maekawataiki maekawataiki commented Jan 11, 2025

変更内容の説明

  • Parameter Handling の改善
    • Construct から Context を読み込んでいたのをトップレベルからの受け渡しに変更
    • Zod を使用した Validation の追加
    • parameter.ts にて複数環境管理できる構成への変更
      • 後方互換性のために、CDK context (cdk.json) → parameter.ts の順で評価し、指定した envparameter.ts でに存在すれば parameter.ts の設定で上書き、なければ CDK context の設定を利用。
  • Snapshot Test の追加
    • 全機能を有効化した Snapshot Test の追加 (Cfn の変更のみ確認。アセットの変更は無視)
    • Pull Request 時のワークフローにて Snapshot 更新が反映されているか確認。

チェック項目

  • npm run lint を実行した
  • 関連するドキュメントを修正した
  • 手元の環境で動作確認済み

関連する Issue

#462
#646

@maekawataiki maekawataiki changed the base branch from main to v3.0.0 January 14, 2025 02:08
timeout: cdk.Duration.minutes(15),
environment: {
KNOWLEDGE_BASE_ID: props.knowledgeBaseId,
KNOWLEDGE_BASE_ID: props.knowledgeBaseId ?? '',
Copy link
Contributor

Choose a reason for hiding this comment

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

これって正しいです?knowledgeBaseId が undefined でも Lambda リソース作ります?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

ありがとうございます。
作らないですね。knowledgeBaseId は ragEnabled 有効時必ず渡されるはずなので optional 外しました。

@tbrand tbrand added the v3 label Jan 15, 2025
@tbrand tbrand merged commit 30aeb62 into v3.0.0 Jan 16, 2025
@tbrand tbrand deleted the refactor/parameter branch January 16, 2025 02:21
@tbrand tbrand mentioned this pull request Jan 16, 2025
3 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants