Skip to content

[Aikido] AI Fix for Overly Broad Permissions in GitHub Actions Workflows is risky - #237

Merged
shouze merged 1 commit into
mainfrom
fix/aikido-security-sast-154907805-rjy4
Oct 23, 2026
Merged

shouze merged 1 commit into
mainfrom
fix/aikido-security-sast-154907805-rjy4

Conversation

@aikido-autofix

Copy link
Copy Markdown
Contributor

What does this PR do?

This patch mitigates excessive workflow-level permissions by replacing the workflow-level contents write permission with an empty permissions block and granting contents read to the build job and contents write to the release job at the job level.

How did you verify your code works?

AI Confidence Level (confidence in the quality of this automated fix): Low confidence: Aikido has tested similar fixes, which indicate the correct approach but may be incomplete. Further validation is necessary.


@github-actions

Copy link
Copy Markdown
Contributor

Coverage after merging fix/aikido-security-sast-154907805-rjy4 into main will be

96.55%

Coverage Report
FileStmtsBranchesFuncsLinesUncovered Lines
src
   aggregate.ts100%100%100%100%
   api-utils.ts92.80%100%91.67%92.92%101–103, 165, 86–87, 92–93
   api.ts94.96%100%100%94.42%340–343, 405, 422, 53, 64–67, 69
   cache.ts94.37%100%100%93.94%139–141, 39
   completions.ts99.44%100%100%99.39%268
   gh-cli.ts100%100%100%100%
   group.ts98.37%100%98.33%98.37%1105, 263–265, 525, 531, 607, 865, 980
   output.ts99.31%100%95.83%99.62%89
   regex.ts99.37%100%100%99.32%360
   render.ts90.35%100%88.24%90.43%175, 177–178, 201–206, 208–210, 213, 234–235, 427–428, 528–532, 562–569, 571–579, 582–584
   scroll-cooldown.ts100%100%100%100%
   style.ts100%100%100%100%
   upgrade.ts86.73%100%92.31%86.36%128, 131, 133–134, 153–154, 167–169, 188–193, 195, 198–202, 209, 214, 250–253
src/render
   filter-match.ts97.22%100%92.31%100%
   filter.ts100%100%100%100%
   highlight.ts96.53%100%90.40%99.28%284–285
   layout-constants.ts100%100%100%100%
   mouse-hit.ts100%100%100%100%
   mouse.ts100%100%100%100%
   rows.ts99.34%100%100%99.29%239
   selection.ts100%100%100%100%
   summary.ts100%100%100%100%
   team-pick.ts100%100%100%100%
   terminal.ts100%100%100%100%

@shouze
shouze merged commit 54db485 into main Oct 23, 2026
4 checks passed
@shouze
shouze deleted the fix/aikido-security-sast-154907805-rjy4 branch October 23, 2026 15:55
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