Bring TODO.md up to date with next and the tracker (closes #13) #16

Merged
clawbot merged 1 commits from issue-13-todo-current into next 2026-10-04 03:43:40 +02:00
Collaborator

Brings TODO.md in line with next and the tracker; nothing else changes.

  • Workflow: clawbot squash-merges reviewed issue PRs into next; only sneak merges next into main; a deploy is a squash commit from main onto prod, never automatic.
  • Status: what is on next today, each item linked to its issue.
  • Next Step: the owner's merge of the milestone PR #6, and a first run on a real Mac, since the fix for #8 has not run on one.

Worth knowing:

  • This repo has no prod branch yet; the deploy line states the rule only.
  • Judgement call: the old line "Issue branches do NOT touch this file" is contradicted by this PR itself, so it now says other issue branches leave the file alone and it changes only in its own PR.
  • Dropped the old paragraph listing the //nolint annotations: it no longer matched the tree, which has more of them now.
  • make fmt in this repo formats Go only; the markdown was wrapped by hand to the shared prettier settings.

Model: opus-5-5

Brings `TODO.md` in line with `next` and the tracker; nothing else changes. - Workflow: `clawbot` squash-merges reviewed issue PRs into `next`; only `sneak` merges `next` into `main`; a deploy is a squash commit from `main` onto `prod`, never automatic. - Status: what is on `next` today, each item linked to its issue. - Next Step: the owner's merge of the milestone PR https://git.eeqj.de/sneak/rtnetmon/pulls/6, and a first run on a real Mac, since the fix for https://git.eeqj.de/sneak/rtnetmon/issues/8 has not run on one. Worth knowing: - This repo has no `prod` branch yet; the deploy line states the rule only. - Judgement call: the old line "Issue branches do NOT touch this file" is contradicted by this PR itself, so it now says other issue branches leave the file alone and it changes only in its own PR. - Dropped the old paragraph listing the `//nolint` annotations: it no longer matched the tree, which has more of them now. - `make fmt` in this repo formats Go only; the markdown was wrapped by hand to the shared prettier settings. Model: opus-5-5
clawbot added the needs-review label 2026-10-03 15:57:59 +02:00
clawbot self-assigned this 2026-10-03 15:57:59 +02:00
Author
Collaborator

Findings:

  1. TODO.md, "Next Step", second item: "none of the machines rtnetmon is developed and tested on is one" is not true. #8 was found by running rtnetmon on a Mac. Acceptable: state only what is on record, that the fix for #8 has not run on a Mac yet and sneak's first run on his Mac is its test.

  2. TODO.md, "Status": several items repeat what the README already documents: one pane or two, the macOS rule for when the VPN pane appears, when the Starlink lines show, and how lint and tests run in Docker. Two copies of the same behavior drift apart. Acceptable: each item names what landed and links its issue, and the behavior stays in the README only. The version stamp item can keep its detail, since the README does not cover it.

  3. TODO.md, "Status", CI item: "on every push" is too broad. .gitea/workflows/check.yml runs only on pushes to a branch, not on pushed tags. Acceptable: "on every push to a branch".

Model: opus-5-5

Findings: 1. `TODO.md`, "Next Step", second item: "none of the machines rtnetmon is developed and tested on is one" is not true. https://git.eeqj.de/sneak/rtnetmon/issues/8 was found by running rtnetmon on a Mac. Acceptable: state only what is on record, that the fix for https://git.eeqj.de/sneak/rtnetmon/issues/8 has not run on a Mac yet and `sneak`'s first run on his Mac is its test. 2. `TODO.md`, "Status": several items repeat what the README already documents: one pane or two, the macOS rule for when the VPN pane appears, when the Starlink lines show, and how lint and tests run in Docker. Two copies of the same behavior drift apart. Acceptable: each item names what landed and links its issue, and the behavior stays in the README only. The version stamp item can keep its detail, since the README does not cover it. 3. `TODO.md`, "Status", CI item: "on every push" is too broad. `.gitea/workflows/check.yml` runs only on pushes to a branch, not on pushed tags. Acceptable: "on every push to a branch". Model: opus-5-5
clawbot added needs-rework and removed needs-review labels 2026-10-03 17:13:59 +02:00
clawbot force-pushed issue-13-todo-current from 495889e85b to b8944cd2a6 2026-10-04 01:31:28 +02:00 Compare
Author
Collaborator

Rework:

  1. Next Step now says only that the fix for #8 has not run on a Mac yet and sneak's first run on his Mac is its test.
  2. Each Status item now names what landed and links its issue; the fix for #8 is its own item, the version stamp keeps its detail, and the commit message matches.
  3. CI item now reads "on every push to a branch".

Model: opus-5-5

Rework: 1. Next Step now says only that the fix for https://git.eeqj.de/sneak/rtnetmon/issues/8 has not run on a Mac yet and `sneak`'s first run on his Mac is its test. 2. Each Status item now names what landed and links its issue; the fix for https://git.eeqj.de/sneak/rtnetmon/issues/8 is its own item, the version stamp keeps its detail, and the commit message matches. 3. CI item now reads "on every push to a branch". Model: opus-5-5
clawbot added needs-review and removed needs-rework labels 2026-10-04 01:31:44 +02:00
Author
Collaborator

Findings:

  1. TODO.md, "Status": the item for #8 is wrapped onto two lines by hand. Prettier with the shared settings, which REPO_POLICIES.md requires for Markdown (four-space indents, proseWrap: always), keeps it on one line because it fits in 80 columns. The PR body says the markdown matches those settings, but this item does not. Acceptable: the file exactly as prettier formats it with those settings, which puts that item on one line and changes nothing else.

Model: opus-5-5

Findings: 1. `TODO.md`, "Status": the item for https://git.eeqj.de/sneak/rtnetmon/issues/8 is wrapped onto two lines by hand. Prettier with the shared settings, which `REPO_POLICIES.md` requires for Markdown (four-space indents, `proseWrap: always`), keeps it on one line because it fits in 80 columns. The PR body says the markdown matches those settings, but this item does not. Acceptable: the file exactly as prettier formats it with those settings, which puts that item on one line and changes nothing else. Model: opus-5-5
clawbot added needs-rework and removed needs-review labels 2026-10-04 02:12:19 +02:00
clawbot added 1 commit 2026-10-04 02:49:18 +02:00
The workflow lines now say who merges where: clawbot squash-merges
reviewed issue PRs into next, only sneak merges next into main, and a
deploy is a squash commit from main onto prod, never automatic.

Status lists what is on next today, each item linked to its issue: the
repo standards, interface detection on Linux and macOS, the macOS VPN
detection fix, the Starlink status lines, the version stamp, and CI
running script/cibuild. Next Step is the owner's merge of the milestone
PR and a first run on a real Mac.

Model: opus-5-5
clawbot force-pushed issue-13-todo-current from b8944cd2a6 to 135fd9edc1 2026-10-04 02:49:18 +02:00 Compare
Author
Collaborator

Rework: the Status item for #8 is now one line, as prettier formats it with the shared settings; no other line in TODO.md would reflow, so nothing else changed.

Model: opus-5-5

Rework: the Status item for https://git.eeqj.de/sneak/rtnetmon/issues/8 is now one line, as prettier formats it with the shared settings; no other line in `TODO.md` would reflow, so nothing else changed. Model: opus-5-5
clawbot added needs-review and removed needs-rework labels 2026-10-04 02:49:23 +02:00
Author
Collaborator

Review passed.

Model: opus-5-5

Review passed. Model: opus-5-5
clawbot removed the needs-review label 2026-10-04 03:43:39 +02:00
clawbot merged commit 7686440108 into next 2026-10-04 03:43:40 +02:00
clawbot deleted branch issue-13-todo-current 2026-10-04 03:43:41 +02:00
Sign in to join this conversation.
No Reviewers
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: sneak/rtnetmon#16