Shim make dev and make build, format backend/ markdown (closes #28) #101

Merged
clawbot merged 1 commits from issue-28-script-models into next 2026-10-04 04:15:00 +02:00
Collaborator

Closes #28; its item 2 landed with #97.

  • make dev now shims to the new script/dev, the Vite dev server. The new make build shims to script/build, which builds only the frontend; the Go server keeps backend/script/build.
  • .prettierignore no longer leaves out backend/. Under backend/ prettier reads only README.md and .golangci.yml; make fmt rewrapped the README.

Decisions on the differences from the org model scripts:

  • script/bootstrap: kept, for Go and gcc (the backend and its race detector), node 22.13.0 or newer (eslint), linking what it installs into ~/.local/bin, and the apt-get update before the first apt install, without which gcc does not install on a fresh apt machine. The date on its pins goes back to the model's, as the pins are the model's.
  • script/fmt, script/fmt-check: kept; the model formats only markdown, and this repo also has JS and Go.
  • script/install-precommit: back to the model byte for byte; its only change was cosmetic.

Disclosures:

  • Judgement call: backend/.golangci.yml is now named in .prettierignore. It is the org standard file whose sha256 backend/script/lint checks, and prettier would reindent it.
  • .claude/ stays in .prettierignore, since git does not ignore that directory; .claude stays in .dockerignore.
  • The model script/install-precommit sets a hook variable it never uses; that is for the prompts repo to fix.
  • script/check is left alone: the model's comment now calls test a Docker step, which it is not here.

Model: opus-5-5

Closes https://git.eeqj.de/sneak/netwatch/issues/28; its item 2 landed with https://git.eeqj.de/sneak/netwatch/pulls/97. - `make dev` now shims to the new `script/dev`, the Vite dev server. The new `make build` shims to `script/build`, which builds only the frontend; the Go server keeps `backend/script/build`. - `.prettierignore` no longer leaves out `backend/`. Under `backend/` prettier reads only `README.md` and `.golangci.yml`; `make fmt` rewrapped the README. Decisions on the differences from the org model scripts: - `script/bootstrap`: kept, for Go and gcc (the backend and its race detector), node 22.13.0 or newer (eslint), linking what it installs into `~/.local/bin`, and the `apt-get update` before the first apt install, without which gcc does not install on a fresh apt machine. The date on its pins goes back to the model's, as the pins are the model's. - `script/fmt`, `script/fmt-check`: kept; the model formats only markdown, and this repo also has JS and Go. - `script/install-precommit`: back to the model byte for byte; its only change was cosmetic. Disclosures: - Judgement call: `backend/.golangci.yml` is now named in `.prettierignore`. It is the org standard file whose sha256 `backend/script/lint` checks, and prettier would reindent it. - `.claude/` stays in `.prettierignore`, since git does not ignore that directory; `.claude` stays in `.dockerignore`. - The model `script/install-precommit` sets a `hook` variable it never uses; that is for the `prompts` repo to fix. - `script/check` is left alone: the model's comment now calls test a Docker step, which it is not here. Model: opus-5-5
clawbot added the needs-review label 2026-10-04 03:18:20 +02:00
clawbot self-assigned this 2026-10-04 03:18:20 +02:00
Author
Collaborator

PASS: make dev and make build are shims over working POSIX sh scripts. Prettier now formats and checks the markdown in backend/ and leaves alone the backend/.golangci.yml that the lint check compares by sha256. Each remaining difference from the model scripts is either reverted or explained, as the plan in #28 (comment) asks.

  • Deviation: gated on a local rebase onto next at 3b12627, which conflicts in TODO.md only.
  • Judgement call: the one-line comment in script/bootstrap names Go, gcc and the newer node, not the ~/.local/bin links or the apt-get update; I read those as serving the same installs.
  • Judgement call: script/check differs from the current model only in a comment that would be false here. The plan names four scripts and the PR discloses this one, so I did not require a comment in the script.

Model: opus-5-5

PASS: `make dev` and `make build` are shims over working POSIX sh scripts. Prettier now formats and checks the markdown in `backend/` and leaves alone the `backend/.golangci.yml` that the lint check compares by sha256. Each remaining difference from the model scripts is either reverted or explained, as the plan in https://git.eeqj.de/sneak/netwatch/issues/28#issuecomment-116126 asks. - Deviation: gated on a local rebase onto `next` at `3b12627`, which conflicts in `TODO.md` only. - Judgement call: the one-line comment in `script/bootstrap` names Go, gcc and the newer node, not the `~/.local/bin` links or the `apt-get update`; I read those as serving the same installs. - Judgement call: `script/check` differs from the current model only in a comment that would be false here. The plan names four scripts and the PR discloses this one, so I did not require a comment in the script. Model: opus-5-5
clawbot added needs-rebase and removed needs-review labels 2026-10-04 03:42:59 +02:00
clawbot added 1 commit 2026-10-04 04:11:17 +02:00
make dev ran yarn dev inline and there was no make build. Both are now
shims, over script/dev and the new script/build. .prettierignore stops
leaving out backend/, so backend/README.md is formatted and checked;
backend/.golangci.yml is left out by name instead: it is the org
standard file whose sha256 backend/script/lint checks, and prettier
would reindent it. .claude/ stays in .prettierignore, as git does not
ignore it. script/install-precommit and the date on bootstrap's pins go
back to the org model; bootstrap, fmt and fmt-check keep what the Go
backend and eslint need, each with a comment saying so.

Model: opus-5-5
clawbot force-pushed issue-28-script-models from 0e007efde3 to 0900de4b95 2026-10-04 04:11:17 +02:00 Compare
clawbot added needs-checks and removed needs-rebase labels 2026-10-04 04:11:21 +02:00
clawbot merged commit d412815953 into next 2026-10-04 04:15:00 +02:00
clawbot deleted branch issue-28-script-models 2026-10-04 04:15:00 +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/netwatch#101