@erclx/canon 5.44.0 → 5.46.0

package/governance/rules/ui/link-behavior.md

@@ -11,9 +11,9 @@ paths:
 
 ## Default target
 
-- Open a link in the same tab. Reserve `target="_blank"` for a destination that would discard in-progress work if it replaced the current page, such as a document a user is midway through elsewhere.
-- Do not add `target="_blank"` as a default for every external link.
+- Open a link in the same tab. Reserve `target="_blank"` for a page the person must consult while finishing a task in the current one.
+- Never add `target="_blank"` as a default for every external link.
 
 ## mailto: links
 
-- Open a `mailto:` link in the same tab. It hands off to the mail client rather than replacing page content, so a new tab leaves an empty tab behind.
+- Open a `mailto:` link in the same tab. It hands off to the mail client rather than replacing the page, so a new tab leaves an empty tab behind.

package/governance/rules/ui/motion.md

@@ -12,12 +12,12 @@ paths:
 
 ## Moving content
 
-- Give any content that moves, blinks, or scrolls automatically for more than five seconds a control to pause, stop, or hide it (WCAG Pause, Stop, Hide, 2.2.2).
+- Give any content that moves, blinks, or scrolls automatically for more than five seconds a control to pause, stop, or hide it (WCAG 2.2.2).
 - Halt an auto-advancing carousel or ticker while it holds keyboard focus or pointer hover.
-- Do not flash content more than three times in any one second.
+- Never flash content more than three times in any one second (WCAG 2.3.1).
 
 ## Reduced motion
 
 - Honor `prefers-reduced-motion: reduce` for every animation and transition that conveys no state. Remove it or replace it with an instant change.
 - Keep essential state changes visible under reduced motion, using a fade or a static indicator instead of movement.
-- Do not start video or animated backgrounds on their own when the preference is set.
+- Start no video or animated background on its own when the preference is set.

package/governance/rules/ui/surface-capture.md

@@ -12,39 +12,27 @@ paths:
 
 ## What a surface is
 
-- A surface is anything the project renders for a person to look at. A framework route is one. A page the project generates is another, whoever generates it.
-- Judge a generated page by the same rule as a route. Reading its markup reports nothing about how it composes.
-- A component the production build strips out is not a surface, such as a dev-only scenario switcher gated behind a build flag. The build removes it before anything renders, so no comparison exists to capture.
+- Treat anything the project renders for a person as a surface, a generated page as much as a route.
+- Skip a component the production build strips out, such as a dev-only switcher.
 
 ## When to capture
 
-- Run the project's capture command after changing what a surface renders. Report it rather than proceeding silently when the project has none.
-- Capture against a running preview server. Do not capture against a dev server.
-- Capture every theme the surface ships. Do not capture the default theme alone.
-- Capture again after fixing a defect a capture found. A repair inside a shared stylesheet can cancel a rule written earlier in the same file.
+- Run the project's capture command against a preview server, never a dev server, after changing a surface. Report it when the project has none.
+- Capture every theme the surface ships, and again after fixing a defect a capture found.
 
 ## What a capture covers
 
-- Capture the full page. Do not capture a component in isolation.
-- Declare the widths a layout case captures. Always include 320, the reflow floor from the reflow criterion in `a11y`, then add each breakpoint bucket the project writes rules for, read from the `## Layout` section of `DESIGN.md`.
-- Fall back to the 320 floor alone when the project has no `DESIGN.md` or its record carries no `## Layout` section. Do not impose another project's breakpoints.
-- Add a layout case at each bound when a surface renders persistent chrome a person can resize, such as a draggable sidebar or a split pane, one with the chrome at its narrowest and one at its widest. Take both at the narrowest declared width where the chrome stays persistent rather than collapsing, since a frame where it has turned into an overlay shows none of the width it takes.
-- Read those bounds from the code that clamps the chrome, since `## Layout` has no slot for them. Judge each frame by the content width left beside the chrome rather than by the window width, against the 320 floor `a11y` holds for that region.
-- Apply the declaration to a case that covers layout only. A case driving a menu open or an answer chosen tests a state, so it takes one width rather than the full set.
-- Read a capture at 320 as proof that a frame was taken there, not that the page reflows correctly. The criterion is stated in `a11y` and nothing here enforces it.
-- Write the capture into a folder named `evidence`. The image is collected for the pull request comment when any segment of its path is literally `evidence` and its filename carries an image extension, so the folder name is fixed and its position in the tree is the project's own.
-- Several `evidence/` folders in one project is the ordinary shape rather than a problem to consolidate. One per surface is how a project with more than one rendered surface ends up.
-- Drive a state a screenshot cannot reach. An initial render reports nothing about a menu that opens, an answer that is chosen, or a rail that tracks scrolling.
-- Add a case to the capture record when adding a surface.
-- Remove a surface's case in the change that removes the surface.
+- Capture the full page, never a component in isolation.
+- Capture a layout case at 320, the reflow floor in `a11y`, plus each breakpoint in the `## Layout` section of `DESIGN.md`, or at 320 alone when there is none.
+- Capture resizable chrome at its narrowest and widest clamp, at the narrowest width keeping it persistent. Judge the content width beside it.
+- Capture a state case at one width rather than the full set.
+- Drive a state an initial render cannot reach, such as an open menu or a rail tracking scroll.
+- Add a surface's case with the surface, and remove it with the surface.
 
 ## Sharing a capture
 
