feat(base2): add meta-rule 'Syntax ≠ logic' to General guidelines - #873
Open
chicagoist wants to merge 1 commit into
Open
feat(base2): add meta-rule 'Syntax ≠ logic' to General guidelines#873chicagoist wants to merge 1 commit into
chicagoist wants to merge 1 commit into
Conversation
Add a general principle: parser passes (node --check, tsc --noEmit, etc.) do NOT mean the code works. Always verify execution, not just parsing. Placed before the JavaScript Debugging rule as the general axiom from which the JS-specific checks follow.
Contributor
|
The instinct here is fine — reminding the agent that a parser pass doesn't guarantee runtime correctness is a legitimate point, and it's a tiny, low-risk diff to
If you separate the general axiom (which is easy to justify) from the JS-specific checklist (which needs either broadening or evidence it prevents a recurring class of bugs), this would be much easier to port as-is. Consider trimming the second bullet to be shorter and more general, or dropping it if it was written for a single incident. |
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
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.
General principle: parser passes (
node --check,tsc --noEmit, etc.)do NOT mean the code works. Always verify execution, not just parsing.
Placed before the JavaScript Debugging rule (#872) as the axiom
from which the JS-specific verification steps follow.
Diff