Skip to content

fix: resolve Vercel build warning and lint errors#20

Merged
amondnet merged 1 commit intomainfrom
fix/vercel-build-and-lint-errors
Oct 16, 2025
Merged

fix: resolve Vercel build warning and lint errors#20
amondnet merged 1 commit intomainfrom
fix/vercel-build-and-lint-errors

Conversation

@amondnet
Copy link
Copy Markdown
Contributor

@amondnet amondnet commented Oct 16, 2025

Summary

  • Add GITHUB_TOKEN environment variable to turbo.json build task
  • Fix ESLint errors in server utilities

Problems

  1. Vercel build warning: GITHUB_TOKEN was set in Vercel project but missing from turbo.json
  2. Multiple lint errors in server files

Solutions

  • turbo.json: Added "env": ["GITHUB_TOKEN"] to build task configuration
  • marketplaces.get.ts: Removed unused AggregatedPlugin import and fixed import order
  • github.ts: Changed process.env.GITHUB_TOKEN to import.meta.env.GITHUB_TOKEN for Nuxt compatibility and added missing newline

Test Plan

  • All lint errors resolved (verified with bun run lint)
  • Build configuration updated for Vercel
  • Verify Vercel build completes without warnings
  • Confirm GITHUB_TOKEN is accessible during build

- Add GITHUB_TOKEN to turbo.json build task env to fix Vercel warning
- Remove unused AggregatedPlugin import in marketplaces.get.ts
- Fix import order in marketplaces.get.ts
- Replace process.env with import.meta.env in github.ts for Nuxt compatibility
- Add missing newline at end of github.ts file
@vercel
Copy link
Copy Markdown

vercel Bot commented Oct 16, 2025

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

Project Deployment Preview Comments Updated (UTC)
claude-code-plugins Ready Ready Preview Comment Oct 16, 2025 2:27pm

@amondnet amondnet merged commit 1b87031 into main Oct 16, 2025
5 checks passed
@amondnet amondnet deleted the fix/vercel-build-and-lint-errors branch October 16, 2025 14:30
@amondnet amondnet self-assigned this Oct 16, 2025
@pleaseai pleaseai Bot mentioned this pull request Oct 16, 2025
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