Skip to content

Harden production and improve performance - #1

Merged
1ove9 merged 11 commits into
mainfrom
agent/site-hardening-performance-evidence
Jul 29, 2026
Merged

Harden production and improve performance#1
1ove9 merged 11 commits into
mainfrom
agent/site-hardening-performance-evidence

Conversation

@1ove9

@1ove9 1ove9 commented Jul 29, 2026

Copy link
Copy Markdown
Owner

What changed

  • Hardened production behavior: upgraded Next.js, removed build-time font downloads, split Sanity read/write tokens, added contact-form abuse protection, repaired signed webhook revalidation, sitemap, environment handling, broken links, and enforced CI builds.
  • Reduced client-side work: narrowed Client Component and i18n message boundaries, removed duplicate GSAP/Motion animation dependencies, and aligned Sanity caching with on-demand revalidation.
  • Strengthened evidence and conversion: added DOI/publication and source-linked case-study fields, filtered homepage evidence to verifiable records, and improved mobile content rails.
  • Added Playwright smoke/E2E coverage for localized routes, internal links, mobile overflow, sitemap, and webhook rejection.

Why

The site needed a production-safe baseline before performance refactoring, plus stronger technical credibility and clearer conversion paths.

Validation

  • npm run build
  • npm run typecheck
  • npm run lint
  • Playwright smoke/E2E: 7 passed
  • Git diff and remote Git tree hashes verified

@vercel

vercel Bot commented Jul 29, 2026

Copy link
Copy Markdown

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

Project Deployment Actions Updated (UTC)
source-sequence Ready Ready Preview, Comment Jul 29, 2026 5:52pm
v0-sourcesequence Ready Ready Preview, Comment, Open in v0 Jul 29, 2026 5:52pm

@1ove9
1ove9 marked this pull request as ready for review July 29, 2026 17:52
@1ove9
1ove9 merged commit e7922e0 into main Jul 29, 2026
5 checks passed

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

💡 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".

Comment thread tests/e2e/smoke.spec.ts
"/en/solutions",
"/en/antenna",
"/en/models",
"/api/revalidate",

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

P1 Badge 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

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

P2 Badge 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 👍 / 👎.

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