Skip to content

chore(security): reduce AppSec scanner Bedrock cost - #5

Merged
abir-vim merged 2 commits into
masterfrom
chore/appsec-cost-controls
Oct 8, 2026
Merged

abir-vim merged 2 commits into
masterfrom
chore/appsec-cost-controls

Conversation

@abir-vim

@abir-vim abir-vim commented Oct 8, 2026

Copy link
Copy Markdown
Collaborator

Automated fleet sync of the Vim AppSec Agent. Scans when review starts instead of after every push, skips high-volume deployment/publish bots, supports the centrally configured Sonnet 5 profile, and avoids retrying permanent Bedrock failures. The fail-closed high-severity gate and PR findings comments remain enabled. See scanner-org for the source template.

@github-actions

github-actions Bot commented Oct 8, 2026 •

Copy link
Copy Markdown

Vim AppSec Agent

Passed — findings are below the gate (blocks at high or dependency BLOCK).

Critical High Medium Low Info Dep BLOCK Dep WARN
0 0 0 0 2 0 0

Findings

Info (2)

New dependency-only policy skips Bedrock SAST phases for vim-gitops-app[bot]

  • Where: .github/workflows/security.yml:80
  • CWE: CWE-693
  • Remediation: No change required; the bypass is correctly constrained to a no-code trigger shape. If stricter assurance is desired, additionally assert the diff is empty of content changes (e.g. verify dummy.txt is the only path AND its hunk is trivial) and emit a visible audit log line (already present) so dependency-only runs remain reviewable.

600s debounce on synchronize relies on cancel-in-progress and can delay the verdict

  • Where: .github/workflows/security.yml:44
  • CWE: CWE-400
  • Remediation: Acceptable as-is. If merge latency becomes a concern, shorten the debounce window or move it to a lighter-weight dedicated step; no security remediation needed since the gate remains fail-closed.

Gate blocks at severity high or a dependency marked BLOCK.

Questions, false positives, or feedback — ping @abir-vim.

@abir-vim
abir-vim merged commit 916cdf7 into master Oct 8, 2026
1 check passed
@abir-vim
abir-vim deleted the chore/appsec-cost-controls branch October 8, 2026 14:26
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