refactor(daemon): compress brief prose and demote the sub-issue playbook (MUL-5442) - #6310
Merged
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
…ook (MUL-5442) Six static edits to the issue brief, -1,249 bytes on the standard fixture (17,977 -> 16,728). All edits are identical for every issue run; prompt-cache byte stability (#6008) is untouched. - BTS persistent-service bullet: keep the handoff contract (deliverable-only, lifecycle detached, readiness verified, best-effort survival), drop the operational walkthrough. Pins re-anchored to concepts (durable logs, recorded PID, verify readiness). - BTS CI-ban bullet: keep the command blacklist and the complete-hand-off rule, drop the auto-merge/snapshot elaboration. - Ownership mode: state the identity-forbids clause once on the header instead of once per status bullet. - Delivery invariant: merge the three sub-bullets into the lead paragraph; also fixes the stale '(below)' — the per-surface delivery line renders above the invariant, not below it. - Sub-issue Creation: demote the todo/backlog/stage playbook to the multica-working-on-issues skill; the brief keeps a one-line flag map plus the skill pointer. Skill-side anchors added to TestWorkingOnIssuesSkillCoversIssueLoopContracts so the pointer cannot dangle. - Attachments: collapse the two-sentence CLI-fetch intro into one line. Every pinned behavioral phrase is either carried verbatim or re-pinned to an equivalent semantic anchor in the same assertion; no assertion is deleted without a replacement. Co-authored-by: multica-agent <github@multica.ai>
Bohan-J
force-pushed
the
agent/steve-jobs/29a60922
branch
from
August 3, 2026 11:08
65c8408 to
f369a90
Compare
Bohan-J
changed the base branch from
refactor/brief-cross-section-dedup
to
main
August 3, 2026 11:08
…L-5442) Review catch by Elon on #6310: the compressed bullet weakened the contract two ways — the reply requirement dropped 'logs' from the URL/logs/stop triple (durable logs alone are unobservable if the user is never told where they are), and the general ownership/cleanup handle narrowed to a bare PID (a supervisor/profile-managed service has no single stable PID). Both test pin sets had been updated to the weakened phrases, which would have made the regression look legitimate. Keeps the compressed sentence shape; restores both halves of the contract and pins them ("cleanup handle such as PID/profile", "URL, logs, and stop instructions") so they cannot be compressed away again. +38 bytes; the PR still nets -1,211 against main. Co-authored-by: multica-agent <github@multica.ai>
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
1 participant
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.
Part of MUL-5442 (GH #4358). Stacked on #6302 — base is
refactor/brief-cross-section-dedup; retarget tomainafter #6302 merges.Implements the two greenlit optimization tracks: prose compression (point 2) and skill demotion (point 3). Six static edits, identical for every issue run — #6008 byte stability untouched.
Measured result (standard issue fixture, no org content)
Per section: BTS 3,378 → 3,065 · Sub-issue Creation 795 → 321 · Output 1,664 → 1,466 · Attachments 451 → 331 · Instruction Precedence 5,245 → 5,101.
Edits
stdio redirected to durable logs,PID/profile,verify readiness before replying) to the concepts (durable logs,recorded PID,verify readiness) — per the pin-inventory disposition discussed on the issue.unless your Agent Identity forbids issue status changes; if it does, skip itclause was stated on all three status bullets; now stated once on the section header. Sentence pins atruntime_config_test.goupdated to the new shape (clause pinned once + three short command sentences).Runtime-local paths are never deliverables,NEVER write an absolute path or a file:// URL as a clickable link,path/to/file.ts:42) survive verbatim with zero test changes. Also fixes a stale cross-reference: the text said "(below)" but the per-surface delivery line renders above the invariant.multica-working-on-issuesbuilt-in skill (which already documents all of it, §"Sub-issues" and §"Stages"); the brief keeps a one-line flag map + skill pointer. Anti-dangling guard: skill-side anchors added toTestWorkingOnIssuesSkillCoversIssueLoopContractsso the playbook cannot silently leave the skill while the brief still points at it.Semantic invariants preserved
Tests
go test ./internal/daemon/...green (daemon, execenv, repocache);go test -run 'Skill|Builtin' ./internal/service/green;go build ./...,go vetclean../internal/service/full-suite failures are pre-existing local schema drift (column "kind" does not exist, needs migrated Postgres) — identical failure set without this change; no SQL touched here.