Skip to content

fix(icp-cli): document valid canister-level fields per config style#122

Merged
marc0olo merged 1 commit intomainfrom
fix/icp-cli-canister-field-validation
Mar 25, 2026
Merged

fix(icp-cli): document valid canister-level fields per config style#122
marc0olo merged 1 commit intomainfrom
fix/icp-cli-canister-field-validation

Conversation

@marc0olo
Copy link
Member

@marc0olo marc0olo commented Mar 25, 2026

Summary

  • Add pitfall #12 warning against mixing canister-level fields across recipe and bare-build config styles (e.g. putting candid: at canister level alongside a recipe, using non-existent init_arg_file instead of init_args.path)
  • Link to the icp-cli configuration reference from the pitfall and the llms.txt index from the Additional References section
  • Add adversarial eval "mixing fields across config styles" — scores 4/4 with skill, 0/4 without

Closes #109

Add pitfall warning against mixing fields across recipe and bare-build
config styles, clarify that init_arg_file does not exist (use
init_args.path), link to the authoritative config reference, and add
an adversarial eval that scores 4/4 with skill vs 0/4 without.

Closes #109
@marc0olo marc0olo requested review from a team and JoshDFN as code owners March 25, 2026 09:49
@github-actions
Copy link

Skill Validation Report

Validating skill: /home/runner/work/icskills/icskills/skills/icp-cli

Structure

  • Pass: SKILL.md found
  • Pass: all files in references/ are referenced

Frontmatter

  • Pass: name: "icp-cli" (valid)
  • Pass: description: (507 chars)
  • Pass: license: "Apache-2.0"
  • Pass: metadata: (2 entries)

Markdown

  • Pass: no unclosed code fences found

Tokens

File Tokens
SKILL.md body 3,947
references/binding-generation.md 764
references/dev-server.md 690
references/dfx-migration.md 1,921
Total 7,322

Content Analysis

Metric Value
Word count 1,987
Code block ratio 0.20
Imperative ratio 0.12
Information density 0.16
Instruction specificity 0.80
Sections 17
List items 48
Code blocks 21

References Content Analysis

Metric Value
Word count 1,627
Code block ratio 0.21
Imperative ratio 0.15
Information density 0.18
Instruction specificity 0.75
Sections 16
List items 29
Code blocks 8

Contamination Analysis

Metric Value
Contamination level low
Contamination score 0.12
Primary language category config
Scope breadth 3
  • Warning: Language mismatch: javascript, shell (2 categories differ from primary)

References Contamination Analysis

Metric Value
Contamination level low
Contamination score 0.03
Primary language category javascript
Scope breadth 2
  • Warning: Language mismatch: shell (1 category differ from primary)

Result: passed

Project Checks


✓ Project checks passed for 1 skills (0 warnings)

@marc0olo marc0olo merged commit d99abfe into main Mar 25, 2026
6 checks passed
@marc0olo marc0olo deleted the fix/icp-cli-canister-field-validation branch March 25, 2026 10:19
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.

fix(icp-cli): document valid canister-level fields per config style

2 participants