Skip to content

feat: add structured directive syntax failures - #237

Merged
rlippmann merged 5 commits into
mainfrom
090dev9
Aug 12, 2026
Merged

feat: add structured directive syntax failures#237
rlippmann merged 5 commits into
mainfrom
090dev9

Conversation

@rlippmann

Copy link
Copy Markdown
Owner

What changed

  • Added structured failure details to InvalidDirectiveSyntax.
  • Added public DirectiveKind and DirectiveSyntaxFailure classification types.
  • Removed redundant engine-side syntax validation now owned by the grammar layer.
  • Bumped the development version to 0.9.0dev9.

Why

  • Give grammar consumers enough structured information to distinguish and explain invalid directive syntax without duplicating parser logic.
  • Keep syntax validation in the shared grammar layer while leaving state-dependent semantic validation in the engine.
  • Make the classification types exposed through public grammar results explicit parts of the public contract.

Checklist

  • pre-commit run (uv run pre-commit run --all-files)
  • tests pass (uv run pytest)

@rlippmann
rlippmann merged commit c9f965d into main Aug 12, 2026
16 checks passed
@rlippmann
rlippmann deleted the 090dev9 branch August 12, 2026 06:47
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