Re-vendor the shared files from sneak/prompts at dd4027b (closes #52) #85

Open
clawbot wants to merge 1 commits from issue-52-revendor-prompts into next
Collaborator

Re-vendors the shared files from sneak/prompts commit dd4027b for #52.

  • .golangci.yml, .gitea/workflows/check.yml and REPO_POLICIES.md are the shared copies unchanged; no test-support packages exist for the depguard deny list. .gitignore, .editorconfig and .dockerignore are the shared copies followed by this repository's own entries.
  • Lint (golangci-lint v2.14.0) and test are Dockerfile phases that script/lint and script/test build alone and uncached; the image build depends on both. The format check runs only on the host.
  • The agent guidance file is renamed to AGENTS.md, content unchanged.
  • The Go code is fixed for the new findings: besides wrapping and blank lines, long functions are split, unused functions removed, tests using unexported names renamed *_internal_test.go, and the streamer, WHOIS fetcher, database maintainer and web server start with a context but still run until stopped.

Worth knowing:

  • make test and make lint now need Docker. make test exceeds the 60-second cap here, mostly compiling the cgo SQLite driver with -race; per sneak/prompts#78 this waits on sneak/prompts#113.
  • Four reads now return an error when row iteration fails, instead of partial results.

Disclosures:

  • Deviation: .dockerignore also keeps out SQLite databases and a Go workspace file, since the image ships a source archive of the build context.
  • Rule suppressed: tagliatelle where JSON names are the RIS Live feed's or the existing API; interfacebloat on database.Store; paralleltest on two goroutine-counting tests.
  • Judgement call: script/lint and script/test are the dd4027b versions, not those from sneak/prompts#123.
  • Judgement call: no markdown formatter added; make fmt stays Go-only.
  • Unverified: not run on the Gitea runner.

Model: opus-5-5

Re-vendors the shared files from `sneak/prompts` commit `dd4027b` for https://git.eeqj.de/sneak/routewatch/issues/52. - `.golangci.yml`, `.gitea/workflows/check.yml` and `REPO_POLICIES.md` are the shared copies unchanged; no test-support packages exist for the `depguard` deny list. `.gitignore`, `.editorconfig` and `.dockerignore` are the shared copies followed by this repository's own entries. - Lint (golangci-lint v2.14.0) and test are `Dockerfile` phases that `script/lint` and `script/test` build alone and uncached; the image build depends on both. The format check runs only on the host. - The agent guidance file is renamed to `AGENTS.md`, content unchanged. - The Go code is fixed for the new findings: besides wrapping and blank lines, long functions are split, unused functions removed, tests using unexported names renamed `*_internal_test.go`, and the streamer, WHOIS fetcher, database maintainer and web server start with a context but still run until stopped. Worth knowing: - `make test` and `make lint` now need Docker. `make test` exceeds the 60-second cap here, mostly compiling the cgo SQLite driver with `-race`; per https://git.eeqj.de/sneak/prompts/issues/78 this waits on https://git.eeqj.de/sneak/prompts/issues/113. - Four reads now return an error when row iteration fails, instead of partial results. Disclosures: - Deviation: `.dockerignore` also keeps out SQLite databases and a Go workspace file, since the image ships a source archive of the build context. - Rule suppressed: `tagliatelle` where JSON names are the RIS Live feed's or the existing API; `interfacebloat` on `database.Store`; `paralleltest` on two goroutine-counting tests. - Judgement call: `script/lint` and `script/test` are the `dd4027b` versions, not those from https://git.eeqj.de/sneak/prompts/issues/123. - Judgement call: no markdown formatter added; `make fmt` stays Go-only. - Unverified: not run on the Gitea runner. Model: opus-5-5
clawbot added the needs-review label 2026-10-14 22:03:11 +02:00
clawbot self-assigned this 2026-10-14 22:03:11 +02:00
clawbot added 1 commit 2026-10-14 22:03:12 +02:00
Takes .golangci.yml, .dockerignore, .gitignore, .editorconfig, the
workflow and REPO_POLICIES.md from sneak/prompts dd4027b, keeping this
repository's own entries after the shared content. Lint (golangci-lint
v2.14.0) and test become Dockerfile phases that script/lint and
script/test build alone and uncached; the image build depends on both,
and script/cibuild bootstraps, runs script/check, then builds the image.
The agent guidance file moves to AGENTS.md. The Go code is fixed for
what the new configuration finds; reads that iterate rows now also
report an iteration error.

Model: opus-5-5
Some checks are pending
check / check (push) Waiting to run
You are not authorized to merge this pull request.
This pull request can be merged automatically.
View command line instructions

Checkout

From your project repository, check out a new branch and test the changes.
git fetch -u origin issue-52-revendor-prompts:issue-52-revendor-prompts
git checkout issue-52-revendor-prompts
Sign in to join this conversation.
No Reviewers
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: sneak/routewatch#85