sneak/dnswatcher
1
0
Fork 0
Code Issues 3 Pull Requests 1 Actions Packages Projects Releases Wiki Activity

REPO_POLICIES compliance audit #40

Merged
sneak merged 5 commits from repo-policies-compliance into main 2026-03-01 21:11:50 +01:00
Conversation 2 Commits 5 Files Changed 7 +236 -1299
clawbot commented 2026-03-01 16:51:58 +01:00
Collaborator
Copy Link

Brings the repository into compliance with REPO_POLICIES standards.

Closes issue #39.

Changes

Added files

  • REPO_POLICIES.md — fetched from sneak/prompts (last_modified: 2026-02-22)
  • .editorconfig — fetched from sneak/prompts, enforces 4-space indents (tabs for Makefile)
  • .dockerignore — standard Go exclusions (.git/, bin/, *.md, LICENSE, .editorconfig, .gitignore)

Makefile updates

  • Added fmt-check target (read-only gofmt check)
  • Added hooks target (installs pre-commit hook running make check)
  • Added docker target (runs docker build .)
  • Added -timeout 30s to both test and check targets
  • Updated .PHONY list with all new targets

Removed files

  • CLAUDE.md — superseded by REPO_POLICIES.md
  • CONVENTIONS.md — superseded by REPO_POLICIES.md

README updates

  • First line now includes project name, purpose, category (daemon), and author per REPO_POLICIES format
  • Updated CONVENTIONS.md reference to REPO_POLICIES.md
  • Added License section (pending author choice)
  • Added Author section: @sneak

Intentionally skipped

  • LICENSE file — not created; license choice (MIT, GPL, or WTFPL) requires sneak's input

Verification

  • docker build . passes (all checks green: fmt, lint, tests, build)
  • No changes to .golangci.yml, test assertions, or linter config
