Skip to content

docs: align identity format spec with implementation (#<N>)#32

Merged
akollegger merged 2 commits into020-subject-serializationfrom
copilot/sub-pr-31
Nov 29, 2025
Merged

docs: align identity format spec with implementation (#<N>)#32
akollegger merged 2 commits into020-subject-serializationfrom
copilot/sub-pr-31

Conversation

Copy link
Copy Markdown
Contributor

Copilot AI commented Nov 29, 2025

The specification documented identity format as _anon_<N> but the implementation in Gram.Transform.generateId uses #<N>. Updated specs to match implementation per reviewer feedback.

Changes

  • data-model.md: Updated format from _anon_<N> to #<N> in identity field description and generation rules
  • contracts/api.md: Updated fromGram docstring and invariant examples
  • research.md: Updated all references including code examples and round-trip examples
-- Implementation format (unchanged)
generateId = do
  i <- get
  put (i + 1)
  return $ S.Symbol ("#" ++ show i)  -- produces #1, #2, etc.

💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.

Co-authored-by: akollegger <53756+akollegger@users.noreply.github.com>
Copilot AI changed the title [WIP] Update subject serialization implementation format docs: align identity format spec with implementation (#<N>) Nov 29, 2025
Copilot AI requested a review from akollegger November 29, 2025 18:12
@akollegger akollegger marked this pull request as ready for review November 29, 2025 21:46
@akollegger akollegger merged commit 9a77666 into 020-subject-serialization Nov 29, 2025
@akollegger akollegger deleted the copilot/sub-pr-31 branch November 29, 2025 21:46
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