chore: remove unused API_TOKEN env var#631
Conversation
API_TOKEN was declared in .env.example and Dockerfile but never referenced anywhere in the source code. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
There was a problem hiding this comment.
Pull request overview
Removes the unused API_TOKEN environment variable from the repository’s configuration scaffolding (example env file and Docker build-time args), keeping env surface area aligned with what the app actually consumes.
Changes:
- Removed
API_TOKENfromDockerfilebuild ARG list and.envgeneration step. - Removed
API_TOKENfrom.env.example.
Reviewed changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.
| File | Description |
|---|---|
| Dockerfile | Drops API_TOKEN ARG and stops writing it into the generated .env during image build. |
| .env.example | Removes the unused API_TOKEN entry from the example environment file. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| RUN echo "CHANGEFEED_ACCESS_TOKEN=${CHANGEFEED_ACCESS_TOKEN}" >> .env && \ | ||
| echo "GITHUB_ACCESS_TOKEN=${GITHUB_ACCESS_TOKEN}" >> .env && \ | ||
| echo "LIVEPEER_COM_API_ADMIN_TOKEN=${LIVEPEER_COM_API_ADMIN_TOKEN}" >> .env && \ | ||
| echo "API_TOKEN=${API_TOKEN}" >> .env && \ | ||
| echo "PINATA_JWT=${PINATA_JWT}" >> .env && \ |
There was a problem hiding this comment.
In the .env generation RUN instruction, the last two lines are malformed: the echo "NEXT_PUBLIC_METRICS_SERVER_URL=..." line is missing && \ before the final echo, and the final variable expansion uses a typo (NEXT_PUBLI_AI_METRICS_SERVER_URL) instead of the declared NEXT_PUBLIC_AI_METRICS_SERVER_URL. As written, this will write an incorrect .env line (or behave unexpectedly) and will not propagate the AI metrics URL correctly. Please add the missing command separator and fix the variable name so the final entry is written as its own line.
There was a problem hiding this comment.
Looked at in follow up.
Summary
API_TOKENfrom .env.example and Dockerfile — it was declared but never referenced in source.Test plan
🤖 Generated with Claude Code