Skip to content

refactor: expand public grammar API - #225

Merged
rlippmann merged 1 commit into
mainfrom
grammar_cleanup
Aug 3, 2026
Merged

refactor: expand public grammar API#225
rlippmann merged 1 commit into
mainfrom
grammar_cleanup

Conversation

@rlippmann

Copy link
Copy Markdown
Owner

What changed

  • Promoted existing grammar directive classification helpers into the public grammar API.
  • Removed redundant private-to-public delegation for grammar helpers and added public docstrings describing their behavior and boundaries.
  • Updated the public grammar API fixture and grammar tests to cover the expanded export surface.

Why

  • Grammar consumers need access to shared language classification behavior instead of maintaining duplicate implementations.
  • The exported grammar API now better reflects the capabilities already owned by the grammar module.
  • Documenting these functions clarifies that they perform grammar classification only and do not perform repair, inference, or state evaluation.

Checklist

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

@rlippmann
rlippmann merged commit 510b0e2 into main Aug 3, 2026
18 checks passed
@rlippmann
rlippmann deleted the grammar_cleanup branch August 3, 2026 20:27
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