Skip to content

feat: add IaC support for high-confidence AWS rules#48

Merged
dannysteenman merged 6 commits intomainfrom
codex/iac-high-confidence-rules
Mar 27, 2026
Merged

feat: add IaC support for high-confidence AWS rules#48
dannysteenman merged 6 commits intomainfrom
codex/iac-high-confidence-rules

Conversation

@axonstone
Copy link
Copy Markdown
Contributor

Summary

  • Added IaC support for the high-confidence AWS discovery rules across @cloudburn/rules and @cloudburn/sdk.
  • Extended the AWS static dataset registry and shared rule metadata so the affected rules now evaluate Terraform and CloudFormation inputs alongside live discovery.
  • This was needed to close the gap between discovery-mode coverage and static scan coverage for rules that can be evaluated conservatively from IaC alone.

Diagram

flowchart LR
  IaC[Terraform / CloudFormation] --> Parser[parseIaC]
  Parser --> Registry[AWS static dataset registry]
  Registry --> Datasets[Normalized static datasets]
  Datasets --> Rules[Dual-mode AWS rules]
  Rules --> Findings[IaC findings]
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

Related Issues

None.

@axonstone axonstone added the enhancement New feature or request label Mar 24, 2026
@axonstone axonstone requested a review from dannysteenman March 24, 2026 18:17
@axonstone
Copy link
Copy Markdown
Contributor Author

roborev: Combined Review (55811d5)

Verdict: No High or Critical findings identified across the combined reviews.

Multiple agents reviewed the diff, including two security-focused reviews, and none reported any High or Critical issues.

Medium-severity concerns were reported by one reviewer, but they are omitted here per the requested severity filter.


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

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 0 new potential issues.

View 13 additional findings in Devin Review.

Open in Devin Review

@dannysteenman dannysteenman merged commit c5da62e into main Mar 27, 2026
5 of 6 checks passed
@dannysteenman dannysteenman deleted the codex/iac-high-confidence-rules branch March 27, 2026 15:52
@github-actions github-actions bot mentioned this pull request Mar 27, 2026
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 0 new potential issues.

View 17 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

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants