Skip to content

trim: tighten 8 skill descriptions for prompt budget - #32361

Closed
JeliTron wants to merge 1 commit into
NousResearch:mainfrom
JeliTron:trim/skill-descriptions
Closed

trim: tighten 8 skill descriptions for prompt budget#32361
JeliTron wants to merge 1 commit into
NousResearch:mainfrom
JeliTron:trim/skill-descriptions

Conversation

@JeliTron

Copy link
Copy Markdown
Contributor

Summary

Trims verbose description fields in 8 SKILL.md frontmatter blocks. The available_skills list is injected into every agent turn — long descriptions waste prompt budget with zero information gain.

Changes

Skill Before After Saved
kanban-orchestrator 315 chars 85 chars -73%
kanban-worker 263 chars 60 chars -77%
pretext 257 chars 88 chars -66%
comfyui 213 chars 76 chars -64%
hermes-s6-container-supervision 199 chars 58 chars -71%
kanban-codex-lane 175 chars 53 chars -70%
touchdesigner-mcp 168 chars 58 chars -65%
teams-meeting-pipeline 155 chars 75 chars -52%

Total: ~1,745 chars → ~553 chars (68% reduction)

Impact

  • Description-only change (+8/-8 lines), zero functional impact
  • Skills are loaded on-demand via skill_view() — descriptions just need to identify, not explain
  • Consistent pattern: {Tool}: {action/scope} or {Tool}: {brief purpose}

Shorten verbose frontmatter descriptions across kanban, creative, devops,
and productivity skills.  Long descriptions bloat the available_skills
list injected into every agent turn — trimming them reduces overhead by
~60-77% per skill with zero information loss.

Files changed:
  - devops/kanban-orchestrator: 315→85 chars (-73%)
  - devops/kanban-worker: 263→60 chars (-77%)
  - creative/pretext: 257→88 chars (-66%)
  - creative/comfyui: 213→76 chars (-64%)
  - software-dev/hermes-s6-container-supervision: 199→58 chars (-71%)
  - autonomous-ai-agents/kanban-codex-lane: 175→53 chars (-70%)
  - creative/touchdesigner-mcp: 168→58 chars (-65%)
  - productivity/teams-meeting-pipeline: 155→75 chars (-52%)
@alt-glitch alt-glitch added type/refactor Code restructuring, no behavior change P3 Low — cosmetic, nice to have tool/skills Skills system (list, view, manage) labels May 26, 2026

@teknium1 teknium1 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for targeting recurring skill-index prompt cost; current main still injects skill descriptions into the system prompt (agent/prompt_builder.py:1392, agent/prompt_builder.py:1691-1693).

Problems

  • Four replacements exceed the repository's current 60-character hard limit for modernized skill descriptions (AGENTS.md:888-900): skills/creative/comfyui/SKILL.md:3 (76), skills/creative/pretext/SKILL.md:3 (88), skills/devops/kanban-orchestrator/SKILL.md:3 (85), and skills/productivity/teams-meeting-pipeline/SKILL.md:3 (75).
  • Four diff targets no longer map to current main. Commit 38d3c49aa removed kanban-codex-lane and moved the s6 skill to optional-skills/devops/; commit 84e1d31e5 removed both Kanban worker/orchestrator skills.

Suggested changes

  • Reduce the four over-limit values to at most 60 characters.
  • Salvage the active paths and repoint the s6 edit to its optional-skill location; omit the deleted Kanban targets.

Automated hermes-sweeper review.

---
name: comfyui
description: "Generate images, video, and audio with ComfyUI — install, launch, manage nodes/models, run workflows with parameter injection. Uses the official comfy-cli for lifecycle and direct REST/WebSocket API for execution."
description: "Generate images/video/audio with ComfyUI via comfy-cli + REST/WebSocket API."

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

AGENTS.md:888-900 requires modernized skill descriptions to be 60 characters or fewer. This replacement is 76 characters; please shorten it while preserving the capability signal.

