mirror of
https://github.com/paperclipai/paperclip.git
synced 2026-11-12 22:56:58 +01:00
## Thinking Path
> - Paperclip is the open source app people use to manage AI agents for
work.
> - The board UI loads each page as a lazy chunk. After a deploy, an
open tab can ask for a chunk from its old build.
> - #17349 added a recovery: the app reloads the page one time when a
chunk fails to load. A loop guard permits only one reload in five
minutes.
> - Staging error monitoring still reports `Failed to fetch dynamically
imported module` from builds that include #17349. The events come 1–10
seconds after page load, and 1–10 minutes after page load.
> - During a deploy, the old server and the new server can both answer
for some seconds. The reloaded page can then miss a chunk again. Also, a
second deploy can occur inside the five-minute window. In both cases the
guard stops the recovery, and the user sees the error page. One more
reload a few seconds later would recover.
> - #18518 keeps old chunks available after a deploy, but only on
deployments that enable shared UI assets. Other deployments still depend
on the reload recovery.
> - This pull request permits up to three reloads in the guard window.
The reloads wait 0 s, 2 s and 5 s, so that the deploy can settle. After
the last reload, the error page shows and reports as before.
> - The benefit is that a stale tab recovers through a normal deploy
without an error page and without error-monitoring noise. A chunk that
stays broken still shows the error after a small, fixed number of
reloads.
## Linked Issues or Issue Description
Refs #17349
Refs #18518
**What happened?**
A browser tab stayed open during a staging deploy. A lazy page chunk
failed to load with `TypeError: Failed to fetch dynamically imported
module`. The route error boundary showed "This page hit an error" and
reported the error. The release included the one-time reload from
#17349. The reload did not occur because the loop guard had already used
its single reload in the last five minutes.
**Expected behavior**
The app reloads again after a short wait while the deploy settles, and
the page loads. The error page shows only when the chunk stays broken.
**Steps to reproduce**
1. Open a board page on a deployment without shared UI assets.
2. Deploy a new build. While the old and new servers both answer, go to
a page that was not loaded yet.
3. The chunk fails, and the page reloads one time.
4. The reloaded page gets its HTML from one server and a chunk request
goes to the other server. The chunk fails again.
5. The guard stops the recovery, and the route error page shows.
**Paperclip version or commit**
`master` at 5ac05871b5
## What Changed
- `ui/src/lib/chunk-load-recovery.ts`: The recovery permits up to three
reloads in the five-minute guard window. The reloads wait 0 s, 2 s and 5
s (`RELOAD_DELAYS_MS`). After the third reload, the recovery returns
`false`, and the boundary shows and reports the error as before.
- Storage stays compatible with older builds.
`paperclip:chunk-load-reload-at` still holds only the bare start time of
the window, so an older build keeps its one-reload guard during a
deploy. The retry count is in a separate key,
`paperclip:chunk-load-reload-attempts` (`<window start>:<attempts>`).
The count applies only to the matching window. A window that an older
build started counts as one reload.
- A delayed retry keeps its timer. The new
`cancelPendingChunkLoadReload()` stops a retry that has not started.
`RouteErrorBoundary` calls it when the user moves to another route, and
when the boundary unmounts. A working page is therefore not reloaded
later.
- The abandonment timer starts after the delay, so a delayed reload gets
the full 10 s before the boundary reports.
- A reload that does not leave the document (for example, the user
cancels it) is not tried again from the same document. This keeps the
behavior from #17349.
- Tests: unit tests for delayed retries, the retry limit, compatibility
with older builds, a stale retry count, and cancellation. Boundary tests
start from a delayed attempt and cancel it on navigation. The e2e spec
for a lazy CSS asset that stays broken now expects 4 page loads before
the error page.
## Verification
- `cd ui && npx vitest run src/lib/chunk-load-recovery.test.ts
src/components/RouteErrorBoundary.test.tsx
src/components/AppErrorBoundary.test.tsx`: 3 files, 32 tests pass.
- Red before the fix: with the new tests, the first PR head fails 5
tests (older-build key, cancellation, cancel on navigation). The
original `master` code fails the retry tests. With this head, all pass.
- `tsc --noEmit -p ui/tsconfig.json` shows no errors in the changed
files.
- `tests/e2e/chunk-load-recovery.spec.ts` runs in CI.
- Manual check: on a deployment without shared UI assets, open a page,
deploy, and go to a page that was not loaded. The page reloads, and it
loads after the deploy settles. If you move to another page during the
wait, the page does not reload.
## Risks
- Low risk. The change affects only the chunk-load recovery path.
- A chunk that stays broken now causes up to three reloads (about 7 s of
waits in total) before the error page shows. Before, it caused one
reload. The limit stays fixed in each five-minute window, so a reload
loop cannot occur, also when old and new builds alternate.
- A retry that the `vite:preloadError` listener starts is not tied to a
route, so navigation does not cancel it. This matches the behavior
before this change, where that reload occurred at once.
- No server, data, or contract changes.
## Model Used
- Claude Opus 5.5 (`claude-opus-5-5`) by Anthropic, used through Claude
Code with tool use (shell, file edits and GitHub connector). Standard
context window, with reasoning.
## Checklist
- [x] I have included a thinking path that traces from project context
to this change
- [x] I have specified the model used (with version and capability
details)
- [x] I have checked ROADMAP.md and confirmed this PR does not duplicate
planned core work
- [x] I have searched GitHub for duplicate or related PRs and linked
them above
- [x] I have either (a) linked existing issues with `Fixes: #` / `Closes
#` / `Refs #` OR (b) described the issue in-PR following the relevant
issue template
- [x] I have not referenced internal/instance-local Paperclip issues or
links (only public GitHub `#NNN` / `github.com/paperclipai/paperclip`
URLs)
- [x] My branch name describes the change (e.g. `docs/...`, `fix/...`)
and contains no internal Paperclip ticket id or instance-derived details
- [x] I have run tests locally and they pass
- [x] I have added or updated tests where applicable
- [x] I have updated relevant documentation to reflect my changes
- [x] I have considered and documented any risks above
- [ ] All Paperclip CI gates are green
- [ ] Greptile is 5/5 with no open P2s, recommendations, or follow-ups
- [x] I will address all Greptile and reviewer comments before
requesting merge
🤖 Generated with [Claude Code](https://claude.com/claude-code)
---------
Co-authored-by: Claude Opus 5.5 <noreply@anthropic.com>
242 lines
10 KiB
TypeScript
242 lines
10 KiB
TypeScript
import { afterEach, describe, expect, it, vi } from "vitest";
|
|
import { createChunkLoadRecovery, isChunkLoadError, RELOAD_ABANDONED_MS, RELOAD_DELAYS_MS } from "./chunk-load-recovery";
|
|
|
|
function memoryStorage(initial: Record<string, string> = {}) {
|
|
const values = new Map(Object.entries(initial));
|
|
return {
|
|
getItem: (key: string) => values.get(key) ?? null,
|
|
setItem: (key: string, value: string) => { values.set(key, value); },
|
|
};
|
|
}
|
|
|
|
const cssPreloadError = new Error(
|
|
"Unable to preload CSS for https://paperclip.example/assets/issue-detail-CIbQo9zW.css",
|
|
);
|
|
|
|
function preloadErrorEvent(payload: unknown) {
|
|
return Object.assign(new Event("vite:preloadError", { cancelable: true }), { payload });
|
|
}
|
|
|
|
describe("isChunkLoadError", () => {
|
|
it("matches Vite CSS preload and browser dynamic import failures", () => {
|
|
expect(isChunkLoadError(cssPreloadError)).toBe(true);
|
|
expect(isChunkLoadError(new TypeError("Failed to fetch dynamically imported module: https://paperclip.example/assets/a.js"))).toBe(true);
|
|
expect(isChunkLoadError(new TypeError("error loading dynamically imported module: https://paperclip.example/assets/a.js"))).toBe(true);
|
|
expect(isChunkLoadError(new TypeError("Importing a module script failed."))).toBe(true);
|
|
});
|
|
|
|
it("ignores ordinary render errors and non-errors", () => {
|
|
expect(isChunkLoadError(new Error("Maximum update depth exceeded"))).toBe(false);
|
|
expect(isChunkLoadError(new Error("Request failed: Unable to preload CSS for x"))).toBe(false);
|
|
expect(isChunkLoadError("Unable to preload CSS for x")).toBe(false);
|
|
expect(isChunkLoadError(undefined)).toBe(false);
|
|
});
|
|
});
|
|
|
|
describe("reloadForChunkLoadError", () => {
|
|
afterEach(() => {
|
|
vi.useRealTimers();
|
|
});
|
|
|
|
it("reloads once and treats later failures in the same document as that reload", () => {
|
|
const reload = vi.fn();
|
|
const { reloadForChunkLoadError } = createChunkLoadRecovery({ storage: () => memoryStorage(), reload });
|
|
|
|
expect(reloadForChunkLoadError(cssPreloadError)).toBe(true);
|
|
expect(reloadForChunkLoadError(cssPreloadError)).toBe(true);
|
|
expect(reload).toHaveBeenCalledTimes(1);
|
|
});
|
|
|
|
it("retries a fresh page that fails again, waiting longer each time for the deploy to settle", () => {
|
|
vi.useFakeTimers();
|
|
const storage = memoryStorage();
|
|
const reload = vi.fn();
|
|
let now = 1_000_000;
|
|
createChunkLoadRecovery({ storage: () => storage, reload, now: () => now }).reloadForChunkLoadError(cssPreloadError);
|
|
expect(reload).toHaveBeenCalledTimes(1);
|
|
|
|
for (const delayMs of RELOAD_DELAYS_MS.slice(1)) {
|
|
now += 10_000;
|
|
const freshPage = createChunkLoadRecovery({ storage: () => storage, reload, now: () => now });
|
|
const reloadsBefore = reload.mock.calls.length;
|
|
expect(freshPage.reloadForChunkLoadError(cssPreloadError)).toBe(true);
|
|
vi.advanceTimersByTime(delayMs - 1);
|
|
expect(reload).toHaveBeenCalledTimes(reloadsBefore);
|
|
vi.advanceTimersByTime(1);
|
|
expect(reload).toHaveBeenCalledTimes(reloadsBefore + 1);
|
|
}
|
|
});
|
|
|
|
it("defers to the error UI once the retries in the guard window are used up", () => {
|
|
const storage = memoryStorage();
|
|
const reload = vi.fn();
|
|
let now = 1_000_000;
|
|
for (let attempt = 0; attempt < RELOAD_DELAYS_MS.length; attempt += 1) {
|
|
expect(createChunkLoadRecovery({ storage: () => storage, reload, now: () => now }).reloadForChunkLoadError(cssPreloadError)).toBe(true);
|
|
now += 10_000;
|
|
}
|
|
|
|
const exhaustedPage = createChunkLoadRecovery({ storage: () => storage, reload, now: () => now });
|
|
expect(exhaustedPage.reloadForChunkLoadError(cssPreloadError)).toBe(false);
|
|
});
|
|
|
|
it("counts a reload recorded by an older build as the first retry", () => {
|
|
vi.useFakeTimers();
|
|
const reload = vi.fn();
|
|
const storage = memoryStorage({ "paperclip:chunk-load-reload-at": "1000000" });
|
|
const { reloadForChunkLoadError } = createChunkLoadRecovery({ storage: () => storage, reload, now: () => 1_010_000 });
|
|
|
|
expect(reloadForChunkLoadError(cssPreloadError)).toBe(true);
|
|
expect(reload).not.toHaveBeenCalled();
|
|
vi.advanceTimersByTime(RELOAD_DELAYS_MS[1]!);
|
|
expect(reload).toHaveBeenCalledTimes(1);
|
|
});
|
|
|
|
it("keeps the stored reload time readable by builds without retries", () => {
|
|
const storage = memoryStorage();
|
|
const reload = vi.fn();
|
|
let now = 1_000_000;
|
|
createChunkLoadRecovery({ storage: () => storage, reload, now: () => now }).reloadForChunkLoadError(cssPreloadError);
|
|
now += 10_000;
|
|
createChunkLoadRecovery({ storage: () => storage, reload, now: () => now }).reloadForChunkLoadError(cssPreloadError);
|
|
|
|
// An older build reads this key with Number() and keeps its one-reload guard.
|
|
expect(Number(storage.getItem("paperclip:chunk-load-reload-at"))).toBe(1_000_000);
|
|
});
|
|
|
|
it("ignores a retry count left from another guard window", () => {
|
|
vi.useFakeTimers();
|
|
const reload = vi.fn();
|
|
const storage = memoryStorage({
|
|
"paperclip:chunk-load-reload-at": "2000000",
|
|
"paperclip:chunk-load-reload-attempts": `1000000:${RELOAD_DELAYS_MS.length}`,
|
|
});
|
|
const { reloadForChunkLoadError } = createChunkLoadRecovery({ storage: () => storage, reload, now: () => 2_010_000 });
|
|
|
|
expect(reloadForChunkLoadError(cssPreloadError)).toBe(true);
|
|
vi.advanceTimersByTime(RELOAD_DELAYS_MS[1]!);
|
|
expect(reload).toHaveBeenCalledTimes(1);
|
|
});
|
|
|
|
it("cancels a delayed retry that has not started", () => {
|
|
vi.useFakeTimers();
|
|
const reload = vi.fn();
|
|
const storage = memoryStorage({ "paperclip:chunk-load-reload-at": "1000000" });
|
|
const recovery = createChunkLoadRecovery({ storage: () => storage, reload, now: () => 1_010_000 });
|
|
expect(recovery.reloadForChunkLoadError(cssPreloadError)).toBe(true);
|
|
const abandoned = vi.fn();
|
|
recovery.onChunkLoadReloadAbandoned(abandoned);
|
|
|
|
expect(recovery.cancelPendingChunkLoadReload()).toBe(true);
|
|
vi.advanceTimersByTime(RELOAD_DELAYS_MS[1]! + RELOAD_ABANDONED_MS);
|
|
expect(reload).not.toHaveBeenCalled();
|
|
expect(abandoned).not.toHaveBeenCalled();
|
|
expect(recovery.cancelPendingChunkLoadReload()).toBe(false);
|
|
});
|
|
|
|
it("cannot cancel a reload that already started", () => {
|
|
const reload = vi.fn();
|
|
const recovery = createChunkLoadRecovery({ storage: () => memoryStorage(), reload });
|
|
expect(recovery.reloadForChunkLoadError(cssPreloadError)).toBe(true);
|
|
expect(recovery.cancelPendingChunkLoadReload()).toBe(false);
|
|
expect(reload).toHaveBeenCalledTimes(1);
|
|
});
|
|
|
|
it("notifies waiting callers and stops claiming the failure when the reload is cancelled", () => {
|
|
vi.useFakeTimers();
|
|
const storage = memoryStorage();
|
|
const reload = vi.fn();
|
|
const { reloadForChunkLoadError, onChunkLoadReloadAbandoned } = createChunkLoadRecovery({
|
|
storage: () => storage,
|
|
reload,
|
|
});
|
|
expect(reloadForChunkLoadError(cssPreloadError)).toBe(true);
|
|
const abandoned = vi.fn();
|
|
const stopped = vi.fn();
|
|
onChunkLoadReloadAbandoned(abandoned);
|
|
onChunkLoadReloadAbandoned(stopped)();
|
|
|
|
vi.advanceTimersByTime(RELOAD_ABANDONED_MS - 1);
|
|
expect(abandoned).not.toHaveBeenCalled();
|
|
expect(reloadForChunkLoadError(cssPreloadError)).toBe(true);
|
|
|
|
vi.advanceTimersByTime(1);
|
|
expect(abandoned).toHaveBeenCalledTimes(1);
|
|
expect(stopped).not.toHaveBeenCalled();
|
|
expect(reloadForChunkLoadError(cssPreloadError)).toBe(false);
|
|
expect(reload).toHaveBeenCalledTimes(1);
|
|
|
|
const late = vi.fn();
|
|
onChunkLoadReloadAbandoned(late);
|
|
expect(late).toHaveBeenCalledTimes(1);
|
|
});
|
|
|
|
it("recovers again after the guard window, such as after a later deploy", () => {
|
|
const storage = memoryStorage();
|
|
const reload = vi.fn();
|
|
let now = 1_000_000;
|
|
createChunkLoadRecovery({ storage: () => storage, reload, now: () => now }).reloadForChunkLoadError(cssPreloadError);
|
|
|
|
now += 5 * 60 * 1000;
|
|
const laterPage = createChunkLoadRecovery({ storage: () => storage, reload, now: () => now });
|
|
expect(laterPage.reloadForChunkLoadError(cssPreloadError)).toBe(true);
|
|
expect(reload).toHaveBeenCalledTimes(2);
|
|
});
|
|
|
|
it("ignores an unreadable stored marker", () => {
|
|
for (const marker of ["not-a-time", "{\"windowStartedAt\":\"soon\"}", "null"]) {
|
|
const reload = vi.fn();
|
|
const storage = memoryStorage({ "paperclip:chunk-load-reload-at": marker });
|
|
const { reloadForChunkLoadError } = createChunkLoadRecovery({ storage: () => storage, reload, now: () => 1_000 });
|
|
expect(reloadForChunkLoadError(cssPreloadError)).toBe(true);
|
|
expect(reload).toHaveBeenCalledTimes(1);
|
|
}
|
|
});
|
|
|
|
it("never reloads for other errors", () => {
|
|
const reload = vi.fn();
|
|
const { reloadForChunkLoadError } = createChunkLoadRecovery({ storage: () => memoryStorage(), reload });
|
|
expect(reloadForChunkLoadError(new Error("boom"))).toBe(false);
|
|
expect(reload).not.toHaveBeenCalled();
|
|
});
|
|
|
|
it("never reloads without a working loop guard", () => {
|
|
const reload = vi.fn();
|
|
expect(createChunkLoadRecovery({ storage: () => null, reload }).reloadForChunkLoadError(cssPreloadError)).toBe(false);
|
|
const throwing = {
|
|
getItem: () => { throw new Error("SecurityError"); },
|
|
setItem: () => { throw new Error("SecurityError"); },
|
|
};
|
|
expect(createChunkLoadRecovery({ storage: () => throwing, reload }).reloadForChunkLoadError(cssPreloadError)).toBe(false);
|
|
expect(reload).not.toHaveBeenCalled();
|
|
});
|
|
});
|
|
|
|
describe("startChunkLoadRecovery", () => {
|
|
it("reloads for a Vite preload failure without cancelling the rejection", () => {
|
|
const target = new EventTarget();
|
|
const reload = vi.fn();
|
|
const { startChunkLoadRecovery } = createChunkLoadRecovery({ storage: () => memoryStorage(), reload });
|
|
const stop = startChunkLoadRecovery(target as unknown as Window);
|
|
|
|
const event = preloadErrorEvent(cssPreloadError);
|
|
target.dispatchEvent(event);
|
|
expect(reload).toHaveBeenCalledTimes(1);
|
|
expect(event.defaultPrevented).toBe(false);
|
|
|
|
stop();
|
|
target.dispatchEvent(preloadErrorEvent(cssPreloadError));
|
|
expect(reload).toHaveBeenCalledTimes(1);
|
|
});
|
|
|
|
it("ignores preload events without a chunk load error", () => {
|
|
const target = new EventTarget();
|
|
const reload = vi.fn();
|
|
const { startChunkLoadRecovery } = createChunkLoadRecovery({ storage: () => memoryStorage(), reload });
|
|
startChunkLoadRecovery(target as unknown as Window);
|
|
target.dispatchEvent(preloadErrorEvent(undefined));
|
|
target.dispatchEvent(preloadErrorEvent(new Error("boom")));
|
|
expect(reload).not.toHaveBeenCalled();
|
|
});
|
|
});
|