Skip to content

feat(bulk-scan): enforce a cost limit for each repository - #379

Open
ianw-oai wants to merge 2 commits into
mainfrom
dev/ianw/bulk-scan-cost-limits-20260813
Open

feat(bulk-scan): enforce a cost limit for each repository#379
ianw-oai wants to merge 2 commits into
mainfrom
dev/ianw/bulk-scan-cost-limits-20260813

Conversation

@ianw-oai

Copy link
Copy Markdown
Collaborator

Changes

  • Add the existing cost limit option to bulk scans and container runs.
  • Apply the limit separately to each repository attempt.
  • Record exhausted budgets without retrying completed spending.
  • Reject resumed scans when the configured budget changes.

Checks

  • All 40 bulk scan tests passed.
  • All 12 container entrypoint tests passed.
  • TypeScript and formatting checks passed.

@github-actions github-actions Bot added the enhancement New feature or request label Aug 13, 2026
kmbroai
kmbroai previously approved these changes Aug 13, 2026
zcrab-oai
zcrab-oai previously approved these changes Aug 13, 2026

@zcrab-oai zcrab-oai left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Looks good. Per-repository bulk cost limits, exhausted-budget receipts, resume consistency, and container handling match the merged internal fix.

…st-limits-20260813

# Conflicts:
#	sdk/typescript/src/multiscan.ts
#	sdk/typescript/tests-ts/multiscan.test.ts
@ianw-oai
ianw-oai dismissed stale reviews from zcrab-oai and kmbroai via 486be45 August 13, 2026 17:31
@ianw-oai
ianw-oai requested review from kmbroai and zcrab-oai August 13, 2026 17:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants