Skip to content

feat(utilities): add zero-token /magpie-skills Claude Code mod command (part of #1497) - #1667

Merged
potiuk merged 1 commit into
apache:mainfrom
Kaap10:feat/claude-code-mod-skills-command-1497
Oct 16, 2026
Merged

potiuk merged 1 commit into
apache:mainfrom
Kaap10:feat/claude-code-mod-skills-command-1497

Conversation

@Kaap10

@Kaap10 Kaap10 commented Oct 16, 2026

Copy link
Copy Markdown
Contributor

Summary

Type of change

  • 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)
  • Documentation (docs/, README.md, CONTRIBUTING.md)
  • Project template (projects/_template/)
  • CI / dev loop (prek, workflows, validators)
  • Other: Claude Code mod hooks (plugins/magpie-utilities/hooks/)

Test plan

  • Ran claude plugin validate --strict plugins/magpie-utilities (Validation passed, minimal API surface: $.command.register, $.process.run, $.session.root).
  • Ran claude plugin test plugins/magpie-utilities (16 pass, 0 fail).
  • Ran claude plugin test plugins/magpie-setup for regression testing (20 pass, 0 fail; 36 total across repo).
  • Verified python plugins/magpie-utilities/skills/list-skills/tests/test_list_skills.py (17 tests pass).

RFC-AI-0004 compliance

  • HITL — any new mutation is gated on explicit user confirmation
  • Sandbox — minimal declared surface area, no node:fs or node:path built-in imports, deterministic execution.
  • Vendor neutrality — mod is strictly additive; existing /magpie-utilities:list-skills skill workflow remains completely untouched for non-Claude harnesses.
  • Conversational + correctable — agentic-override path documented if behaviour is adopter-tunable
  • Write-access discipline — no autonomous outbound messages; drafts only, sent on confirmation
  • Privacy LLM — zero model calls made; operates 100% locally on the host machine.

Linked issues

Notes for reviewers (optional)

  • Mirrors the maintainer-refined execution pattern from PR feat(setup): add zero-token /magpie-status Claude Code mod command (part of #1497) #1601 (30s PROCESS_TIMEOUT_MS, PYTHONIOENCODING: utf-8, python3 with python fallback, and cwd: workspaceDir).
  • Argument parsing safely handles double/single quoted paths with spaces in --root, prevents flag collisions (e.g. --root --verbose), and forwards -h/--help directly to the underlying list_skills.py CLI.

…d (part of apache#1497)

Registers /magpie-skills via Claude Code mod hooks ($.command.register and command.run) in plugins/magpie-utilities to provide instant, zero-token skill discovery.

- Hooks session.start to register /magpie-skills with immediate: true
- Intercepts command.run to execute list_skills.py locally without model turns
- Enforces 30s timeout, UTF-8 output encoding, and python3 -> python fallback
- Handles --verbose, -v, --help, -h, and --root (including paths with spaces and quotes)
- Adds comprehensive unit test suite covering all execution and error handling branches
@github-actions github-actions Bot added the family:utilities utilities skills label Oct 16, 2026
@Kaap10

Kaap10 commented Oct 16, 2026

Copy link
Copy Markdown
Contributor Author

Implemented Pilot 3 (zero-token /magpie-skills command in plugins/magpie-utilities) for #1497 following the #1601 pattern.
Ready for review!

@potiuk
potiuk merged commit 861c84e into apache:main Oct 16, 2026
10 checks passed
potiuk added a commit that referenced this pull request Oct 17, 2026
Hand-sync of tools/spec-loop/specs/ with the functionality that landed
between d58ecb7 and 43e4c6e.

Specs changed:
- release-management-lifecycle: verify-rc classification as code
  (abd5969, #1619), vote-draft / promote / announce-draft renders
  (c87ad08, d5fe7fd), the per-outcome / backend / classification
  file split across the family, and the SKILL.md size gap closed (all
  ten now under 500 lines).
- adoption-and-setup: the utilities first-run welcome guide and the
  text-file mirror fallback in the family scan (9f8c479, #1669).
- meta-and-quality-tooling: the zero-token /magpie-skills Claude Code
  mod in magpie-utilities (861c84e, #1667).
- agent-isolation-sandbox: the agent-guard --junie adapter (6c24f30,
  #1666) and sandbox-lint refusing a tracked .claude/settings.json
  (4a07965, #1675).
- adapters, security-issue-lifecycle, project-agnosticism: the canonical
  apache-magpie status-rollup marker, preferred over and replacing the
  legacy tracker-named one for new rollups (ccead70, #1665).
- .last-sync bumped to 43e4c6e.

No spec change needed (specs already updated in the commit itself):
43e4c6e, 399ab1b, 622bfbc, 5fc6a32, a75734c, e44b44a,
a93a7d1, c67e3c6, a3d9b71, ec015bb.

Exempt (mechanical): 0a79a33 (dev version bump), 80a0f1f (npm pin
bump in prek hooks).

Generated-by: Claude Opus 5
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

family:utilities utilities skills

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants