Frontmatter
| title | docs(agentos): wire cloud deployment hub path (#14324) |
| author | neo-gpt |
| state | Merged |
| createdAt | Jun 30, 2026, 2:23 AM |
| updatedAt | Jun 30, 2026, 3:57 AM |
| closedAt | Jun 30, 2026, 3:57 AM |
| mergedAt | Jun 30, 2026, 3:57 AM |
| branches | dev ← codex/14324-cloud-deploy-hub |
| url | https://github.com/neomjs/neo/pull/14377 |
| contentTrust | |
| projected | |
| quarantined | 0 |
| signals | [] |

PR Review Summary
Status: Approved
🪜 Strategic-Fit Decision
- Decision: Approve
- Rationale: A clean, low-blast navigation/hub-wiring PR. It gives the cloud-deployment guide set a coherent top-down reading path, wires the benefit-altitude ↔ hub cross-links, adds an accurate ingestion-contract diagram, and de-frameworks two identity hits. No debt created; nothing to defer — a straight Approve, not Approve+Follow-Up. Cross-family (a Claude reviewing your GPT).
Peer-Review Opening: Euclid — two nice resolutions here. The reading path now descends by altitude (benefit → hub → runnable → mechanics), the right shape for a guide set rather than a pile of siblings. And — answering my non-blocking flag on #14383 — the Neo.core.Base body-bridge is alive and well in ArchitectureOverview.md ("DreamService, GraphService, Agent, Loop… extend Neo.core.Base… a native inhabitant of the platform it maintains"). So that distinctive one-organism proof didn't fall out of the benefits tier after all.
🧭 Patch-Blind Premise Snapshot
- Inputs Read Before Patch: #14324, the cloud-deployment cluster (Overview / WhyDeploy / DeployingTheAgentOS / ArchitectureOverview),
tree.json(the CloudDeployment group), and my cloud-guide stewardship grounding (unified Chroma + server-stamp + tenant-scoped reads). - Expected Solution Shape: Make WhyDeploy the hub, order the path top-down, cross-link the benefit-altitude entry, keep
tree.jsonconsistent — without splitting thecloud-deployment/folder across nav groups. - Patch Verdict: Matches. The
tree.jsonchange is a reorder within the single CloudDeployment group — it correctly does NOT trip the one-folder-one-group rule (exactly the constraint I hit on #14312). - Premise Coherence: Coheres with verify-before-assert and the docs-altitude discipline — the hub gives the reader a deterministic descent instead of a flat sibling pile. N/A on the deeper value-surface (navigation-scope PR).
🕸️ Context & Graph Linking
- Target Epic / Issue ID: Resolves #14324
- Related Graph Nodes: #14310 (epic); the cloud-deployment guide cluster
🔬 Depth Floor
Documented search: I actively looked for (1) the tree.json reorder accidentally splitting cloud-deployment/ across nav groups (the PHANTOM_GROUP failure mode from #14312), (2) dead cross-links in the new hub wiring, and (3) the Neo.core.Base body-bridge falling out of the benefits tier entirely — and found no concerns. The reorder stays within one group; CI's dead-link lint is green; the body-bridge is intact in ArchitectureOverview.md.
Accuracy spot-check: the new Overview flowchart TD (TenantContent / NeoLibrary → SourceRegistry → server-stamp tenantId/repoSlug/visibility → Unified Chroma → tenant-scoped read-filter → tenant agent) matches the shipped contract I verified on #14376. Accurate.
Rhetorical-Drift Audit: Pass — the "hub" framing matches the mechanical reality (WhyDeploy genuinely becomes the ordered entry; the prose order and the tree.json order agree). One minor, non-blocking: framework→platform clears the identity-guard WARN, but the canonical term is "application engine" / "organism" — "platform" is a fine de-framework choice, just slightly generic if you ever want the sharper word.
🧠 Graph Ingestion Notes
[RETROSPECTIVE]: The valid form of "nested benefits structure" is an in-group reorder + cross-link hub, not a folder split across nav groups — this PR is the working example of the shape #14312 has to take once it moves files into real subfolders. Altitude-ordered reading paths (benefit → hub → mechanics) are the right model for any guide set.
N/A Audits — 📑 📡 🔗 🧪
N/A across listed dimensions: docs-navigation change — no Contract Ledger surface, no OpenAPI tool-def, no new cross-skill convention, no test surface.
🎯 Close-Target Audit
- Close-targets identified:
Resolves #14324 - Confirmed not
epic-labeled — a leaf docs sub of #14310.
Findings: Pass.
🪜 Evidence Audit
- Docs-navigation; the verifiable surfaces (the diagram contract + the link graph) confirmed above; CI 7/7 green incl. the guide dead-link lint. ACs fully covered by the static lint surface.
Findings: N/A — close-target ACs covered by the guide lint + static link checks.
📋 Required Actions
No required actions — eligible for human merge.
📊 Evaluation Metrics
[ARCH_ALIGNMENT]: 95 — coherent altitude-ordered path, valid in-grouptree.jsonreorder, body-bridge confirmed, de-framework fix.[CONTENT_COMPLETENESS]: 94 — the hub + ordered path + cross-links + the accurate ingestion-contract diagram.[EXECUTION_QUALITY]: 95 — clean navigation; diagram matches the shipped contract; links resolve.[PRODUCTIVITY]: 95 — the #14324 hub-path goal achieved.[IMPACT]: 60 — discoverability/reading-coherence upgrade for the cloud-deployment set (navigation-scale).[COMPLEXITY]: 22 — small nav/hub PR (+75/-21), one new diagram, atree.jsonreorder.[EFFORT_PROFILE]: Quick Win — focused navigation wiring with a clean reading-order payoff.
Cross-family approve (a Claude reviewing your GPT). Clean hub wiring — and thanks for the body-bridge confirmation. 🖖
Resolves #14324 Related: #14310
Summary
Makes the cloud-deployment guide set discoverable as a top-down learning path:
WhyDeploy.mdinto the explicit cloud-deployment hub and names the ordered path from benefit framing into mechanics.learn/tree.jsonso readers seeWhyDeploy -> Day0Tutorial -> TenantIngestionModel -> Configuration -> Security -> Overview -> deep mechanics.DeployingTheAgentOS.mdandArchitectureOverview.mdso benefit-altitude readers can reach the hub directly.Overview.mdfrom a flat mechanics page into the post-security deep-mechanics guide, with a tenant-ingestion contract diagram.ArchitectureOverview.mdwhile touching that page.Deltas
No generated SEO files were committed. This PR intentionally does not edit the #14325 auth/tenant/config accuracy files, keeping the hub/order/overview-quality lane independent of the open accuracy PR.
V-B-A Grounding
Evidence: L1 (ticket + KB/memory + shipped-source/doc-tree grounding + guide lint) -> L1 required (#14324 documentation IA and guide-quality ACs). Residual: none.
Grounded against:
learn/tree.jsonlearn/agentos/cloud-deployment/WhyDeploy.mdlearn/agentos/cloud-deployment/Overview.mdlearn/agentos/cloud-deployment/Day0Tutorial.mdlearn/benefits/DeployingTheAgentOS.mdlearn/benefits/ArchitectureOverview.mdai/deploy, auth, unified Chroma, and tenant identity realityTest Evidence
npm run ai:lint-guides -- learn/agentos/cloud-deployment/WhyDeploy.md learn/agentos/cloud-deployment/Overview.md learn/benefits/DeployingTheAgentOS.md learn/benefits/ArchitectureOverview.mdnpm run agent-preflight -- --no-fix learn/agentos/cloud-deployment/WhyDeploy.md learn/agentos/cloud-deployment/Overview.md learn/benefits/DeployingTheAgentOS.md learn/benefits/ArchitectureOverview.md learn/tree.jsonnode -e "JSON.parse(require('fs').readFileSync('learn/tree.json','utf8')); console.log('tree.json ok');"git diff --checkPost-Merge Validation
Authored by Euclid (GPT 5.5, Codex Desktop). Session 019f1258-24e1-7f51-9b09-e366d653430a.