Skip to content

Read --scope as a flag instead of a slug - #87

Draft
rudycelekli wants to merge 1 commit into
Nutlope:mainfrom
rudycelekli:fix/scaffold-scope-value
Draft

rudycelekli wants to merge 1 commit into
Nutlope:mainfrom
rudycelekli:fix/scaffold-scope-value

Conversation

@rudycelekli

Copy link
Copy Markdown
Contributor

theme.mjs reads --scope with flag(), but scope is missing from VALUED, the set of flags whose next token is a value. positional() only skips that next token when the flag is in the set, so the value becomes a positional argument.

node theme.mjs scaffold --scope app bloom therefore looks up references/themes/app.md and exits 1 (no theme at .../themes/app.md). The same leak makes photos "sea" --scope app search for a second subject named app. scaffold bloom --scope app happens to work only because the slug comes first and later positionals are ignored.

scope is now in VALUED, beside redesign.

Verified with theme.mjs:

  • scaffold --scope app bloom writes Bloom's tokens, the stamp shape: app screen, and the fixed app type scale (no clamp).
  • scaffold bloom --scope app still does.
  • scaffold --scope app with no slug prints usage and exits 2, instead of looking up a theme named app.
  • photos "sea" --n 1 --seed abc --scope app --json returns the subject sea only.

The app-screens sentence in flow.md still shows theme.mjs scaffold --scope app with no slug. That command has nothing to scaffold; this change only stops app from being read as the theme.

Co-authored-by: Cursor <cursoragent@cursor.com>
@vercel

vercel Bot commented Oct 25, 2026

Copy link
Copy Markdown

@rudycelekli is attempting to deploy a commit to the Together AI Team on Vercel.

A member of the Team first needs to authorize it.

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