Greptile Summary

This PR replaces the Unix-only | xargs oxfmt --check pipeline in format:docs:check with a small Node.js script (scripts/check-docs-format.mjs) that uses git ls-files and chunks the file list to stay within Windows command-line limits. The overall approach is sound and the design is clean.

Key findings:

• Critical: spawn without shell: true breaks on Windows — oxfmt is an npm dependency ("oxfmt": "0.41.0"). npm installs it as a .cmd shim on Windows (node_modules/.bin/oxfmt.cmd). Node.js spawn without { shell: true } does not resolve .cmd extensions, so the script will throw ENOENT on Windows and fail to run at all — the exact problem this PR is meant to fix.
• format:docs (write variant) still uses xargs — the sibling script that writes formatted output was not updated. This may be intentional, but if Windows support is the goal it is worth addressing consistently.

Confidence Score: 2/5

• The PR has a critical bug that prevents it from working on the very platform it targets; do not merge until the spawn shell issue is resolved.
• The approach is conceptually correct and the chunking logic is well-implemented, but spawn("oxfmt", ...) without shell: true will throw ENOENT on Windows because npm-installed CLIs are .cmd shims. This directly breaks the primary cross-platform goal of the PR.
• scripts/check-docs-format.mjs — specifically the spawn call in runOxfmt needs shell: process.platform === "win32" (or use cross-spawn).

This is a comment left during a code review.
Path: scripts/check-docs-format.mjs
Line: 27-29

Comment:
**`spawn` without `shell: true` breaks on Windows for npm-installed CLIs**

`oxfmt` is an npm dependency (`"oxfmt": "0.41.0"` in `package.json`). On Windows, npm puts a `.cmd` shim into `node_modules/.bin/` (i.e., `oxfmt.cmd`), not a bare `oxfmt` executable. Node.js `spawn` without `shell: true` does **not** resolve `.cmd` extensions, so this call will throw an `ENOENT` error on Windows — the very platform this PR is intended to support.

The simplest fix is to enable the shell only on Windows:

```suggestion
    const child = spawn("oxfmt", ["--check", ...files], {
      stdio: "inherit",
      shell: process.platform === "win32",
    });
```

Alternatively, use the well-known `cross-spawn` package, which handles this transparently on all platforms.

How can I resolve this? If you propose a fix, please make it concise.

---

This is a comment left during a code review.
Path: package.json
Line: 542

Comment:
**`format:docs` (write variant) still uses `xargs`**

This PR fixes `format:docs:check` for Windows but the sibling `format:docs` script on this line still uses the Unix-only `| xargs` pipeline. If Windows support is the goal, the write-variant may need the same treatment (e.g., a counterpart `write-docs-format.mjs`, or a shared script that accepts `--write` vs `--check`). Consider whether this is intentional or an oversight.

How can I resolve this? If you propose a fix, please make it concise.

_{Last reviewed commit: "docs: make format ch..."}

Heads up: latest CI run fails before lint/tests because pnpm install can't build @tloncorp/[email protected]. The package's prepare -> npm run build step shells out to vite build (vite isn't bundled with the tarball) and then hits readonly ParameterSpec type errors, so every PR against main is blocked right now. I reproduced the same failure locally with pnpm install --force. I'm preparing a repo-level workaround while we wait on an upstream fix.