{{ message }}
Repository navigation
fix(website): restore Cloudflare Worker after the Next.js upgrade - #5698
Merged
Merged
Conversation
Contributor
Contributor
|
Build artifacts for PR #5698: Run: https://github.com/marktext/marktext/actions/runs/38110693417 |
8 of 9 tasks
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.

Related: #5687 (dependency upgrade that introduced the regression).
Summary
Requests to marktext.me returned HTTP 500 / Cloudflare 1101 after #5687 upgraded Next.js to 16.4.0. Production Worker logs and a local reproduction both showed
Unexpected loadManifest(/.next/server/preview-props.json) call!: Next.js loads this new manifest during server initialization, but OpenNext 1.20.10 does not inline it. Builds and deployments still succeeded.Pin Next.js to 15.5.27, the latest 15.x patch and a version accepted by the current OpenNext adapter. Keep the other dependency upgrades. Use Next 15's required
jsx: preservesetting and add the CSS import declaration needed by TypeScript 6.Add a local Workers HTTP smoke test before production and preview uploads in GitHub Actions. It checks the homepage, documentation, robots.txt, sitemap.xml, and the search index, including response status, content type, and expected content. This catches the original runtime failure that lint, type checking, and builds missed.
Type of change
Test plan
cf:smokepassed.pnpm run lintpassed (existing warnings).pnpm run typecheck: blocked during Muya declaration generation by missing local dependencies (marked,marked-highlight,fuse.js, and others); only the website dependency tree was installed for this incident. Website type checking passes independently.Notes for reviewers
The fixed Worker was already deployed directly to restore service (version
adc71ba3-0809-4b03-b5ba-c49c6924aff6). Merge this PR to persist the fix in the GitHub Actions deployment source. The smoke test validates local runtime behavior; it does not check production DNS, credentials, or domain routing. Revisit the version pin when OpenNext supports the new manifest.Before:
After:
By submitting this pull request, I confirm that my contribution is made under the terms of the MIT license.