Skip to content

feat(icp-cli): route web identity flows to specialized skills#225

Merged
raymondk merged 1 commit into
mainfrom
chore/icp-cli-web-identity-routing-218
Jun 17, 2026
Merged

feat(icp-cli): route web identity flows to specialized skills#225
raymondk merged 1 commit into
mainfrom
chore/icp-cli-web-identity-routing-218

Conversation

@marc0olo

Copy link
Copy Markdown
Member

Summary

Resolves #218. Replaces the thin web-identity stub in skills/icp-cli/SKILL.md with a routing pointer to the two specialized skills that now own icp identity link web (both prerequisites — #212 deploy-to-cloud-engine and #217 agent-web-identity — have merged).

The old stub had two bare command examples with no --auth vs --app distinction, no stdin "Press Enter" warning, and no pitfalls — worse than nothing, since an agent could pick them up and run the command blind.

Change

-### Using the cli with a web identity
-...two bare `icp identity link web ... --app` examples...
+### Web identity flows
+- `deploy-to-cloud-engine` — link the CLI to a cloud engine console with `--auth <console-origin>`, then deploy to the engine's subnet
+- `agent-web-identity` — obtain a delegation for an app-specific principal with `--app <domain>`, then make canister calls as the user

Verification

  • Confirmed each target skill's actual flag against its source:
    • deploy-to-cloud-engine uses --auth <console-origin>
    • agent-web-identity uses --app <domain>
  • npm run validate → all 26 skills pass (only pre-existing unrelated warnings).

Evals

No case in evaluations/icp-cli.json referenced the removed content, so nothing breaks. No new cases added: this is a pure scope reduction with no new agent-facing behavior to regression-test, and a should_not_trigger guard for cloud-engine deploys would fight icp-cli's intentionally broad description ("deploying any IC project", "identity management").

Closes #218

Replace the thin `icp identity link web` stub with a routing pointer to
the deploy-to-cloud-engine and agent-web-identity skills. The two bare
examples had no --auth vs --app distinction, no stdin "Press Enter"
warning, and could mislead an agent into running the command blind.
Both flags are fully documented (with pitfalls) in the specialized
skills.

Closes #218
@marc0olo marc0olo requested review from a team and JoshDFN as code owners June 17, 2026 19:28
@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 6606 tokens (spec recommends < 5000)

Markdown

  • Pass: no unclosed code fences found

Tokens

File Tokens
SKILL.md body 6,606
references/binding-generation.md 1,031
references/dev-server.md 690
references/dfx-migration.md 2,620
Total 10,947

Content Analysis

Metric Value
Word count 3,570
Code block ratio 0.16
Imperative ratio 0.11
Information density 0.14
Instruction specificity 0.85
Sections 20
List items 70
Code blocks 34

References Content Analysis

Metric Value
Word count 2,133
Code block ratio 0.25
Imperative ratio 0.13
Information density 0.19
Instruction specificity 0.80
Sections 18
List items 33
Code blocks 12

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 low
Contamination score 0.03
Primary language category javascript
Scope breadth 2
  • Warning: Language mismatch: shell (1 category differ from primary)

Result: 1 warning

Project Checks


✓ Project checks passed for 1 skills (0 warnings)

@raymondk raymondk merged commit a63f551 into main Jun 17, 2026
6 checks passed
@raymondk raymondk deleted the chore/icp-cli-web-identity-routing-218 branch June 17, 2026 20:26
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.

icp-cli skill: replace web identity stub with routing pointer (after #212 and #217 land)

2 participants