test: run repository-mutating tests against per-test isolated repositories

[angeloashmore]

Resolves:

Description

Removes the serial test project. Tests that mutate whole-repository state (locales, push, sync, status) opt into a throwaway repository per test, so the entire suite runs concurrently:

it("supports --help", async ({ expect, prismic }) => {
	// Runs against the shared repository. No extra repository is created.
});

describe("with an isolated repository", () => {
	it.scoped({ isolateRepo: true });

	it("adds a locale", async ({ expect, prismic, repo }) => {
		// `repo` is a fresh repository unique to this test, deleted afterwards.
	});
});

Checklist

• A comprehensive Linear ticket, providing sufficient context and details to facilitate the review of the PR, is linked to the PR.
• If my changes require tests, I added them.
• If my changes affect backward compatibility, it has been discussed.
• If my changes require an update to the CONTRIBUTING.md guide, I updated it.

Preview

How to QA ¹

Run node --run test. All 83 files (288 tests) run concurrently; there are no *.serial.test.ts files left.

🤖 Generated with Claude Code

---

Note

Low Risk
Changes are almost entirely test and Vitest configuration; runtime CLI behavior is unchanged aside from a whitespace-only edit in push.ts.

Overview
Replaces serial e2e with per-test isolated Prismic repositories so the whole suite can run concurrently. Vitest’s separate serial project and all *.serial.test.ts files are removed; a single config enables sequence.concurrent for every *.test.ts.

The shared test harness gains an isolateRepo fixture (via it.scoped({ isolateRepo: true }) in nested describe blocks). When enabled, repo is a freshly created repository with en-us as master and is deleted after the test; otherwise tests keep using the injected shared repo. Locale, push, status, and sync mutation tests are wrapped in this pattern and no longer call resetLocales or other shared-repo teardown helpers.

Test helpers removeLocale, resetLocales, and deleteAccessToken are removed from test/prismic.ts as unused. locale-set-master keeps the “already master” case on the shared repo; only the mutating case uses isolation. init / repo-create tests take onTestFinished from the Vitest callback instead of a top-level import.

Production change is limited to formatting of a console.warn in push when screenshot deletion fails.

^{Reviewed by Cursor Bugbot for commit 709ed5a. Bugbot is set up for automated code reviews on this repo. Configure here.}

Footnotes

1. Please use these labels when submitting a review:
   ❓ #ask: Ask a question.
   💡 #idea: Suggest an idea.
   ⚠️ #issue: Strongly suggest a change.
   🎉 #nice: Share a compliment. ↩

[cursor]

Cursor Bugbot has reviewed your changes using default effort and found 1 potential issue.

^{❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.}

Want higher recall? High effort reviews run extra passes and find more bugs. A team admin can switch effort levels in the Cursor dashboard.

^{Reviewed by Cursor Bugbot for commit 9fbb2a0. Configure here.}

[lihbr]

Looks nice to me :)