[sea-snake]

checks / validate failed on the push of #454 to main (run), so deploy-ic.yml never ran and the five Internet Identity skills are not on https://skills.internetcomputer.org/.

The error

Validating skill: skills/internet-identity
Frontmatter
  ✗ description exceeds 1024 characters (1029)

Dropping two words ("It also covers" → "Also covers", "wallet integration or ICRC signer flows" → "wallet or ICRC signer flows") brings it to 1014, in line with cloud-engine-canisters (1014) and deploy-to-cloud-engine (1010). No routing term is lost: wallet-integration still appears in the same sentence as the skill to load instead.

Why CI did not catch it on the PR

skill-validator check takes exactly one path:

$ skill-validator check skills/internet-identity skills/internet-identity-sso --skip links
Error: accepts 1 arg(s), received 2
$ echo $?
3

The changed-skills step builds a space-separated list of paths, so on any PR touching two or more skills the validator exited 3 (usage error) with an empty report, and the gate only failed on exit code 1. Every such PR has been going green without being validated. On a push to main the same step passes the single path skills/, which is why the error only surfaced after the merge.

Two changes:

• each path gets its own invocation, aggregating the exit codes with an error (1) winning over a warning (2);
• the gate fails on any code other than 0 (clean) or 2 (warnings), so a usage error can never read as a pass again.

Verification

Per-path loop over the five skills from #454, with the description fix applied:

skills/internet-identity                  -> 2
skills/internet-identity-attributes       -> 0
skills/internet-identity-notifications    -> 0
skills/internet-identity-shared-sessions  -> 0
skills/internet-identity-sso              -> 0
aggregate=2        # warnings only, gate passes

Same loop before the description fix returns 1 for skills/internet-identity, so the new gate catches exactly the failure that got through. skill-validator check skills/ --skip links now exits 2 (was 1), and node scripts/check-project.js passes for 39 skills with the two pre-existing eval-file warnings.

No skill body changed, so no evals were re-run.

🤖 Generated with Claude Code

[github-actions]

Skill Validation Report

Validating skill: /home/runner/work/icskills/icskills/skills/internet-identity

Structure

• Pass: SKILL.md found

Frontmatter

• Pass: name: "internet-identity" (valid)
• Pass: description: (1014 chars)
• Pass: license: "Apache-2.0"
• Pass: compatibility: (31 chars)
• Pass: metadata: (2 entries)

Tokens

• Warning: SKILL.md body is 6255 tokens (spec recommends < 5000)

Markdown

• Pass: no unclosed code fences found

Tokens

File	Tokens
SKILL.md body	6,255
Total	6,255

Content Analysis

Metric	Value
Word count	4,034
Code block ratio	0.10
Imperative ratio	0.06
Information density	0.08
Instruction specificity	0.87
Sections	26
List items	57
Code blocks	11

Contamination Analysis

Metric	Value
Contamination level	medium
Contamination score	0.41
Primary language category	javascript
Scope breadth	3

• Warning: Language mismatch: config (1 category differ from primary)
• Multi-interface tool detected: aws

Result: 1 warning

Project Checks

✓ Project checks passed for 1 skills (0 warnings)

[Copilot]

🟡 Changes recommended

The exit-code aggregation lets an earlier warning mask later usage or execution errors, allowing the final gate to pass even when a skill was not successfully validated.

1 open issue

• Warnings mask later skill validation failures · New

What changed in this PR

This PR addresses the validation failure that blocked deployment of the Internet Identity skills after #454 and updates CI to validate multi-skill changes individually.

Changes:

• Runs the validator separately for each changed skill and aggregates results.
• Updates the failure gate to reject exit codes other than success or warnings.
• Shortens the Internet Identity description to fit the 1,024-character limit without changing the skill body.

File	Description
skills/​internet-identity/​SKILL.md	Trims the description while preserving routing terms.
.github/​workflows/​_checks.yml	Adds per-path validation, result aggregation, and stricter failure conditions.

_{Review effort: Balanced}