Skip to content

feat(pr-workflow): add /mms-debug orchestrator - #98

Open
MajorLift wants to merge 1 commit into
mainfrom
jongsun/add/debug-orchestrator
Open

feat(pr-workflow): add /mms-debug orchestrator#98
MajorLift wants to merge 1 commit into
mainfrom
jongsun/add/debug-orchestrator

Conversation

@MajorLift

@MajorLift MajorLift commented Jul 31, 2026

Copy link
Copy Markdown
Contributor

Summary

  • Adds debug, the symptom-first sibling of evidence. evidence is handed a claim and looks for the observation that would falsify it; debug is handed a symptom and has to generate the hypothesis before anything can falsify it.
  • Routes a symptom to the engine that owns its defect class — memory-leak, race-condition-repro, react-render-delta, sentry-grafana-correlation, extension-errors-debugging, tsc-blindspots, supply-chain-audit. It chooses among engines; it does not reimplement any investigation, so both orchestrators share one engine set.
  • Carries evidence's trust gates, which bind harder here: a weak instrument in review produces a claim someone challenges, in debugging it produces a theory nobody checks.
  • Killed hypotheses are part of the output — deleting them makes the survivor look inevitable and hands the next investigator the same dead ends.

Note: extension-errors-debugging (#44) is currently routed to by no orchestrator, since it is not evidence for a PR claim. This gives it an entry point.

Test plan

  • node .github/scripts/lint-skill-entry.mjs domains/pr-workflow/skills/debug/skill.md — 0 errors, 0 warnings
  • Frontmatter: name matches directory, maturity: experimental, description within the 1536-char budget
  • Every engine named in the routing table exists or has an open PR

`pr-validate` is handed a claim and looks for the observation that would falsify
it. Debugging starts from a symptom and has to generate the hypothesis first,
which is where the expensive failure lives: the theory is yours, nobody else is
positioned to challenge it, and confirmation is cheap.

Routes the symptom to the engine that owns its defect class rather than
reimplementing any investigation, so both orchestrators share one set of engines.
Carries `pr-validate`'s trust gates, which bind harder here — a weak instrument
in review yields a claim someone challenges, in debugging it yields a theory
nobody checks.
@MajorLift MajorLift changed the title feat(pr-workflow): add debug orchestrator feat(pr-workflow): add /mms-debug orchestrator Jul 31, 2026
@MajorLift

Copy link
Copy Markdown
Contributor Author

Context budget

What this PR costs an agent, measured from an install rather than read from the diff. Three tiers, and only the first is unavoidable.

Skill Frontmatter Selected + refs & knowledge
debug 1,151 chars ~1,716 tok ~1,716 tok

Frontmatter is the only tier paid unconditionally — every agent loads it on every run once the skill is installed, used or not, because it is what the agent reads to decide relevance. The 28 skills across the eleven open skill PRs sit at a median of ~1,716 tokens selected and ~1,860 with references followed. All are within the 1,536-character description budget.

Selected is paid only when the agent picks the skill. + refs & knowledge is the ceiling if every bundled reference is then read; it is a worst case, not an expectation.

Method

tools/install --repo metamask-extension --maturity experimental against this branch at 703242599, measured per installed skill directory. Repo overlays are merged into the emitted SKILL.md, so they land in the selected tier rather than being missed by a source-byte count. Token figures are bytes/4 — a proxy for scale, not accounting.

These figures are pinned to the commit above and drift on every push; #96 tracks automating them.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant