Skip to content

fix(browser): parse flags after text values and fail on missing sessions - #115

Merged
junhsss merged 1 commit into
mainfrom
jun/browser-session-flag
Oct 1, 2026
Merged

junhsss merged 1 commit into
mainfrom
jun/browser-session-flag

Conversation

@junhsss

@junhsss junhsss commented Oct 1, 2026

Copy link
Copy Markdown
Contributor

fill, type, setvalue and set useragent captured every argument after the value, including --session. The command then ran on the default session. If no default daemon was running, the CLI silently started a new cloud session. Agents saw "Unknown ref" or a blank page while their real session sat idle.

  • Drop trailing_var_arg from the four value args. Flags now parse anywhere, and unquoted multi-word values still work.
  • --session <name> no longer starts a new session when its daemon is not running. It fails and suggests steel browser start --session <name>. This also covers sessions that expired or idled out. Those used to come back silently with default settings.
  • Starting the implicit default session prints a warning to stderr, also in JSON mode.

@junhsss
junhsss merged commit 6ea1953 into main Oct 1, 2026
4 of 5 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