Skip to content

fix(ci): resolve Vercel deploy failure caused by empty VERCEL_TOKEN#173

Open
Copilot wants to merge 32 commits intomainfrom
copilot/fix-help-step-issue
Open

fix(ci): resolve Vercel deploy failure caused by empty VERCEL_TOKEN#173
Copilot wants to merge 32 commits intomainfrom
copilot/fix-help-step-issue

Conversation

Copy link
Copy Markdown
Contributor

Copilot AI commented Apr 9, 2026 •
edited
Loading

The Deploy to Vercel (Production) job was failing at "Pull Vercel environment" because VERCEL_TOKEN was not configured as a repository secret, causing every vercel command to receive --token= (empty string) and bail with "No existing credentials found."

Changes

  • Token via env var — moved VERCEL_TOKEN (plus VERCEL_ORG_ID / VERCEL_PROJECT_ID) to a job-level env: block; Vercel CLI picks it up automatically, eliminating the repeated --token=${{ secrets.VERCEL_TOKEN }} on every command
  • Early validation — added a "Verify Vercel token" step before any vercel invocation that emits a clear ::error:: annotation and exits immediately if the secret is absent
env:
  VERCEL_TOKEN: ${{ secrets.VERCEL_TOKEN }}
  VERCEL_ORG_ID: ${{ secrets.VERCEL_ORG_ID }}
  VERCEL_PROJECT_ID: ${{ secrets.VERCEL_PROJECT_ID }}

- name: Verify Vercel token
  run: |
    if [ -z "$VERCEL_TOKEN" ]; then
      echo "::error::VERCEL_TOKEN secret is not configured. Add it under Settings → Secrets and variables → Actions."
      exit 1
    fi

Action required: add VERCEL_TOKEN (and optionally VERCEL_ORG_ID, VERCEL_PROJECT_ID) under Repository Settings → Secrets and variables → Actions for deployments to succeed.

Copilot AI and others added 30 commits April 9, 2026 02:41
Initial plan
3a96e01
Initial plan
bc5baf5
Initial plan
1b81a5e
@LVT-ENG
feat: implement Bunker_Stirpe_V10 module from issue #145
116b31e 
Agent-Logs-Url: https://github.com/Tryonme-com/tryonyou-app/sessions/d22ccf17-c970-45f4-b4d2-d149b6a41c47

Co-authored-by: LVT-ENG <214667862+LVT-ENG@users.noreply.github.com>
@LVT-ENG
Merge pull request #150 from Tryonme-com/copilot/update-sovereign-bun…
38d5781 
…ker-audit-method
@LVT-ENG
Merge pull request #148 from Tryonme-com/copilot/fix-issue-145b
6cef073
@LVT-ENG
Merge pull request #151 from Tryonme-com/copilot/update-user-profile-ui
1a5e350
@cursoragent @LVT-ENG
@CertezaAbsoluta @lo+erestu PCT/EP2025/067317 - Inyección soberana de…
7aa10e0 
…ploy_divineo Bajo Protocolo de Soberanía V10 - Founder: Rubén

Co-authored-by: Tryonme <LVT-ENG@users.noreply.github.com>
@LVT-ENG
Merge pull request #163 from Tryonme-com/cursor/protocolo-omega-inyec…
d7c9327 
…cion-a675
@cursoragent @LVT-ENG
@CertezaAbsoluta @lo+erestu PCT/EP2025/067317 - Cierre total Omega Di…
be23bee 
…vineo (orquestador+tests+docs) Bajo Protocolo de Soberanía V10 - Founder: Rubén

Co-authored-by: Tryonme <LVT-ENG@users.noreply.github.com>
@cursoragent @LVT-ENG
feat: deployall — full CI/CD pipeline + local deploy script
809ba41 
- Enhanced .github/workflows/ci.yml: parallel Python tests + Vite build + Vercel deploy on main
- Added TypeScript type-check step (tsc --noEmit) to CI
- Added src/vite-env.d.ts for Vite client types (import.meta.env, Window augmentation)
- Added scripts/deployall.sh: local build+test+deploy pipeline (--dry for skip deploy)
- Added npm scripts: typecheck, deployall, deployall:dry

All 141 Python tests pass. Vite build + type-check clean.

@CertezaAbsoluta @lo+erestu PCT/EP2025/067317
Bajo Protocolo de Soberanía V10 - Founder: Rubén

