Skip to content

feat: Add encrypted-maps skill for vetKeys EncryptedMaps#158

Open
andreacerulli wants to merge 2 commits intomainfrom
add-encrypted-maps-skill
Open

feat: Add encrypted-maps skill for vetKeys EncryptedMaps#158
andreacerulli wants to merge 2 commits intomainfrom
add-encrypted-maps-skill

Conversation

@andreacerulli
Copy link
Copy Markdown
Collaborator

New skill covering end-to-end encrypted on-chain storage using the ic-vetkeys EncryptedMaps library. Includes complete Rust backend, Motoko backend reference, frontend TypeScript examples, icp-cli configuration, and pitfalls from real-world testing.

New skill covering end-to-end encrypted on-chain storage using the
ic-vetkeys EncryptedMaps library. Includes complete Rust backend,
Motoko backend reference, frontend TypeScript examples, icp-cli
configuration, and pitfalls from real-world testing.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@andreacerulli andreacerulli requested review from a team and JoshDFN as code owners April 9, 2026 12:45
@github-actions
Copy link
Copy Markdown

github-actions bot commented Apr 9, 2026

Skill Validation Report

Validating skill: /home/runner/work/icskills/icskills/skills/encrypted-maps

Structure

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

Frontmatter

  • Pass: name: "encrypted-maps" (valid)
  • Pass: description: (521 chars)
  • Pass: license: "Apache-2.0"
  • Pass: compatibility: (16 chars)
  • Pass: metadata: (2 entries)

Markdown

  • Pass: no unclosed code fences found

Tokens

File Tokens
SKILL.md body 4,744
references/motoko-backend.md 2,159
Total 6,903

Content Analysis

Metric Value
Word count 1,976
Code block ratio 0.52
Imperative ratio 0.09
Information density 0.31
Instruction specificity 1.00
Sections 13
List items 19
Code blocks 11

References Content Analysis

Metric Value
Word count 899
Code block ratio 0.93
Imperative ratio 0.00
Information density 0.47
Instruction specificity 1.00
Sections 3
List items 2
Code blocks 3

Contamination Analysis

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

References Contamination Analysis

Metric Value
Contamination level low
Contamination score 0.00
Primary language category config
Scope breadth 1

Result: passed

Project Checks


WARNINGS (1):
  ⚠ encrypted-maps/SKILL.md: missing evaluations/encrypted-maps.json — see CONTRIBUTING.md for evaluation guidance

✓ Project checks passed for 1 skills (1 warnings)

@andreacerulli andreacerulli changed the title Add encrypted-maps skill for vetKeys EncryptedMaps feat: Add encrypted-maps skill for vetKeys EncryptedMaps Apr 9, 2026
Comment on lines +74 to +83
"@dfinity/agent": "^3.4.0",
"@dfinity/principal": "^3.4.0",
"@dfinity/vetkeys": "^0.4.0",
"@icp-sdk/auth": "^5.0.0",
"@icp-sdk/core": "^5.0.0"
}
}
```

`@dfinity/agent` and `@dfinity/principal` are required because `@dfinity/vetkeys` depends on them. `@icp-sdk/auth` provides Internet Identity login — import from `@icp-sdk/auth/client`, not the root module (the root has no exports). `@icp-sdk/core` provides canister ID discovery via the `ic_env` cookie set by `icp deploy`.
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

same question here. can we get rid of using @dfinity/agent and @dfinity/principal ?

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