Skip to content

Register Graft structural observer role - #22

Merged
flyingrobots merged 2 commits into
mainfrom
docs/graft-structural-observer-role
May 12, 2026
Merged

Register Graft structural observer role#22
flyingrobots merged 2 commits into
mainfrom
docs/graft-structural-observer-role

Conversation

@flyingrobots

Copy link
Copy Markdown
Owner

Summary

  • Adds Graft to the Continuum README as the structural observer and review engine.
  • Keeps Echo and git-warp described as sibling Continuum runtime implementations, with git-warp Git-backed posture treated as deployment metadata.
  • Adds a Graft row to the contract family registry role matrix without promoting Graft into Family Matrix primary-consumer status before evidence exists.

Why

Graft now has a NORTHSTAR that positions it as the code-aware structural reading layer over witnessed causal history. Continuum should mirror that role in coordination truth without implying that Graft is a runtime, debugger product, shadow semantic owner, or already-proven primary consumer of shared families.

Review hygiene

This PR supersedes #21. It is based directly on current origin/main and excludes the already-landed no-graph branch history from PR review.

Validation

  • git diff --check origin/main..HEAD
  • npx --no-install markdownlint-cli2 README.md docs/contract-family-registry.md
  • node --test wesley/test/*.test.mjs

@coderabbitai

coderabbitai Bot commented May 12, 2026

Copy link
Copy Markdown

Review Change Stack
No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: ASSERTIVE

Plan: Pro

Run ID: 8bf83719-9c3d-45bc-897d-30a7c903aebb

📥 Commits

Reviewing files that changed from the base of the PR and between 37f5783 and 0205422.

📒 Files selected for processing (2)
  • README.md
  • docs/contract-family-registry.md

Summary by CodeRabbit

  • Documentation
    • Introduced Graft as a first-class structural observer and review engine component in ecosystem and stack documentation
    • Added Graft to repository role matrix describing its structural responsibilities and interaction expectations
    • Updated maintenance rule compatibility checklist to include Graft as a registry update trigger

Walkthrough

README.md and contract-family-registry.md receive documentation updates establishing Graft as a first-class structural observer and review-engine component. Graft is introduced in the ecosystem overview, specified in the Current Stack section with ownership boundaries clarified, and formalized in the repo role matrix and maintenance rules.

Changes

Graft Ecosystem Documentation

Layer / File(s) Summary
README ecosystem overview
README.md
Initial repository/stack bullet list reworked to add Graft and adjust surrounding Continuum ecosystem descriptions to frame Graft explicitly as a structural observer and review-engine component.
README Graft specification and boundaries
README.md
Current Stack section adds detailed Graft entry describing it as a structural observer/review engine over causal history that owns code-aware structural readings until shared Continuum semantics are required. Ownership boundaries clarified by explicitly excluding Graft-local structural reading payloads from Continuum ownership. Related repositories section links the Graft repository with structural observer/review engine description.
Registry role matrix and maintenance rules
docs/contract-family-registry.md
Repo Role Matrix adds Graft row specifying planned responsibilities (structural observer/review engine), expected consumer/producer nouns (runtime-boundary observer nouns, app-local structural payloads), and evidence conditions before Graft can be treated as primary consumer. Maintenance Rule expanded to include Graft as a compatibility-metadata publishing trigger alongside Echo, git-warp, warp-ttd, and app repos.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~8 minutes

Possibly related PRs

  • flyingrobots/continuum#19: Both PRs modify the same documentation files (README.md, docs/contract-family-registry.md) to update cross-repo contract-family registry and repo role matrix entries.

Poem

Graft joins the Continuum fold so bright,
A structural observer's keen sight,
Reading causal echoes from git's deep well,
Until semantics need their tale to tell.
🌿 First-class citizen, sealed by registry's hand ✨

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Title check ✅ Passed The title 'Register Graft structural observer role' accurately and specifically summarizes the main change: adding Graft to documentation with its defined structural observer role.
Description check ✅ Passed The description comprehensively relates to the changeset, clearly explaining what Graft is being added as, why it matters, and explicitly clarifying what it is NOT being promoted to.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch docs/graft-structural-observer-role

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@flyingrobots flyingrobots changed the title [codex] Register Graft structural observer role Register Graft structural observer role May 12, 2026
@flyingrobots
flyingrobots marked this pull request as ready for review May 12, 2026 17:56
@flyingrobots

Copy link
Copy Markdown
Owner Author
Issue Severity Addressed by Regression coverage Outcome
No unresolved actionable PR feedback found after flat comment and thread-aware review inspection. Docs N/A gh pr view --comments; GraphQL reviewThreads query; git diff --check origin/main..HEAD; npx --no-install markdownlint-cli2 README.md docs/contract-family-registry.md; node --test wesley/test/*.test.mjs CodeRabbit reports no actionable comments, reviewThreads is empty, reviewDecision is APPROVED, branch is pushed and up to date.

@flyingrobots
flyingrobots merged commit 60727ae into main May 12, 2026
1 check passed
@flyingrobots
flyingrobots deleted the docs/graft-structural-observer-role branch May 12, 2026 19:14
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