Format Markdown with prettier in make fmt and make fmt-check (closes #7) #11

Merged
clawbot merged 3 commits from issue-7-prettier-markdown into next 2026-10-06 12:24:53 +02:00
Collaborator

script/fmt now also runs prettier over every Markdown file, and script/fmt-check checks them, after the Go check, without writing. prettier 3.8.1 is pinned in package.json and yarn.lock. script/bootstrap keeps its Go and apt handling and gains the canonical pinned node (nvm from a hash-checked archive) and yarn (corepack) install, then yarn install --frozen-lockfile. Both fmt scripts find yarn the canonical way: the one on PATH if there is one, otherwise nvm is sourced for the pinned node. package.json, yarn.lock, .prettierrc, .prettierignore and the yarn lookup are the copies from sneak/prompts at cc440118c876.

Two commits for review, squashed on merge: the tooling, then what make fmt produced (README.md and TODO.md rewrapped, the TODO.md Workflow list switched to - markers, no wording changes). The vendored REPO_POLICIES.md already passes prettier, so it is not in .prettierignore.

The README Entrypoints lines for script/bootstrap, script/fmt and script/fmt-check, and the Participation sentence on formatting, now say what the scripts do.

The CI runner image ships its own node and corepack, so CI takes the corepack path; the nvm path is what a host without node takes.

  • Deviation: package.json drops the canonical "license": "MIT" line, since this repo is WTFPL; sneak/secret and sneak/upaas do the same.
  • Judgement call: TODO.md gains a Completed Steps entry and loses this item from Future Steps; Next Step stays #6, which is in flight.

Model: opus-5-5

`script/fmt` now also runs prettier over every Markdown file, and `script/fmt-check` checks them, after the Go check, without writing. prettier 3.8.1 is pinned in `package.json` and `yarn.lock`. `script/bootstrap` keeps its Go and apt handling and gains the canonical pinned node (nvm from a hash-checked archive) and yarn (corepack) install, then `yarn install --frozen-lockfile`. Both fmt scripts find yarn the canonical way: the one on `PATH` if there is one, otherwise nvm is sourced for the pinned node. `package.json`, `yarn.lock`, `.prettierrc`, `.prettierignore` and the yarn lookup are the copies from `sneak/prompts` at `cc440118c876`. Two commits for review, squashed on merge: the tooling, then what `make fmt` produced (`README.md` and `TODO.md` rewrapped, the `TODO.md` Workflow list switched to `-` markers, no wording changes). The vendored `REPO_POLICIES.md` already passes prettier, so it is not in `.prettierignore`. The README Entrypoints lines for `script/bootstrap`, `script/fmt` and `script/fmt-check`, and the Participation sentence on formatting, now say what the scripts do. The CI runner image ships its own node and corepack, so CI takes the corepack path; the nvm path is what a host without node takes. - Deviation: `package.json` drops the canonical `"license": "MIT"` line, since this repo is WTFPL; `sneak/secret` and `sneak/upaas` do the same. - Judgement call: `TODO.md` gains a Completed Steps entry and loses this item from Future Steps; Next Step stays https://git.eeqj.de/sneak/bsdaily/issues/6, which is in flight. Model: opus-5-5
clawbot added the needs-review label 2026-10-06 09:16:08 +02:00
clawbot self-assigned this 2026-10-06 09:16:08 +02:00
clawbot added 2 commits 2026-10-06 09:16:08 +02:00
script/fmt now also runs prettier over every Markdown file, and
script/fmt-check checks them without writing. prettier is pinned in
package.json and yarn.lock. script/bootstrap keeps its Go and apt
handling and gains the canonical pinned node (nvm from a hash-checked
archive) and yarn (corepack) install, then installs the locked
packages. Both fmt scripts find yarn the canonical way, sourcing nvm
when yarn is not on PATH. The new files and the yarn lookup come from
sneak/prompts at cc440118c876. The vendored REPO_POLICIES.md already
passes prettier, so it is not ignored. The Markdown is reformatted in
the next commit.

Deviation: package.json drops the canonical "license": "MIT" line; this
repo is WTFPL.

Model: opus-5-5
The output of make fmt after the previous commit: README.md and TODO.md
are rewrapped at 80 columns, and the TODO.md Workflow list uses -
markers. No wording changes. REPO_POLICIES.md was already formatted.

Model: opus-5-5
Author
Collaborator

Review failed.

  1. README.md (the script/bootstrap line under Entrypoints) and TODO.md (the 2026-10-06 entry under Completed Steps) say script/bootstrap installs "a pinned node and yarn". It installs them at the pinned versions only when they are absent; a node or yarn already present is used whatever its version, so in the CI runner image the run uses the image's own node 24, not the pinned 22.17.0. Acceptable: both places say node and yarn are installed at pinned versions when absent, as the header comment of script/bootstrap already does.

Judgement call: the "license": "MIT" line dropped from package.json is accepted, since this repo is WTFPL and sneak/secret and sneak/upaas do the same.

Model: opus-5-5

Review failed. 1. `README.md` (the `script/bootstrap` line under Entrypoints) and `TODO.md` (the 2026-10-06 entry under Completed Steps) say `script/bootstrap` installs "a pinned node and yarn". It installs them at the pinned versions only when they are absent; a node or yarn already present is used whatever its version, so in the CI runner image the run uses the image's own node 24, not the pinned 22.17.0. Acceptable: both places say node and yarn are installed at pinned versions when absent, as the header comment of `script/bootstrap` already does. Judgement call: the `"license": "MIT"` line dropped from `package.json` is accepted, since this repo is WTFPL and `sneak/secret` and `sneak/upaas` do the same. Model: opus-5-5
clawbot added needs-rework and removed needs-review labels 2026-10-06 10:12:34 +02:00
clawbot added 1 commit 2026-10-06 10:26:00 +02:00
The README Entrypoints line and the TODO.md Completed Steps entry said
script/bootstrap installs a pinned node and yarn. It installs them at the
pinned versions only when they are missing and uses a node or yarn already
installed whatever its version, as its header comment says.

Model: opus-5-5
Author
Collaborator

The script/bootstrap line under Entrypoints in README.md and the 2026-10-06 Completed Steps entry in TODO.md now say node and yarn are installed at pinned versions only when absent, and a node or yarn already installed is used whatever its version.

Model: opus-5-5

The `script/bootstrap` line under Entrypoints in `README.md` and the 2026-10-06 Completed Steps entry in `TODO.md` now say node and yarn are installed at pinned versions only when absent, and a node or yarn already installed is used whatever its version. Model: opus-5-5
clawbot added needs-review and removed needs-rework labels 2026-10-06 10:45:49 +02:00
Author
Collaborator

Review passed.

Model: opus-5-5

Review passed. Model: opus-5-5
clawbot merged commit 925a3896f7 into next 2026-10-06 12:24:53 +02:00
Sign in to join this conversation.
No Reviewers
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: sneak/bsdaily#11