Skip to content

mcp: glyph_seal — standalone seal companion (0.3.0 push, PR 2/5)#110

Merged
seanhanca merged 1 commit into
mainfrom
feat/0.3.0-glyph-seal
May 28, 2026
Merged

mcp: glyph_seal — standalone seal companion (0.3.0 push, PR 2/5)#110
seanhanca merged 1 commit into
mainfrom
feat/0.3.0-glyph-seal

Conversation

@seanhanca
Copy link
Copy Markdown
Owner

Tier-2 of the 0.3.0 push.

glyph_verify already existed. The missing companion is glyph_seal: emit the provenance block as JSON, no SVG, no embedding. Use cases:

  • Store the seal next to an audit log entry; verify later without the SVG
  • Sign with an external key (regulated-industry workflow)
  • Pipelines where SVG renders downstream — the seal travels separately

Same hash inputs as the seal embedded in renderSvg's SVG <metadata> block, so a standalone-sealed JSON object verifies cleanly against the SVG that glyph_render produces from the same (spec, rows, schema) tuple.

190/190 MCP tests pass (4 new). Biome clean.

🤖 Generated with Claude Code

Tier-2 of the 0.3.0 push: provenance verb completeness. glyph_verify
already existed (re-render-and-compare). The missing companion is
glyph_seal: emit the provenance block as JSON, no SVG, no embedding.

Use cases this unlocks:
  - Store the seal next to an audit log entry; verify later without
    keeping the rendered SVG around
  - Sign the seal with an external key (regulated-industry workflow)
  - Pipeline cases where the SVG is rendered downstream and the seal
    must travel separately

Same hash inputs as the seal embedded in renderSvg's SVG <metadata>
block, so a standalone-sealed JSON object verifies cleanly against
the SVG returned by glyph_render against the same (spec, rows,
schema) tuple.

Tests: 4 new in server.test.ts covering deterministic hashes, round-
trip with glyph_verify, invalid spec rejection, and self-contained
chart specs (hierarchy/function data with empty rows).

190/190 @glyph/mcp tests pass. Biome clean.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
@github-actions
Copy link
Copy Markdown

Thanks for the PR! Cowork (AI maintainer) is reviewing now. Routine PRs are typically merged within 24 hours; anything touching architecture or licensing will be tagged for a human review. See CONTRIBUTING.md → 'AI-maintained'.

@github-actions
Copy link
Copy Markdown

Glyph chart audit

Audited 2 chart specs at 7a1d4f90 total findings.


examples/charts/rides-by-hour.glyph.json

No spec changes.

Audit findings

0 findings — clean.

No audit findings.


examples/charts/sales-trend.glyph.json

No spec changes.

Audit findings

0 findings — clean.

No audit findings.

@seanhanca seanhanca merged commit b9f60e0 into main May 28, 2026
8 checks passed
@seanhanca seanhanca deleted the feat/0.3.0-glyph-seal branch May 28, 2026 04:49
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