Add deterministic Vercel publishing to readout - #45
Conversation
Co-Authored-By: Oz <oz-agent@warp.dev>
|
I'm starting a first review of this pull request. You can view the conversation on Warp. I completed the review and no human review was requested for this pull request. Comment Powered by Oz |
There was a problem hiding this comment.
Overview
This PR updates the readout skill to require explicit public Vercel consent and adds helper scripts for Vercel scope discovery, publishing, and anonymous verification.
Concerns
- Scope discovery omits the authenticated user's personal Vercel account, so users without teams cannot select any deployment scope and users with teams cannot choose personal publishing.
Verdict
Found: 0 critical, 1 important, 0 suggestions
Request changes
Comment /oz-review on this pull request to retrigger a review (up to 3 times on the same pull request).
Powered by Oz
| if cursor is None: | ||
| break | ||
| teams = list(teams_by_slug.values()) | ||
| if not teams: |
There was a problem hiding this comment.
vercel teams ls only returns teams, so the personal Vercel account is never offered as a scope; users without teams hit this error even though they could deploy to their own account.
Summary
Testing
publish_vercel.pyin dry-run modegit diff --checkwarpdotdev/readouts-sathvik-9321projectAgent Mode: conversation
Co-Authored-By: Oz oz-agent@warp.dev