Skip to content

fix(flow-ai): trim description under 1024-char install limit, add CI guard - #6

Merged
mhusbynflow merged 1 commit into
mainfrom
mhusbynflow/fix-skill-description-and-ci
Jul 1, 2026
Merged

fix(flow-ai): trim description under 1024-char install limit, add CI guard#6
mhusbynflow merged 1 commit into
mainfrom
mhusbynflow/fix-skill-description-and-ci

Conversation

@mhusbynflow

Copy link
Copy Markdown
Contributor

What

  • Fix the install blocker: the plugin installer rejects a SKILL.md whose frontmatter description exceeds 1024 characters. flow-ai's had grown to 1121, causing install errors. Trimmed to 988 by removing only redundant phrasing — every trigger keyword is preserved, so activation behaviour is unchanged.
  • Add a lean CI guard: scripts/validate_skills.py (stdlib-only, no deps) checks that every plugins/**/SKILL.md has name + description and that description ≤ 1024 chars. Wired via .github/workflows/validate-skills.yml on PRs and pushes to main.
  • Version bump 0.7.0 → 0.7.1 across plugin.json and all flow-ai/<version> User-Agent references, per the CHANGELOG's bump-both-together rule. CHANGELOG entry added.

Verification

  • Validator is green on the current repo (Checked 1 skill(s), 0 error(s)., exit 0).
  • Validator is red on an injected 1200-char description (description is 1200 chars (max 1024), exit 1).

Out of scope

  • Making the workflow a required status check — that's a branch-protection ruleset setting and requires the workflow to have run on main first.
  • Broader skill quality/best-practice linting (e.g. tessl scenario evals) — deliberately kept to the bare-minimum guard for now.

🤖 Generated with Claude Code

…guard

The plugin installer rejects a SKILL.md whose frontmatter `description`
exceeds 1024 characters. flow-ai's description had grown to 1121 chars,
blocking installation. Trim it to 988 by removing redundant phrasing only —
all trigger keywords are preserved, so activation behaviour is unchanged.

Add a lean, dependency-free CI guard (scripts/validate_skills.py, run by
.github/workflows/validate-skills.yml) that fails the build if any skill's
description exceeds the limit, so this cannot regress silently.

Bump the version 0.7.0 -> 0.7.1 across plugin.json and the flow-ai/<version>
User-Agent references, per the CHANGELOG's bump-both-together rule.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
@mhusbynflow

Copy link
Copy Markdown
Contributor Author

@claude please review this PR

@mhusbynflow
mhusbynflow merged commit 7bce3db into main Jul 1, 2026
1 check passed
@mhusbynflow
mhusbynflow deleted the mhusbynflow/fix-skill-description-and-ci branch July 1, 2026 12:23
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.

1 participant