Skip to content

Commit eba70ca

Browse files
fix: address PR review comments
- Keep gitleaks and osv-scanner enabled in CI build (only disable trivy/grype/semgrep which need GitHub API downloads) - Remove unused @aws-sdk/client-bedrock-agentcore mock from orchestrate-task.test.ts (SDK is no longer imported by orchestrator) - Update PR description to note additive strategy_type event field
1 parent 41e2e2d commit eba70ca

2 files changed

Lines changed: 3 additions & 7 deletions

File tree

.github/workflows/build.yml

Lines changed: 3 additions & 1 deletion
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

cdk/test/handlers/orchestrate-task.test.ts

Lines changed: 0 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -27,12 +27,6 @@ jest.mock('@aws-sdk/lib-dynamodb', () => ({
2727
UpdateCommand: jest.fn((input: unknown) => ({ _type: 'Update', input })),
2828
}));
2929

30-
jest.mock('@aws-sdk/client-bedrock-agentcore', () => ({
31-
BedrockAgentCoreClient: jest.fn(() => ({ send: jest.fn() })),
32-
InvokeAgentRuntimeCommand: jest.fn((input: unknown) => ({ _type: 'InvokeAgentRuntime', input })),
33-
StopRuntimeSessionCommand: jest.fn((input: unknown) => ({ _type: 'StopRuntimeSession', input })),
34-
}));
35-
3630
const mockHydrateContext = jest.fn();
3731
jest.mock('../../src/handlers/shared/context-hydration', () => ({
3832
hydrateContext: mockHydrateContext,

0 commit comments

Comments
 (0)