-- Do not commit the sweep. It stays ignored.
-- Commit a flagged case's evidence output so the pull request carries the comparison, rather than attaching it by hand.
-- A project whose own established convention already commits that same comparison may route a flagged case's evidence there instead of a separate evidence folder. Only an established convention already committing the same comparison qualifies, not a preference for skipping a second folder.
-- Commit an evidence case for the first time only after running the capture twice with no code change between the runs and confirming the two outputs are byte-identical.
-- Mask the element carrying a value that moves between runs, such as an elapsed-time counter, a clock, or a relative date, in the state's capture case before the double capture, so the check still runs on everything else in the frame. Mask the smallest element holding the value, never the state.
-- Do not commit the state as evidence when the masked outputs still differ, which happens when the value's width reflows its neighbors. Name the variance in the pull request instead, and never re-run toward a match.
-- Recommitting an unbounded sweep on every run reaches a gigabyte of repository history inside a hundred merges. A small, committed evidence set is what a reviewer needs and what lets GitHub draw its own before-and-after comparison on the pull request.
-- `git-pr` and `git-followup` maintain a comparison comment on the pull request automatically once a case is committed, so a reviewer never has to open Files Changed to see it.
+- Write a capture into a folder named `evidence`, which the pull request comment collects from.
+- Commit a flagged case's evidence output and never the sweep, unless the project already commits the same comparison elsewhere.
+- Commit a new evidence case only after two captures with no code change between them match byte for byte.
+- Mask the smallest element holding a value that moves between runs, such as a clock, before the double capture.
+- Name the variance in the pull request when masked outputs still differ, never re-running toward a match.

package/governance/rules/ui/ux-completeness.md

@@ -10,56 +10,33 @@ paths:
 
 # UX completeness standards
 
-## Mutation feedback
+## Feedback
 
-- Every user-initiated mutation (save, delete, sync, import, export) must show inline feedback on success.
-- Feedback must be transient: auto-dismiss after 2-3 seconds without requiring user action.
-- Show inline error feedback when a mutation fails. Do not fail silently.
+- Confirm every user-initiated mutation, such as a save, delete, or import, inline on success, and show an inline error when it fails.
+- Auto-dismiss a success message only when its outcome stays visible elsewhere. Keep a message carrying the only copy of its information until dismissed (WCAG 2.2.1).
+- Show no optimistic status, such as a green dot or "connected", while unsaved edits are pending.
 
 ## Empty states
 
-- Every list, table, or collection view must handle the empty state explicitly.
-- Empty states must include a next action: a button, link, or instruction that tells the user what to do.
-- Distinguish between "nothing exists yet" (onboarding) and "nothing matches your filter" (recovery). Use different copy for each.
-- Filter-driven empty states must include a way to clear the filter (link or button).
+- Give every list, table, or collection an explicit empty state carrying a next action, such as a button or link.
+- Write different copy for "nothing exists yet" and "nothing matches your filter", and give the filtered state a way to clear the filter.
 
 ## Loading states
 
-- Every async operation visible to the user must show a loading indicator (spinner, skeleton, or disabled state).
-- Do not show stale content while new content is loading. Either show a loading state or keep the previous content with a progress indicator.
+- Show a progress indicator for any operation a person waits on for longer than about one second.
+- Mark previous content as updating while new content loads, rather than showing it unmarked as current.
 
 ## Destructive actions
 
