Skip to content

Rewrite bundled short_description brief lines for agent routing - #356

Open
Voyagerroc-Lab wants to merge 1 commit into
ARPAHLS:mainfrom
Voyagerroc-Lab:feat/issue-355-brief-lines
Open

Rewrite bundled short_description brief lines for agent routing#356
Voyagerroc-Lab wants to merge 1 commit into
ARPAHLS:mainfrom
Voyagerroc-Lab:feat/issue-355-brief-lines

Conversation

@Voyagerroc-Lab

@Voyagerroc-Lab Voyagerroc-Lab commented Sep 16, 2026 •
edited
Loading

Copy link
Copy Markdown

Summary

  • Rewrite agent-routing short_description values for the 21 skills covered by this PR, including optimization/context_optimizer added in feat(optimization): add context_optimizer for query-aware extractive … #357.
  • Add a guard for presence, single-line format, and the 160-character limit.
  • Patch-bump the changed manifests and synchronize skill-page and catalog version metadata; context_optimizer is now v0.1.1.
  • Expand contributor guidance for brief-line routing.

Fixes #355

Validation

  • python -m black --check . — 228 files unchanged.
  • python -m flake8 . — passed.
  • pytest -p no:cacheprovider --basetemp .pytest-tmp-skillware-20260920 -q tests/test_short_descriptions.py tests/test_registry_docs.py tests/test_skill_docs.py — 16 passed.
  • git diff --check — passed.

@rosspeili

rosspeili commented Sep 16, 2026

Copy link
Copy Markdown
Contributor

Thanks for #356, good sweep on brief lines, docs sync, and the guard test.

Please before merge:

  1. **Comment on and claim [Skill Upgrade]: Improve manifest short_description brief lines for agent routing #355
  2. Rebase on latest main, conflicts expected in CHANGELOG.md and docs/skills/README.md
  3. Add optimization/context_optimizer (feat(optimization): add context_optimizer for query-aware extractive … #357 landed after your branch), rewrite its short_description per spec, patch-bump, sync catalog version metadata
  4. Update PR text, 21 skills, not 20, but nw, minor
  5. Re-run black --check ., flake8 ., and the registry/skill-doc guard tests on the rebased branch

Once those are done and CI is green, good to merge. Thanks! <3

@mrmasa88 mrmasa88 mentioned this pull request Sep 17, 2026
Open
22 tasks
@ARPAHLS ARPAHLS deleted a comment from mrmasa88 Sep 17, 2026
docs(skills): rewrite bundled short_description lines as agent-routin…
ca9b108 
…g hints

Rewrite every bundled manifest short_description into a routing hint that
states what the skill does (verb + object) and when to use it, disambiguated
from sibling skills (target <= 120 chars). Patch-bump each touched skill
version.

Sync docs to match: docs/skills/README.md version rows and per-skill catalog
version metadata, the CONTRIBUTING manifest bullet, the agent workflow
checklist, and the cli.md short_description section (80-char display
truncation vs 160-char brief-line cap).

Add tests/test_short_descriptions.py to assert every bundled manifest declares
a single-line short_description within the 160-char brief-line cap.

Fixes ARPAHLS#355
@Voyagerroc-Lab
Voyagerroc-Lab force-pushed the feat/issue-355-brief-lines branch from 930af13 to ca9b108 Compare September 20, 2026 19:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

[Skill Upgrade]: Improve manifest short_description brief lines for agent routing

2 participants

@Voyagerroc-Lab @rosspeili