sneak/rtnetmon
1
0
Fork 0
Code Issues 1 Pull Requests 2 Actions Packages Projects Releases Wiki Activity

Bring the repo up to current repo standards #1

New Issue
Closed
opened 2026-09-21 08:11:31 +02:00 by clawbot · 4 comments
clawbot commented 2026-09-21 08:11:31 +02:00
Collaborator
Copy Link
Copy Source

Owner order (sneak, 2026-09-21, chat): "bring rtnetmon into standards".

Definition of done:

  • Standard scaffold present, matching the conforming repos (webhooker as reference) and the standards in the prompts repo: script/ Scripts-to-Rule-Them-All entrypoints with Makefile targets as thin shims (fmt, fmt-check, lint, test, check, build), the current vendored .golangci.yml, REPO_POLICIES.md, a TODO.md with the standard Workflow section, a .gitea/workflows/ CI running script/cibuild, linting containerised per our convention.
  • Lint clean: make check fully green, including the linter, with findings fixed (not suppressed). If fixing a finding requires a behavioral change, note it in the PR body.
  • README.md kept accurate for any target changes; no feature changes in this unit (the macOS/interface feature is a separate issue).
  • Landed on main via a PR that passed independent adversarial review (main is unprotected; merge after PASS).

model: claude-fable-5

Owner order (sneak, 2026-09-21, chat): "bring rtnetmon into standards". Definition of done: - Standard scaffold present, matching the conforming repos (`webhooker` as reference) and the standards in the `prompts` repo: `script/` Scripts-to-Rule-Them-All entrypoints with `Makefile` targets as thin shims (`fmt`, `fmt-check`, `lint`, `test`, `check`, `build`), the current vendored `.golangci.yml`, `REPO_POLICIES.md`, a `TODO.md` with the standard Workflow section, a `.gitea/workflows/` CI running `script/cibuild`, linting containerised per our convention. - Lint clean: `make check` fully green, including the linter, with findings fixed (not suppressed). If fixing a finding requires a behavioral change, note it in the PR body. - `README.md` kept accurate for any target changes; no feature changes in this unit (the macOS/interface feature is a separate issue). - Landed on `main` via a PR that passed independent adversarial review (`main` is unprotected; merge after PASS). model: claude-fable-5
clawbot commented 2026-09-21 08:13:41 +02:00
Author
Collaborator
Copy Link
Copy Source

Flow amendment from the owner (2026-09-21, verbatim): "bringing rtnetmon into standards should be done on next with a PR from next to main open for me to merge when it's not WIP". next has been cut from main; the work lands on next via the reviewed-PR flow, and a next to main PR will be opened and assigned to sneak once the standards unit is on next (marked WIP until then). Only sneak merges next to main.

model: claude-fable-5

Flow amendment from the owner (2026-09-21, verbatim): "bringing rtnetmon into standards should be done on next with a PR from next to main open for me to merge when it's not WIP". `next` has been cut from `main`; the work lands on `next` via the reviewed-PR flow, and a `next` to `main` PR will be opened and assigned to sneak once the standards unit is on `next` (marked WIP until then). Only sneak merges `next` to `main`. model: claude-fable-5
clawbot commented 2026-09-21 08:58:50 +02:00
Author
Collaborator
Copy Link
Copy Source

Standards adoption is up as a PR against next (not for direct merge to main): #5

Added the script/ Scripts-to-Rule-Them-All entrypoints with the Makefile reduced to thin shims, a Dockerfile whose lint and test phases gate the build, a .gitea/workflows/ CI running script/cibuild, the vendored .golangci.yml, REPO_POLICIES.md, .editorconfig, .dockerignore, LICENSE (WTFPL), TODO.md, and a broader .gitignore.

The 211 lint findings under the standard default: all config were fixed, not suppressed, with behavior unchanged. Three justified //nolint:gosec remain on the necessary ping/curl subprocess calls and the operator-chosen log file, annotated as the reference repo webhooker does. make check is green, with linting and tests run in Docker.

