fix(skills): restore the /learn operational skill (regression from #77)#105
Merged
Conversation
Phase 3 plugin packaging (#77) migrated every .claude/commands/*.md slash command into an operational skill under skills/ and the bundled mureo/_data/skills/, EXCEPT it dropped `learn` entirely (deleted .claude/commands/learn.md, never created a learn skill). README/docs still document /learn, so /learn became uninvocable while every other command kept working. Restore it as an operational skill (skills/learn + bundled copy, byte-identical per the manifest parity test), ported from the deleted command and adapted to the current layout: - name: learn (no _ prefix — operational, surfaces in the picker) - PREREQUISITE + frontmatter match peer skills (daily-check/sync-state) - writes to ../_mureo-pro-diagnosis/SKILL.md (sibling install path); pro-diagnosis is intentionally canonical-only (not shipped, grows per account) so the skill scaffolds it on first use, keeping the _ foundation prefix - approval-required, append-only, no secrets/PII, never Claude memory Tests: EXPECTED_PACKAGED_SKILLS 15->16 + learn install assert (test_setup_cmd); learn added to _BUNDLE_NAMES (remove symmetry); stale in-tree-count comments de-numbered. Parser/discovery/matcher/ manifest-parity auto-cover the new SKILL.md. Test plan: - [x] full suite 3209 passed - [x] code-reviewer APPROVE (no CRITICAL/HIGH) - [x] byte-identical skills/learn vs mureo/_data/skills/learn
Merged
hyoshi
added a commit
that referenced
this pull request
May 16, 2026
- pyproject 0.8.0 -> 0.9.0; Development Status 3 - Alpha -> 4 - Beta - mureo/__init__.py __version__ 0.7.1 -> 0.9.0 (sync; was stale) - .claude-plugin/plugin.json 0.8.0 -> 0.9.0 - CHANGELOG: [Unreleased] -> [0.9.0] - 2026-05-16, plus entries for the /learn restore (#105), configure UI visual refresh + official logo, and Google Ads OAuth-scope guidance (post-#99 merges) Highlights: mureo configure Web UI (Phase 1) + visual refresh & logo; mureo providers CLI; plugin->MCP tool exposure; Meta on Claude Code via the Claude.ai connector; /learn restored; BYOD/demo date-anchor fix. Test plan: - [x] full suite 3209 passed - [x] python -m build + twine check PASSED (0.9.0 wheel + sdist) - [x] plugin manifest version-match test green
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.
Problem
/learnstopped working while every other workflow command (/daily-check,/rescue, …) still works. Root cause: Phase 3 plugin packaging (#77) migrated every.claude/commands/*.mdinto an operational skill underskills/+ bundledmureo/_data/skills/, but droppedlearnentirely (deleted.claude/commands/learn.md, never created alearnskill). README/docs still document/learn.Fix
skills/learn/SKILL.md+ byte-identicalmureo/_data/skills/learn/SKILL.md, ported from the deleted command and adapted to the current layout/conventions.name: learn(no_prefix → operational, appears in the picker); PREREQUISITE + frontmatter match peer skills.../_mureo-pro-diagnosis/SKILL.md(sibling install path)._mureo-pro-diagnosisis intentionally canonical-only (not shipped, grows per account, test-enforced opt-out), so the skill scaffolds it on first use, preserving the_foundation prefix.Tests
EXPECTED_PACKAGED_SKILLS15→16 +learninstall assert (test_setup_cmd)learnadded to_BUNDLE_NAMES(remove symmetry)