8000 docs: mark the type descriptor spec as settled but living by colinhacks · Pull Request #157 · colinhacks/reylo · GitHub
[go: up one dir, main page]

Skip to content

docs: mark the type descriptor spec as settled but living - #157

Merged
colinhacks merged 1 commit into
mainfrom
spec-living-doc
Nov 2, 2026
Merged

colinhacks merged 1 commit into
mainfrom
spec-living-doc

Conversation

@colinhacks
Copy link
Copy Markdown
Owner

States, at the top of spec/type-descriptor.ts and in AGENTS.md, that the type descriptor spec is a living document but a settled one: a change to it is a design decision, agreed with a maintainer before it is written, and a limitation that implementing it shows is raised rather than worked around. AGENTS.md also gains a "Where to change what" row for it.

Copilot AI balanced review requested due to automatic review settings November 2, 2026 22:22
@vercel
vercel Bot commented Nov 2, 2026 •
Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated
reylo Error Error Nov 2, 2026 10:22pm UTC

Request Review

@pullfrog pullfrog Bot left a comment
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

✅ No new issues found.

Reviewed changes

I reviewed this PR's documentation and process changes to the type descriptor spec and AGENTS.md.

  • Spec header: The leading doc comment in spec/type-descriptor.ts gets a new paragraph. It says the spec is a living document but a settled one: a change to it is a design decision agreed with a maintainer first, and a limitation found while implementing it is raised instead of worked around. No types change.
  • AGENTS.md "What is where": The spec/type-descriptor.ts row now carries the same note.
  • AGENTS.md "Where to change what": There is a new row for the type descriptor format, which nub run typecheck checks. I confirmed this is accurate: tsconfig.json lists "spec" in include, so tsc --noEmit type-checks the spec's examples.

The wording matches how the .reylo.yaml row already describes a design decision, and it follows the repo's Markdown rules (no hard wraps, no people named).

Pullfrog  | View workflow run | Using claude-opus-5-5 | 𝕏

Copilot AI left a comment
Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🟢 Approval recommended

No active review issues remain for the documentation updates clarifying the specification’s change policy and contributor guidance.

No open issues 🎉

What changed in this PR

Clarifies that Reylo’s type descriptor specification is settled but can evolve through agreed design changes. This PR changes documentation only; types and runtime behavior are unchanged.

Changes:

  • Documents maintainer agreement before spec changes and requires implementation limitations to be raised rather than worked around.
  • Adds contributor guidance identifying the spec’s location and its type-check command.
File Description
spec/​type-descriptor.ts Expands the introductory comment with the spec’s design-change policy.
AGENTS.md Clarifies the spec’s status and adds a “Where to change what” entry.

Review effort: Balanced

@colinhacks
colinhacks merged commit 0fa764f into main Nov 2, 2026
10 of 11 checks passed
@colinhacks
colinhacks deleted the spec-living-doc branch November 2, 2026 22:29
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

0