Skip to content

fix(cli): correct output messages for state load and path-based actions#109

Merged
ctate merged 6 commits into
vercel-labs:mainfrom
timwhitez:main
Jan 22, 2026
Merged

fix(cli): correct output messages for state load and path-based actions#109
ctate merged 6 commits into
vercel-labs:mainfrom
timwhitez:main

Conversation

@timwhitez

Copy link
Copy Markdown
Contributor

fix(cli): correct output messages for state load and path-based actions

Previously, the CLI output layer treated any response containing data.path as a "Screenshot saved" event. This caused state load to incorrectly print screenshot success messages.

Changes:

  • Refactor print_response to accept the command action.
  • Implement specific success messages based on the action (e.g., PDF, Trace, HAR, Video, State save/load).
  • Fix state_load to print "State path set to {path}" instead of screenshot feedback.

fix(cli): correct output messages for state load and path-based actions
@vercel

vercel Bot commented Jan 15, 2026

Copy link
Copy Markdown
Contributor

@timwhitez is attempting to deploy a commit to the Vercel Labs Team on Vercel.

A member of the Team first needs to authorize it.

@ctate

ctate commented Jan 18, 2026

Copy link
Copy Markdown
Collaborator

Nice fix! Can you please fix the merge conflicts?

Comment thread cli/src/output.rs Outdated
@timwhitez

Copy link
Copy Markdown
Contributor Author

Nice fix! Can you please fix the merge conflicts?

Done

@ctate

ctate commented Jan 22, 2026

Copy link
Copy Markdown
Collaborator

Perfect ty!

@ctate ctate merged commit ea17db8 into vercel-labs:main Jan 22, 2026
9 of 10 checks passed
timothywangdev pushed a commit to timothywangdev/banana-browser that referenced this pull request Mar 17, 2026
…ns (vercel-labs#109)

* Add files via upload

fix(cli): correct output messages for state load and path-based actions

* Add files via upload

* Update output.rs

* fix crlf

---------

Co-authored-by: Chris Tate <chris@ctate.dev>
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.

2 participants