Skip snapshot seed file commits (#28405)

This commit is contained in:
Kit Langton
2026-05-19 17:45:55 -04:00
committed by GitHub
parent a8f7c5ec93
commit 59c99dc62d
2 changed files with 1 additions and 2 deletions

View File

@@ -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 }
})

View File

@@ -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