@plot-pm/board 0.8.0-rc.9 → 0.8.1-rc.0

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@plot-pm/board",
-  "version": "0.8.0-rc.9",
+  "version": "0.8.1-rc.0",
   "description": "Local Kanban board for Plot — a glanceable view of plan phases from docs/plans, with sprint and story filters",
   "type": "module",
   "license": "MIT",

package/plot-config.sh

@@ -27,6 +27,21 @@
 #   Project board | Branch prefixes | Plan directory | Active index |
 #   Delivered index | Sprint directory | Story directory | Story index |
 #   Plan template | Main branch | Board command
+#   Worker bound        seconds a single prompt run may take in the worker loop
+#                       before it is ended and the worker exits (no hop). Read by
+#                       plot-worker-loop.sh; default 3600 (~1h), `0` disables it.
+#                       Non-numeric or empty falls back to the default. It bounds
+#                       a HUNG agent — one whose CLI crashed without exiting — so
+#                       one dead worker cannot hold a slot for hours.
+#   Agent registry      the directory the dispatcher writes agent manifests to,
+#                       read by the board's registry. Default `.plot/agents`
+#                       (repo-relative, gitignored, hence per-worktree). A board
+#                       served from a worktree the dispatcher never wrote to
+#                       reads an empty directory and synthesizes the whole fleet
+#                       with no session ids; point this at a shared location so
+#                       the board finds the registry wherever it was started.
+#                       Absent = the default, so a single-checkout project is
+#                       unaffected.
 # Agent-runner keys (optional; Plot hardcodes no agent tooling, Principle 5):
 #   Worker command      how /plot-dispatch runs an agent headless on a worktree.
 #                       `none` = asked, and this repo starts workers by hand —

package/plot-plan-meta.sh

@@ -128,6 +128,18 @@
 #                  is a note to a reviewer rather than a release note. Comment
 #                  interiors stay non-content, as everywhere else in this parser,
 #                  so the template's guidance block contributes nothing.
+#   long_wave_names wave names too long to be labels — a report, not a refusal.
+#                  A wave name is a label (`Shaped`, `Gated`, `Offered first`);
+#                  a sentence-length heading is a plan-authoring mistake the
+#                  board can only render badly. Names past a threshold judgement
+#                  (LONG_WAVE_NAME_MAX, set from the estate's longest legitimate
+#                  name) are listed here in document order, so the reconcile
+#                  sweep can surface them for a fix. Reads the SAME wave names
+#                  waves[] carries — never a second scan of the file — so both
+#                  the ## Branches and ## Waves spellings are covered. [] when
+#                  every wave name is a label; ALWAYS present, so a consumer
+#                  never reads undefined. The plan still parses in full: waves[]
+#                  is unchanged and no name is shortened or dropped.
 #   issues         tracker issue numbers this plan answers, from the `## Status`
 #                  `Issue:` line or front matter `issue:` (sorted, unique).
 #                  A DEDICATED field, never a scan of the body for `#NNN`: a
@@ -169,15 +181,18 @@
 #                  canonical entries and the front-matter entry merge
 #                  (canonical first, front matter appended)
 #   rounds         how many rounds of /plot:challenge-the-plan the plan has been
-#                  through, read from the `CHALLENGE-THE-PLAN-METADATA` block
-#                  the skill writes. **OMITTED ENTIRELY when the plan carries no
-#                  such block** — absent is not zero. A plan nobody has
-#                  interrogated and a plan interrogated to no effect want
-#                  opposite reactions from a reader, so the field is missing
-#                  rather than 0, the same rule `claimed`/`eligible` follow on
-#                  the board side. A malformed or truncated block is reported
-#                  the same way: the round is simply absent, and every other
-#                  field still parses.
+#                  through, read from `## Status` `Rounds:` first, YAML front
+#                  matter `rounds:` second, and the `CHALLENGE-THE-PLAN-METADATA`
+#                  block last. The field wins a disagreement (the file states
+#                  what the reader sees), and the block remains readable so the
+#                  40 plans carrying only a block go on reporting correctly.
+#                  **OMITTED ENTIRELY when no source carries a readable round**
+#                  — absent is not zero. A plan nobody has interrogated and a
+#                  plan interrogated to no effect want opposite reactions from a
+#                  reader, so the field is missing rather than 0, the same rule
+#                  `claimed`/`eligible` follow on the board side. A malformed or
+#                  truncated block is reported the same way: the round is simply
+#                  absent, and every other field still parses.
 #
 # title/sprint/story/assignee are the board-facing surface (`@plot-pm/board`
 # consumes this script instead of parsing plans itself). Front matter wins over
