Skip to content

quest: split the macOS and Windows app quests - #13

Merged
kixelated merged 1 commit into
mainfrom
quest-plan/split-app
Nov 3, 2026
Merged

kixelated merged 1 commit into
mainfrom
quest-plan/split-app

Conversation

@kixelated

Copy link
Copy Markdown
Contributor

Summary

quest/m0/app.md covered macOS and Windows in one quest, but no single host can run both. It becomes two independent quests, so a Mac can finish the macOS half now:

  • [M] Studio on macOS (app-macos.md): adds the known --features capture build failure on macOS (a ScreenCaptureKit listing future that is not Send, held across runtime.spawn in ui.rs). It was moved here from quest: plan follow-ups of the 2026-11-02 quest-complete merges #12.
  • [M] Studio on Windows (app-windows.md): the Windows half, unchanged.

Links are updated in the m0 README, packaging.md (now needs both), and vcam/macos.md and vcam/windows.md (each needs its own platform). #8 adds a link to app.md in browser-source.md, so whichever of the two lands second points it at both new quests.

Decisions (2026-11-02, maintainer)

  • Split app.md before spawning on a Mac:
    • ✅ Split and start macOS
    • Start app.md whole
    • Wait

Review

Skipped: this moves existing quest text into two files and updates links, with no new plan content beyond the build failure already recorded on #3, #6, and #7. quest check passes.

🤖 Generated with Claude Code

(Written by Claude Opus 5.5)

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
kixelated added a commit that referenced this pull request Nov 3, 2026
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@coderabbitai

coderabbitai Bot commented Nov 3, 2026

Copy link
Copy Markdown

Warning

Review limit reached

You've used all free OSS reviews for now. Wait for the free limit to reset to keep reviewing this public repository.

Next included review available in 24 minutes.

Check out review usage here.

View limit details

Limit details: You’ve used the included review currently available.

Learn how review limits work.

Review configuration:

Run configuration
  • Source mode: Auto
  • Repository: Unavailable
  • Branch: Unavailable
  • File: No configuration file from this repository applied
  • Commit: Unavailable
  • Layers: Organization UI
  • Review profile: CHILL
  • Plan: Advanced
  • Run ID: 047e3f56-30f1-4d69-85ae-e543c3864d7b
Commits

Reviewing files that changed from the base of the PR and between d8535b5 and 2fb926f.

Files selected for processing: 7
  • quest/m0/README.md
  • quest/m0/app-macos.md
  • quest/m0/app-windows.md
  • quest/m0/app.md
  • quest/m0/packaging.md
  • quest/m0/vcam/macos.md
  • quest/m0/vcam/windows.md
  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

Share

CodeRabbit Verify

  • Run CodeRabbit Verify to test the changes in this pull request.

@kixelated kixelated left a comment

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Automated review by review (OpenAI)

Reviewed commit: 2fb926f

No actionable defects found. Splitting platform quests preserves the original goals and maps packaging and virtual-camera dependencies clearly; the macOS capture blocker remains explicit.

Verification: static GitHub review only; no tests or platform builds run.

@kixelated
kixelated merged commit 11b3f41 into main Nov 3, 2026
2 checks passed
@kixelated
kixelated deleted the quest-plan/split-app branch November 3, 2026 03:56
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