mosaicstack/stack
25
0
Fork 1
Code Issues 365 Pull Requests 13 Actions Packages Projects Releases 15 Wiki Activity

Review loop through the bus: push_working, request_review, fetch_review, review_verdict #1587

New Issue
Open
opened 2026-10-12 19:50:03 +00:00 by jarvis · 0 comments
jarvis commented 2026-10-12 19:50:03 +00:00
Contributor
Copy Link
Copy Source

The coder holds git.push.working and review.request and the reviewer holds review.verdict, but no broker verb or typed tool exists for them (PM findings faults 3 and 5). The host pushes a coder branch to a per-project working repository under the data root, records the review request, fetches it into the reviewer's clone, and records the verdict on the task. Owner Filbert after rows 91 and 95; reviewers Darkwing and Dewey.

Brief: docs/plans/2026-10-12_pm-test-fixes.md, section "A review loop through the bus". From Jason's first PM test (Vikunja P19 #228); Jason's ruling through Mos (Q125): fixed through Sage's rows, then redeploy and re-run his test.

The coder holds git.push.working and review.request and the reviewer holds review.verdict, but no broker verb or typed tool exists for them (PM findings faults 3 and 5). The host pushes a coder branch to a per-project working repository under the data root, records the review request, fetches it into the reviewer's clone, and records the verdict on the task. Owner Filbert after rows 91 and 95; reviewers Darkwing and Dewey. Brief: `docs/plans/2026-10-12_pm-test-fixes.md`, section "A review loop through the bus". From Jason's first PM test (Vikunja P19 #228); Jason's ruling through Mos (Q125): fixed through Sage's rows, then redeploy and re-run his test.
Sign in to join this conversation.
No Branch/Tag Specified
Branches Tags
Labels
Clear labels
bug
Defect in framework tooling or delivered code
 fleet-enhancement
Fleet-wide reliability/speed/cost improvements surfaced by the nightly enhance pass
No labels
Milestone
No items
No Milestone
Projects
Clear projects
No projects
Assignees
Clear assignees
code-be-01 (Mosaic fleet seat code-be-01) code-be-02 (Mosaic fleet seat code-be-02) code-dogfood-01 (Mosaic fleet seat code-dogfood-01) code-infra-01 (Mosaic fleet seat code-infra-01) darkwing (Mosaic fleet seat darkwing) dewey (Mosaic fleet seat dewey) fargo filbert (Mosaic fleet seat filbert) fred gate-merge-01 (Mosaic fleet seat gate-merge-01) happy jason.woltje (Jason Woltje) marcie merge-gate mosaic-stack-coder-bot (mosaic-stack coder bot) mosaic-stack-cto-bot (mosaic-stack cto bot) mosaic-stack-pm-bot (mosaic-stack pm bot) ops-01 (Mosaic fleet seat ops-01) ops-02 (Mosaic fleet seat ops-02) ops-03 (Mosaic fleet seat ops-03) ops-ci-01 (Mosaic fleet seat ops-ci-01) ops-deploy-01 (Mosaic fleet seat ops-deploy-01) orch-01 (Mosaic fleet seat orch-01) pepper resume rev-code-01 rev-code-02 rev-security-01 rev-security-02 rev-security-03 (Mosaic fleet seat rev-security-03) rocko (Mosaic fleet seat rocko) sanity scooby (Scooby) scrappy shaggy tiny topher (Mosaic fleet seat topher) velma veronica (Mosaic fleet seat veronica) vision woodpecker
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: mosaicstack/stack#1587