Skip to content
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
3 changes: 2 additions & 1 deletion CLAUDE.md
Original file line number Diff line number Diff line change
Expand Up @@ -5,6 +5,7 @@
```bash
npm clean-install
npm run build # Bundles src/index.ts → dist/index.js via esbuild
npm test # Builds, then runs action.test.mjs with vitest — needs ghcup on PATH
npm run format # Format with js-beautify — enforced in CI, run before committing
```

Expand All @@ -26,7 +27,7 @@ Suggestions are welcome across all layers:

## Testing

No unit test framework, and none should be added. All tests are integration tests that run the action end-to-end using fixture files in `examples/`. To cover new behaviour, add a fixture under `examples/` and a corresponding step in `test.yaml`, following the existing pattern.
Tests run with vitest. All tests are integration tests that run the action end-to-end: each one starts `dist/index.js` as its own process against a fixture file in `examples/`, which is why `npm test` builds first. To cover new behaviour, add a fixture under `examples/` and a corresponding `test` in `action.test.mjs`, following the existing pattern.

## Ideas Worth Considering

Expand Down
2 changes: 1 addition & 1 deletion test.mjs → action.test.mjs
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
import { test } from "node:test";
import { test } from "vitest";
import assert from "node:assert/strict";
import { spawnSync } from "node:child_process";
import { mkdtempSync, readFileSync, writeFileSync, rmSync } from "node:fs";
Expand Down
Loading
Loading