Skip to content

chore(skills): sync simplify skill to tiered version#598

Merged
coseto6125 merged 1 commit into
mainfrom
feat/sync-simplify-skill
Jun 23, 2026
Merged

chore(skills): sync simplify skill to tiered version#598
coseto6125 merged 1 commit into
mainfrom
feat/sync-simplify-skill

Conversation

@coseto6125

Copy link
Copy Markdown
Owner

The repo's simplify skill source (skill_sample/{claude,codex}/simplify) lagged the installed copy. The installed version is the tiered dispatch rewrite; the repo still carried the old unconditional three-agent split.

What changed

  • Tier ladder (0–3) replaces always-fan-out:
    • Tier 0: docs/comments/lockfile-only diff → no review
    • Tier 1: <3 files or <100 LOC → zero agents, single-pass self-review (~50–70k tokens saved vs fan-out)
    • Tier 2: ≤10 files and ≤400 LOC → one agent, merged checklist
    • Tier 3: bigger / cross-crate / HIGH-risk → dimension-selected parallel fan-out (2–4 agents, not unconditional)
  • Adds a dedicated Correctness (bugs) review dimension (logic/boundary/error-handling/concurrency/contract/security).
  • Confidence-gated aggregation (≥70 fix, 50–69 surface, <50 drop).
  • Codex variant synced to match.

Why

So ecp admin claude install skills simplify installs the current tiered version instead of downgrading users to the three-agent one. Pure skill-doc change; no Rust touched.

The repo's simplify skill source (skill_sample/{claude,codex}/simplify)
lagged the installed copy. The installed version is the tiered dispatch
rewrite: a Tier 0–3 ladder (Tier 0 docs-only → no review; Tier 1 small
diff → zero-agent single-pass self-review; Tier 2 → one agent; Tier 3 →
dimension-selected parallel fan-out) plus a dedicated correctness/bugs
review dimension and confidence-gated aggregation. This replaces the old
unconditional three-agent split, saving ~50–70k tokens on the common
small-diff pre-push path.

Sync the repo source to match so `ecp admin claude install skills simplify`
installs the current version instead of downgrading to the three-agent one.
@coseto6125 coseto6125 enabled auto-merge (squash) June 23, 2026 10:50
@coseto6125 coseto6125 added the merge-queue Opt-in to Mergify merge queue label Jun 23, 2026
@github-actions

Copy link
Copy Markdown
Contributor
ecp impact cache (0 symbols) — internal, used by ecp dev pr-analyze

[]

@github-actions github-actions Bot added the ecp:risk-low ecp signal label Jun 23, 2026
@coseto6125 coseto6125 merged commit 653561d into main Jun 23, 2026
18 checks passed
@coseto6125 coseto6125 deleted the feat/sync-simplify-skill branch June 23, 2026 11:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ecp:risk-low ecp signal merge-queue Opt-in to Mergify merge queue

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant