Format Markdown with prettier in make fmt and make fmt-check #7

New Issue

2026-10-06T00:54:04+02:00

clawbot commented

2026-10-06 00:54:04 +02:00

REPO_POLICIES.md says Markdown is formatted with prettier (default settings, four-space indents, proseWrap: always). bsdaily formats only Go: script/fmt runs go fmt and script/fmt-check runs gofmt -l, so README.md and TODO.md are never checked.

Definition of done

script/fmt also formats every Markdown file with prettier, and script/fmt-check fails when one is not formatted, writing nothing.
prettier comes from package.json and yarn.lock with exact versions, installed by script/bootstrap the way the canonical sneak/prompts scripts do it (pinned node via nvm from a hash-checked archive, yarn through corepack), and script/fmt and script/fmt-check find yarn the same way the canonical ones do. .prettierrc and .prettierignore are the canonical copies.
The vendored REPO_POLICIES.md is left as fetched (it goes in .prettierignore if prettier would change it).
The reformatting of the existing Markdown is its own commit in the PR, separate from the tooling change.
make check and script/cibuild pass on a fresh clone on a host carrying only docker and git.

Starts after #1, which rewrites the same scripts.

Model: opus-5-5

`REPO_POLICIES.md` says Markdown is formatted with prettier (default settings, four-space indents, `proseWrap: always`). bsdaily formats only Go: `script/fmt` runs `go fmt` and `script/fmt-check` runs `gofmt -l`, so `README.md` and `TODO.md` are never checked. ## Definition of done 1. `script/fmt` also formats every Markdown file with prettier, and `script/fmt-check` fails when one is not formatted, writing nothing. 2. prettier comes from `package.json` and `yarn.lock` with exact versions, installed by `script/bootstrap` the way the canonical `sneak/prompts` scripts do it (pinned node via nvm from a hash-checked archive, yarn through corepack), and `script/fmt` and `script/fmt-check` find yarn the same way the canonical ones do. `.prettierrc` and `.prettierignore` are the canonical copies. 3. The vendored `REPO_POLICIES.md` is left as fetched (it goes in `.prettierignore` if prettier would change it). 4. The reformatting of the existing Markdown is its own commit in the PR, separate from the tooling change. 5. `make check` and `script/cibuild` pass on a fresh clone on a host carrying only docker and git. Starts after https://git.eeqj.de/sneak/bsdaily/issues/1, which rewrites the same scripts. Model: opus-5-5

clawbot referenced this issue

2026-10-06 00:54:16 +02:00

Bring the repo up to the standard layout: config files, Docker lint and test phases, scripts #1

clawbot referenced this issue

2026-10-06 01:14:15 +02:00

Bring the repo up to the standard layout (closes #1) #9

clawbot referenced this issue

2026-10-06 01:14:23 +02:00

Bring the repo up to the standard layout: config files, Docker lint and test phases, scripts #1

clawbot referenced this issue

2026-10-06 08:26:23 +02:00

Adopt the shared .golangci.yml and fix the code to it #6

clawbot commented

2026-10-06 08:26:23 +02:00

Plan. Canonical copies come from sneak/prompts at the fixed commit cc440118c87605583f9a5ac621fe7f5ab74342e2, the commit #9 vendored the rest from. script/bootstrap keeps its own Go and apt handling and gains the canonical pinned node and yarn install. The PR has two commits, tooling then reformatting, and is squashed on merge, so the split is for review only. It is proven where CI runs: a fresh clone passes script/cibuild inside the CI runner image. Runs alongside #6, which owns the Go code and the Dockerfile.

Model: opus-5-5

Plan. Canonical copies come from `sneak/prompts` at the fixed commit `cc440118c87605583f9a5ac621fe7f5ab74342e2`, the commit https://git.eeqj.de/sneak/bsdaily/pulls/9 vendored the rest from. `script/bootstrap` keeps its own Go and apt handling and gains the canonical pinned node and yarn install. The PR has two commits, tooling then reformatting, and is squashed on merge, so the split is for review only. It is proven where CI runs: a fresh clone passes `script/cibuild` inside the CI runner image. Runs alongside https://git.eeqj.de/sneak/bsdaily/issues/6, which owns the Go code and the `Dockerfile`. Model: opus-5-5

clawbot referenced this issue from a commit

2026-10-06 08:31:06 +02:00

Format Markdown with prettier in script/fmt and script/fmt-check (refs #7)

clawbot referenced this issue from a commit

2026-10-06 08:31:16 +02:00

Reformat the existing Markdown with prettier (closes #7)

clawbot referenced a pull request that will close this issue

2026-10-06 09:16:08 +02:00

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

clawbot commented

2026-10-06 09:16:14 +02:00

Built as #11, covering the definition of done; the PR body lists the one deviation (package.json without the canonical MIT licence line, since this repo is WTFPL) and the TODO.md judgement call.

Model: opus-5-5

Built as https://git.eeqj.de/sneak/bsdaily/pulls/11, covering the definition of done; the PR body lists the one deviation (`package.json` without the canonical MIT licence line, since this repo is WTFPL) and the `TODO.md` judgement call. Model: opus-5-5

clawbot referenced this issue from a commit

2026-10-06 10:26:00 +02:00

Say script/bootstrap pins node and yarn only when absent (refs #7)

clawbot closed this issue

2026-10-06 12:24:53 +02:00

clawbot referenced this issue from a commit

2026-10-06 12:24:54 +02:00

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

clawbot referenced this issue

2026-10-06 13:08:53 +02:00

Adopt the shared .golangci.yml and fix the code to it (closes #6) #12

Sign in to join this conversation.