4 Commits

Include renames

Author	SHA1	Message	Date
clawbot	30b36dabe4	Make the golangci-lint settings take effect: v2 config layout, linter pinned at v2.12.2 (closes #10) ... golangci-lint v2 ignores a top-level linters-settings block without a warning, so limits such as the 88-column line length were never applied. .golangci.yml is now the canonical copy from sneak/prompts, verbatim, which keeps its settings under linters.settings and also configures depguard and gomodguard_v2. golangci-lint is pinned at v2.12.2 by commit in the Dockerfile and script/bootstrap. The long lines in attrsum.go are rewrapped and one nolint directive moves to its own line; behaviour is unchanged. Model: opus-5-5	2026-10-06 03:26:51 +02:00
clawbot	9f3af05c91	Report the version: docker build . stamps the git tag or short commit (closes #7) ... attrsum --version now prints the git tag or short commit. make build stamps it with -X from git describe, and the Dockerfile takes the VERSION build argument when given, otherwise git describe --tags --always on the .git in the build context, failing if .git is present and no version comes out. A new .dockerignore, the canonical one, keeps .git/config out of the context, and also this repo's host-built /attrsum. CI checks out full history so it sees the 1.0.0 tag and stamps what a full clone does. script/docker is replaced with the canonical copy. Model: opus-5-5	2026-10-02 11:02:38 +02:00
sneak	7f75f2ee72	Clear all golangci-lint findings and make the CI build green (closes #1) ... Fix every golangci-lint finding under the repo's standard .golangci.yml (from ~192 down to 0 under the pinned v2.10.1) without changing program behavior: - gochecknoglobals: replace the verbose/quiet/exclude* package globals with an options struct threaded through the command implementations. - err113: introduce package-level sentinel errors and wrap them with %w. - errcheck: check or explicitly discard every previously unchecked error (bar.Add/Finish, deferred Close, verbose writes). - forbidigo: route verbose output through os.Stdout instead of fmt.Print*. - noinlineerr / wsl_v5 / nlreturn / gofmt: split inline error checks and normalize whitespace. - complexity (cyclop/gocognit/nestif): extract small behavior-preserving helpers (runOverPaths, countAndBar, walkSkip, clearOne, checkOne, missingChecksum, reportCheck). - mnd/lll/nonamedreturns/revive/modernize/nilnil: named constants, wrapped lines, unnamed returns, doc comments, SplitSeq, non-(nil,nil) returns. - paralleltest/thelper: mark tests parallel (now race-safe with no shared globals) and add t.Helper(); probe the real xattr key so the guard is accurate. Also make the tests actually runnable in CI rather than skipping: - move the xattr keys into the user.* namespace (user.berlin.sneak.app.*), which Linux requires for regular-file xattrs; macOS treats the whole string as an opaque name, so behavior is unchanged there. - run the Docker builder's checks as an unprivileged user so the permission tests are meaningful (root bypasses file mode bits).	2026-07-27 00:55:46 +07:00
sneak	6230bb1c3a	Add scripts-to-rule-them-all scaffold (refs #1) ... Add the standard STRTA scaffold, mirroring the conformant Go repos: - script/ POSIX-sh entrypoints (bootstrap, setup, projectname, test, lint, fmt, fmt-check, check, docker, cibuild, precommit, install-precommit). - Makefile rewritten as thin shims: .PHONY plus the nine standard targets each delegating to script/NAME; repo-specific build, clean, and try targets retained. - .golangci.yml matching the org-standard Go lint config. - Dockerfile whose build runs make check then make build, so the image fails on any check failure. - .gitea/workflows/check.yml running script/cibuild. make fmt-check and make test are green. make check is not yet green because of pre-existing lint findings in the application code, which are out of scope for this scaffold change; hence refs (not closes).	2026-07-25 18:36:57 +07:00