Co-authored-by: Tryonme <LVT-ENG@users.noreply.github.com>
@LVT-ENG
Merge pull request #165 from Tryonme-com/cursor/deployall-ci-cd-4899
d2ad284
Initial plan
e7a2426
@LVT-ENG
Protocolo Soberanía V10: mirror + Pau Marais + Firebase applet @Certe…
6a6c91a 
…zaAbsoluta @lo+erestu PCT/EP2025/067317 Bajo Protocolo de Soberanía V10 - Founder: Rubén

Co-authored-by: LVT-ENG <214667862+LVT-ENG@users.noreply.github.com>
@LVT-ENG
Protocolo Soberanía V10: mirror + Pau Marais + Firebase applet @Certe…
de7f6d3 
…zaAbsoluta @lo+erestu PCT/EP2025/067317 Bajo Protocolo de Soberanía V10 - Founder: Rubén

Co-authored-by: LVT-ENG <214667862+LVT-ENG@users.noreply.github.com>
@LVT-ENG
Protocolo Soberanía V10: mirror + Pau Marais + Firebase applet @Certe…
fef4922 
…zaAbsoluta @lo+erestu PCT/EP2025/067317 Bajo Protocolo de Soberanía V10 - Founder: Rubén

Co-authored-by: LVT-ENG <214667862+LVT-ENG@users.noreply.github.com>
@LVT-ENG
Merge pull request #166 from Tryonme-com/copilot/supercommit-max-sh
7a498ae
@cursoragent @LVT-ENG
feat: supercommit_max v10.17 VIVOS — full pipeline consolidado @Certe…
1477368 
…zaAbsoluta @lo+erestu PCT/EP2025/067317 Bajo Protocolo de Soberanía V10 - Founder: Rubén

Co-authored-by: Tryonme <LVT-ENG@users.noreply.github.com>
@cursoragent @LVT-ENG
🧹 Hulk cleanup: purge 2953 tracked node_modules/ files — .gitignore a…
bd5e68b 
…lready excludes them @CertezaAbsoluta @lo+erestu PCT/EP2025/067317 Bajo Protocolo de Soberanía V10 - Founder: Rubén

Co-authored-by: Tryonme <LVT-ENG@users.noreply.github.com>
@cursoragent @LVT-ENG
Merge: supercommit_max v10.17 VIVOS + Hulk cleanup (2953 node_modules…
a5d0f3c 
… purged) @CertezaAbsoluta @lo+erestu PCT/EP2025/067317 Bajo Protocolo de Soberanía V10 - Founder: Rubén

Co-authored-by: Tryonme <LVT-ENG@users.noreply.github.com>
@cursoragent @LVT-ENG
🔱 OMEGA_DEPLOY: Full CI/CD Pipeline Merged.
444a370 
🥀 Honor Protocol: Goldschmied/Valentino/Ospina.
✨ Status: VIVOS.
🚀 Divineo Absoluto en París.
@CertezaAbsoluta @lo+erestu PCT/EP2025/067317 Bajo Protocolo de Soberanía V10 - Founder: Rubén

Co-authored-by: Tryonme <LVT-ENG@users.noreply.github.com>
@cursoragent @LVT-ENG
🔱 OMEGA_DEPLOY: Full CI/CD Pipeline Merged.
7400abd 
🥀 Honor Protocol: Goldschmied/Valentino/Ospina.
✨ Status: VIVOS.
🚀 Divineo Absoluto en París.
@CertezaAbsoluta @lo+erestu PCT/EP2025/067317 Bajo Protocolo de Soberanía V10 - Founder: Rubén

Co-authored-by: Tryonme <LVT-ENG@users.noreply.github.com>
@cursoragent @LVT-ENG
@CertezaAbsoluta @lo+erestu PCT/EP2025/067317 - Integración logo ofic…
0b1b22d 
…ial + copy FR traducible + videos venta EUR Bajo Protocolo de Soberanía V10 - Founder: Rubén

Co-authored-by: Tryonme <LVT-ENG@users.noreply.github.com>
@cursoragent @LVT-ENG
@CertezaAbsoluta @lo+erestu PCT/EP2025/067317 - Fix interacción idiom…
55b9b29 
…a y shell overlay para venta FR/EUR Bajo Protocolo de Soberanía V10 - Founder: Rubén

Co-authored-by: Tryonme <LVT-ENG@users.noreply.github.com>
@LVT-ENG
Merge pull request #164 from Tryonme-com/cursor/protocolo-omega-inyec…
c180338 
…cion-a675
Initial plan
f45bb98
@LVT-ENG
feat: add Stripe secret key, public key, and webhook secret support
9ea7261 
Agent-Logs-Url: https://github.com/Tryonme-com/tryonyou-app/sessions/5c24d097-2c3f-4879-bef1-e33aa8d9d2b6

