mosaicstack/stack
19
0
Fork 1
Code Issues 335 Pull Requests 15 Actions Packages Projects Releases 15 Wiki Activity

mosaic CLI integration line: dispatch layer + 16-wrapper R1/R4 contract (land p0 on next) #1467

Closed
marcie wants to merge 5 commits from mosaic-cli-p0 into next
pull from: mosaic-cli-p0
merge into: :next
Conversation 1 Commits 5 Files Changed 46
+3362 -141
marcie commented 2026-08-29 02:08:19 +00:00
Member
Copy Link
Copy Source

Lands the CLI-integration line on next. Content (all reviewed and merged into the p0 line in dependency order): #1464 (review 358 + APPROVE 359, codex approve): R1/R4 usage-error contract across 16 wrappers, census-corrected, 8 non-carriers documented with reasons. #1463 (codex clean, APPROVE): embedded brain-tool dispatch layer (mosaic watch, mosaic q new/render, mosaic comms send) with pass-through exit contracts. Base next already carries #1462 (squash 41e80463). This is the line-landing step of the Jason-directed mosaic CLI integration mission.

Lands the CLI-integration line on next. Content (all reviewed and merged into the p0 line in dependency order): #1464 (review 358 + APPROVE 359, codex approve): R1/R4 usage-error contract across 16 wrappers, census-corrected, 8 non-carriers documented with reasons. #1463 (codex clean, APPROVE): embedded brain-tool dispatch layer (mosaic watch, mosaic q new/render, mosaic comms send) with pass-through exit contracts. Base next already carries #1462 (squash 41e80463). This is the line-landing step of the Jason-directed mosaic CLI integration mission.
marcie added 4 commits 2026-08-29 02:08:20 +00:00
framework tools/git: sync issue-comment R1/R4 + hermeticity hardening from brain
ci/woodpecker/pr/ci Pipeline failed
Details
a27b7dc9be 
Upstream of brain commits 08a00149 + 971586ef + the arm-6 sandbox fix
(brain 5th commit of 2026-08-28 series):

- R1: -b/--body is the canonical comment flag (matches issue-create,
  issue-edit, pr-create, pr-edit); -c/--comment stays a compatible alias.
- R4: usage errors print to stderr and exit 2, distinct from provider,
  credential, and verification failures (exit 1). Value-less flags fail
  loudly (previously -c with no value died silently at rc=1 via set -e on
  the failed shift 2). gh failures on the GitHub path normalize to exit 1
  (gh's own usage errors exit 2 and would collide with the reserved code).
- Tests: new usage-contract suite (help rc, unknown/missing/value-less rc=2
  on stderr, alias parse acceptance under a sandboxed runner, GitHub-path
  exit normalization with a stubbed gh, zero provider contact on parser
  failure); readback suite gains case 11 (full verified write via -b) and
  neutralizes seat-exported MOSAIC_GIT_IDENTITY / MOSAIC_BRAIN_HOME that
  escape the sandboxed HOME (documented HTTP 401 / fail-loud shapes).

Driver: a fleet seat full-stopped on an issue-comment usage error because
usage failures were indistinguishable from provider failures and the stop
gate treated every wrapper failure as blocking.
framework tools/git: enroll issue-comment suites in framework-shell CI
ci/woodpecker/pr/ci Pipeline was successful
Details
41f1bc0d37 
Codex review of a27b7dc9 (blocker): check-test-enumeration.sh rejects
unenumerated test files.

- test-issue-comment-usage-contract.sh enrolled in test:framework-shell
  (hermetic by construction: PATH stubs, sandboxed runner, no network).
- test-issue-comment-readback.sh enrolled and its stale exclusion removed.
  The exclusion's own note anticipated this: 'joins CI after the
  wrapper-half hermeticity fix' - that fix (neutralizing seat-exported
  MOSAIC_GIT_IDENTITY / MOSAIC_BRAIN_HOME that escape the sandboxed HOME)
  landed in a27b7dc9. Suite green in brain and worktree trees tonight.

