You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@magpie.apache.org by "Kaap10 (via GitHub)" <gi...@apache.org> on 2026/10/02 07:45:39 UTC

[PR] perf(contributor-growth): trim contributor-to-committer body budget [magpie]

Kaap10 opened a new pull request, #1487:
URL: https://github.com/apache/magpie/pull/1487

   <!-- SPDX-License-Identifier: Apache-2.0
        https://www.apache.org/licenses/LICENSE-2.0 -->
   
   ## Summary
   
   - Trim `contributor-to-committer` body budget from **5,741 tokens** to **4,591 tokens** (-20.0%, saving **1,150 tokens**), bringing the skill under the framework's $\le 5,000$ token ceiling.
   - Extract Step 5 brief template layout, bar charts, and rendering rules byte-identically into companion file `render-brief.md` with an explicit pointer in `SKILL.md`.
   - Tighten frontmatter routing metadata to **~58 tokens** ($\le 200$ budget) while strictly preserving all 4 quoted routing triggers and boundary rules.
   
   ## Type of change
   
   - [x] Skill change (`.claude/skills/<name>/`) — eval fixtures updated below
   - [ ] Tool / bridge contract (`tools/<system>/*.md`)
   - [ ] Python package (`tools/*/` with `pyproject.toml`)
   - [ ] Groovy reference impl
   - [ ] Cross-cutting (RFC, AGENTS.md, sandbox, privacy-LLM)
   - [x] Documentation (`docs/`, `README.md`, `CONTRIBUTING.md`)
   - [ ] Project template (`projects/_template/`)
   - [ ] CI / dev loop (`prek`, workflows, validators)
   - [ ] Other:
   
   ## Test plan
   
   - [x] `prek run --all-files` passes
   - [ ] For Python packages touched: `uv run pytest` / `ruff check` / `mypy` passes
   - [ ] For Groovy bridges touched: command-line invocation tested end-to-end
   - [x] For skill changes: eval suite passes for the affected skill
         (`PYTHONPATH=tools/skill-evals/src python3 -m skill_evals.runner tools/skill-evals/evals/contributor-to-committer/`)
   - [ ] For skill *behaviour* changes: a new or updated eval fixture is included in this PR
         (a regression test for the bug fixed / the behaviour added — see CONTRIBUTING.md)
   - [x] Other: `measured_tokens` stamped at 4591 (`tiktoken cl100k_base`), `surface_hash` stamped at `sha256:b702cd2ce6ef9120`, and `docs/setup/marketplace.md` synced to `~0.6k`.
   
   ## RFC-AI-0004 compliance
   
   - [x] **HITL** — any new mutation is gated on explicit user confirmation
   - [x] **Sandbox** — no new unrestricted host access; network reach declared in the adapter
   - [x] **Vendor neutrality** — placeholders (`<PROJECT>`, `<tracker>`, `<upstream>`, `<security-list>`) used in all skill / tool prose (the `check-placeholders` prek hook is the mechanical gate)
   - [x] **Conversational + correctable** — agentic-override path documented if behaviour is adopter-tunable
   - [x] **Write-access discipline** — no autonomous outbound messages; drafts only, sent on confirmation
   - [ ] **Privacy LLM** — private content does not reach a non-approved LLM; redactor invoked where needed
   
   ## Linked issues
   
   Refs #1348, Refs #1342
   
   ## Notes for reviewers (optional)
   
   Part of #1348 Phase 2 (Body Budget Trimming) under the #1342 optimization umbrella:
   - Preserves all 4 quoted routing triggers (`"how close is <handle> to being a committer"`, `"is <handle> approaching the bar"`, `"track <handle>'s path to committer"`, `"what does <handle> still need for nomination"`).
   - Step headings and interactive prompt (`### After presenting the brief`) remain directly in `SKILL.md` for complete eval fixture compatibility.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@magpie.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org