Co-authored-by: LVT-ENG <214667862+LVT-ENG@users.noreply.github.com>
Initial plan
1dcd9b5
@LVT-ENG
Merge pull request #172 from Tryonme-com/copilot/run-sync-lafayette-node
8f96721
@LVT-ENG
Merge pull request #169 from Tryonme-com/copilot/add-stripe-keys-to-t…
debfeb5 
…ryonyou-app
@vercel
Copy link
Copy Markdown

vercel bot commented Apr 9, 2026 •
edited
Loading

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

Project Deployment Actions Updated (UTC)
tryonyou-app Ready Ready Preview, Comment Apr 9, 2026 5:10pm
tryonyou-pilot Ready Ready Preview, Comment Apr 9, 2026 5:10pm
workspace Ready Ready Preview, Comment Apr 9, 2026 5:10pm

Request Review

@LVT-ENG
fix: resolve Vercel deploy CI failure due to missing VERCEL_TOKEN
112b70b 
The deploy job was running `vercel pull --token=` with an empty value
because the VERCEL_TOKEN secret was not configured. Fix:
- Move VERCEL_TOKEN (+ VERCEL_ORG_ID/VERCEL_PROJECT_ID) to job-level
  env block — the standard Vercel CLI authentication pattern
- Remove redundant --token= flags from all vercel commands
- Add "Verify Vercel token" pre-check step with a clear error message

Agent-Logs-Url: https://github.com/Tryonme-com/tryonyou-app/sessions/c15c7495-1db1-442a-8fba-b61409e3b997

Co-authored-by: LVT-ENG <214667862+LVT-ENG@users.noreply.github.com>
Copilot AI changed the title [WIP] Fix issue in help step of workflow fix(ci): resolve Vercel deploy failure caused by empty VERCEL_TOKEN Apr 9, 2026
Copilot AI requested a review from LVT-ENG April 9, 2026 17:11
@LVT-ENG LVT-ENG marked this pull request as ready for review April 11, 2026 17:10
@LVT-ENG LVT-ENG requested a review from Copilot April 12, 2026 11:56
LVT-ENG
LVT-ENG approved these changes Apr 12, 2026
View reviewed changes
Copy link
Copy Markdown
Member

@LVT-ENG LVT-ENG left a comment

Choose a reason for hiding this comment

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

Yes

Copilot AI reviewed Apr 12, 2026
View reviewed changes
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR updates the GitHub Actions production deploy workflow to avoid Vercel CLI failures when VERCEL_TOKEN is missing, and to make token usage less repetitive across steps.

Changes:

  • Moves VERCEL_TOKEN, VERCEL_ORG_ID, and VERCEL_PROJECT_ID to a job-level env: block so Vercel CLI can read them implicitly.
  • Adds an early “Verify Vercel token” step that fails fast with a clear error if VERCEL_TOKEN is unset.
  • Removes repeated --token=... flags from vercel pull/build/deploy commands.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread .github/workflows/ci.yml
LVT-ENG
LVT-ENG approved these changes Apr 12, 2026
View reviewed changes
Copy link
Copy Markdown
Member

@LVT-ENG LVT-ENG left a comment

Choose a reason for hiding this comment

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

Yes

LVT-ENG
LVT-ENG approved these changes Apr 12, 2026
View reviewed changes
Copy link
Copy Markdown
Member

@LVT-ENG LVT-ENG left a comment

Choose a reason for hiding this comment

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

Yes

LVT-ENG
LVT-ENG approved these changes Apr 12, 2026
View reviewed changes
Copy link
Copy Markdown
Member

@LVT-ENG LVT-ENG left a comment

Choose a reason for hiding this comment

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

Yes

LVT-ENG
LVT-ENG approved these changes Apr 12, 2026
View reviewed changes
Copy link
Copy Markdown
Member

@LVT-ENG LVT-ENG left a comment

Choose a reason for hiding this comment

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

Yes

LVT-ENG
LVT-ENG approved these changes Apr 12, 2026
View reviewed changes
Copy link
Copy Markdown
Member

@LVT-ENG LVT-ENG left a comment

Choose a reason for hiding this comment

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

Yes

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

Copilot code review Copilot Copilot left review comments

+1 more reviewer

@LVT-ENG LVT-ENG LVT-ENG approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

Copilot code review Copilot

@LVT-ENG LVT-ENG

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@LVT-ENG @cursoragent