Skip to content

docs(diagnostics): E001 reserved (never emitted) + parse-example column#445

Merged
InauguralPhysicist merged 1 commit into
mainfrom
docs/diagnostics-e001-column
Jul 5, 2026
Merged

docs(diagnostics): E001 reserved (never emitted) + parse-example column#445
InauguralPhysicist merged 1 commit into
mainfrom
docs/diagnostics-e001-column

Conversation

@InauguralPhysicist

Copy link
Copy Markdown
Collaborator

Two DIAGNOSTICS.md accuracy fixes from a documentation review:

  • E001 documented but never emitted. The table listed E001 = Syntax error (tokenizer), but the source only ever emits E000 and E002 (grep of lint.c/eigenlsp.c). Marked E001 reserved — lexer-level failures surface through the parser as E002 — rather than removing it, preserving the stable-code contract.
  • Parse-example missing the column. The worked example showed Parse error line 1:, but since Runtime-error carets + token-precise LSP ranges (residual of #407) #407 the binary prints Parse error line 1:9:. Updated to the real output (verified: if x > 0 with no colon → line 1:9:).

Doc-only.

🤖 Generated with Claude Code

…e example

- E001 was documented as "Syntax error (tokenizer)" but grep of the source
  shows only E000/E002 are ever emitted — no code path produces E001. Marked
  it *reserved* (lexer failures surface as E002) rather than removing it, so
  the stable-code contract keeps E002's meaning intact.
- The parse-error example still showed the pre-#407 format `Parse error
  line 1:` — the binary now prints `Parse error line 1:9:` (column added).
  Updated to match actual output.

Doc-only. Both verified against the running binary.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
@InauguralPhysicist InauguralPhysicist merged commit b880942 into main Jul 5, 2026
17 checks passed
@InauguralPhysicist InauguralPhysicist deleted the docs/diagnostics-e001-column branch July 5, 2026 06:12
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