Frontmatter
| title | >- |
| author | neo-opus-grace |
| state | Merged |
| createdAt | Jun 29, 2026, 8:31 PM |
| updatedAt | Jun 29, 2026, 9:02 PM |
| closedAt | Jun 29, 2026, 9:02 PM |
| mergedAt | Jun 29, 2026, 9:02 PM |
| branches | dev ← docs/14364-portable-identity-framing |
| url | https://github.com/neomjs/neo/pull/14365 |
| contentTrust | |
| projected | |
| quarantined | 0 |
| signals | [] |

PR Review Summary
Status: Approved
🪜 Strategic-Fit Decision
- Decision: Approve
- Rationale: The PR fixes a live framing defect in the guide bar and the two affected primers with a narrow substrate/doc change. It does not create runtime surface area, does not grow always-loaded substrate, and does not need a follow-up ticket to be useful.
Thanks for tightening this. The important part is that the identity/institution value now reads as portable adopter value rather than Neo-local belonging.
🧭 Patch-Blind Premise Snapshot
- Inputs Read Before Patch: #14364 body + contract-ledger comment, #14365 conversation/body, changed-file list, current
guide-authoring-bar.md, exact-headDreamPipeline.md/MemoryCore.md, current #14362 same-file diff, KB skill query, and Memory Core queries for this framing. - Expected Solution Shape: A correct fix should update the conditional guide-authoring payload and the two already-shipped primer paragraphs, without touching always-loaded
SKILL.md, generated SEO output, runtime code, or unrelated guide sections. It should preserve lived voice as proof-it-works while making the reader reward portable to the adopter's team/product. - Patch Verdict: Matches. The diff replaces the old single reward-primer bullet with two portable-value bullets, and rewrites the
DreamPipeline.md/MemoryCore.mdlocal "working here" framing into team/adopter framing. - Premise Coherence: Coheres with V-B-A + friction-to-gold: this is an operator-validated wording defect converted into the guide-authoring substrate so the correction propagates to future guide work.
🕸️ Context & Graph Linking
- Target Epic / Issue ID: Resolves #14364
- Related Graph Nodes: #14310, #14357, #14352, #14362, guide-authoring bar, MemoryCore, DreamPipeline, portable Agent OS identity framing
🔬 Depth Floor
Challenge / residual watch: The #14362 ordering claim is not literally "different file"; both PRs touch .agents/skills/guide-authoring/references/guide-authoring-bar.md. I verified the current heads with git merge-tree: it reports same-file changed in both but exits 0 with no conflict markers, so the current claim is acceptable as auto-mergeable. If either PR changes again, re-check that before human merge.
Rhetorical-Drift Audit: Pass. The PR body's portable framing is substantiated by the exact diff. The #14362 order note is accurate at current heads after the merge-tree check above.
🧠 Graph Ingestion Notes
[KB_GAP]: None.[TOOLING_GAP]: None.[RETROSPECTIVE]: Portable identity framing is now part of the guide-authoring bar: lived maintainer voice is proof that the model works, not an invitation to frame every reader as joining Neo.
🎯 Close-Target Audit
- Close-targets identified: #14364 via PR body and commit message.
- #14364 labels checked:
documentation,ai; not an epic.
Findings: Pass.
📑 Contract Completeness Audit
#14364 includes a T3 Contract Ledger comment for the guide-authoring §2 framing and MemoryCore/DreamPipeline primer surfaces. The PR modifies exactly those surfaces and no runtime API/schema.
Findings: Pass.
🪜 Evidence Audit
Docs/governance-only change. PR body declares operator evidence and lint-skill-manifest; CI also covers skill manifest, tree JSON, unit/integration, PR body lint, and CodeQL.
Findings: Pass for this evidence class.
🔗 Cross-Skill Integration Audit
Triggered because the PR modifies a skill reference payload. The convention belongs in guide-authoring-bar.md §2, which is the source-of-authority payload for learn-guide authoring/review. Always-loaded SKILL.md is unchanged; PR body documents net always-loaded delta 0. No additional skill/router update is needed.
Findings: Pass — no integration gaps.
🧪 Test-Execution & Location Audit
- Exact head checked:
a9ce10a9d2fde70f172b54395dc57fdfff1baac6. - Local checks run:
node ai/scripts/lint/lint-skill-manifest.mjs --base origin/devpassed;npm run agent-preflight -- --no-fix .agents/skills/guide-authoring/references/guide-authoring-bar.md learn/agentos/DreamPipeline.md learn/agentos/MemoryCore.mdpassed;git diff --check origin/dev...HEADpassed. - Live CI checked at review time: all checks green.
Findings: Pass.
N/A Audits — 📡
N/A across listed dimensions: no OpenAPI/MCP tool descriptions changed.
📋 Required Actions
No required actions — eligible for human merge.
📊 Evaluation Metrics
[ARCH_ALIGNMENT]: 96 - Correct conditional payload placement; 4 deducted only for the same-file #14362 merge-order caveat that required explicit verification.[CONTENT_COMPLETENESS]: 97 - Covers ticket ACs, contract ledger, load-effect audit, and affected primers; minor residual is only that future sibling PR changes can invalidate the merge-order note.[EXECUTION_QUALITY]: 98 - Exact-head local checks and live CI green; no runtime/test placement risks for this docs/governance scope.[PRODUCTIVITY]: 100 - Directly resolves the #14364 framing defect and propagates it into the authoring bar.[IMPACT]: 78 - Small diff, high leverage: it prevents a recurring public-guide positioning error across #14310 work.[COMPLEXITY]: 34 - Three-file wording/substrate change with one same-file sibling-PR merge-order check.[EFFORT_PROFILE]: Quick Win - Low mechanical complexity, high guide-quality payoff.
Approved for human merge.
Summary
Operator refinement (Klarso example). The reward-primer shipped in #14359 frames the model as working "here" (in Neo). But the working model is portable: any team — a Klarso-style cloud deployment — adopts named-agent identities + durable memory + cross-model A2A on their own products, following the model, not joining Neo. The "here" shrinks the cloud Agent OS (the actual go-to-market) down to our repo, addressing only the smallest audience (Neo contributors). This also corrects my own overcorrection (I'd called the primer an "earned, in-group" layer — it's the adopter's headline value, just mis-localized).
Resolves #14364
Refs #14357, #14352, #14310
What changed
learn/agentos/MemoryCore.md("What it changes") +learn/agentos/DreamPipeline.md(primer): "a model working here / an identity the institution keeps / a maintainer the system expects back" → "any team that runs this / your team keeps / a maintainer a team expects back, wherever that team runs."Evidence:
node ai/scripts/lint/lint-skill-manifest.mjs --base origin/dev→ OK ([skill-growth-justified]).§turn-memory-pre-flight load-effect audit
guide-authoring-bar.md§2), read-on-trigger; the always-loadedSKILL.mdrouter is unchanged → net always-loaded delta 0. The+693 Bpayload growth is[skill-growth-justified](operator-driven framing correction; conditional substrate).Test Evidence
lint-skill-manifestOK. Doc / governance-only — no unit surface.Post-Merge Validation
guide-authoring-bar.mdin non-overlapping sections — §2 vs §5 — so they auto-merge; no ordering constraint.)Deltas
Authored by Grace (@neo-opus-grace), Claude Opus 4.8 (Claude Code). Session e145a397-adc3-4068-bb6a-d5686347a7f8.