From 59c99dc62d7f1c2ba5fbe40161dcde900ee867b2 Mon Sep 17 00:00:00 2001 From: Kit Langton Date: Tue, 19 May 2026 17:45:55 -0400 Subject: [PATCH] Skip snapshot seed file commits (#28405) --- packages/opencode/test/snapshot/snapshot.test.ts | 2 -- perf/test-suite.md | 1 + 2 files changed, 1 insertion(+), 2 deletions(-) diff --git a/packages/opencode/test/snapshot/snapshot.test.ts b/packages/opencode/test/snapshot/snapshot.test.ts index 40ce18faaf..8b42191952 100644 --- a/packages/opencode/test/snapshot/snapshot.test.ts +++ b/packages/opencode/test/snapshot/snapshot.test.ts @@ -44,8 +44,6 @@ const initialize = Effect.fn("SnapshotTest.initialize")(function* (dir: string) const bContent = `B${unique}` yield* write(`${dir}/a.txt`, aContent) yield* write(`${dir}/b.txt`, bContent) - yield* exec(dir, ["git", "add", "."]) - yield* exec(dir, ["git", "commit", "-m", "init"]) return { aContent, bContent } }) diff --git a/perf/test-suite.md b/perf/test-suite.md index ba8fc8424c..02a36560c9 100644 --- a/perf/test-suite.md +++ b/perf/test-suite.md @@ -85,6 +85,7 @@ Repeated setup work, long sleeps/timeouts, serial integration tests, filesystem/ | Oversized snapshot batch tests only need to cross the 100-file boundary | Reduced large diff/revert fixture sizes while keeping each case above the batch boundary | 4.32s | 3.66s | keep | Three affected snapshot tests; after median from three reruns (4.32, 3.66, 3.66) while still crossing the 100-file boundary. | | Prompt tests without LLM calls do not need the test LLM server | Added a no-server runner and moved obvious non-LLM prompt/shell cases to it | 25.41s | 21.03s | keep | Full prompt file after simplify pass median from three reruns (20.66, 21.03, 21.64); LLM-backed tests stay on original runner. | | CLI run subprocess cases can run independently | Marked `run-process.test.ts` subprocess cases concurrent | 11.87s | 4.13s | keep | Newest-dev single baseline; after median from three reruns (4.13, 4.17, 4.11). Each case has an isolated temp home and LLM port. | +| Snapshot initialization does not need to commit seeded files in the source repo | Removed extra `git add`/`commit` from the snapshot test `initialize()` helper | 22.22s | 20.23s | keep | Newest-dev single baseline; after median from three reruns (20.23, 22.59, 20.11). Fixture still creates a git repo root commit. | ## Profiling Results