Aller au contenu
login
arrow_backRetour aux issues
medici-finance/assay #82

Recurring: workers write PR ref into board Status cell -> malformed row fails lint (PRs #74, #76, #80)

ecoDébutant help wanted

descriptionDescription

## Recurring: workers write the PR ref into the board Status cell, producing a malformed row that fails `lint` Three separate stalled PRs this cycle carried the **identical** defect in their `docs/streams//README.md` board row, each firing the `lint` (statusgen-board) job red: - PR #74 (desk-containers/02) — `docs/streams/desk-containers/README.md` - PR #76 (statusgen/02) — `docs/streams/statusgen/README.md` - PR #80 (derived-board/02) — `docs/streams/derived-board/README.md` ### The defect The implementing worker set the Status cell to something like `implemented (#NN)` and, in doing so, **prepended a stray leading cell plus an empty cell** ahead of the `#` column: ``` | implemented (#80) || 02 | [brief](...) | 0 | M | ... | ``` The stray `| implemented (#NN) |` shifts every column one place right and injects an empty cell, so statusgen emits 5–7 PROBLEMs and the `lint` job exits 1. ### Root cause The board row's Status column accepts ONLY a bare vocabulary token (`todo` / `implemented` / `verified` / `done`). Workers are trying to record the PR association *in the cell* (dispatch prompts that say "set Status → implemented + link the PR" invite this). statusgen rejects any decorated Status value. ### Fix 1. **Convention:** the PR association lives in the PR (the `Brief:` trailer / the PR body), never in the board cell. The row is edited to bare `implemented`, columns unchanged. 2. **Prompt fix (dispatch side):** worker/board-flip instructions should say "set the Status cell to the bare token `implemented` — do NOT add the PR number or a link to the cell." (Adopted in this session's later dispatches.) 3. **Optional tooling:** statusgen's lint error for a decorated/'||' status could name the fix explicitly ("Status must be a bare token; PR link belongs in the Brief: trailer") to shorten the diagnosis loop, since this recurs. All three PRs were corrected to a well-formed single row this cycle and their `lint` job went green.
codeOuvre sur GitHub