Skip to content
Merged
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
88 changes: 88 additions & 0 deletions docs/hygiene-history/ticks/2026/05/14/2131Z.md
Original file line number Diff line number Diff line change
@@ -0,0 +1,88 @@
# Tick 2026-05-14T21:31Z — PR #3256 `+`-at-line-start fix per Copilot review

## Refresh (step 1)

- Cron sentinel `12fb713e` live.
- 2 in-flight PRs polled (#3256 / #3258):
- **#3256** (shard 2123Z) — BLOCKED + 1 unresolved Copilot thread
- **#3258** (shard 2128Z) — wait-ci, autoMerge armed

## Holding discipline (step 2)

Per `blocked-green-ci-investigate-threads.md`: investigate threads on
PR #3256.

## Work (step 3) — fix `+`-at-line-start MD032 false-trigger

Copilot catch on `docs/hygiene-history/ticks/2026/05/14/2123Z.md:45`:

> P1: This wrapped continuation starts with `+` inside a `-` bullet,
> which markdownlint parses as a nested list item using a different
> bullet style. The repository guideline explicitly calls this out as
> CI-breaking; move the plus to the previous line or reword it as prose.

Substantive catch — real lint hazard, not nit. Line was:

```text
- Authored `docs/backlog/P1/B-0461-...md`
following B-0459's shape (123 lines: frontmatter + AC + scope clarification
+ dependency chain + composes_with + pre-start checklist)
```

The leading `+` on line 45 reads as a new list item starting a `+`
bullet. The repo even has a dedicated audit tool
(`tools/hygiene/audit-md032-plus-linestart.ts`) specifically for
detecting this class.

Fix: rewrote as prose — "(123 lines covering frontmatter, AC, scope
clarification, dependency chain, composes_with, and pre-start
checklist)" — no line starts with `+` or any bullet char.

Pushed commit `9c4590e` to #3256's branch; thread resolved with reply.

## Proactive sweep

`bun tools/hygiene/audit-md032-plus-linestart.ts --list | grep
"ticks/2026/05/14"` after fix: **empty**. No other shards today have
the pattern.

## Verify (step 4)

- `markdownlint-cli2` clean on the fixed shard
- `audit-md032-plus-linestart --list` reports 0 violations across today's shards
- Thread resolved via GraphQL mutation
- Composite branch-guard + `gh pr create --head` used

## Shard (step 5)

This file.

## CronList (step 6)

Sentinel `12fb713e` armed; one entry, recurring.

## Visibility (step 7)

- **PR #3256** (shard 2123Z) — fix `9c4590e` pushed, thread resolved,
CI re-running; autoMerge stays armed
- **PR #3258** (shard 2128Z) — still wait-ci, autoMerge armed

## Notes for future-Otto

The repo has TWO markdownlint-adjacent audit tools for shard-authoring
hygiene:

| Tool | Catches |
|---|---|
| `audit-md032-plus-linestart.ts` | continuation lines that accidentally start with `+` |
| `markdownlint-cli2` (with repo config) | MD032 blanks-around-lists + other configured rules |

Running both before pushing a tick shard prevents the same class of
finding from reaching CI / Copilot review. The 2119Z shard added MD032
discipline; this tick adds the `+`-at-line-start companion.

Pattern: when authoring a tick shard with structured content (bulleted
descriptions of file structure, summaries of frontmatter sections), use
plain prose for the description rather than `+`-separated lists inside
existing `-` bullets. The structure can still be communicated; the lint
gate stops complaining.
Loading