Skip to content

ci: post-merge fixes — Vercel deploy cwd, try-runtime snapshots, gittensor indexer#2865

Closed
unarbos wants to merge 3 commits into
mainfrom
fix/deploy-docs-vercel-cwd
Closed

ci: post-merge fixes — Vercel deploy cwd, try-runtime snapshots, gittensor indexer#2865
unarbos wants to merge 3 commits into
mainfrom
fix/deploy-docs-vercel-cwd

Conversation

@unarbos

@unarbos unarbos commented Jul 12, 2026

Copy link
Copy Markdown
Contributor

Summary

Consolidated fixes for the CI breakages surfaced by the first post-merge runs on main:

  • Deploy Docs failed (ENOENT): vercel build ran from website/, but the Vercel project's root directory (website/apps/bittensor-website) is resolved against the CLI's cwd, doubling the path. Dropped working-directory: website from the staging deploy (deploy-docs.yml) and the production promotion (watch-mainnet-release.yml).
  • All three try-runtime snapshot jobs failed: in refresh-mainnet-snapshot.yml, create-snapshot --uri <ws> <name>.snap fails because --uri is variadic and swallows the trailing snapshot path ("not a valid WS(S) url"). Moved the positional path before --uri. (The mainnet clone snapshot job in the same workflow succeeded and already published its artifact.)
  • ai-review-index-gittensor fails every scheduled run: the pinned gittensor commit declares requires-python >=3.12; bumped the job's Python from 3.11 to 3.12 (pin unchanged).

Related cleanup handled outside this PR:

Test plan

  • After merge, workflow_dispatch Deploy Docs → staging deploy succeeds
  • After merge, workflow_dispatch Refresh Mainnet Snapshot → all four snapshot artifacts publish
  • Next scheduled ai-review-index-gittensor run installs and indexes cleanly

@vercel

vercel Bot commented Jul 12, 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 12, 2026 6:04pm

Request Review

@unarbos unarbos closed this Jul 12, 2026
@unarbos unarbos force-pushed the fix/deploy-docs-vercel-cwd branch from 5937706 to 647ca2b Compare July 12, 2026 16:57
The Vercel project root directory (website/apps/bittensor-website) is resolved against the CLI cwd, so running from website/ doubles the path and vercel build fails with ENOENT.
@github-actions

github-actions Bot commented Jul 12, 2026

Copy link
Copy Markdown
Contributor

🛡️ AI Review — Skeptic (security review)

VERDICT: SAFE

MEDIUM scrutiny by account age, mitigated by repository write access and substantial contributions; no Gittensor association. CI/deployment hotfix targets main.

Static review found the workflow-only changes consistent with the documented failures. Since the prior SAFE verdict, the only addition changes the indexer from Python 3.11 to the pinned Gittensor dependency’s required Python 3.12. No review-policy files, runtime code, dependencies, permissions, triggers, or secret handling are changed.

Findings

No findings.

Conclusion

The patch remains small and mechanical, with no apparent security vulnerability or malicious behavior.


🔍 AI Review — Auditor (domain review)

VERDICT: 👍

UNKNOWN Gittensor association; established high-volume contributor with repository write access and substantial merged work.

This deployment-only CI hotfix appropriately targets main. The workflow changes consistently address the documented Vercel cwd issue, variadic --uri parsing, and gittensor's Python requirement.

The overlap with broad monorepo PR #2841 is incidental rather than duplicate work. Static inspection was sufficient; runtime code, dependencies, and spec_version are unaffected.

Findings

No findings.

Conclusion

The changes are minimal, internally consistent, and match the reported CI failures. Ready to merge.

@unarbos unarbos reopened this Jul 12, 2026
@github-actions

Copy link
Copy Markdown
Contributor

🔄 AI review updated — Skeptic: SAFE Auditor: 👍

@github-actions

Copy link
Copy Markdown
Contributor

🔄 AI review updated — Skeptic: SAFE Auditor: 👍

--uri is variadic in try-runtime-cli, so a trailing positional snapshot path is consumed as a second URI value and fails validation. All three try-runtime snapshot jobs failed this way on the first dispatch.
@unarbos unarbos changed the title ci: run Vercel deploys from the repo root ci: fix Vercel deploy cwd and try-runtime snapshot invocation Jul 12, 2026
@github-actions

Copy link
Copy Markdown
Contributor

🔄 AI review updated — Skeptic: SAFE Auditor: 👍

The pinned gittensor commit declares requires-python >=3.12, so every scheduled run fails at pip install under the 3.11 setup.
@unarbos unarbos changed the title ci: fix Vercel deploy cwd and try-runtime snapshot invocation ci: post-merge fixes — Vercel deploy cwd, try-runtime snapshots, gittensor indexer Jul 12, 2026
@github-actions

Copy link
Copy Markdown
Contributor

🔄 AI review updated — Skeptic: SAFE Auditor: 👍

@unarbos

unarbos commented Jul 12, 2026

Copy link
Copy Markdown
Contributor Author

Consolidated into #2867 (same commits merged there); closing to keep a single next-PR into main.

@unarbos unarbos closed this Jul 12, 2026
@IntiTechnologies IntiTechnologies deleted the fix/deploy-docs-vercel-cwd branch July 14, 2026 17:35
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