Skip to content

Add interactive open-in-browser and approve flow#671

Merged
potiuk merged 1 commit intomainfrom
offer-open-and-approve
Apr 13, 2026
Merged

Add interactive open-in-browser and approve flow#671
potiuk merged 1 commit intomainfrom
offer-open-and-approve

Conversation

@potiuk
Copy link
Copy Markdown
Member

@potiuk potiuk commented Apr 6, 2026

Summary

  • After verify-action-build completes verification, offers the user quick links (source tree, commit, repository)
  • Interactive prompt: o to open in browser, a to approve the action, Enter/q to dismiss
  • Skipped automatically in CI mode

Test plan

  • Run uv run utils/verify-action-build.py <action-ref> and verify the prompt appears after results
  • Verify o opens the browser, a returns approve, Enter/q dismisses
  • Verify --ci flag suppresses the prompt

Generated-by: Claude Opus 4.6 (1M context) noreply@anthropic.com

Copy link
Copy Markdown
Member

@dave2wave dave2wave left a comment

Choose a reason for hiding this comment

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

Looks ok to me, but wait for another approval

After verify-action-build completes, offer the user quick links to the
source tree, commit, and repository on GitHub, with options to open in
browser or approve the action directly.

Generated-by: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@potiuk potiuk force-pushed the offer-open-and-approve branch from 0de5663 to 3f8fb8f Compare April 12, 2026 19:57
@potiuk
Copy link
Copy Markdown
Member Author

potiuk commented Apr 12, 2026

Rebased to latest main

@potiuk
Copy link
Copy Markdown
Member Author

potiuk commented Apr 13, 2026

I will cautiously merge it. It only adds a new option

@potiuk potiuk merged commit 2b7bd80 into main Apr 13, 2026
9 checks passed
@potiuk potiuk deleted the offer-open-and-approve branch April 13, 2026 16:41
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