kernel->instance vocab migration lane anchor (T218 D6)#38
Open
MSD21091969 wants to merge 2 commits into
Open
Conversation
Branch anchor for the runtime side of 4.0 delta D6 (kernel->instance). No code rename in this commit; surveys the ~59-site footprint and records alias-first + build-gate (Doctor + go test) discipline. Glued to ffs0 z440-vscode-lead/manifold-instances-vocab and moos-router feat/manifold-instances-vocab via the shared purpose-slug. authored-by: agent:vscode.hp-z440.primary / session:sam.z440-vscode-projection-lead / manifold-instances-vocab Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Collaborator
Author
Glued lane — shared purpose-slug
|
There was a problem hiding this comment.
Pull request overview
Adds a runtime-lane design/intent note anchoring the upcoming kernel → instance vocabulary migration work (delta D6) on the feat/manifold-instances-vocab feature branch, documenting scope and migration discipline without changing any code.
Changes:
- Introduces a new design note documenting the purpose of the branch and the (future) runtime footprint of the rename.
- Records migration discipline (alias-first compatibility, build gate requirements, and explicit “docs-only” boundary).
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| @@ -0,0 +1,38 @@ | |||
| # Runtime lane — `kernel` → `instance` vocab migration (T=218) | |||
Comment on lines
+34
to
+35
| - **Build gate = apply gate (T218).** No rename merges to `master` until `Doctor` + `go test ./...` pass | ||
| on this branch. |
Address Copilot review nits on the kernel-side rename anchor: - Standardize T=218 notation. - Clarify the Doctor gate is the ffs0 Test-MoosFederation.ps1 -Mode Doctor federation health check (not a command in this repo). - Spell out "hypergraph (HG) rewrite" to avoid Mercurial ambiguity. authored-by: agent:vscode.hp-z440.primary / session:sam.z440-vscode-projection-lead / manifold-instances-vocab Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Runtime lane —
kernel→instancevocab migration (kernel side)Lane:
feat/manifold-instances-vocab(T218: runtime work always feature-branched, merged only on build gate).Authored by
agent:vscode.hp-z440.primary/session:sam.z440-vscode-projection-lead.Branch anchor for the runtime side of 4.0 delta D6 (
kernel→instance). This PR adds an intent/design note only — no code rename. It surveys the footprint to migrate (internal/kernel/,/healthz+/state/*payload keys,cmd/moosflags/logs, README/CLAUDE prose, URN handling — ~59 sites) and records the discipline:kernelandinstanceforms across the transition so federation/readback never breaks; URN form stays readable until a reviewed URN migration.masteruntil Doctor +go test ./...pass.Gluing
Shared purpose-slug
manifold-instances-vocabglues this to:ffs0z440-vscode-lead/manifold-instances-vocab(the 4.0 delta doc, D6/D7)moos-routerfeat/manifold-instances-vocab(router side)manifold= colimit of these per-repo branches sharing one purpose (T218 E4).Boundary
Docs only.
go build ./...passes on this branch. No code rename, HG rewrite, deploy, or secret handling in this PR.authored-by: agent:vscode.hp-z440.primary / session:sam.z440-vscode-projection-lead / manifold-instances-vocab