Brings the repository into compliance with REPO_POLICIES standards. Closes [issue #39](https://git.eeqj.de/sneak/dnswatcher/issues/39). ## Changes ### Added files - **REPO_POLICIES.md** — fetched from `sneak/prompts` (last_modified: 2026-02-22) - **.editorconfig** — fetched from `sneak/prompts`, enforces 4-space indents (tabs for Makefile) - **.dockerignore** — standard Go exclusions (.git/, bin/, *.md, LICENSE, .editorconfig, .gitignore) ### Makefile updates - Added `fmt-check` target (read-only gofmt check) - Added `hooks` target (installs pre-commit hook running `make check`) - Added `docker` target (runs `docker build .`) - Added `-timeout 30s` to both `test` and `check` targets - Updated `.PHONY` list with all new targets ### Removed files - **CLAUDE.md** — superseded by REPO_POLICIES.md - **CONVENTIONS.md** — superseded by REPO_POLICIES.md ### README updates - First line now includes project name, purpose, category (daemon), and author per REPO_POLICIES format - Updated CONVENTIONS.md reference to REPO_POLICIES.md - Added **License** section (pending author choice) - Added **Author** section: [@sneak](https://sneak.berlin) ### Intentionally skipped - **LICENSE file** — not created; license choice (MIT, GPL, or WTFPL) requires sneak's input ## Verification - `docker build .` passes (all checks green: fmt, lint, tests, build) - No changes to `.golangci.yml`, test assertions, or linter config
clawbot added the needs-reviewbot labels 2026-03-01 16:51:58 +01:00
clawbot added 4 commits 2026-03-01 16:51:58 +01:00
Add REPO_POLICIES.md, .editorconfig, and .dockerignore 80c6236cf1 
- REPO_POLICIES.md fetched from sneak/prompts (last_modified: 2026-02-22)
- .editorconfig fetched from sneak/prompts
- .dockerignore with standard Go exclusions
Add fmt-check, hooks, docker targets and test timeout to Makefile da312f5c7b 
- Add fmt-check target for read-only format checking
- Add hooks target to install pre-commit hook running make check
- Add docker target to run docker build
- Add 30-second timeout to test and check targets
- Add new targets to .PHONY list
Remove CLAUDE.md and CONVENTIONS.md b6f3ed314e 
Both files are superseded by REPO_POLICIES.md which is the
authoritative standard from sneak/prompts.
Update README for REPO_POLICIES compliance
All checks were successful
check / check (push) Successful in 34s
Details
c7e5c03239 
- Add project description first line with name, purpose, category, author
- Replace CONVENTIONS.md reference with REPO_POLICIES.md
- Add License section (pending author choice)
- Add Author section
clawbot reviewed 2026-03-01 16:54:33 +01:00
clawbot left a comment
Author
Collaborator
Copy Link

Review: PASS ✅

Reviewed all changes in PR #40 (closing issue #39).

Checklist

  • ✅ REPO_POLICIES.md — exact match with authoritative source at sneak/prompts (last_modified: 2026-02-22)
  • ✅ .editorconfig — standard: 4-space indent, LF line endings, tab for Makefile
  • ✅ .dockerignore — sensible for Go project (excludes .git/, bin/, *.md, LICENSE, .editorconfig, .gitignore)
  • ✅ Makefile — all required targets present: test, lint, fmt, fmt-check, check, hooks, docker, build, clean. 30s timeout added to both test and check. .PHONY updated.
  • ✅ CLAUDE.md and CONVENTIONS.md — properly deleted
  • ✅ README.md — REPO_POLICIES-compliant first line (name, purpose, category, author), License section (pending), Author section. Reference to CONVENTIONS.md updated to REPO_POLICIES.md.
  • ✅ No stale references — no remaining mentions of CLAUDE.md or CONVENTIONS.md in any project file
  • ✅ No functional code changes — only housekeeping files touched (no .go files)
  • ✅ No cheating — .golangci.yml unmodified, no test changes
  • ✅ docker build . — passes (includes make check: formatting, linting, tests, build)
  • ✅ Branch up-to-date with main, no rebase needed

Clean housekeeping PR. Ready to merge.

## Review: PASS ✅ Reviewed all changes in [PR #40](https://git.eeqj.de/sneak/dnswatcher/pulls/40) (closing [issue #39](https://git.eeqj.de/sneak/dnswatcher/issues/39)). ### Checklist - ✅ **REPO_POLICIES.md** — exact match with authoritative source at `sneak/prompts` (`last_modified: 2026-02-22`) - ✅ **.editorconfig** — standard: 4-space indent, LF line endings, tab for Makefile - ✅ **.dockerignore** — sensible for Go project (excludes `.git/`, `bin/`, `*.md`, `LICENSE`, `.editorconfig`, `.gitignore`) - ✅ **Makefile** — all required targets present: `test`, `lint`, `fmt`, `fmt-check`, `check`, `hooks`, `docker`, `build`, `clean`. 30s timeout added to both `test` and `check`. `.PHONY` updated. - ✅ **CLAUDE.md** and **CONVENTIONS.md** — properly deleted - ✅ **README.md** — REPO_POLICIES-compliant first line (name, purpose, category, author), License section (pending), Author section. Reference to `CONVENTIONS.md` updated to `REPO_POLICIES.md`. - ✅ **No stale references** — no remaining mentions of CLAUDE.md or CONVENTIONS.md in any project file - ✅ **No functional code changes** — only housekeeping files touched (no `.go` files) - ✅ **No cheating** — `.golangci.yml` unmodified, no test changes - ✅ **`docker build .`** — passes (includes `make check`: formatting, linting, tests, build) - ✅ **Branch up-to-date** with main, no rebase needed Clean housekeeping PR. Ready to merge.
clawbot commented 2026-03-01 16:54:56 +01:00
Author
Collaborator
Copy Link

Review: PASS ✅

Reviewed all changes in PR #40 (closing issue #39).

Checklist

  • ✅ REPO_POLICIES.md — exact match with authoritative source at sneak/prompts (last_modified: 2026-02-22)
  • ✅ .editorconfig — standard: 4-space indent, LF line endings, tab for Makefile
  • ✅ .dockerignore — sensible for Go project (excludes .git/, bin/, *.md, LICENSE, .editorconfig, .gitignore)
  • ✅ Makefile — all required targets present: test, lint, fmt, fmt-check, check, hooks, docker, build, clean. 30s timeout added to both test and check. .PHONY updated.
  • ✅ CLAUDE.md and CONVENTIONS.md — properly deleted
  • ✅ README.md — REPO_POLICIES-compliant first line (name, purpose, category, author), License section (pending), Author section. Reference to CONVENTIONS.md updated to REPO_POLICIES.md.
  • ✅ No stale references — no remaining mentions of CLAUDE.md or CONVENTIONS.md in any project file
  • ✅ No functional code changes — only housekeeping files touched (no .go files)
  • ✅ No cheating — .golangci.yml unmodified, no test changes
  • ✅ docker build . — passes (includes make check: formatting, linting, tests, build)
  • ✅ Branch up-to-date with main, no rebase needed

Clean housekeeping PR. Ready to merge.

## Review: PASS ✅ Reviewed all changes in [PR #40](https://git.eeqj.de/sneak/dnswatcher/pulls/40) (closing [issue #39](https://git.eeqj.de/sneak/dnswatcher/issues/39)). ### Checklist - ✅ **REPO_POLICIES.md** — exact match with authoritative source at `sneak/prompts` (`last_modified: 2026-02-22`) - ✅ **.editorconfig** — standard: 4-space indent, LF line endings, tab for Makefile - ✅ **.dockerignore** — sensible for Go project (excludes `.git/`, `bin/`, `*.md`, `LICENSE`, `.editorconfig`, `.gitignore`) - ✅ **Makefile** — all required targets present: `test`, `lint`, `fmt`, `fmt-check`, `check`, `hooks`, `docker`, `build`, `clean`. 30s timeout added to both `test` and `check`. `.PHONY` updated. - ✅ **CLAUDE.md** and **CONVENTIONS.md** — properly deleted - ✅ **README.md** — REPO_POLICIES-compliant first line (name, purpose, category, author), License section (pending), Author section. Reference to `CONVENTIONS.md` updated to `REPO_POLICIES.md`. - ✅ **No stale references** — no remaining mentions of CLAUDE.md or CONVENTIONS.md in any project file - ✅ **No functional code changes** — only housekeeping files touched (no `.go` files) - ✅ **No cheating** — `.golangci.yml` unmodified, no test changes - ✅ **`docker build .`** — passes (includes `make check`: formatting, linting, tests, build) - ✅ **Branch up-to-date** with main, no rebase needed Clean housekeeping PR. Ready to merge.
clawbot added merge-ready and removed botneeds-review labels 2026-03-01 16:55:06 +01:00
sneak was assigned by clawbot 2026-03-01 16:55:15 +01:00
sneak added 1 commit 2026-03-01 21:10:48 +01:00
Merge branch 'main' into repo-policies-compliance
All checks were successful
check / check (push) Successful in 44s
Details
a66d17ad1b
sneak merged commit 2835c2dc43 into main 2026-03-01 21:11:50 +01:00
sneak deleted branch repo-policies-compliance 2026-03-01 21:11:50 +01:00
Sign in to join this conversation.
Reviewers
No Reviewers
Labels
Clear labels
bot
Agent's turn to work
 merge-ready
All checks pass, reviewed, rebased, ready for merge
 needs-checks
make check does not pass cleanly
 needs-rebase
PR has merge conflicts or is behind main
 needs-review
Code review not yet done
 needs-rework
Code review found issues to fix
No Label merge-ready
Milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
clawbot sneak
No Assignees sneak
2 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: sneak/dnswatcher#40
Delete Branch "repo-policies-compliance"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?