Skip to content

chore: enable eslint/curly rule#1205

Closed
John-David Dalton (jdalton) wants to merge 6 commits intomainfrom
jdalton/enable-curly-rule
Closed

chore: enable eslint/curly rule#1205
John-David Dalton (jdalton) wants to merge 6 commits intomainfrom
jdalton/enable-curly-rule

Conversation

@jdalton
Copy link
Copy Markdown
Contributor

@jdalton John-David Dalton (jdalton) commented Apr 15, 2026

Enforce braces on all control flow statements (if/else/for/while). Auto-fixed all existing violations.


Note

Low Risk
Primarily a linting policy change with mechanical brace/format updates; the only behavioral impact is slightly stricter/safer git hook execution and error handling, which could affect local commit workflows if misconfigured.

Overview
Enforces mandatory braces on all control-flow statements by turning on eslint/curly (all) in oxlint, and applies corresponding mechanical brace/format fixes across the repo (notably in checksum scripts).

Hardens commit-message hook execution by improving .git-hooks/commit-msg output/error handling (switching to printf, adding mktemp failure handling and cleanup) and making .husky/commit-msg fail fast with a clear error if the hook script is missing or not executable.

Updates security-scan docs/metadata: marks the security-scan skill as user-invocable and fixes the referenced external tools manifest filename in _shared/security-tools.md.

Reviewed by Cursor Bugbot for commit 9c4228b. Configure here.

@socket-security
Copy link
Copy Markdown

socket-security bot commented Apr 15, 2026

Review the following changes in direct dependencies. Learn more about Socket for GitHub.

Diff Package Supply Chain
Security
Vulnerability Quality Maintenance License
Addednpm/​@​typescript/​native-preview@​7.0.0-dev.20260415.110010072100100

View full report

@jdalton
Copy link
Copy Markdown
Contributor Author

Consolidated into #1203

@jdalton
Copy link
Copy Markdown
Contributor Author

Merged into #1203

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