Skip to content

Commit 590bbff

Browse files
philoserfclaude
andcommitted
refactor(cc-lint): remove stale Portability dimension, fold into Correctness
Portability was really just field conformance (use only documented frontmatter fields) — not cross-platform portability. Fold it into Correctness where it belongs, add trigger synonyms, align approach steps with evaluation-process.md, and add conflict resolution guidance. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
1 parent 5b9d000 commit 590bbff

5 files changed

Lines changed: 15 additions & 44 deletions

File tree

skills/cc-check/references/common-failures.md

Lines changed: 3 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -24,16 +24,15 @@ Reference this file when debugging test failures. Each pattern includes symptoms
2424

2525
**Fix**: Add trigger phrase variations to description. Include both formal terms and casual language users might use.
2626

27-
### Portability Issue
27+
### Field Conformance Issue
2828

29-
**Symptom**: Skill uses non-standard frontmatter fields or agent-specific coupling
29+
**Symptom**: Skill uses non-standard frontmatter fields
3030

3131
**Common causes**:
3232

3333
- Non-standard fields in frontmatter (beyond documented fields)
34-
- Instructions tightly coupled to a specific implementation
3534

36-
**Diagnosis**: Check frontmatter for documented fields only.
35+
**Diagnosis**: Check frontmatter against documented fields: `name`, `description`, `argument-hint`, `disable-model-invocation`, `user-invocable`, `allowed-tools`, `model`, `effort`, `context`, `agent`, `hooks`.
3736

3837
**Fix**: Remove non-standard frontmatter fields.
3938

skills/cc-lint/SKILL.md

Lines changed: 7 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,7 @@
11
---
22
disable-model-invocation: true
33
allowed-tools: Read, Grep, Glob, Bash
4-
description: Performs quick structural validation of Claude Code customizations. Use when linting, validating structure, checking frontmatter, or reviewing any skill, agent, or hook for correctness. Checks YAML frontmatter, required fields, naming conventions, file organization, and settings.json health.
4+
description: Performs quick structural validation of Claude Code customizations. Use when linting, validating structure, checking frontmatter, auditing customizations, reviewing skill structure, or reviewing any skill, agent, or hook for correctness. Checks YAML frontmatter, required fields, naming conventions, file organization, and settings.json health.
55
---
66

77
## Reference Files
@@ -41,16 +41,13 @@ If a specific file or directory is passed as an argument, lint that target direc
4141

4242
## Approach
4343

44-
When evaluating a Claude Code customization, this skill follows a systematic process:
44+
When evaluating a Claude Code customization, this skill follows the process in [evaluation-process.md](references/evaluation-process.md):
4545

46-
1. Read and parse target file(s) to extract structure and content
47-
2. Validate YAML frontmatter for documented fields and correct syntax
48-
3. Apply type-specific validation criteria (agent/skill/hook)
49-
4. Assess context economy and progressive disclosure usage
50-
5. Verify only documented frontmatter fields are used
51-
6. Check integration with settings.json and other customizations
52-
7. Generate structured report with specific findings and recommendations
53-
8. Prioritize issues by severity (correctness > clarity > effectiveness)
46+
1. Determine scope and identify extension type
47+
2. Apply type-specific validation (frontmatter, structure, fields)
48+
3. Check integration with settings.json
49+
4. Assess context economy and progressive disclosure
50+
5. Generate structured report with prioritized findings
5451

5552
Detailed criteria, process steps, and examples are available in the reference files above.
5653

skills/cc-lint/assets/report-format.md

Lines changed: 0 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -20,7 +20,6 @@ This document defines the standardized structure for evaluation reports.
2020
- **Correctness**: PASS | NEEDS WORK | FAIL
2121
- **Clarity**: PASS | NEEDS WORK | FAIL
2222
- **Effectiveness**: PASS | NEEDS WORK | FAIL
23-
- **Portability**: PASS | NEEDS WORK | N/A
2423

2524
## Correctness Findings
2625

@@ -52,16 +51,6 @@ This document defines the standardized structure for evaluation reports.
5251
- {issue 2}
5352
- ...
5453

55-
## Portability Findings
56-
57-
{Field conformance and structural quality issues}
58-
59-
**Issues Found**: {count}
60-
61-
- {issue 1}
62-
- {issue 2}
63-
- ...
64-
6554
## Context Usage
6655

6756
- **File Size**: {bytes or "N/A for multiple files"}

skills/cc-lint/references/evaluation-criteria.md

Lines changed: 4 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -5,9 +5,11 @@ This document defines the correctness, clarity, and effectiveness standards for
55
## Contents
66

77
- [Correctness Criteria](#correctness-criteria) — Agents, Skills, Hooks
8-
- [Clarity Criteria](#clarity-criteria) — Description quality, structure, portability
8+
- [Clarity Criteria](#clarity-criteria) — Description quality, structure
99
- [Effectiveness Criteria](#effectiveness-criteria) — Context economy, triggering, integration
1010

11+
When dimensions conflict (e.g., brevity aids context economy but hurts trigger coverage), prioritize correctness first, then effectiveness, then clarity.
12+
1113
## Correctness Criteria
1214

1315
### Agents
@@ -37,7 +39,7 @@ Per the [Claude Code skills docs](https://docs.anthropic.com/en/docs/claude-code
3739

3840
- All fields are optional; only `description` is recommended
3941
- Documented fields: `name`, `description`, `argument-hint`, `disable-model-invocation`, `user-invocable`, `allowed-tools`, `model`, `effort`, `context`, `agent`, `hooks`
40-
- Any other field is non-standard — flag as a warning
42+
- Any other field is non-standard — flag as a warning (field conformance)
4143

4244
**Structure validation**:
4345

@@ -70,14 +72,6 @@ Per the [Claude Code skills docs](https://docs.anthropic.com/en/docs/claude-code
7072
- References used when content exceeds ~500 lines
7173
- References clearly linked from SKILL.md, one level deep
7274

73-
### Portability (Skills)
74-
75-
Per the [Claude Code skills docs](https://docs.anthropic.com/en/docs/claude-code/skills):
76-
77-
- Only documented frontmatter fields used (`name`, `description`, `argument-hint`, `disable-model-invocation`, `user-invocable`, `allowed-tools`, `model`, `effort`, `context`, `agent`, `hooks`)
78-
- No unnecessary assumptions baked into structure
79-
- Tool names documented as implementation details, not hard requirements
80-
8175
## Effectiveness Criteria
8276

8377
### Context Economy

skills/cc-lint/references/evaluation-process.md

Lines changed: 1 addition & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -69,14 +69,6 @@ Calculate approximate size and efficiency:
6969
3. Check progressive disclosure usage
7070
4. Estimate token count impact
7171

72-
## Step 5: Assess Portability
73-
74-
For skills, confirm only documented frontmatter fields are used (verified in Step 2) and assess broader quality:
75-
76-
1. Check for non-standard frontmatter fields
77-
2. Assess whether content is clear and well-structured
78-
3. Note any non-documented fields as warnings
79-
80-
## Step 6: Generate Structured Report
72+
## Step 5: Generate Structured Report
8173

8274
Create comprehensive evaluation following the report format (see [report-format.md](../assets/report-format.md#report-template)).

0 commit comments

Comments
 (0)