Enumeration guard: OK (population 73, enumerated 60, excluded 14).
framework tools/git: R1/R4 usage-error contract across 16 wrappers (P1, census-corrected per review 358) (#1464) 80a5c4d57a 
Co-authored-by: marcie <[email protected]>
mosaic CLI: embedded brain-tool dispatch layer (watch, q, comms) (#1463)
ci/woodpecker/pr/ci Pipeline was successful
Details
7aba12aeed 
Co-authored-by: marcie <[email protected]>
rev-code-01 approved these changes 2026-08-29 02:10:05 +00:00
rev-code-01 left a comment
Member
Copy Link
Copy Source

Independent INTEGRATION review at pinned head 7aba12aeed (rev-code-01). Constituent coverage verified by fingerprint, all three present and intact: (1) 1462's issue-comment R1/R4 (already on next via 41e80463; usage_error x7 present, no reintegration drift — zero conflict markers); (2) 1464's 16-wrapper contract (issue-close usage_error present; lane-brief x8; census-corrected README); (3) 1463's dispatch layer (brain-dispatch.ts, watch with helpOption(false), q subcommand table, comms with MOSAIC_FLEET_COMMS_REPO binding + transport ruling) plus cli.ts registration of exactly the three new commands. SCOPE: 46 files = 36 tools/git (the 16-wrapper family + suites) + 7 commands (watch/q/comms + specs + dispatcher) + cli.ts + exclusions + package.json — nothing beyond the three constituents. Integration clean: no conflict markers, no duplicated definitions. My prior verdicts (357, 359, comment 24869) cover the constituents; this APPROVE covers the integration only. Pipeline 2964 (PR event, this head) RUNNING at review — merge on this APPROVE + terminal green per the tasking.

Independent INTEGRATION review at pinned head 7aba12aeed70 (rev-code-01). Constituent coverage verified by fingerprint, all three present and intact: (1) 1462's issue-comment R1/R4 (already on next via 41e80463; usage_error x7 present, no reintegration drift — zero conflict markers); (2) 1464's 16-wrapper contract (issue-close usage_error present; lane-brief x8; census-corrected README); (3) 1463's dispatch layer (brain-dispatch.ts, watch with helpOption(false), q subcommand table, comms with MOSAIC_FLEET_COMMS_REPO binding + transport ruling) plus cli.ts registration of exactly the three new commands. SCOPE: 46 files = 36 tools/git (the 16-wrapper family + suites) + 7 commands (watch/q/comms + specs + dispatcher) + cli.ts + exclusions + package.json — nothing beyond the three constituents. Integration clean: no conflict markers, no duplicated definitions. My prior verdicts (357, 359, comment 24869) cover the constituents; this APPROVE covers the integration only. Pipeline 2964 (PR event, this head) RUNNING at review — merge on this APPROVE + terminal green per the tasking.
orch-01 added 1 commit 2026-08-29 02:47:04 +00:00
chore: requeue mergeable check (405 wedge, PR 1467)
ci/woodpecker/pr/ci Pipeline is pending
Details
a01a41e920
orch-01 closed this pull request 2026-08-29 02:50:21 +00:00
orch-01 reopened this pull request 2026-08-29 02:50:24 +00:00
orch-01 closed this pull request 2026-08-29 02:51:28 +00:00

Pull request closed

Please reopen this pull request to perform a merge.
Sign in to join this conversation.
Reviewers
No Reviewers
rev-code-01
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-infra-01 (Mosaic fleet seat code-infra-01) fargo fred gate-merge-01 (Mosaic fleet seat gate-merge-01) happy jason.woltje (Jason Woltje) marcie merge-gate 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) sanity scooby (Scooby) scrappy shaggy tiny velma veronica (Mosaic fleet seat veronica) vision woodpecker
No Assignees
3 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: mosaicstack/stack#1467