Skip to content

fix: alias staging docs via the deployments API#2897

Closed
unarbos wants to merge 1 commit into
mainfrom
fix-staging-alias
Closed

fix: alias staging docs via the deployments API#2897
unarbos wants to merge 1 commit into
mainfrom
fix-staging-alias

Conversation

@unarbos

@unarbos unarbos commented Jul 13, 2026

Copy link
Copy Markdown
Contributor

Summary

  • Every Deploy Docs run on main has failed at the final step with Error: You don't have access to the domain staging.bittensor.com under rao-foundation, because vercel alias set only accepts domains in the team-level domain registry while staging.bittensor.com is attached (and verified) at the project level.
  • Replace the CLI alias with the Vercel deployments API (GET /v13/deployments/<url> to resolve the id, POST /v2/deployments/<id>/aliases), which honors project domains. curl -sf fails the step on any HTTP error.

Test plan

  • Ran the exact resolve-and-alias pipeline locally against the latest deployment with the same token scope; the alias was applied and https://staging.bittensor.com serves it.
  • After merge, confirm the Deploy Docs run goes green and staging.bittensor.com picks up the new deployment.

Made with Cursor

`vercel alias set` only accepts domains registered at the team level,
but staging.bittensor.com is attached and verified on the project, so
every staging deploy failed at the alias step. The deployments API
honors project domains, so resolve the deployment id and set the alias
with curl instead.

Co-authored-by: Cursor <cursoragent@cursor.com>
@vercel

vercel Bot commented Jul 13, 2026

Copy link
Copy Markdown

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

Project Deployment Actions Updated (UTC)
subtensor Ready Ready Preview, Comment Jul 13, 2026 9:09pm

Request Review

@github-actions

github-actions Bot commented Jul 13, 2026

Copy link
Copy Markdown
Contributor

🛡️ AI Review — Skeptic (security review)

VERDICT: SAFE

BASELINE scrutiny: author has repository write access and substantial merged contribution history; no Gittensor association found. Branch fix-staging-alias → main.

Static analysis found the workflow change narrowly scoped to resolving the Vercel deployment ID and assigning the existing staging alias. Inputs interpolated into the requests come from trusted secrets, repository variables, or Vercel’s deployment response; no credential-exposure or command-injection path was introduced.

Findings

No findings.

Conclusion

The API-based alias operation preserves the workflow’s existing trust boundary and fails on HTTP or parsing errors. No security vulnerability or malicious behavior found.


🔍 AI Review — Auditor (domain review)

VERDICT: 👍

Gittensor association: UNKNOWN; established contributor with repository write access and substantial prior contributions.

The implementation matches the PR description: staging deployments now resolve the Vercel deployment ID and assign the project-level alias through the deployments API, while the production path remains unchanged. HTTP and response-parsing failures correctly fail the workflow step. No overlapping PRs were identified, and no runtime/spec-version change is involved.

Findings

No findings.

Conclusion

This is a focused, well-explained workflow fix with an appropriate test account and no substantive correctness issues found.

@github-actions

Copy link
Copy Markdown
Contributor

🔄 AI review updated — Skeptic: SAFE Auditor: 👍

@IntiTechnologies

Copy link
Copy Markdown
Contributor

Superseded by #2898 (v431 release): these changes were reviewed and integrated there.

@IntiTechnologies IntiTechnologies deleted the fix-staging-alias branch July 14, 2026 17:27
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