-- Destructive actions (delete, disconnect, discard) must require confirmation before executing.
-- Confirmation UI must be dismissible via Escape in addition to a cancel button.
+- Confirm a destructive action, such as a delete, disconnect, or discard, before it runs, unless it can be undone afterward.
 
-## Inline confirmations
+## Element states
 
-- All inline confirmation rows (delete, discard, cancel) must be dismissible via Escape.
-- Confirmation labels and destructive buttons must have sufficient contrast in both light and dark modes.
+- Give every interactive element distinct hover, active, focus, and disabled styles.
+- Never flash an intermediate state, such as a light frame before a dark theme or a layout jump before data arrives. Reserve the space or settle it first.
 
-## Status indicators
+## Overflow
 
-- Status indicators (dots, badges, connection status) must reset to a neutral state when their source input changes.
-- Do not show optimistic status (green dot, "connected") while unsaved edits are pending.
-
-## State transitions
-
-- Do not flash intermediate states during transitions (e.g., white flash on theme load, layout jump before data arrives).
-- Use CSS transitions or deferred rendering to prevent visible flicker between states.
-
-## Interactive element states
-
-- All interactive elements must visually respond to hover, active, focus, and disabled states.
-- Disabled elements must use reduced opacity or muted colors and `cursor: not-allowed`.
-- Do not rely on a single state (e.g., focus only) to communicate interactivity.
-
-## Overflow and truncation
-
-- Long text in constrained containers must truncate with ellipsis, not overflow or wrap unexpectedly.
-- Truncated text must be accessible via `title` attribute or expandable interaction.
-
-## Scroll and clipping
-
-- Focus rings and interactive borders must not be clipped by parent `overflow: hidden` or `overflow: auto` containers.
-- Add padding or adjust overflow to prevent clipping on edge items in scrollable lists.
+- Truncate long text in a constrained container with an ellipsis rather than letting it overflow.
+- Expose truncated text through an expand control or full text on focus, never through a `title` tooltip alone.
+- Keep focus rings and interactive borders clear of a parent's `overflow` clipping, padding the edge items of a scrollable list.

package/governance/stacks/base.toml