---
name: pretext
description: "Use when building creative browser demos with @chenglou/pretextDOM-free text layout for ASCII art, typographic flow around obstacles, text-as-geometry games, kinetic typography, and text-powered generative art. Produces single-file HTML demos by default."
description: "Browser demos w/ @chenglou/pretext: DOM-free text layout, ASCII art, kinetic typography."

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

AGENTS.md:888-900 requires modernized skill descriptions to be 60 characters or fewer. This replacement is 88 characters; please reduce it to the required limit.

---
name: kanban-orchestrator
description: Decomposition playbook + anti-temptation rules for an orchestrator profile routing work through Kanban. The "don't do the work yourself" rule and the basic lifecycle are auto-injected into every kanban worker's system prompt; this skill is the deeper playbook when you're specifically playing the orchestrator role.
description: "Kanban orchestrator playbook: decomposition + anti-temptation rules for routing work."

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

AGENTS.md:888-900 requires modernized skill descriptions to be 60 characters or fewer. This replacement is 85 characters; please shorten it if this skill is restored or otherwise retained during salvage.

---
name: teams-meeting-pipeline
description: "Operate the Teams meeting summary pipeline via Hermes CLI — summarize meetings, inspect pipeline status, replay jobs, manage Microsoft Graph subscriptions."
description: "Teams meeting summaries: pipeline status, replay jobs, Graph subscriptions."

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

AGENTS.md:888-900 requires modernized skill descriptions to be 60 characters or fewer. This replacement is 75 characters; please reduce it to the required limit.

@teknium1 teknium1 added the sweeper:blast-broad Sweeper blast radius: broad — a core path most sessions hit label Jul 13, 2026
teknium1 added a commit that referenced this pull request Jul 24, 2026
… budget

Every SKILL.md description over 60 chars was silently truncated to
57 chars + '...' in the system-prompt skill index
(extract_skill_description, agent/skill_utils.py), destroying the
routing signal for 69 of 179 skills — some descriptions ran to
1,005 chars.

Rewrites follow the authoring standard: <=60 chars, one sentence,
ends with a period, trigger front-loaded, no marketing words, no
skill-name repetition. Excess detail already lives in each skill's
body.

Includes the touchdesigner-mcp trim from PR #32361 (credit:
@JeliTron) and aligns with the router-precision direction of PR
#48780 (@John-Lussier). Docs catalogs + per-skill pages regenerated
via website/scripts/generate-skill-docs.py.

Co-authored-by: JeliTron <287797501+JeliTron@users.noreply.github.com>
@teknium1

Copy link
Copy Markdown
Contributor

Closing with credit — your touchdesigner-mcp trim landed verbatim in PR #70531 with you as co-author on the commit.

The rest of the PR aged out from under you: 3 of the 8 targets (the kanban skills) were deleted from main, one moved to optional-skills/, and the repo has since adopted a hard ≤60-char description standard that the remaining proposed texts exceed (they were written against an earlier looser budget). Those skills got fresh ≤60 trims in #70531 rather than asking you to redo the branch.

Thanks for being early on the prompt-budget problem — the direction was right, the standard just tightened after you submitted.

@teknium1 teknium1 closed this Jul 24, 2026
randlee pushed a commit to randlee/hermes-agent that referenced this pull request Aug 11, 2026
… budget

Every SKILL.md description over 60 chars was silently truncated to
57 chars + '...' in the system-prompt skill index
(extract_skill_description, agent/skill_utils.py), destroying the
routing signal for 69 of 179 skills — some descriptions ran to
1,005 chars.

Rewrites follow the authoring standard: <=60 chars, one sentence,
ends with a period, trigger front-loaded, no marketing words, no
skill-name repetition. Excess detail already lives in each skill's
body.

Includes the touchdesigner-mcp trim from PR NousResearch#32361 (credit:
@JeliTron) and aligns with the router-precision direction of PR
NousResearch#48780 (@John-Lussier). Docs catalogs + per-skill pages regenerated
via website/scripts/generate-skill-docs.py.

Co-authored-by: JeliTron <287797501+JeliTron@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

P3 Low — cosmetic, nice to have sweeper:blast-broad Sweeper blast radius: broad — a core path most sessions hit tool/skills Skills system (list, view, manage) type/refactor Code restructuring, no behavior change

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants