Skip to content

A keyword is a legal variable name; the parser dropped it and emitted 'var ;' - #4644

Merged
gHashTag merged 2 commits into
masterfrom
queen-2703
Sep 23, 2026
Merged

gHashTag merged 2 commits into
masterfrom
queen-2703

Conversation

@gHashTag

Copy link
Copy Markdown
Owner

Closes #2703

Written by a bee on queen-2703 and published by tools/queen/publish.py. The branch itself is the bee's; the second commit is the coordination entry every pull request must add, which a bee has no way to know about.

1 file changed, 1 insertion(+), 1 deletion(-)

🤖 Generated with Claude Code

Trinity Bee and others added 2 commits September 23, 2026 15:36
The local-decl parser only accepted Ident tokens for variable names, but
keywords like 'module' should also be allowed when used as variable names.
This fixes the issue where 'var module = node_new(NodeKind::Module);' would
result in an empty name being emitted.

Changed the condition in parse_local_decl to accept KwModule tokens in
addition to Ident tokens for variable names.

Closes #2703
A pull request must add exactly one docs/now entry and a bee has no way
to know that: its brief names a boundary file and acceptance criteria,
and docs/now/ is neither. The publisher adds it rather than failing the
gate.

Closes #2703

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
@github-actions

Copy link
Copy Markdown
Contributor

PR Dashboard

Generated at: 2026-09-23 15:46:08 UTC

Summary

Status Count
Total Open PRs 38
PRs with Failing Checks 31
PRs with All Checks Green 7
READY 3
FAILING 31
PENDING 0
NO CHECKS YET 0

These columns do not partition: 3 + 31 + 0 + 0 = 34, and there are 38 open PRs. A PR is being counted twice or not at all.

Seal Status

  • ⚠️ STALE -- sha256(compiler.rs)=403499176a5d != manifest seal=87e5cbd3ad94.
    The committed NMSE numbers were certified against an older compiler.rs.
    Run scripts/reseal-check.sh locally for the two-step reseal command (advisory; not a merge gate).

@gHashTag
gHashTag merged commit b9d763f into master Sep 23, 2026
23 of 31 checks passed
@github-actions

Copy link
Copy Markdown
Contributor

📓 NotebookLM Notebook linked to this PR

This notebook contains session context, decisions, and artifacts for this work.

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.

A keyword is a legal variable name; the parser dropped it and emitted 'var ;'

2 participants