Skip to content

Feat/move grants to common grammar - #55

Draft
JosteinGj wants to merge 2 commits into
mainfrom
feat/move-grants-to-common-grammar
Draft

JosteinGj wants to merge 2 commits into
mainfrom
feat/move-grants-to-common-grammar

Conversation

@JosteinGj

Copy link
Copy Markdown
Contributor

What this changes

one common implementation of the type/subject/verb grants logic for both settings and datasets. still want to test this fully with the sdk.

How it was verified

Checklist

  • Pull requests this one depends on are named above and merged first, or there are none
  • Commits are signed off (git commit -s), per CONTRIBUTING.md
  • ./gradlew build passes
  • Behaviour visible outside this repo is reflected in the documentation, or does not need to be
    (see AGENTS.md for which of the two documentation sites it belongs in)

…rammar

DatasetGrants and SettingsGrants are now thin facades over a shared
parser bound to a subject and verb set; public APIs unchanged, both
test suites pin the parser's behaviour.

Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
One platform-wide verb vocabulary; each grammar declares the subset
that applies to its subject. No wire or behaviour change.

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
@JosteinGj
JosteinGj force-pushed the feat/move-grants-to-common-grammar branch from fa91e1d to a56893c Compare September 9, 2026 09:17
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