Repository navigation
Closed
Feature
3 / 33 of 3 issues completed
Copy link
Labels
priority: mediumReal value, no hard deadline; normal backlog flow.Real value, no hard deadline; normal backlog flow.status: readyTriaged, unblocked, and fully specified; eligible to pick up.Triaged, unblocked, and fully specified; eligible to pick up.work-class: structuralRefactors, migrations, contract changes; cross-cutting and hard to reverse.Refactors, migrations, contract changes; cross-cutting and hard to reverse.
Description
Refs: #7575
Agent Brief
Type: Feature
Summary: Rename /session-flow:show-options to /session-flow:recommend-skills, the one skill router: a script indexes the frontmatter of every skill, the agent shortlists, reads the shortlisted SKILL.md files, asks at most one question, and never enables or installs anything. This is the parent's recommend-skills slice.
Current behavior:
plugins/session-flow/skills/show-options/exists; norecommend-skillsdirectory exists.session-flow:show-optionsis referenced in 18 files underpluginsanddocsoutside changelogs.
Desired behavior (from #7575's decisions D80, Q157, Q189, Q190):
- The skill directory is
plugins/session-flow/skills/recommend-skills/, invoked as/session-flow:recommend-skills. - A script builds an index from the frontmatter of every skill: installed, disabled and not installed.
- The agent shortlists from the index, reads the shortlisted SKILL.md files, may ask one question, and never enables or installs a skill.
- A user setting controls whether the router excludes disabled or uninstalled skills; the default includes all.
Done when
- The
recommend-skillsskill directory exists,show-optionsdoes not, andrg -n 'session-flow:show-options' plugins docs --glob '!**/CHANGELOG.md' --glob '!.changes/**'prints nothing. - The index script has a test that passes under
scripts/run-test-file.sh, covering one installed, one disabled and one not-installed skill. - Eval cases cover: a shortlist drawn from the index; at most one question asked; no enable or install action taken; the exclude setting dropping disabled and uninstalled skills.
- The skill-body change cites before/after eval results in the PR.
-
bash plugins/skill-quality/scripts/check-skill.sh plugins/session-flow/skillsexits 0. -
scripts/check-changelog-fragments.shpasses with a session-flow fragment.
Out of scope:
- The retro-absorb and trends-move slice (retro still has
codifyandtrendsmodes); see the corrections: new plugin with /corrections:codify #7573 absorb slice, which also edits retro. - Description trims.
Metadata
Metadata
Assignees
Labels
priority: mediumReal value, no hard deadline; normal backlog flow.Real value, no hard deadline; normal backlog flow.status: readyTriaged, unblocked, and fully specified; eligible to pick up.Triaged, unblocked, and fully specified; eligible to pick up.work-class: structuralRefactors, migrations, contract changes; cross-cutting and hard to reverse.Refactors, migrations, contract changes; cross-cutting and hard to reverse.
Type
Fields
No fields configured for Feature.
Activity
github-actions commentedon Oct 30, 2026
Proposed work class: work-class: structural
Basis: automated intake triage; the lanes App applied this class.
Size: Large
Complexity: Medium
Uncertainty: Known
Flags: none
Blocked by: none
Understood: rename
/session-flow:show-optionsto/session-flow:recommend-skills, add a script that indexes the frontmatter of every skill (installed, disabled and not installed), and have the agent shortlist, read the shortlisted SKILL.md files, ask at most one question, and never enable or install anything. A user setting controls whether disabled and uninstalled skills are excluded, and the default includes all. The done-when list is concrete. No duplicates found among the open issues.Automated triage, applied by the lanes App.
melodic-automation-lanes commentedon Oct 30, 2026
Not admitted to the issue lane yet: Size large: the split into sub-issues at intake is pending (owner Q93).
melodic-automation-lanes commentedon Oct 30, 2026
Split at intake (owner Q93). This issue is Size Large, so its done-when is split into 3 sub-issues, each holding its task-list lines as written.
Automated split by the lanes App. Admission waits on this issue while a sub-issue is open; work-unblock closes it as completed once every sub-issue is closed by a merge or a listed human.
melodic-automation-lanes commentedon Oct 31, 2026
Every sub-issue of this split is closed by a listed human or a merge (#11130, #11131, #11132), so the lanes App closes it as completed.