chore: sync to node-red-standards v0.3.3 (AGENTS block + .gitattributes) - #493
Merged
Conversation
Refresh the managed AGENTS.md block from the central standard and add the standard's .gitattributes: - .gitattributes pins the working tree to LF (`* text=auto eol=lf`). With core.autocrlf=true on Windows, files checked out as CRLF fail `npm run format:check` locally even though the blob in git is LF and CI stays green on Linux. This makes local and CI results identical on every platform. - AGENTS.md managed block updated to v0.3.3: the expanded single-exit rule (no guard clauses; validate by nesting; throw is the one allowed non-return exit), "no defensive programming", the ESLint 10 recommended additions (no-unassigned-vars / no-useless-assignment), and the test-discovery note explaining why the test script takes no path args. Dev-facing only: neither file ships in the npm package (not in `files`), so no CHANGELOG entry or version bump. Supersedes the conflicted automated PR #489. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
Dependency Review✅ No vulnerabilities or license issues or OpenSSF Scorecard issues found.Scanned FilesNone |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Supersedes #489 (which conflicted because an earlier manual sync applied an older, simpler rule set).
Regenerated from the central standard v0.3.3 via
nrstd sync --writeon top of current master:.gitattributes(* text=auto eol=lf) — fixesformat:checkgoing red locally on Windows (core.autocrlf=true) while CI stays green on Linux. Local and CI results now match on every platform.throwis the one allowed non-return exit), "no defensive programming", the ESLint 10 recommended additions (no-unassigned-vars/no-useless-assignment), and the test-discovery note.The devDependency bumps #489 also wanted (
eslint-config-prettier^10,globals^17) already landed via #487/#488.eslint.config.js,CLAUDE.md,.claude/settings.json, and the workflows are intentionally kept as repo customizations (the standard's own rollout leaves them alone too).Dev-facing only — neither changed file ships in the npm package, so no CHANGELOG/version bump.
🤖 Generated with Claude Code