Skip to content

0.9.0dev6 - #232

Merged
rlippmann merged 3 commits into
mainfrom
remove_is_canonical_directive
Aug 7, 2026
Merged

0.9.0dev6#232
rlippmann merged 3 commits into
mainfrom
remove_is_canonical_directive

Conversation

@rlippmann

Copy link
Copy Markdown
Owner

What changed

  • Removed the redundant is_canonical_directive() grammar API helper.
  • Clarified grammar API boundaries:
    • use validate_directive() for canonical directive classification;
    • use decompose_directive() when operands are required;
    • use match_canonical_directive_start() only for shallow syntax detection.
  • Documented that directive text fields preserve accepted input text and are not canonical rendered output.
  • Bumped the development release version.

Why

  • Reduce redundant public API surface.
  • Keep Python and TypeScript grammar contracts smaller and easier to maintain.
  • Make parsing, validation, and rendering responsibilities explicit.

Checklist

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

@rlippmann
rlippmann merged commit 39f039d into main Aug 7, 2026
16 checks passed
@rlippmann
rlippmann deleted the remove_is_canonical_directive branch August 7, 2026 07:04
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