Skip to content

fix(skills): keep personal skills out of curator#19257

Closed
LeonSGP43 wants to merge 1 commit into
NousResearch:mainfrom
LeonSGP43:fix/curator-skip-personal-skills-19040
Closed

fix(skills): keep personal skills out of curator#19257
LeonSGP43 wants to merge 1 commit into
NousResearch:mainfrom
LeonSGP43:fix/curator-skip-personal-skills-19040

Conversation

@LeonSGP43

Copy link
Copy Markdown
Contributor

Summary

  • exclude the user-owned skills/me/ namespace from curator candidate discovery
  • keep personal slash-command skills out of agent_created_report() while preserving ordinary agent-created skill handling
  • add regression coverage for the personal namespace filter

Closes #19040

Verification

  • scripts/run_tests.sh tests/tools/test_skill_usage.py -> 40 passed, 4 warnings
  • git diff --check -> clean

Non-goals

  • Does not restore skills that were already consolidated or archived; this prevents future curator passes from treating personal skills as agent-created candidates.

@alt-glitch alt-glitch added type/bug Something isn't working tool/skills Skills system (list, view, manage) P2 Medium — degraded but workaround exists labels May 3, 2026
@teknium1

teknium1 commented May 4, 2026

Copy link
Copy Markdown
Contributor

Closing in favor of PR #19618 (salvaged from your earlier #19235). Your two PRs solved #19040 with different strategies — the provenance-marker approach from #19235 subsumes the path-based filter here: any skill the user authored manually (whether in skills/me/ or anywhere else) has no created_by: "agent" marker in .usage.json, so the curator never touches it. I verified via E2E that a skill at skills/me/<name>/SKILL.md is correctly excluded under #19618's rules. Thanks for two thoughtful attempts — credit preserved in the merged commit from #19235.

@teknium1 teknium1 closed this May 4, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

P2 Medium — degraded but workaround exists tool/skills Skills system (list, view, manage) type/bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Bug]: Can't invoke my peronal skills anymore

3 participants