Add create-testvectors skill

[quexten]

🎟️ Tracking

n/a

📔 Objective

Adds a create-testvectors Claude Code skill that documents how to record a serialized/encrypted format as a permanent test vector.

🚨 Breaking Changes

None.

[github-actions]

🤖 Bitwarden Claude Code Review

Overall Assessment: APPROVE

Reviewed the addition of the create-testvectors Claude Code skill (.claude/skills/create-testvectors/SKILL.md), a documentation-only change. Verified the YAML frontmatter, the allowed-tools scoping, and that every referenced file, function, constant, and code example matches the actual implementations in bitwarden-crypto (data_envelope.rs and password_protected_key_envelope.rs). No security, correctness, or breaking-change concerns found.

Code Review Details

No actionable findings.

• Frontmatter is valid; allowed-tools is appropriately scoped to Bash(cargo test:*) and Bash(npm run lint).
• All cited references (generate_test_vectors, test_data_envelope_test_vector, seal_ref/unseal_ref, ExampleNamespace, TESTVECTOR_* consts, test_testvector_cosekey/test_testvector_legacykey) exist and match the codebase.
• Code examples faithfully mirror the real reference tests; no broken links, no hardcoded secrets, no committed local settings.
• The existing thread suggesting allowed-tools was already addressed by the author.

[github-actions]

🔍 SDK Breaking Change Detection

SDK Version: km/testvector-skill (cb2530b)

⚠️ If breaking changes are detected, a corresponding pull request addressing them must be ready for merge in the affected client repository.

Client	Status	Details
typescript	✅ No breaking changes detected	Compilation passed with new SDK version - View Details
android	✅ No breaking changes detected	Compilation passed with new SDK version - View Details

---

Breaking change detection uses the build of the SDK from this branch, including any incompatibities pre-existing on or merged into this branch. Check the workflow logs to confirm.
Results update as workflows complete.

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 85.09%. Comparing base (4b41c1c) to head (2966cbb).
⚠️ Report is 4 commits behind head on main.

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #1208      +/-   ##
==========================================
+ Coverage   85.08%   85.09%   +0.01%     
==========================================
  Files         464      464              
  Lines       63781    63866      +85     
==========================================
+ Hits        54267    54348      +81     
- Misses       9514     9518       +4     

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
• 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

[SaintPatrck]

Great skill. Overall this is excellent. Well scoped and concise. 👏

One minor suggestion to remove a couple permission prompts while the skill is doing its thing.

[SaintPatrck]

🎨 Adding Bash(cargo test:*) and Bash(npm run lint) to allowed-tools will make these command executions more seamless. Users won't have to manually approve them.

[quexten]

Good point. Applied!

[sonarqubecloud]

Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud