Skip to content

chore: version packages#39

Merged
dannysteenman merged 1 commit intomainfrom
changeset-release/main
Mar 19, 2026
Merged

chore: version packages#39
dannysteenman merged 1 commit intomainfrom
changeset-release/main

Conversation

@github-actions
Copy link
Copy Markdown
Contributor

This PR was opened by the Changesets release GitHub action. When you're ready to do a release, you can merge this and the packages will be published to npm automatically. If you're not ready to do a release yet, that's fine, whenever you add more changesets to main, this PR will be updated.

Releases

cloudburn@0.9.3

Patch Changes

  • Updated dependencies []:
    • @cloudburn/sdk@0.17.1

@cloudburn/rules@0.17.1

Patch Changes

  • #38 b46ad3b Thanks @dannysteenman! - Flag CloudWatch unused log streams when they have never received events or when their last ingestion was more than 90 days ago.

@cloudburn/sdk@0.17.1

Patch Changes

  • Updated dependencies [b46ad3b]:
    • @cloudburn/rules@0.17.1

@axonstone
Copy link
Copy Markdown
Contributor

roborev: Combined Review (e32e9ca)

Verdict: No high or critical findings from the completed reviews; this PR looks clean.

Completed codex reviews (security and default) reported no issues.

claude-code reviews failed and produced no findings.


Synthesized from 4 reviews (agents: claude-code, codex | types: default, security)

@dannysteenman dannysteenman merged commit 241b219 into main Mar 19, 2026
@dannysteenman dannysteenman deleted the changeset-release/main branch March 19, 2026 19:44
Copy link
Copy Markdown
Contributor

@devin-ai-integration devin-ai-integration Bot left a comment

Choose a reason for hiding this comment

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

Devin Review found 1 new potential issue.

⚠️ 1 issue in files not directly in the diff

⚠️ completion command bypasses the shared CliResponse / renderResponse formatter pipeline (packages/cloudburn/src/commands/completion.ts:37-41)

The completion command at packages/cloudburn/src/commands/completion.ts:37-41 writes directly to process.stdout.write(output) without building a typed CliResponse or routing through renderResponse. The REVIEW.md rule states: "All stdout-producing commands should build a typed CliResponse and render it through the shared renderResponse(response, format) pipeline." Every other stdout-producing command in the CLI (scan, discover, init, estimate, rules list) follows this pattern — for example, init uses the document kind for its raw YAML output (packages/cloudburn/src/commands/init.ts:26-33). The completion command could use the same document kind with a default format of text to stay consistent with the established contract.

View 8 additional findings in Devin Review.

Open in Devin Review

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