sneak/bsdaily

Merge next into main: standard layout, lint config, version stamp, command line in internal/ #10

Open

clawbot wants to merge 5 commits from next into main

Author	SHA1	Message	Date
clawbot	bcf2c8550d	Move the command line from cmd/bsdaily into internal/cli (closes #8 ) check / check (push) Successful in 4m15s Details cmd/bsdaily/main.go now only passes Version to cli.Main and exits with the status it returns, as REPO_POLICIES.md requires of cmd/. The cobra command, the flag rules and the date parsing moved unchanged into internal/cli, so flags, help text, error messages, exit status and the first log line stay as they were. The fallback to dev for an empty version moved with them into cli.Main. parseTargetDates is exported as ParseTargetDates so that a table test outside the package can reach it; the test never touches the filesystem or runs the extraction. README describes the new layout and TODO.md records the step. Model: opus-5-5 Co-authored-by: clawbot <sneak+clawbot@sneak.cloud>	2026-10-06 23:07:50 +02:00
clawbot	f5c7614768	Stamp the git tag or short commit into the binary (closes #4 ) check / check (push) Successful in 3m35s Details A plain `docker build .` now stamps bsdaily's version: the VERSION build argument when one is given, otherwise `git describe --tags --always` on the .git in the build context, as the canonical Dockerfile does. The build fails if .git is there and the version still comes out empty, dev or unknown. A host `make` build stamps the same `git describe`, or dev. bsdaily logs the version on the first line of every run and prints it with --version; a build that stamps nothing, or an empty value, reports dev. Judgement call: the build line also takes the canonical -trimpath and -s -w. One //nolint (gochecknoglobals): -X can only set a package-level variable. Model: opus-5-5 Co-authored-by: clawbot <sneak+clawbot@sneak.cloud>	2026-10-06 18:41:51 +02:00
clawbot	c16f177575	Adopt the shared .golangci.yml and fix the code to it (closes #6 ) check / check (push) Successful in 5m14s Details Vendor .golangci.yml byte-identical from sneak/prompts at cc440118 and move the Dockerfile lint phase to golangci-lint v2.14.0 by the digest REPO_POLICIES.md names. Fix the code to that config with flags, help text, output files, SQL and the order of steps unchanged; long functions are split into named steps. Judgement call: the extraction transaction is now rolled back on every early return; the old deferred rollback missed most failures and could dereference a nil transaction. Thirteen //nolint directives (gosec, unconvert, mnd, unqueryvet), each with its reason. Model: opus-5-5	2026-10-06 14:24:48 +02:00
clawbot	925a3896f7	Format Markdown with prettier in make fmt and make fmt-check (closes #7 ) check / check (push) Successful in 5m9s Details 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 node and yarn install (nvm from a hash-checked archive, yarn through corepack), at the pinned versions when they are absent; a node or yarn already installed is used as is. Both fmt scripts find yarn the canonical way. The new files come from sneak/prompts at cc440118c876. README.md and TODO.md are rewrapped by make fmt, with no wording changes. Deviation: package.json drops the canonical "license": "MIT" line; this repo is WTFPL. Model: opus-5-5 Co-authored-by: clawbot <sneak+clawbot@sneak.cloud>	2026-10-06 12:24:51 +02:00
clawbot	17b6b84322	Bring the repo up to the standard layout (closes #1 ) check / check (push) Successful in 5m6s Details Vendors .gitignore, .dockerignore, .editorconfig, REPO_POLICIES.md, script/cibuild, script/docker, script/lint, script/test and the CI workflow byte-identical from sneak/prompts commit cc440118c876; the two ignore files then add this repo's own build output, root-anchored so cmd/bsdaily/ stays in. The Dockerfile gains a lint phase on the golangci-lint image already pinned and a test phase on the Debian Go image with sqlite3 and zstd; the build stage copies a file from each, so a plain docker build cannot skip them. Nothing installs golangci-lint on the host. On apt, script/bootstrap refreshes the package lists once before its first install. .golangci.yml and the lint cleanup: #6 Model: opus-5-5	2026-10-06 08:25:03 +02:00