Skip to content

Revert #390 (ja build/automation and build/interface) so search fits the Worker again - #392

Merged
zhuangjianguo merged 1 commit into
mainfrom
claude/project-thread-9aiaji
Oct 29, 2026
Merged

zhuangjianguo merged 1 commit into
mainfrom
claude/project-thread-9aiaji

Conversation

@zhuangjianguo

Copy link
Copy Markdown
Collaborator

Reverts #390 (PR 2 of 7 for #388, 11 ja pages under build/automation/ and build/interface/) so that main can deploy again.

Why

After #390 merged as 3bbc710, the post-deploy smoke check failed on main CI run 40862958838, on the first attempt and again on a re-run. Both times the deploy rolled back on its own, so docs.objectos.ai is still serving the #389 version. The failure card is #391.

The search route (apps/docs/app/api/search/route.ts) keeps one Orama index per locale, built on that locale's first search, in a module-level map that is never emptied. Once an isolate has built several locales' indexes, it hits the Worker's resource limit, and #390's extra ja content was enough to tip it over. Until search builds less per isolate, main cannot deploy, and that blocks every docs PR, not only the ja ones.

Then

  1. File and fix the search card: bound the per-isolate index cache.
  2. Re-land these 11 pages unchanged.
  3. Continue Other locales, phase 2: ja to full coverage (the remaining 79 pages, seven serial PRs) #388 with PR 3.

The seat reviewed these pages, and they pass every translation check (#390's ACCEPT stands). They are reverted only for the deploy.


Generated by Claude Code

…into ja (#390)"

This reverts commit 3bbc710.

After #390, the post-deploy smoke check failed twice, on the first run and on a re-run.
/api/search answered 503 (Cloudflare's HTML error page, after 9–12 s) in two or three
locales per run, once several locale indexes had been built in one Worker isolate. The
deploy rolled back both times. Before #390, the same check passed with each search
taking 3.6–5.3 s. The extra ja pages push search over the Worker's resource limit, so
main cannot deploy until search builds less per isolate. The pages come back after that fix.

Co-Authored-By: Claude <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01QsR2jTaZ2GeyfjANMwstiw
@zhuangjianguo
zhuangjianguo marked this pull request as ready for review October 29, 2026 11:21
@zhuangjianguo
zhuangjianguo added this pull request to the merge queue Oct 29, 2026
Merged via the queue into main with commit 09784e7 Oct 29, 2026
4 checks passed
zhuangjianguo added a commit that referenced this pull request Oct 29, 2026
…388, PR 2 of 7)

Reverts #392, which reverted #390 because /api/search ran out of Worker
memory. #394 (#393) fixed that, so the same 11 ja pages come back
unchanged.

Refs #388

Claude-Session: https://claude.ai/code/session_01QsR2jRsjbApbf83iyTDDXm

Co-Authored-By: Claude <noreply@anthropic.com>
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.

2 participants