Skip to content

Offer existing-project setup and portable compose examples - #23

Merged
JorchCortez merged 2 commits into
mainfrom
fix/choose-existing-or-new-project
Oct 3, 2026
Merged

JorchCortez merged 2 commits into
mainfrom
fix/choose-existing-or-new-project

Conversation

@JorchCortez

@JorchCortez JorchCortez commented Oct 3, 2026 •

Copy link
Copy Markdown
Collaborator

Summary

  • Ask a fresh or disconnected CLI installation whether to connect to an existing project or create a new one before suggesting enrollment. Keep pending requests resumable and align agent status, doctor, help, and README with that choice.
  • Explain when a new-project invitation must be accepted and prompt the agent to verify its selected project before writes.
  • Make compose catalog and JSON examples usable on machines without Liberation fonts. Omitting the optional font field selects an installed fallback; callers can still request a specific installed family.

What was already on main

Behavior Main commit (Pacific time)
Project, invitation, and sign-in reset contract ec442ce, Sep 23 at 7:45 PM
Player-style spaced pairing codes and publish playback acknowledgment 12e4854, Sep 29 at 5:36 PM
Authenticated project-media fetch for file playlist previews, with explicit placeholder warnings 7045a79, Oct 1 at 4:19 PM
Latest backend contract refresh c8c67c6, Oct 2 at 5:47 PM

Our original local CLI checkout was at Sep 22's 6dca9f8. This PR was based on current main. The preview implementation on main warns about unresolved placeholders; it does not implement the older local checkout's strict failure behavior.

Verification

  • npm ci, npm run typecheck, npm run build, npm run check:public, npm run smoke:mock, npm run pack:dry, SCREENRIG_VERSION=26.10.0-dev npm run pack:release, and npm run check:npm-install passed.
  • Full local compiled suite excluding vendor-drift.test.js: 887 passed, 1 skipped, 0 failed. The compose subset passed 34/34 after the font change. git diff --check passed.
  • npm run check stops at vendor:check because this workspace's sibling backend checkout is older than the CLI main branch's vendored contract. No vendored files changed here; public CI verifies vendor integrity without that sibling checkout.

Follow-up

After review and merge, rebuild the official plugin from the merged CLI artifact, align its guidance with current CLI commands, and test clean existing-project and new-project setup. This PR does not publish the CLI or plugin.

@JorchCortez
JorchCortez marked this pull request as ready for review October 3, 2026 01:35
@JorchCortez JorchCortez changed the title Ask which project to use before CLI enrollment Offer existing-project setup and portable compose examples Oct 3, 2026
@JorchCortez
JorchCortez merged commit 96d6224 into main Oct 3, 2026
6 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.

1 participant