chore(plans): record sweep re-read + GroupToTag summary follow-ups as closed
This commit is contained in:
@@ -511,12 +511,26 @@ Notable findings made along the way:
|
|||||||
other's child nodes (MSB4166) — `-m:1` avoids it; ssh can transiently refuse with "Too
|
other's child nodes (MSB4166) — `-m:1` avoids it; ssh can transiently refuse with "Too
|
||||||
many authentication failures" under agent concurrency (back off and retry).
|
many authentication failures" under agent concurrency (back off and retry).
|
||||||
|
|
||||||
Follow-ups recorded, not started (deliberately small):
|
Follow-ups recorded, not started (deliberately small) — *first two closed 2026-08-18 on
|
||||||
|
`feat/sweep-reread-tag-summary`*:
|
||||||
|
|
||||||
- The `fca978d` sweep re-read (above) — the one genuinely ticket-worthy item.
|
- The `fca978d` sweep re-read (above) — the one genuinely ticket-worthy item.
|
||||||
|
*Closed in `f82dac1`: a mechanical pre-pass narrowed the sweep's 1,383 deletions to
|
||||||
|
68 files / 816 residual prose lines; a judged review of every one found 17 collateral
|
||||||
|
deletions across 10 files (rationale prose with no surviving equivalent — SessionManager
|
||||||
|
metrics invariants, HubTokenService hollow-token guard, AcknowledgeAlarm routing
|
||||||
|
remarks, LmxSubtagAlarmSource unsecured-write/idempotency notes, WnWrapAlarmConsumer's
|
||||||
|
WIN-911 precedent, DashboardSessionAdminService error-boundary catches,
|
||||||
|
WorkerPipeSession factory-null throw, three test-design rationales) — all restored,
|
||||||
|
markers left stripped. Everything else flagged verified benign (`<inheritdoc/>`
|
||||||
|
resolves to equal-or-richer interface docs, or the substance survives relocated).
|
||||||
|
Verified: NonWindows slnx 0W/0E, touched gateway test classes 65/65; windev worker
|
||||||
|
x86 build 0W/0E, worker suite 524/11/0.*
|
||||||
- `DashboardGroupTagMapping`'s class-level `<summary>` still describes the lookup as
|
- `DashboardGroupTagMapping`'s class-level `<summary>` still describes the lookup as
|
||||||
"full DN first, leading-RDN fallback" — accurate mechanics, but it could point at the
|
"full DN first, leading-RDN fallback" — accurate mechanics, but it could point at the
|
||||||
short-name-keys consequence the inline comment now records.
|
short-name-keys consequence the inline comment now records.
|
||||||
|
*Closed in `4b6f907`: the summary now states full-DN keys can never match and points
|
||||||
|
at the lookup comment in `MapGroupsToTags`.*
|
||||||
- wwtools `mxa read` human-readable formatter throws `RuntimeBinderException` on a failed
|
- wwtools `mxa read` human-readable formatter throws `RuntimeBinderException` on a failed
|
||||||
read (`ReadCommand.cs:137`); `--llm-json` works. Different repo, noted here so it isn't
|
read (`ReadCommand.cs:137`); `--llm-json` works. Different repo, noted here so it isn't
|
||||||
lost.
|
lost.
|
||||||
|
|||||||
Reference in New Issue
Block a user