@@ -4,7 +4,6 @@ extends = ""
 # `canon gov detect`. Every other rule is opt-in.
 rules = [
   "output",
-  "task-scope",
   "scratch",
   "worktrees",
   "git",

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "@erclx/canon",
   "type": "module",
-  "version": "5.44.0",
+  "version": "5.46.0",
   "description": "Runs your agent's work and keeps its code good, on one setup for every repo.",
   "license": "MIT",
   "bin": {

package/scripts/core/install-check.sh

@@ -100,7 +100,7 @@ log_step "Assert scaffold"
 # Authoring references name no path because they ship inside the plugin's
 # skills rather than installing into a target. The two
 # rule paths stand for the `claude` and `canon` groups `base` takes whole.
-for path in "CLAUDE.md" ".claude/wiki/index.md" ".claude" "canon/context/index.md" \
+for path in "CLAUDE.md" ".claude" "canon/context/index.md" \
   ".oxfmtrc.json" ".lintstagedrc" ".husky/pre-commit" ".github/workflows/verify.yml" "scripts/verify.ts" \
   ".claude/rules/canon/claude/output.md" ".claude/rules/canon/canon/scratch.md"; do
   if [ ! -e "$TARGET_DIR/$path" ]; then

package/src/claude/cases/workflow.ts

@@ -146,6 +146,18 @@ export const WORKFLOW_CASES: readonly SkillCase[] = [
       'Are our requirements and architecture docs still true, and when were they last reviewed?',
     expect: 'document-health',
   },
+  // A repair rather than a report, and folder by folder until each gates,
+  // which is what separates it from `document-health` above.
+  {
+    prompt:
+      'Our docs folder fails the paragraph and bullet limits everywhere, so rewrite it until it passes and lock it in.',
+    expect: 'document-repair',
+  },
+  {
+    prompt:
+      'Bring the context entries under the density limits one folder at a time and gate each once it is clean.',
+    expect: 'document-repair',
+  },
   {
     prompt: 'Open a new entry on the task board for this piece of work.',
     expect: 'task-board',

package/src/cli/help.ts

@@ -29,7 +29,6 @@ export const COMMAND_GROUPS: { title: string; rows: CommandRow[] }[] = [
       ['gov [cmd]', 'Governance rules'],
       ['tooling [cmd]', 'Tooling stacks'],
       ['claude [cmd]', 'Claude workflow'],
-      ['wiki [cmd]', 'Wiki pages'],
       ['indexes [cmd]', 'Regenerate index.md files'],
       ['docs [cmd|topic]', 'Emit toolkit reference docs'],
     ],

package/src/cli.ts

@@ -7,7 +7,6 @@ import { register as sync } from '@/commands/sync'
 import { register as gov } from '@/commands/gov'
 import { register as tooling } from '@/commands/tooling'
 import { register as claude } from '@/commands/claude'
-import { register as wiki } from '@/commands/wiki'
 import { register as indexes } from '@/commands/indexes'
 import { register as docs } from '@/commands/docs'
 import { register as design } from '@/commands/design'
@@ -68,7 +67,6 @@ sync(program)
 gov(program)
 tooling(program)
 claude(program)
-wiki(program)
 indexes(program)
 docs(program)
 design(program)

package/src/commands/context.ts

@@ -56,6 +56,11 @@ import {
   presentNames,
   resolveFolders,
 } from '@/context/folders'
+import {
+  type CoverageReport,
+  measureCoverage,
+  readRoots,
+} from '@/context/coverage'
 import { isGating } from '@/context/gate'
 import { isShapeGated, SHAPE_GATED_FOLDERS } from '@/context/shape-gated'
 import { currentWorktreeRoot } from '@/git/worktree'
@@ -101,7 +106,7 @@ export function register(program: Command): void {
   context
     .command('audit')
     .description(
-      'Report required sections, entry length, citations, reference form, catalog tables, provenance, index drift, and the requirements record against its own word cap',
+      'Report required sections, entry length, citations, reference form, catalog tables, provenance, index drift, code folders no entry names, and the requirements record against its own word cap',
     )
     .argument('[path]', 'Project root, defaulting to the current directory')
     .helpOption('-h, --help', 'Show this help message')
@@ -113,7 +118,7 @@ export function register(program: Command): void {
     .option('--citations-only', 'Run the gating citation check alone')
     .option(
       '--gate',
-      'Also fail on a missing required section, an H2 outside the set, a shape break in a gated folder, or index drift, the findings that are facts',
+      'Also fail on a missing required section, an H2 outside the set, a shape break in a gated folder, index drift, or a code folder no entry names, the findings that are facts',
     )
     .addHelpText(
       'after',
@@ -129,7 +134,9 @@ export function register(program: Command): void {
         '--citations-only, which never measures it. A record stating no',
         'cap is reported and never gated. --gate widens the gate to the',
         'findings that are facts rather than judgments: a missing',
-        'required section, an H2 outside the closed set, and index drift.',
+        'required section, an H2 outside the closed set, index drift, and',
+        'a code folder no Layout names or two entries claim. The roots come',
+        'from canon/config/context.toml and default to src when present.',
         'A context entry requires Overview and Layout and allows Gotchas.',
         'A shape break, such as a Decisions section or a sixth gotcha, is',
         'reported everywhere and gated only under a shape-gated folder.',
@@ -667,6 +674,9 @@ async function runAudit(
         isShapeGated(relative(top, join(root, rel))),
       )
   const length = gateOnly ? undefined : lengthFindings(entries)
+  const coverage: CoverageReport | undefined = gateOnly
+    ? undefined
+    : measureCoverage(root, readRoots(root), folders)
   // Absent under `--citations-only` and null when the project carries no
   // record. A run that never looked and a project with nothing to look at are
   // different answers, and one value for both reports the second as the first.
@@ -685,6 +695,7 @@ async function runAudit(
     reportTables(entries)
     reportProvenance(entries, folders)
     reportDrift(drift)
+    reportCoverage(coverage ?? { kind: 'no-root' })
     reportRequirements(requirements, root)
     outro()
   }
@@ -721,6 +732,8 @@ async function runAudit(
         extraSections: extras,
         shapeFindings: shape,
         indexDrift: drift,
+        // Absent under `--citations-only`, which never measures it.
+        coverage,
         // Null says the run opened the project and found no record, which a
         // target that never wrote one is entitled to. Absent says the run
         // never looked, which is `--citations-only`.
@@ -753,6 +766,7 @@ async function runAudit(
     extraSections: extras,
     shape,
     drift,
+    coverage: coverage ?? { kind: 'no-root' },
     widened,
   })
 
@@ -1210,6 +1224,46 @@ function reportProvenance(
   )
 }
 
+function reportCoverage(report: CoverageReport): void {
+  logStep('Coverage')
+
+  if (report.kind === 'no-root') {
+    logInfo(
+      'No code root found, since no canon/config/context.toml declares one and there is no src/, so nothing was measured.',
+    )
+    return
+  }
+
+  if (report.kind === 'invalid') {
+    logWarn(report.reason)
+    return
+  }
+
+  if (report.kind === 'opted-out') {
+    logInfo(
+      'canon/config/context.toml declares roots = [], so coverage was not measured.',
+    )
+    return
+  }
+
+  const { unnamed, doubled, missingRoots } = report.findings
+  const lines = [
+    ...missingRoots.map((root) => `${root}  matches no folder`),
+    ...unnamed.map((folder) => `${folder}  named in no Layout`),
+    ...doubled.map(
+      ({ folder, rels }) => `${folder}  Layout H3 in ${rels.join(', ')}`,
+    ),
+  ]
+
+  if (lines.length === 0) {
+    logInfo(`Every folder under ${report.roots.join(', ')} is named once.`)
+    return
+  }
+
+  logWarn(plural(lines.length, 'coverage finding'))
+  pipeOutput(lines.join('\n'))
+}
+
 /** Reports the requirements record against the word cap it states, if any. */
 function reportRequirements(
   report: RequirementsReport | undefined,

package/src/commands/gov.ts

@@ -37,6 +37,7 @@ import {
   hasFinding,
   PRACTICE_FOLDERS,
   readRuleAudit,
+  RULE_GATE_EXEMPT,
   RULE_GATED_FOLDERS,
   RULE_LIMITS,
   type RuleAuditReport,
@@ -631,6 +632,7 @@ export function register(program: Command): void {
         '  inside it.',
         '',
         `Gated folders: ${RULE_GATED_FOLDERS.length === 0 ? 'none yet' : RULE_GATED_FOLDERS.join(', ')}`,
+        `Exempt from the gate: ${RULE_GATE_EXEMPT.length === 0 ? 'none' : RULE_GATE_EXEMPT.join(', ')}`,
         '',
         'Exit codes:',
         '  0  no finding, or under --gate no finding in a gated folder',
@@ -691,6 +693,7 @@ async function runAudit(
         root,
         limits: RULE_LIMITS,
         gated: RULE_GATED_FOLDERS,
+        exempt: RULE_GATE_EXEMPT,
         ...report,
       })}\n`,
     )
@@ -699,6 +702,7 @@ async function runAudit(
   return auditExit(report, {
     gate: opts.gate ?? false,
     gated: RULE_GATED_FOLDERS,
+    exempt: RULE_GATE_EXEMPT,
   })
 }
 

package/src/commands/init.ts

@@ -39,8 +39,8 @@ export function register(program: Command): void {
         '',
         'Runs `canon tooling sync base --write` first, which replaces every',
         'base golden config the target holds, then `canon claude init`,',
-        '`canon gov install`, and `canon wiki init`. Refuses a target that is',
-        'not a directory or is the toolkit root.',
+        'and `canon gov install`. Refuses a target that is not a directory or is',
+        'the toolkit root.',
         '',
         'Exit codes:',
         '  0  every domain installed, or the prompt was cancelled',

package/src/commands/labels.ts

@@ -5,6 +5,7 @@ import { execa } from 'execa'
 import { gitEnv } from '@/git/env'
 import { diffLabels } from '@/labels/apply'
 import { type LabelAuditRefusal, auditLabels } from '@/labels/audit'
+import { checkClosing } from '@/labels/closing'
 import { resolveScanInput } from '@/labels/event'
 import { checkTitleFormat, type TitleFormatIssue } from '@/labels/format'
 import { MAP_REL } from '@/labels/map'
@@ -34,6 +35,7 @@ interface ScanOptions {
   readonly body?: string
   readonly bodyFile?: string
   readonly head?: string
+  readonly closes?: string
   readonly json?: boolean
 }
 
@@ -196,6 +198,10 @@ export function register(program: Command): void {
       'Body to scan, read from a file, overriding the event payload',
     )
     .option('--head <ref>', 'Head branch, overriding the event payload')
+    .option(
+      '--closes <numbers>',
+      'Issues the body must close, a comma list or `none`, graded when given',
+    )
     .option('--json', 'Add a machine-readable record on stdout')
     .addHelpText(
       'after',
@@ -239,12 +245,20 @@ export function register(program: Command): void {
         'character, an em dash or a semicolon, in the title or body of a pull',
         'request or a review, outside a code span or a link.',
         '',
+        'With --closes, it also grades the closing lines. Each issue named needs',
+        'its own `Closes #NNN` keyword in the body, and a closing keyword for an',
+        'issue the list leaves out fails. `none` expects no closing line.',
+        'Without the flag the check does not run, so the pull request job and a',
+        'followup keep their behavior. A keyword in a code span or a fence is',
+        'not read, as GitHub does not read it.',
+        '',
         'Exit codes:',
-        '  0  none of the six found',
+        '  0  none of the eight found',
         '  1  refused, with the reason on stderr or in the JSON record',
         '  2  the title or body carries a phase label, a board identifier, a',
         '     session link, a banned character, a heading outside the closed',
-        '     list or a required one missing, or a title',
+        '     list or a required one missing, a missing or stray closing line',
+        '     under --closes, or a title',
         "     breaking claude/skills/git-pr/references/pr.md's format, casing, or length",
         '',
         'Examples:',
@@ -536,11 +550,42 @@ async function runApply(number: string, opts: ApplyOptions): Promise<number> {
   return owed.length === 0 ? 0 : 2
 }
 
+/**
+ * Reads `--closes` as the issues the body must close. Undefined when the flag
+ * is absent, which skips the check, and a message when the value is neither
+ * `none` nor a comma list of numbers, since a guessed list would grade the
+ * body against the wrong issues.
+ */
+function parseCloses(value: string | undefined): number[] | string | undefined {
+  if (value === undefined) return undefined
+  if (value === 'none') return []
+  if (!/^\d+(,\d+)*$/.test(value)) {
+    return `--closes takes a comma list of issue numbers or none, not ${value}`
+  }
+
+  return [...new Set(value.split(',').map(Number))]
+}
+
 async function runScan(opts: ScanOptions): Promise<number> {
   const emitJson = opts.json ?? false
 
   intro('canon labels scan')
 
+  const expected = parseCloses(opts.closes)
+
+  if (typeof expected === 'string') {
+    logStep('Refused')
+    logWarn(expected)
+    outro()
+
+    if (emitJson) {
+      process.stdout.write(
+        `${JSON.stringify({ reason: 'bad-input', message: expected })}\n`,
+      )
+    }
+    return 1
+  }
+
   const resolved = resolveScanInput(opts)
 
   if (resolved.kind === 'refused') {
@@ -693,6 +738,37 @@ async function runScan(opts: ScanOptions): Promise<number> {
     for (const issue of titleFormatIssues) logWarn(TITLE_FORMAT_MESSAGES[issue])
   }
 
+  const closing =
+    expected === undefined
+      ? { missing: [], stray: [] }
+      : checkClosing(resolved.body, expected)
+
+  logStep(
+    expected === undefined
+      ? 'Closing lines not checked'
+      : closing.missing.length === 0 && closing.stray.length === 0
+        ? 'Clean'
+        : 'Closing line issue found',
+  )
+  if (expected === undefined) {
+    logInfo('no --closes list, so the closing lines are not graded')
+  } else if (closing.missing.length === 0 && closing.stray.length === 0) {
+    logInfo('the closing lines match the issues the task closes')
+  } else {
+    if (closing.missing.length > 0) {
+      logWarn(
+        `${plural(closing.missing.length, 'issue')} with no closing line. Add one \`Closes #NNN\` per issue.`,
+      )
+      for (const number of closing.missing) logWarn(`#${number}`)
+    }
+    if (closing.stray.length > 0) {
+      logWarn(
+        `${plural(closing.stray.length, 'closing line')} for an issue this pull request does not close. Delete each.`,
+      )
+      for (const number of closing.stray) logWarn(`#${number}`)
+    }
+  }
+
   outro()
 
   if (emitJson) {
@@ -706,6 +782,8 @@ async function runScan(opts: ScanOptions): Promise<number> {
         titleFormatIssues,
         sections: { checked: sectionsChecked, ...sections },
         bannedCharacters,
+        missingClosures: closing.missing,
+        strayClosures: closing.stray,
       })}\n`,
     )
   }
@@ -716,7 +794,9 @@ async function runScan(opts: ScanOptions): Promise<number> {
     titleFormatIssues.length === 0 &&
     sections.outside.length === 0 &&
     sections.missing.length === 0 &&
-    banFindings.length === 0
+    banFindings.length === 0 &&
+    closing.missing.length === 0 &&
+    closing.stray.length === 0
     ? 0
     : 2
 }