Skip to content

fix: streamline discovery targeting and live catalog loading#55

Merged
dannysteenman merged 5 commits intomainfrom
codex/fix-discovery-cli-resource-explorer
Apr 1, 2026
Merged

fix: streamline discovery targeting and live catalog loading#55
dannysteenman merged 5 commits intomainfrom
codex/fix-discovery-cli-resource-explorer

Conversation

@dannysteenman
Copy link
Copy Markdown
Member

Summary

  • What changed?
    • Reduced AWS live discovery pressure by batching Resource Explorer requests, adding throttling-aware retries, and reusing memoized dataset loads where possible.
    • Redesigned CLDBRN-AWS-CLOUDWATCH-2 to evaluate log groups from latest stream activity summaries instead of enumerating every log stream.
    • Added SDK/provider debug tracing, restored the CLI discover --region single-region flag, and kept multi-region discovery support in the SDK.
  • Why was this needed?
    • Broad discovery runs were easy to throttle in Resource Explorer, large CloudWatch log-stream inventories could become very slow, and the CLI targeting/debug surface needed to be simpler and more predictable.

Diagram

flowchart LR
  CLI["CLI discover --region"] --> SDK["CloudBurnClient / SDK"]
  SDK --> RE["Resource Explorer batched catalog"]
  RE --> Datasets["Memoized dataset loaders"]
  Datasets --> CW["CloudWatch latest-stream activity summary"]
  Datasets --> Rules["Discovery rules"]
  CW --> Rules
Loading

Scope

  • cloudburn (cli)
  • @cloudburn/sdk
  • @cloudburn/rules
  • docs/community files

Release Notes

  • Added a .changeset/*.md file for published package changes
  • No published package changes in this PR

Verification

  • pnpm lint
  • pnpm typecheck
  • pnpm test
  • pnpm build
  • pnpm verify

Boundary Checks

  • No engine/parser/provider logic added to @cloudburn/rules
  • CLI delegates scan logic to SDK
  • README/CONTRIBUTING/docs updated when behavior changed

@dannysteenman dannysteenman added the bug Something isn't working label Mar 31, 2026
@dannysteenman dannysteenman requested a review from axonstone March 31, 2026 19:32
devin-ai-integration[bot]

This comment was marked as resolved.

@dannysteenman dannysteenman merged commit becbfa3 into main Apr 1, 2026
5 of 6 checks passed
@dannysteenman dannysteenman deleted the codex/fix-discovery-cli-resource-explorer branch April 1, 2026 06:55
@github-actions github-actions bot mentioned this pull request Apr 1, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant