chore: harden env handling and package manager config#11
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 4b0c3f0d28
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
| @@ -1 +0,0 @@ | |||
| VITE_GA_ID=G-0XMXRJ5KWP No newline at end of file | |||
There was a problem hiding this comment.
Keep default GA ID available for pageview tracking
Removing the only committed VITE_GA_ID value makes usePageview in src/App.tsx exit early (if (!id || !(window as any).gtag) return) on any fresh clone or deployment that does not explicitly inject this env var, so page_view events stop being sent. Because index.html still hardcodes GA initialization, click events can still fire, which creates silently incomplete analytics rather than a clean disable; adding a non-secret default or documenting/validating required env setup would prevent this regression.
Useful? React with 👍 / 👎.
Summary
Validation
Notes