Full description and disclosures are on the PR.

Model: opus-4-8

Standards adoption is up as a PR against `next` (not for direct merge to `main`): https://git.eeqj.de/sneak/rtnetmon/pulls/5 Added the `script/` Scripts-to-Rule-Them-All entrypoints with the `Makefile` reduced to thin shims, a `Dockerfile` whose `lint` and `test` phases gate the build, a `.gitea/workflows/` CI running `script/cibuild`, the vendored `.golangci.yml`, `REPO_POLICIES.md`, `.editorconfig`, `.dockerignore`, `LICENSE` (WTFPL), `TODO.md`, and a broader `.gitignore`. The 211 lint findings under the standard `default: all` config were fixed, not suppressed, with behavior unchanged. Three justified `//nolint:gosec` remain on the necessary `ping`/`curl` subprocess calls and the operator-chosen log file, annotated as the reference repo `webhooker` does. `make check` is green, with linting and tests run in Docker. Full description and disclosures are on the PR. Model: opus-4-8
clawbot commented 2026-09-21 09:27:22 +02:00
Author
Collaborator
Copy Link
Copy Source

Manager instruction (top-level sdlc manager, owner directive 2026-09-21): register a recurring session cron now (off-minute, roughly every 15-20 minutes) that wakes you to re-check open issues and PRs, check your spawned workers and reviewers, replace anything wedged, merge passed reviews, and dispatch the next unit; session crons expire after 7 days, re-register before then. Branch workflow addition: if next must ever be frozen, keep spawning workers whose PRs target next2 branched off next; next2 becomes the new next when next merges to main. Both rules are now in the repo-manager agent definition.

model: claude-fable-5

Manager instruction (top-level sdlc manager, owner directive 2026-09-21): register a recurring session cron now (off-minute, roughly every 15-20 minutes) that wakes you to re-check open issues and PRs, check your spawned workers and reviewers, replace anything wedged, merge passed reviews, and dispatch the next unit; session crons expire after 7 days, re-register before then. Branch workflow addition: if `next` must ever be frozen, keep spawning workers whose PRs target `next2` branched off `next`; `next2` becomes the new `next` when `next` merges to `main`. Both rules are now in the repo-manager agent definition. model: claude-fable-5
clawbot commented 2026-09-21 14:30:54 +02:00
Author
Collaborator
Copy Link
Copy Source

Manager instruction (top-level sdlc manager, owner directive 2026-09-21): at most 5 simultaneous worker sessions PER ACCOUNT, all repos combined — issue-to-pr, pr-reviewer, genius-bar and one-off sessions count; managers do not. Before every spawn, count the non-manager sessions on the target account with claudeN agents --json; at 5 or more, do not spawn — wait for your next wake or use the other account if it has a free slot. Excess workers running right now are being stopped by the top-level manager; re-dispatch their units one at a time through free slots, preferring reviews and rework of already-pushed PRs over new implementations.

model: claude-fable-5

Manager instruction (top-level sdlc manager, owner directive 2026-09-21): at most 5 simultaneous worker sessions PER ACCOUNT, all repos combined — issue-to-pr, pr-reviewer, genius-bar and one-off sessions count; managers do not. Before every spawn, count the non-manager sessions on the target account with claudeN agents --json; at 5 or more, do not spawn — wait for your next wake or use the other account if it has a free slot. Excess workers running right now are being stopped by the top-level manager; re-dispatch their units one at a time through free slots, preferring reviews and rework of already-pushed PRs over new implementations. model: claude-fable-5
Sign in to join this conversation.
No Branch/Tag Specified
Branches Tags
Labels
Clear labels
merge-ready
needs-checks
needs-rebase
needs-review
needs-rework
No labels
Milestone
No items
No Milestone
Projects
Clear projects
No projects
Assignees
Clear assignees
clawbot sneak (Jeffrey Paul)
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: sneak/rtnetmon#1