Skip to content

docs: recover AR redesign docs onto freeze main (supersedes #245) - #251

Merged
scrimshawlife-ctrl merged 2 commits into
mainfrom
docs/ar-redesign-recover-245
Jul 29, 2026
Merged

scrimshawlife-ctrl merged 2 commits into
mainfrom
docs/ar-redesign-recover-245

Conversation

@scrimshawlife-ctrl

Copy link
Copy Markdown
Owner

Summary

Workstream E / Prabu rebase request for #245 — recovered onto current freeze main without merging stale branch history.

Field Value
Base current main (post #249/#250 freeze)
Authority SUPPORTING / REFERENCE_ONLY only
Binding law Unchanged (SOLO_MVP_SCOPE, WAYKIN_SPEC)
Mesh/runtime Inventory updated for Ember Fox (#246); #242/#243 noted closed
Supersedes #245 stale branch cut pre-mesh

Why not rebase #245 in place

Per freeze plan and safety hold: #245 was cut before Ember Fox / pressure on main. Recreating from current main avoids force-merging stale history and index conflicts with freeze re-baseline docs.

Documents recovered

  • docs/design/AR_PRODUCT_REDESIGN_MAP.md
  • docs/design/AR_SYSTEM_INVENTORY.md (Ember Fox runtime corrections)
  • docs/design/AR_SESSION_IA_CONFLICTS.md
  • docs/plans/AR_APP_REDESIGN_PLAN.md
  • docs/design/PRODUCT_VISION_NORTH_STAR.md (REFERENCE_ONLY)
  • docs/canonical/MVP_TO_VISION_LADDER.md

Corrections vs original #245

  • Evidence vocabulary: removed unsupported SPECULATIVE labels
  • Inventory: Ember Fox package, async load/replace, anchor detach, clip sidecars, tests/gates
  • Placement: assumed-ground fallback noted
  • Authority metadata retained: SUPPORTING / does_not_override binding docs
  • Does not rewrite CONTINUATION freeze lane into redesign-first sequencing

Non-goals

After merge

Close #245 as superseded. Device indoor smoke remains next for product evidence before redesign implementation.

Test plan

  • Docs only
  • CI green
  • Write-access approval

Fresh branch from current main recovers SUPPORTING redesign map,
inventory (Ember Fox runtime), IA conflicts, phased plan, north star,
and MVP→vision ladder. Index wiring only on freeze-era docs; no stale
#245 history, no binding-law rewrite, no mesh/code changes.
@scrimshawlife-ctrl

Copy link
Copy Markdown
Owner Author

Recovery of #245 onto freeze main for Prabu's rebase request. Docs-only; SUPPORTING authority. Please review when CI is green.

@chatgpt-codex-connector chatgpt-codex-connector Bot 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.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: ac2efb514b

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread docs/plans/AR_APP_REDESIGN_PLAN.md Outdated
Comment thread docs/design/AR_PRODUCT_REDESIGN_MAP.md Outdated
Comment thread docs/collaboration/ACTIVE_WORK.md Outdated
Require Phase 1 indoor+outdoor device evidence before Phase 2 AR-default
UX; document assumed-ground then camera placement fallback; mark recovery
as landing via #251 (complete after merge).

@prabu-openclaw prabu-openclaw left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Approving. This is the clean recovery of #245 onto the freeze baseline — #245 can be closed.

Verified rather than assumed:

Scope. Docs only. The two files outside docs/ are README.md and ROADMAP.md, both pointer wiring. No Swift, no assets.

The freeze is not lifted. AR_MVP_FREEZE.md has exactly one added line and zero removals — the redesign map is registered as SUPPORTING with "does not lift freeze" stated inline. That was the thing worth checking most carefully, and it holds.

Binding law untouched. WAYKIN_SPEC, SOLO_MVP_SCOPE, ARCHITECTURE and the capability matrix appear only in the does_not_override list. North star stays REFERENCE_ONLY; the map is scoped "near-term only, chapter one, not the full vision."

Runtime description is accurate. recovered_on_main_base: 28bea09, companion_runtime: MESHY_EMBER_FOX_WALK_V1, and the inventory correctly describes the post-#246 loader — versioned async template load, procedural fallback, live authored replacement, explicit scene-anchor detach. That matches what actually shipped, including the two bugs that caused the placeholder to persist on device.

Only carry-over note from my #245 review, unchanged and not blocking: WAYKIN_SPEC still opens "audio-first adaptive walking experience" while this maps AR as the primary surface. C1–C11 and Phase 0 are the right place for that; flagging so the eventual spec edit is deliberate rather than drift. Worth remembering audio is currently the most reliable layer on device — .playback plus background mode, cues confirmed audible on real walks.

@scrimshawlife-ctrl
scrimshawlife-ctrl merged commit 90de100 into main Jul 29, 2026
3 checks passed
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