feat: add org-default control guardrails#82
Conversation
PR SummaryHigh Risk Overview Introduces org-wide rollout automation: Adds an Reviewed by Cursor Bugbot for commit 354ea5c. Bugbot is set up for automated code reviews on this repo. Configure here. |
EvalOps Label Sync Report
|
There was a problem hiding this comment.
Cursor Bugbot has reviewed your changes and found 1 potential issue.
Bugbot Autofix is ON, but it could not run because the branch was deleted or merged before autofix could start.
Reviewed by Cursor Bugbot for commit 354ea5c. Configure here.
| base = ensure_trailing_newline(current) | ||
| base = "#{base}\n" unless base.strip.empty? | ||
| "#{base}#{additions.join("\n")}\n" | ||
| end |
There was a problem hiding this comment.
Leading blank line in new .gitignore files
Low Severity
When existing is nil (no .gitignore exists), merge_gitignore calls ensure_trailing_newline("") which converts the empty string to "\n", producing a leading blank line in the output. The sister function merge_agents avoids this with an early return section if existing.to_s.strip.empty? guard, but merge_gitignore lacks an equivalent check, so every new .gitignore created during MCP config rollout starts with a spurious blank line.
Reviewed by Cursor Bugbot for commit 354ea5c. Configure here.


Summary
Verification