Skip to content

[finding] dispatch-gates: NO_CHECK_FAMILIES_MARKER and INHERITED_POPULATION_MARKER's reason half carry the same first-newline capture #18422 repaired for the three population markers — no live cut today, exposure only #18662

Description

@os-justin

Filed by the domain:skills execution seat (session_01Gqi43smmqjJ5sUrhfoPeKu, seat post #7623) out of the #18422 round (PR #18660), from the dev's out-of-scope findings, re-measured by the seat before filing. ⛔ Filed bare: finding only, domain:* / type / priority are triage's.

Dedupe words: no-check-families reason cut · inherited-population reason · population marker wholeness · first newline capture · dispatch-gates marker roster.

The reading, on origin/main 14d254411

#18422 / PR #18660 repaired the three POPULATION markers: a reason that does not end on the marker line is refused (populationReasonCutRefusal), with the grammar in one spelling (populationMarkerPattern). Two markers outside that roster capture a reason with the same (\S.*)$-under-m shape and are NOT covered by the new reading:

  • NO_CHECK_FAMILIES_MARKER (workflow files, # dispatch-gates: no-check-families -- <reason>);
  • INHERITED_POPULATION_MARKER's reason half (// dispatch-gates: inherited-population <path> … -- <reason>).

Live declarations at this sha (seat's git grep): 3 no-check-families (.github/workflows/merged-branch-reaper.yml:212, os-create-smoke.yml:48, scaffold-e2e.yml:23) and 2 inherited-population (scripts/cli-build-prerequisite.mjs:111, scripts/pm/dispatch-gates.mjs:702). The #18422 dev measured all five as one-liners followed by a blank line ⇒ no reason is cut in the tree today. This is exposure, not a live defect: the next author who wraps one of these reasons hands a seat a sentence that stops, with nothing red — the #18422 failure mode on a marker the repair did not reach.

Shape (⛔ a proposal, not a prescription)

The helper PR #18660 adds is generic: register the two markers with populationReasonContinuation / a cut refusal, one registration line each, and pin a wrapped-reason case per marker in the file's --self-test. The inherited-population marker's path list is its own grammar; only its reason half is in question.

⛔ Not measured

  • Whether either marker's consumer ever renders the reason to a seat (if a reason is read by nothing, a cut costs nothing — the card assumes it is rendered, as the population markers' reasons are).

Refs: #18422 · PR #18660

domain:skills execution seat · seat post #7623 · readings taken on origin/main 14d254411


Generated by Claude Code

Activity

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Type

    No type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions