@erclx/canon 5.46.0 → 5.47.0

package/src/tasks/archive.ts

@@ -107,6 +107,7 @@ export interface ArchiveSuccess {
   readonly from: string
   readonly to: string
   readonly priorityRowRemoved: boolean
+  readonly backlogRowRemoved: boolean
   readonly indexRegenerated: boolean
   /** Every live plan the task cited that no other live task still holds, in line order. */
   readonly plans: readonly PlanMove[]
@@ -556,7 +557,7 @@ function isRowFor(line: string, target: string): boolean {
 }
 
 /**
- * Drops the declined task's bullet from the backlog. A backlog line is a
+ * Drops the task's bullet from the backlog. A backlog line is a
  * bullet carrying a link rather than a table row, so the match is a bullet
  * prefix and the link target rather than `isRowFor`'s pipe-delimited cell.
  */
@@ -571,9 +572,17 @@ export function removeBacklogRow(
   return { text: kept.join('\n'), removed: kept.length !== lines.length }
 }
 
+/**
+ * A bullet is about a task when its first link targets it. A later link is a
+ * reference, such as a note on what the bullet waits for, and matching anywhere
+ * would drop that other task's line from a file no history backs.
+ */
 function isBulletFor(line: string, target: string): boolean {
   const trimmed = line.trimStart()
-  return /^[-*]\s/.test(trimmed) && trimmed.includes(target)
+  if (!/^[-*]\s/.test(trimmed)) return false
+
+  const first = trimmed.indexOf('](')
+  return first !== -1 && trimmed.startsWith(target, first)
 }
 
 /**
@@ -896,8 +905,8 @@ async function resolveStem(
 }
 
 /**
- * Archives one task as a single unit: the move, the ordering-row removal, and
- * the index regen. The hook and `task-complete` both call this, so every gate
+ * Archives one task as a single unit: the move, the ordering-row and backlog
+ * bullet removal, and the index regen. The hook and `task-complete` both call this, so every gate
  * refuses rather than reports. A caller with nobody watching cannot act on a
  * warning, and two callers gating differently is the drift this exists to stop.
  */
@@ -996,6 +1005,7 @@ export async function archiveTask(
 
   const { relinked, failed } = await repointInboundLinks(root, from, to)
   const priorityRowRemoved = await clearPriorityRow(dir, stem)
+  const backlogRowRemoved = await clearBacklogRow(dir, stem)
   const regen = await regenOne(dir, { dryRun: false })
 
   return {
@@ -1004,6 +1014,7 @@ export async function archiveTask(
     from,
     to,
     priorityRowRemoved,
+    backlogRowRemoved,
     indexRegenerated: regen.action === 'written',
     plans: plans.moves,
     ready: ready || undefined,

package/tooling/base/seeds/.github/pull_request_template.md

@@ -8,7 +8,7 @@
 - <Verb> <specific component/file/function>
 
 <!-- ## Technical context (optional, omit for docs/config/trivial changes) -->
-<!-- - <Why this approach was taken, not what changed> -->
+<!-- **<Decision in a few words>.** <Why this approach, what the code can't show, and any shortcoming> -->
 
 ## Testing