Skip to content
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Original file line number Diff line number Diff line change
@@ -0,0 +1,12 @@
---
schema_version: 1
id: "iss-132"
slug: "marker-scan-quoted-delimiter-doc-fps"
severity: "nitpick"
category: "tech-debt"
source: "impl-review"
found_during: "iss-111 fix (2026-07-24 run queue, burst 9)"
found_at: "internal/core/lifeboat/sources_conventions.go"
---

three documentation lines that literally quote a marker with its delimiter (TODO followed by colon) still self-match after the iss-111 fix and are irreducible under the issue's two bounded options; closing them needs a different mechanism, e.g. skipping fenced or inline-code marker examples in Markdown
Original file line number Diff line number Diff line change
@@ -0,0 +1,12 @@
---
schema_version: 1
id: "iss-133"
slug: "truncation-message-overstates-scope"
severity: "nitpick"
category: "tech-debt"
source: "impl-review"
found_during: "iss-112/114/116 review (2026-07-24 run queue, burst 9)"
found_at: "internal/core/lifeboat/sources_conventions.go"
---

the probe adapters' truncation message says the rest of the tree was not walked, but the per-directory read bound added for iss-112 can set truncated while the walk completed everything else; the over-warning is in the conservative direction, not a loud-staging violation, but the message is imprecise about WHAT was truncated
Original file line number Diff line number Diff line change
@@ -0,0 +1,12 @@
---
schema_version: 1
id: "iss-134"
slug: "readdirbounded-determinism-doc-claim"
severity: "nitpick"
category: "tech-debt"
source: "impl-review"
found_during: "iss-112/114/116 review (2026-07-24 run queue, burst 9)"
found_at: "internal/core/lifeboat/probe.go"
---

readDirBounded's doc comment claims sorted-by-name determinism, which holds only at or under the bound; a directory exceeding the bound yields a readdir-order subset that is then sorted, so WHICH entries survive is nondeterministic (loud via truncated, consistent with ListDir, pathological trigger only)
Original file line number Diff line number Diff line change
@@ -0,0 +1,12 @@
---
schema_version: 1
id: "iss-135"
slug: "walkfiles-non-dot-start-divergence"
severity: "nitpick"
category: "tech-debt"
source: "impl-review"
found_during: "iss-112/114/116 review (2026-07-24 run queue, burst 9)"
found_at: "internal/core/lifeboat/probe.go"
---

WalkFiles start-boundary divergence for a non-dot start: a start dir named in the skip set would now be walked and a regular-file start returns nil; unreachable today (both callers pass dot) — flag for any future caller passing a non-dot start
Original file line number Diff line number Diff line change
@@ -0,0 +1,12 @@
---
schema_version: 1
id: "iss-136"
slug: "provenance-lacks-pass-b-exemption-marker"
severity: "minor"
category: "inconsistency"
source: "agent-finding"
found_during: "itd-88 fidelity review (2026-07-24 run queue, burst 10)"
found_at: "internal/core/lifeboat/plan.go"
---

itd-88's fidelity gap audit found a missing press-release claim: Pass B is promised to ship as a declared exemption in _provenance.json, never a silent gap, but no exemption field or marker exists anywhere in the lifeboat package or the Provenance struct — a promise with no implementing code, recorded in itd-88's Audit Notes (receipt rcp-4d07032fc6ab)
Original file line number Diff line number Diff line change
@@ -0,0 +1,12 @@
---
schema_version: 1
id: "iss-137"
slug: "disembark-plugin-surface-missing-probe-coverage"
severity: "minor"
category: "inconsistency"
source: "agent-finding"
found_during: "itd-88 fidelity review (2026-07-24 run queue, burst 10)"
found_at: "commands/abcd/disembark.md"
---

probe and coverage ship as CLI verbs only: the plugin markdown surface (commands/abcd/disembark.md) documents plan and pack alone, while itd-88's press release headlines /abcd:disembark probe and coverage — either the surface doc grows the two verbs or the intent's diverged claim is the accepted record (dispositioned in itd-88's Audit Notes)
21 changes: 20 additions & 1 deletion .abcd/work/issues/open/iss-24-changelog-fragments.md
Original file line number Diff line number Diff line change
Expand Up @@ -8,4 +8,23 @@ source: "user-observation"
found_during: "sources-ingest session 2026-07-08"
---

changelog fragments for abcd-managed repos: per-change entries land as files in a fragments directory (towncrier pattern) and are assembled into CHANGELOG.md at release, so concurrent PRs never conflict on the Unreleased block; candidate abcd verb/discipline — assembly could hook the release gate
changelog fragments for abcd-managed repos: per-change entries land as files in a fragments directory (towncrier pattern) and are assembled into CHANGELOG.md at release, so concurrent PRs never conflict on the Unreleased block; candidate abcd verb/discipline — assembly could hook the release gate

---

**Evidence (2026-07-26, the 2026-07-24 run-queue merge window):** the monolithic
`[Unreleased]` block produced three server-side merge conflicts in a single day
while the run's eleven PRs merged. The `.gitattributes` `CHANGELOG.md merge=union`
stopgap resolves every one of them locally but is ignored by the hosting
platform's server-side merge engine, so each round needed a manual
merge-main-into-branch commit (the 2026-07-21 record already names this
asymmetry). Any two open PRs that both carry an Unreleased entry conflict at the
same insertion point; the run protocol already assumes `changelog.d/<slug>.md`
fragments — the repo has never grown the directory or the assembly step.

**Design constraint discovered since capture:** the release gate now *reads* the
changelog records to derive the release tier (`releaseImpact` →
`ShippedSince(base).Impact()`, landed with iss-122), so fragment assembly cannot
be cosmetic — it must preserve the impact-derivation seam those readers depend
on. Together with the original "verb or discipline?" question this makes iss-24
a grill-then-implement item, not an autonomous fix.