Skip to content

Claude/add zig zon parsing lkt cp#2

Merged
rjrodger merged 4 commits into
mainfrom
claude/add-zig-zon-parsing-LktCp
Apr 21, 2026
Merged

Claude/add zig zon parsing lkt cp#2
rjrodger merged 4 commits into
mainfrom
claude/add-zig-zon-parsing-LktCp

Conversation

@rjrodger
Copy link
Copy Markdown
Contributor

No description provided.

claude added 4 commits April 18, 2026 22:08
Point the devDependency and Go module at the current tip of the upstream
jsonic repository rather than the last published tag, so development
tracks unreleased changes. TypeScript 19/19 and Go tests pass.

https://claude.ai/code/session_01TLYwCQbxYQNqzzZstKtRCx
Pass { rule: { alt: { g: 'zon' } } } to the jsonic grammar method
(supported on main) so every alt in the ZON grammar is auto-tagged with
the 'zon' group. This makes the tags user-configurable at a single
site, enables `rule.exclude: 'zon'` for callers who want to opt out,
and removes redundant 'zon,' prefixes from each alt in the grammar
file.

https://claude.ai/code/session_01TLYwCQbxYQNqzzZstKtRCx
The plugin previously made a separate `jsonic.options(...)` /
`j.SetOptions(...)` call before applying the grammar. Attach the same
options to the GrammarSpec (`grammarDef.options` / `gs.Options`) so
jsonic applies them atomically as part of `jsonic.grammar(...)` /
`j.Grammar(...)`. The behaviour and test results are identical
(TS 19/19, Go tests pass) but the plugin now has a single
configuration-application site.

https://claude.ai/code/session_01TLYwCQbxYQNqzzZstKtRCx
- npm jsonic dep and Go go.mod both track jsonic@main (commit 33f522a;
  Go resolves to v0.1.19). TS 19/19 and Go tests pass.
- Add Parse(src, opts...) and MakeJsonic(opts...) convenience wrappers
  to the Go package so callers do not have to build a Jsonic instance
  manually (mirrors jsonicjs/ini). A typed ZonOptions struct is
  provided alongside the existing Zon/Defaults plugin entry points.
- Restructure documentation to follow the Diataxis framework
  (Tutorials / How-to guides / Explanation / Reference) in
  doc/zon-ts.md and doc/zon-go.md, and link them from the README.
  The Makefile already matches the jsonicjs/ini template.

https://claude.ai/code/session_01TLYwCQbxYQNqzzZstKtRCx
@rjrodger rjrodger merged commit bb26c85 into main Apr 21, 2026
9 checks passed
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.

2 participants