Skip to content

fix(help): say --launch-console is for local iOS simulator launches - #4028

Merged
thymikee merged 1 commit into
mainfrom
fix/launch-console-help-local
Oct 28, 2026
Merged

thymikee merged 1 commit into
mainfrom
fix/launch-console-help-local

Conversation

@thymikee

@thymikee thymikee commented Oct 28, 2026 •

Copy link
Copy Markdown
Member

Summary

Follow-up to the #3938 review. The debugging workflow help (src/commands/schema/cli-help-workflows.ts) still said --launch-console is "only for direct iOS simulator app launches". Since #3938 a provider-owned iOS simulator refuses the flag, and open's flag help and the docs say "local". The workflow help now says "local iOS simulator" too, and the two tests that pin the wording are updated. Scope expansion: none.

Behavior note from the same review, accepted rather than changed: on a provider-owned device the launch-console refusal runs before the request-shape checks, so open --launch-console <path> with no app target, or with a URL, returns UNSUPPORTED_OPERATION (launch-console-owner-unadmitted) instead of INVALID_ARGS. Both answers refuse before the device is touched, and the owner refusal is the actionable one there: the flag cannot work on that device whatever the arguments are.

Validation

  • pnpm check:affected --run at 47c3569e7b: format, lint, typecheck, layering, fallow, build, and vitest-related passed (exactCommit=true).
  • Help text only; not device-facing.

🤖 Generated with Claude Code

View guided diff Turn on auto-fix

The debugging workflow help still said direct iOS simulator app launches,
while open's flag help and the docs say local since #3938, where a
provider-owned simulator refuses the flag.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@vercel

vercel Bot commented Oct 28, 2026 •

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

1 Skipped Deployment
Project Deployment Actions Updated
agent-device-website Ignored Ignored Preview Oct 28, 2026 3:54pm UTC

Request Review

@cubic-dev-ai cubic-dev-ai Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

No issues found across 3 files

View guided diff | Turn on auto-fix | Re-trigger cubic

@github-actions

Copy link
Copy Markdown
Contributor

Size Report

Metric Base Current Diff
Installed (including dependencies) 5.56 MB 5.56 MB -1 B
Package (unpacked) 5.56 MB 5.56 MB -1 B
Package (download) 1.69 MB 1.69 MB 0 B

Startup median (7 runs, lower is better):

Scenario Base Current Diff
CLI --version 33.4 ms 33.2 ms -0.2 ms
CLI --help 67.9 ms 67.7 ms -0.1 ms

@thymikee

Copy link
Copy Markdown
Member Author

The change at 47c3569 looks good: it only updates the --launch-console help string and the two help-text assertions, and both assertions match the new wording. I read the code but did not run the help tests locally. The diff touches no device or smoke route, so the Smoke Tests run, which was still queued when I looked, is unlikely to fail because of it. There are no conflicts.

@thymikee thymikee added the ready-for-human Valid work that needs human implementation, judgment, or maintainer merge label Oct 28, 2026
@thymikee
thymikee merged commit f4787a2 into main Oct 28, 2026
21 of 22 checks passed
@thymikee
thymikee deleted the fix/launch-console-help-local branch October 28, 2026 17:05
@github-actions

Copy link
Copy Markdown
Contributor
PR Preview Action v1.8.1
Preview removed because the pull request was closed.
2026-10-28 17:06 UTC

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ready-for-human Valid work that needs human implementation, judgment, or maintainer merge

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant