Skip to content

parity(eventbridge): fix all parity.md findings#2376

Closed
agbishop wants to merge 1 commit into
mainfrom
polecat/garnet/parity-eventbridge
Closed

parity(eventbridge): fix all parity.md findings#2376
agbishop wants to merge 1 commit into
mainfrom
polecat/garnet/parity-eventbridge

Conversation

@agbishop

Copy link
Copy Markdown
Collaborator

EventBridge parity. Validated. GH006-safe.

🤖 Generated with Claude Code

…ta, ListEventBuses Limit, Shutdown cancellation, tag leak

- paginate: encode next-page tokens as base64 so they are opaque to callers,
  matching real AWS behavior; parseNextToken retains a plain-integer fallback
  for tokens produced before this change
- paginateN: new generic helper that respects a caller-supplied page size (0 = default 100)
- ListEventBuses: add limit parameter to StorageBackend interface and InMemoryBackend
  so the Limit field from API requests is honored instead of always paging at 100
- CreateEventBus quota: count custom buses across all regions in b.buses to enforce
  the per-account limit (200), not per-region
- StartWorker: derive a cancellable context for scheduler and archiveJanitor goroutines;
  store the cancel func so Shutdown can terminate both workers before closing the backend
- DeleteEventBus / DeleteRule: describe the resource before deletion to obtain its ARN,
  then call clearResourceTags so the handler tags map does not grow without bound

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@agbishop agbishop enabled auto-merge (squash) June 25, 2026 23:41
@agbishop agbishop disabled auto-merge June 25, 2026 23:50
@agbishop

Copy link
Copy Markdown
Collaborator Author

Folded into parity-sweep #2342 (single accumulating PR per workflow)

@agbishop agbishop closed this Jun 25, 2026
@agbishop

Copy link
Copy Markdown
Collaborator Author

📊 Code Coverage Report

Metric Value Status
Total Coverage 0.0%
82.4%
New Code Coverage N/A (0/0 stmts)

Tip

This project maintains a minimum coverage threshold of 85%. Maintain or improve coverage on new code to ensure long-term stability.


Last updated: Fri, 26 Jun 2026 00:12:42 GMT

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.

1 participant