6 Commits

Include renames

Author	SHA1	Message	Date
sneak	2c3c35095b	Bring main's fixes into next (closes #16) ... Merges main into next as a two-parent merge, so main becomes an ancestor of next and the later merge of next into main does not conflict. This brings in golangci-lint v2.12.2 with .golangci.yml, and the fix that makes every handler emit slog attributes. README.md and TODO.md conflicted. README.md keeps main's Attribute output section, followed by next's Entrypoints section. TODO.md keeps the completed steps from both sides, newest first. The other files merged cleanly: the Makefile is still next's script/ shims, and the Dockerfile has main's v2.12.2 lint image. Model: opus-5-5	2026-10-05 23:22:45 +00:00
sneak	86436449c5	Emit slog attributes from every handler (closes #19, closes #24) ... Handle never read the record's attributes, and WithAttrs and WithGroup returned the receiver unchanged. The JSON and webhook handlers also marshaled slog.Record itself, whose attributes are unexported. Each handler now carries a handlerAttrs value, copied rather than mutated, so sibling loggers cannot leak attributes into each other. JSON and webhook output nest groups as objects; the console appends key=value pairs with dotted group keys, quoted as slog.NewTextHandler quotes them, invalid UTF-8 included. Logged values are never written to. In JSON the record's own fields win a key collision, and a repeated key keeps its last value unless both are groups, which merge; the README documents both. Deviation: DEL is quoted on the console; the stdlib leaves it bare. Model: opus-5-5	2026-09-28 10:49:46 +00:00
sneak	403ba4c42e	build: update golangci-lint to v2.12.2 with canonical config ... Add the canonical .golangci.yml (v2 schema, all linters enabled with a small documented disable list) and pin the Dockerfile lint stage to golangci/golangci-lint:v2.12.2 by tag and digest, replacing the old v1.64.8 digest-only pin. Fix all findings surfaced by the v1 to v2 jump without changing any exported signatures or behavior: - add package and exported-symbol doc comments (revive) - rename unused handler parameters to underscore (revive) - check or explicitly discard error returns (errcheck, errchkjson) - wrap errors with %w instead of %v (err113) - use http.NewRequestWithContext instead of http.Post (noctx) - replace fmt.Println with fmt.Fprintln(os.Stdout, ...) (forbidigo) - name magic numbers as constants (mnd) - add explicit slog.LevelDebug case (exhaustive) - interface{} to any (modernize) - move tests to the simplelog_test package (testpackage) and add t.Parallel() (paralleltest) - move NewWebhookHandler above its methods (funcorder) - whitespace, line-length, and blank-line fixes (wsl_v5, whitespace, nlreturn, lll, embeddedstructfieldcheck) - nolint with justification for the intentional init/global design (gochecknoinits, gochecknoglobals) and interface-returning constructor (ireturn)	2026-08-07 17:10:03 +00:00
sneak	5cb4f827d2	Adopt scripts-to-rule-them-all: script/ entrypoints, Makefile shims	2026-07-07 01:54:40 +02:00
sneak	6cb690bfb7	Add standard Workflow section to TODO.md	2026-07-06 21:06:43 +02:00
sneak	403d853d27	Add TODO.md	2026-07-06 20:35:50 +02:00