@@ -204,7 +219,7 @@ if [ ${#files[@]} -eq 0 ] && [ ${#missing[@]} -eq 0 ]; then
 fi
 
 for f in ${missing[@]+"${missing[@]}"}; do
-  printf '{"file":"%s","format":"none","error":"file not found","phase_raw":"","phase":"NONE","phase_alt_raw":"","phase_alt":"NONE","type":"","title":"","sprint":"","story":"","assignee":"","branches":[],"prs":[],"issues":[],"malformed_prs":[],"changelog":[],"review_raw":"","review":"NONE","impl_raw":"","impl":"NONE","design_raw":"","approved_raw":"","released_raw":"","delivered_raw":"","started_raw":[]}\n' \
+  printf '{"file":"%s","format":"none","error":"file not found","phase_raw":"","phase":"NONE","phase_alt_raw":"","phase_alt":"NONE","type":"","title":"","sprint":"","story":"","assignee":"","branches":[],"prs":[],"issues":[],"malformed_prs":[],"changelog":[],"long_wave_names":[],"review_raw":"","review":"NONE","impl_raw":"","impl":"NONE","design_raw":"","approved_raw":"","released_raw":"","delivered_raw":"","started_raw":[]}\n' \
     "$(printf '%s' "$f" | sed 's/\\/\\\\/g; s/"/\\"/g')"
 done
 
@@ -278,15 +293,17 @@ function reset_state() {
   fm_title = ""; fm_sprint = ""; fm_story = ""; fm_assignee = ""
   fm_review = ""; fm_impl = ""; fm_approved = ""; fm_started = ""; fm_released = ""
   fm_delivered = ""; fm_design = ""
+  fm_rounds = ""
   canon_phase = ""; canon_type = ""
   canon_sprint = ""; canon_story = ""; canon_assignee = ""
   canon_review = ""; canon_impl = ""; canon_approved = ""; canon_released = ""
   canon_delivered = ""; canon_design = ""
+  canon_rounds = ""
   h1_title = ""
   # "" means the plan carries no readable round — NOT zero. Emitted by omitting
   # the field, so a consumer cannot mistake "never interrogated" for "asked
   # nothing".
-  rounds = ""
+  block_rounds = ""
   in_fm = 0; section = ""; in_comment = 0; in_challenge = 0; in_fence = 0; branches_seen = 0; waves_seen = 0
   delete branches; n_branches = 0
   delete prs; n_prs = 0
@@ -399,6 +416,27 @@ function emit_record(   fmt, praw, palt_raw, traw, title, sprint, story, assigne
     out = out "]}"
   }
   out = out "]"
+  # long_wave_names[]: wave names past LONG_WAVE_NAME_MAX, in document order. A
+  # wave name is a label (`Shaped`, `Gated`, `Offered first`); a sentence-length
+  # heading is a plan-authoring mistake the board can only render badly. This
+  # REPORTS the offenders — a top-level field, never a change to waves[], so the
+  # plan still parses and every existing consumer of waves[] is untouched. The
+  # threshold is a JUDGEMENT baked into one constant (see LONG_WAVE_NAME_MAX):
+  # the longest legitimate name in the estate is `Offered first` at 13, the
+  # offender that motivated this is 53. Measured over the whole array so BOTH
+  # spellings (## Branches and ## Waves) are covered from one place. length()
+  # here is the awk byte length, which can exceed the character count for a name
+  # with a multi-byte dash — harmless: the threshold clears every real name by
+  # tens of bytes either way, so the verdict never turns on the last byte.
+  out = out ",\"long_wave_names\":["
+  lwn = 0
+  for (w = 1; w <= n_waves; w++) {
+    if (length(wave_names[w]) > LONG_WAVE_NAME_MAX) {
+      out = out (lwn > 0 ? "," : "") "\"" jesc(wave_names[w]) "\""
+      lwn++
+    }
+  }
+  out = out "]"
   out = out ",\"review_raw\":\"" jesc(review) "\",\"review\":\"" norm_review(review) "\""
   out = out ",\"impl_raw\":\"" jesc(impl) "\",\"impl\":\"" norm_impl(impl) "\""
   out = out ",\"design_raw\":\"" jesc(design) "\""
@@ -408,12 +446,24 @@ function emit_record(   fmt, praw, palt_raw, traw, title, sprint, story, assigne
   out = out ",\"started_raw\":["
   for (i = 1; i <= n_started; i++) out = out (i > 1 ? "," : "") "\"" jesc(started[i]) "\""
   out = out "]"
-  # OMITTED, not zeroed, when no block was found. The absence is the message.
+  # Rounds: preference order: ## Status, front matter, CHALLENGE block fallback.
+  # A placeholder ("<!-- optional -->") counts as absent, like Sprint/Story.
+  # OMITTED, not zeroed, when no source carries a readable round.
+  rounds = ""
+  _r = strip_placeholder(canon_rounds)
+  if (_r == "") _r = strip_placeholder(fm_rounds)
+  if (_r == "") _r = block_rounds
+  if (_r != "" && _r ~ /^[0-9]+$/) rounds = _r + 0
   if (rounds != "") out = out ",\"rounds\":" rounds
   out = out "}"
   print out
 }
-BEGIN { branch_re = "`(" PREFIXES ")/[^`]+`" }
+# The longest wave name that still reads as a label, not prose. A JUDGEMENT, not
+# a measurement: the longest legitimate name in the estate is `Offered first`
+# (13), and the offender this exists to catch is a 53-character sentence, so the
+# line sits well clear of both. Reported, never enforced — a name past it makes
+# `long_wave_names`, and nothing refuses the plan.
+BEGIN { branch_re = "`(" PREFIXES ")/[^`]+`"; LONG_WAVE_NAME_MAX = 40 }
 FNR == 1 {
   if (NR > 1) emit_record()
   reset_state()
@@ -442,6 +492,7 @@ in_fm {
   # here: front matter in this repo is a flat key/value surface, and guessing a
   # list grammar the format never promised would invent a contract.
   else if (lower ~ /^changelog:/ && fm_changelog == "") fm_changelog = val_after_colon($0)
+  else if (lower ~ /^rounds:/ && fm_rounds == "") fm_rounds = val_after_colon($0)
   next
 }
 # Interior of multi-line HTML comments is non-content (template guidance
@@ -454,11 +505,11 @@ in_fm {
 # block (question history, category coverage) stays non-content — the script
 # collects, the skill interprets.
 in_comment {
-  if (in_challenge && rounds == "" && $0 ~ /^[ \t]*"round"[ \t]*:[ \t]*[0-9]+/) {
+  if (in_challenge && block_rounds == "" && $0 ~ /^[ \t]*"round"[ \t]*:[ \t]*[0-9]+/) {
     _r = $0
     sub(/^[ \t]*"round"[ \t]*:[ \t]*/, "", _r)
     sub(/[^0-9].*$/, "", _r)
-    if (_r != "") rounds = _r + 0
+    if (_r != "") block_rounds = _r + 0
   }
   if ($0 ~ /-->/) { in_comment = 0; in_challenge = 0 }
   next
@@ -518,6 +569,7 @@ section == "status" {
   else if (lower ~ /^[ \t]*[-*]?[ \t]*\**review[:*]/ && canon_review == "") canon_review = val_after_colon($0)
   else if (lower ~ /^[ \t]*[-*]?[ \t]*\**impl[:*]/ && canon_impl == "") canon_impl = val_after_colon($0)
   else if (lower ~ /^[ \t]*[-*]?[ \t]*\**design[:*]/ && canon_design == "") canon_design = val_after_colon($0)
+  else if (lower ~ /^[ \t]*[-*]?[ \t]*\**rounds[:*]/ && canon_rounds == "") canon_rounds = val_after_colon($0)
   else if (lower ~ /^[ \t]*[-*]?[ \t]*\**approved[:*]/ && canon_approved == "") canon_approved = val_after_colon($0)
   else if (lower ~ /^[ \t]*[-*]?[ \t]*\**released[:*]/ && canon_released == "") canon_released = val_after_colon($0)
   else if (lower ~ /^[ \t]*[-*]?[ \t]*\**delivered[:*]/ && canon_delivered == "") canon_delivered = val_after_colon($0)