Skip to content

docs(Mobius): plan persistent PR workspaces with child windows - #497

Open
kyleve wants to merge 13 commits into
mainfrom
codex/mobius-plan
Open

kyleve wants to merge 13 commits into
mainfrom
codex/mobius-plan

Conversation

@kyleve

@kyleve kyleve commented Nov 14, 2026 •

Copy link
Copy Markdown
Owner

Posted by an AI agent on kve's behalf.

Summary

Plan Mobius as a persistent code-review app for iPhone, iPad, and Mac Catalyst. A PR owns one logical workspace with independently navigable review, source, research, and preview views. Workspaces can appear as tabs or outer windows. Returning restores the arrangement and last-focused view; a resumed review starts its selected file at the top.

Closing a workspace removes its presentation while retaining its views, navigation, drafts, and review/skip evidence. A persistent Workspaces picker reopens it. Closing an inactive workspace preserves the current review; closing the last workspace provides an explicit empty state. Cross-device open/closed behavior remains a later Q-09 decision.

Within each PR, remaining work shows new edits since each file's last-reviewed version and the full PR diff for unreviewed files. Kind-based groups and content-appropriate comparisons make quieter changes easier to scan. Explicit PR-local skip/undo stays separate from reviewed: new or changed files return after a push, unchanged skips persist, and skipping never advances a review baseline. Full diffs and source/metadata remain accessible.

The first release includes GitHub comments, approvals, requests for changes, and private iCloud continuity using the same Apple ID. Stack relationships are informational. Embedded review and live execution remain a later feasibility experiment.

Planning horizon

Start with the overview, requirements, decisions, and continuation record.

The current deliverable is 01a — Workspace prototype, executing in #500. It is a lightweight, synthetic browser mockup across phone, tablet, and desktop sizes. Its current artifact includes compact chrome, grouped changes and skip/undo, truthful workspace/view layouts, title-bar dragging, and retained workspace close/reopen. The prototype now runs through the repository’s pinned Astro toolchain with automatic source updates and local-network device access. CSS updates preserve the sample session; page/script updates reload and reset it. The Astro migration preserves the interaction code and prior standalone artifact. User evaluation remains open.

Native plans 01–07 remain outlines: review-state foundation, application workspace, GitHub/revision comparison, repository exploration, review feedback, iCloud/release, and the embedded experiment. Prototype evaluation precedes their detailed methods. Authentication, rendering/index budgets, format coverage, sync merge/publication policy, Catalyst distribution, and live-context feasibility retain explicit refinement gates.

Review focus

  • Workspace close/reopen ownership: retained workspace identity is separate from open presentation; child state and shared PR evidence survive closure.
  • Change grouping and skipping: group skip snapshots current remaining files, preserves reviewed files and future additions, and retains the actual review baseline. Rich and source views share comparison identity; unavailable bytes cannot imply a clean diff.
  • Compact controls preserve identity, comparison meaning, immediate actions, readable text, and usable targets. Workspace presentation and child-view arrangement have distinct scopes; selected layouts must match rendered behavior at every width.
  • This PR owns the shared requirements and native outlines. feat(Mobius): prototype grouped review workspaces #500 owns the current 01a method, implementation evidence, and publication checkpoint.

Validation

  • Parent revision ecb3c017d: reviewed the nine-file planning series and coordinated close/reopen method; 182 local Markdown links/anchors, balanced fences, all 14 requirement owners, the acyclic eight-plan graph, and whitespace checks pass. No plan-consistency or method-readiness finding remains.
  • Workspace lifecycle evidence: 11 lifecycle scenarios across three sizes and both workspace presentations pass, with fresh independent review, touch/hit checks, and selected automated accessibility scans. Scoped Chromium and WebKit checks independently verify close/reopen retention and title-bar behavior on phone and desktop. The coordinator verified the tested HTML and all nine capture hashes and inspected representative final captures.
  • Astro migration evidence: check/build, actual CSS/markup/script automatic updates, three-size served scenarios, and scoped Chromium/WebKit dev/build checks pass. Initial before/after renders match exactly at all three sizes. Fresh migration review has no actionable findings; actual phone Wi-Fi remains untested. The owned review server stays running, with lifecycle commands in the prototype README.
  • Earlier artifact/evidence history remains in 01a. These are isolated, in-memory browser checks; user evaluation, native/VoiceOver behavior, GitHub integration, persistence, and sync acceptance remain open. No native tests or SwiftFormat were run for this Markdown-only parent revision. Swift examples are pseudocode; Mermaid was source-reviewed.

Stack and tracking

Stack #501: main → this plan PR → prototype #500.

STU-329 and STU-330 remain In Review. The prototype's current 01a record owns the delivery receipt and requested completion notice. Native implementation is outside this work. The reusable Astro prototype process is proposed separately in #503, based directly on main.

Define the product scope, revision-aware progress, concurrent workspace model, GitHub feedback, private iCloud continuity, and a later embedding experiment. Detail the local state foundation and preserve later work as six outlines with refinement gates. Validation: reviewed the complete plan, verified 103 local links and anchors, checked the seven-plan dependency graph and all 12 requirement owners, and passed git diff --cached --check. Product tests and SwiftFormat were skipped because only planning Markdown changed. Mermaid was source-reviewed; no local renderer was available.
Link PR #497, record STU-329 as In Review, and preserve the next planning or execution resume action. Reviewed the handoff diff and passed git diff --check; no product code changed.
Record independent PR workspaces, file-top resumption, and remaining work based on each file last marked reviewed, while keeping full PR diffs accessible. Insert a lightweight interactive mockup across phone, tablet, and desktop before durable storage, and return the foundation to outline. Preserve the tentative answers and subsequent clarifications in Q-13 through Q-18. Validation: reviewed the revised nine-file series and current prototype horizon, checked 118 local links and anchors, balanced fences, eight acyclic plan dependencies, all 12 requirement owners, and git diff --cached --check. No prototype was implemented. Product tests and SwiftFormat were skipped for Markdown-only changes; Mermaid was source-reviewed without a local renderer.
@kyleve kyleve changed the title docs(Mobius): plan persistent code-review workspaces docs(Mobius): plan PR workspaces and prototype-first delivery Nov 15, 2026
Record the user-authorized 01a execution branch and isolated worktree while keeping production-plan discussion in the parent checkout. Reviewed the checkpoint and passed git diff --check; Markdown-only change, so app tests and SwiftFormat were not run.
Integrate main at 96f5bf0, including the planning-skill update from PR #498 and the previously merged Device Layout Lab. The resulting branch differs from main only in Mobius planning Markdown. Verified the merged skill, all 118 plan links and anchors, no unresolved index entries, and git diff --cached --check; regenerated agent mirrors. App tests were skipped because this branch introduces no app-code difference from main.
@kyleve kyleve changed the title docs(Mobius): plan PR workspaces and prototype-first delivery docs(Mobius): plan persistent PR workspaces with child windows Nov 15, 2026

This branch has not been deployed

No deployments
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