script/lint requires buildx; document the prerequisite #40

New Issue

2026-08-10T15:35:16+02:00

clawbot commented

2026-08-10 15:35:16 +02:00

script/lint passes --output type=cacheonly, which is a BuildKit/buildx flag. On a docker without buildx, or with BuildKit disabled, it fails: DOCKER_BUILDKIT=0 script/lint gives unknown flag: --output, exit 125.

This is not a defect — the failure is loud and can never produce a false green, and CI is green on 25b6c0a — but the prerequisite is undocumented, so the failure is unexplained when someone hits it.

Found during review of #39 ; accepted there rather than reworked.

Why the flag is there

It avoids an image export per lint run. The main image's exporting to image step measures 121.8s, so this is a substantial saving, not the dangling-image tidiness the original commit body claims.

Definition of done

script/lint states buildx as a prerequisite, and states what the flag buys, in place of the weaker dangling-image rationale.
The README Entrypoints entry for script/lint says the same.
script/bootstrap is checked: decide and record whether it should verify buildx is present, or whether failing loudly at first use is sufficient. Either answer is fine; an undecided one is not.
make check green.

`script/lint` passes `--output type=cacheonly`, which is a BuildKit/buildx flag. On a docker without buildx, or with BuildKit disabled, it fails: `DOCKER_BUILDKIT=0 script/lint` gives `unknown flag: --output`, exit 125. This is not a defect — the failure is loud and can never produce a false green, and CI is green on `25b6c0a` — but the prerequisite is undocumented, so the failure is unexplained when someone hits it. Found during review of https://git.eeqj.de/sneak/lora.vegas/pulls/39 ; accepted there rather than reworked. ## Why the flag is there It avoids an image export per lint run. The main image's `exporting to image` step measures 121.8s, so this is a substantial saving, not the dangling-image tidiness the original commit body claims. ## Definition of done - `script/lint` states buildx as a prerequisite, and states what the flag buys, in place of the weaker dangling-image rationale. - The README Entrypoints entry for `script/lint` says the same. - `script/bootstrap` is checked: decide and record whether it should verify buildx is present, or whether failing loudly at first use is sufficient. Either answer is fine; an undecided one is not. - `make check` green.

clawbot referenced this issue

2026-08-10 15:36:21 +02:00

Makefile has no cibuild or precommit shims, so two documented entrypoints have no make target #34

clawbot referenced this issue

2026-08-10 15:40:27 +02:00

next: accumulated work for the current cycle #41

clawbot referenced this issue

2026-08-10 15:40:38 +02:00

Makefile has no cibuild or precommit shims, so two documented entrypoints have no make target #34

clawbot referenced this issue

2026-08-10 15:49:52 +02:00

next: accumulated work for the current cycle #41