Skip to content

Implement token-saving skill and related evaluation files#5

Merged
miroslavpojer merged 6 commits into
masterfrom
feature/3-Implement-the-token-saving-skill
Jun 1, 2026
Merged

Implement token-saving skill and related evaluation files#5
miroslavpojer merged 6 commits into
masterfrom
feature/3-Implement-the-token-saving-skill

Conversation

@miroslavpojer
Copy link
Copy Markdown
Contributor

@miroslavpojer miroslavpojer commented May 20, 2026

Summary

Adds the token-saving skill — an always-active response-discipline layer that enforces brevity and structured output across all Copilot replies. Also ships the full evaluation suite for the skill.

Changes

Area Detail
skills/SKILL.md New token-saving skill definition with trigger rules, line-limit rules, and code-output footer format
skills/token-saving/evals/ Trigger eval + scenario evals covering 5 real-world input files
README.md Minor clarification
docs/getting-started.md Minor clarification

Release Notes

  • token-saving skill — always-active formatting rules that reduce response verbosity without compromising accuracy. Covers filler suppression, line caps, table/bullet preference, and a mandatory What changed / Why / How to verify footer for code-output responses.

Closes #3

Comment thread docs/token-saving.md
Comment thread skills/token-saving/SKILL.md
Comment thread skills/token-saving/SKILL.md Outdated
Comment thread skills/token-saving/SKILL.md Outdated
Copy link
Copy Markdown

@lsulak lsulak left a comment

Choose a reason for hiding this comment

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

Review finished. One more thing, a conceptual level below.

The skill says "shortest response that fully answers" but doesn't explicitly say correctness is more important than brevity. An LLM under aggressive conciseness pressure may skip important caveats. Add: "Never sacrifice correctness or safety-critical information for brevity."

Comment thread .github/workflows/check_pr_release_notes.yml Outdated
Comment thread docs/getting-started.md Outdated
Copy link
Copy Markdown

@lsulak lsulak left a comment

Choose a reason for hiding this comment

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

LGTM, good job!

@miroslavpojer miroslavpojer merged commit 7b804d1 into master Jun 1, 2026
@miroslavpojer miroslavpojer deleted the feature/3-Implement-the-token-saving-skill branch June 1, 2026 11:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Implement the token-saving skill

3 participants