Skip to content

SRE-415: Fix vercel not building#8342

Merged
TimDiekmann merged 1 commit intomainfrom
t/sre-415-fix-vercel-not-building
Feb 1, 2026
Merged

SRE-415: Fix vercel not building#8342
TimDiekmann merged 1 commit intomainfrom
t/sre-415-fix-vercel-not-building

Conversation

@TimDiekmann
Copy link
Member

🌟 What is the purpose of this PR?

Remove the sourcing of Cargo environment variables in the Vercel build script, as it's no longer needed with mise handling environment setup.

🔗 Related links

🔍 What does this change?

  • Removes source "$HOME/.cargo/env" from the Vercel build script
  • Adds a reference to H-3212 in the environment variable cleanup comments

Pre-Merge Checklist 🚀

🚢 Has this modified a publishable library?

This PR:

  • does not modify any publishable blocks or libraries, or modifications do not need publishing

📜 Does this require a change to the docs?

The changes in this PR:

  • are internal and do not require a docs change

🕸️ Does this require a change to the Turbo Graph?

The changes in this PR:

  • do not affect the execution graph

🛡 What tests cover this?

  • Vercel build process will validate this change works correctly

❓ How to test this?

  1. Checkout the branch
  2. Verify the Vercel build completes successfully without the Cargo environment sourcing

@vercel
Copy link

vercel bot commented Feb 1, 2026

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

2 Skipped Deployments
Project Deployment Actions Updated (UTC)
hashdotdesign Ignored Ignored Preview Feb 1, 2026 0:03am
hashdotdesign-tokens Ignored Ignored Preview Feb 1, 2026 0:03am

@vercel vercel bot temporarily deployed to Preview – petrinaut February 1, 2026 12:02 Inactive
@cursor
Copy link

cursor bot commented Feb 1, 2026

PR Summary

Low Risk
Build-script-only change removing Cargo env sourcing; primary risk is Vercel builds failing if Rust tooling was implicitly required.

Overview
Removes source "$HOME/.cargo/env" from apps/hash-frontend/vercel-build.sh, relying solely on mise to set up the build environment for Vercel.

Adds an additional internal issue link to the existing .env cleanup comments.

Written by Cursor Bugbot for commit 66a0436. This will update automatically on new commits. Configure here.

@github-actions github-actions bot added area/apps > hash* Affects HASH (a `hash-*` app) type/eng > frontend Owned by the @frontend team area/apps labels Feb 1, 2026
Copy link
Member Author

TimDiekmann commented Feb 1, 2026

@augmentcode
Copy link

augmentcode bot commented Feb 1, 2026

🤖 Augment PR Summary

Summary: Stops sourcing $HOME/.cargo/env during Vercel builds now that mise handles environment setup.

Changes: Removes the Cargo env sourcing step and adds a reference to H-3212 for ongoing environment variable cleanup.

🤖 Was this summary useful? React with 👍 or 👎

Copy link

@augmentcode augmentcode bot left a comment

Choose a reason for hiding this comment

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

Review completed. No suggestions at this time.

Comment augment review to trigger a new review at any time.

@TimDiekmann TimDiekmann added this pull request to the merge queue Feb 1, 2026
Merged via the queue into main with commit a469ab7 Feb 1, 2026
47 checks passed
@TimDiekmann TimDiekmann deleted the t/sre-415-fix-vercel-not-building branch February 1, 2026 21:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area/apps > hash* Affects HASH (a `hash-*` app) area/apps type/eng > frontend Owned by the @frontend team

Development

Successfully merging this pull request may close these issues.

2 participants