Skip to content

docs(icp-cli): clarify automatic PUBLIC_CANISTER_ID env vars for inte…#248

Merged
raymondk merged 1 commit into
mainfrom
cli-canister-env
Jul 16, 2026
Merged

docs(icp-cli): clarify automatic PUBLIC_CANISTER_ID env vars for inte…#248
raymondk merged 1 commit into
mainfrom
cli-canister-env

Conversation

@raymondk

Copy link
Copy Markdown
Collaborator

No description provided.

…r-canister wiring

The Canister Environment Variables section understated the feature: the
injected IDs are runtime-readable canister settings, per-environment, and
refreshed on every deploy — so setter methods, init args, and post-deploy
wiring scripts are unnecessary. Add Pitfall 22 for that anti-pattern, move
the (corrected) Motoko example to references/canister-env-vars.md — envVar
requires the <system> capability and returns ?Text — and document two
settings caveats: create-class installs re-apply YAML env vars over
manually-set values, and the 128-char value cap.

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
@raymondk raymondk requested review from a team and JoshDFN as code owners July 15, 2026 01:18
@github-actions

Copy link
Copy Markdown

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: (626 chars)
  • Pass: license: "Apache-2.0"
  • Pass: metadata: (2 entries)

Tokens

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

Markdown

  • Pass: no unclosed code fences found

Tokens

File Tokens
SKILL.md body 7,511
references/binding-generation.md 1,031
references/canister-env-vars.md 298
references/dev-server.md 690
references/dfx-migration.md 2,620
Total 12,150

Content Analysis

Metric Value
Word count 4,066
Code block ratio 0.16
Imperative ratio 0.12
Information density 0.14
Instruction specificity 0.80
Sections 20
List items 76
Code blocks 34

References Content Analysis

Metric Value
Word count 2,291
Code block ratio 0.24
Imperative ratio 0.14
Information density 0.19
Instruction specificity 0.73
Sections 21
List items 35
Code blocks 14

Contamination Analysis

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

References Contamination Analysis

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

Result: 1 warning

Project Checks


✓ Project checks passed for 1 skills (0 warnings)

@raymondk raymondk merged commit 3ff755f into main Jul 16, 2026
6 checks passed
@raymondk raymondk deleted the cli-canister-env branch July 16, 2026 05:16
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.

2 participants