Harden production and improve performance - #1
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: 718dd16db3
ℹ️ 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".
| "/en/solutions", | ||
| "/en/antenna", | ||
| "/en/models", | ||
| "/api/revalidate", |
There was a problem hiding this comment.
Remove the POST-only route from the GET health check
The updated .github/workflows/ci.yml unconditionally runs this smoke suite, but /api/revalidate exports only a POST handler, so this request.get() receives HTTP 405 and response.ok() is false on every CI run. Test the route through its signed POST contract or omit it from this GET-only loop.
Useful? React with 👍 / 👎.
|
|
||
| const projectId = process.env.NEXT_PUBLIC_SANITY_PROJECT_ID | ||
| const dataset = process.env.NEXT_PUBLIC_SANITY_DATASET | ||
| const token = process.env.SANITY_API_WRITE_TOKEN |
There was a problem hiding this comment.
Use a separate privileged credential for content seeding
When operators follow .env.example/README and make SANITY_API_WRITE_TOKEN narrowly scoped to creating and querying inquiry documents, this seed fails authorization because it uses that token for createOrReplace calls on capabilities, solution groups, solutions, and case studies. The same credential was also wired into the migration/model-seeding scripts, so content administration needs a separate privileged token rather than the public-form credential.
Useful? React with 👍 / 👎.
What changed
Why
The site needed a production-safe baseline before performance refactoring, plus stronger technical credibility and clearer conversion paths.
Validation
npm run buildnpm run typechecknpm run lint