Repository navigation
task-monitor 3/6: PM short-term queue (task-monitor next …): add task-monitor-queue - #281
Merged
Merged
Conversation
Owner
Author
Spec complianceChecked against #275 (scope bullets):
Code-review findingsNo blockers. Two small nits, both fixable in this PR:
Not blocking, noting for awareness: Verdictclean-with-nits — nit 1 is worth the one-line fix; nothing blocks merge. |
…-monitor-queue New canonical root binary `bin/task-monitor-queue` (named by the PM: the user-facing `task-monitor` is the #276 front end). The PM writes the board's Next section with `set <ref>... [--note <ref>=<text>]`, `add`, `rm`, `clear`, or no subcommand to show it. Each write answers with one line, so the PM's board cost is one short call. It owns the queue file format that `task-monitor-snapshot` already reads: `$TASK_FORCE_HOME/monitor/<repo>/queue`, one ref per line with an optional tab-separated note. Refs are compared by their last path segment, so `267`, `#267` and an issue URL are one item. `task-work` uses that to drop the ref it dispatches, best effort, so a dispatched item never returns as next once it leaves In flight. Every PM prompt (four claude `commands/pm.md`, this repo's copy, three kiro `agents/pm.json`) gains one paragraph on when to call it. All seven installers link it; README, workflow templates, schema doc and CHANGELOG updated. New tests/task_monitor_queue.bats (21 tests under /bin/bash, mutation-checked) and a symlink test. Closes #275 Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
…review round 1 - Every leading `#` of a ref is stripped, so `##5` is stored as `5` instead of `#5`, which the reader skipped as a comment. - `--` ends the options, so a ref starting with `-` can be passed. - A URL ref's query string and fragment are dropped from its key: a Notion page URL with and without `?pvs=4` is one item. - Documented that `--note` splits on the first `=`, so a ref containing `=` cannot take a note. - One comment naming the accepted race between a PM `set` and the task-work `rm`. Three new tests, each mutation-checked. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
…k-monitor-queue in the task-monitor docs After rebasing onto #282: the README's snapshot and `task-monitor` sections and the `task-monitor` block in all seven workflow templates and this repo's workflow doc now say the Next section is the queue the PM writes with `task-monitor-queue`. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
martin-conur
force-pushed
the
task/monitor-queue
branch
from
October 20, 2026 19:14
7bd8228 to
334b8bb
Compare
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
The PM now has a one-line command,
task-monitor-queue, to record what comes next in a repo; that list is the "Next" section of the task-monitor board.task-worktakes an item off the list when it dispatches it, so the PM never has to tidy the board. Every PM prompt learns when to use it, in one short paragraph.Closes #275
What changed
bin/task-monitor-queue(new, canonical, bash 3.2):set <ref>... [--note <ref>=<text>],add,rm [--quiet],clear, andshow(the default);--repo DIRon all. Each write prints one line (queue: 267 266 180). It writes$TASK_FORCE_HOME/monitor/<repo>/queuein exactly the formattask-monitor-snapshotalready reads, and now owns it. Namedtask-monitor-queueper the PM's naming note (the user-facingtask-monitoris task-monitor 4/6: ratatui front end (repo view + all-repos overview, real-time) #276's front end).#and any URL/path prefix are dropped, so267,#267and.../issues/267are one item;tasks/012-x.mdand012-x.mdlikewise.setkeeps the note of a ref that stays queued;--note <ref>=clears it.bin/task-work: after writing the sidecar, best-efforttask-monitor-queue rm --quietof the dispatched ref. Not literally in the spec, but without it a merged item falls out of In flight and reappears in Next until the PM spends a call removing it — the token cost the spec says to avoid.commands/pm.md, this repo's.claude/commands/pm.md, and the three kiroagents/pm.json, with a tracker-appropriate example ref..claude/gh-workflow.md, schema doc, snapshot comment, CHANGELOG (with upgrading note: re-runinstall.shandtask-init <loadout>).Tests
tests/task_monitor_queue.bats: 24 tests under/bin/bash— each subcommand, key matching, notes, refusals, linked-worktree and--reporesolution, a round trip throughtask-monitor-snapshot --local, task-work's drop, and the PM prompts. Mutation-checked: disabling task-work's drop, key matching, note carry-over and note cleaning each turn the matching tests red.tests/install_symlinks.bats: link test../run_tests.sh: 1460/1460 ok.tools/check-drift.shandshellcheck -xclean.Not covered
fix-auth) does not match the dispatchedtasks/012-fix-auth.md; the PM prompts say to queue it in the form passed totask-work.setracing atask-workrmcan lose one update (accepted, commented in the code).Review round 1
Leading
#s all stripped (##5no longer vanishes),--ends options, URL query string and fragment ignored when keying (Notion?pvs=4),=-in-ref note limit documented, race comment added. Three new tests, mutation-checked; queue suite now 24, full suite 1460/1460.🤖 Generated with Claude Code