camunda / camunda/spm-cli

vendor: materialize transitively resolved skills with inherited targets

Open
#82 1 comment 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

enhancement
Dominant language
Rust
Stars
8
Forks
0
Avg merge
6h 12m
Merged PRs (30d)
19

Description

Summary

Phase 3 of #68 (depends on the recursive resolution sub-issue landing). Wire the recursively-resolved, deduplicated skill set into vendor materialization so transitive skills actually get written out for each configured target.

Scope

  • Confirm/implement that a transitively-resolved skill inherits the top-level targets (Claude/Copilot) rather than declaring its own subset, per the design decision.
  • Ensure src/vendor/mod.rs's for_target dispatch and the Claude (src/vendor/claude.rs) / Copilot (src/vendor/copilot.rs) adapters materialize each deduplicated skill exactly once, even if reached via multiple paths in the graph.
  • End-to-end test in tests/cli.rs: a manifest with one skill whose repo declares a nested ai.json with a further skill, asserting both are materialized for both targets without duplication.

Relevant code

  • src/vendor/mod.rs, src/vendor/claude.rs, src/vendor/copilot.rs, tests/cli.rs

Part of #68.

Contributor guide

No contributing guide indexed for this repository

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Research direction

Start with the recursive resolution dependency and read src/vendor/mod.rs, then follow the Claude and Copilot adapters in src/vendor/claude.rs and src/vendor/copilot.rs. Run the relevant end-to-end coverage in tests/cli.rs with a nested ai.json manifest; done means both transitively resolved skills are materialized for Claude and Copilot exactly once.

Written by the indexing model from the issue text.

Assessment

Tech stack
rust
Domain
cli
Issue type
Feature
Difficulty
4/5
Estimated time
3-5 days
Activity status
Active
Clarity
Mostly clear
Newbie friendliness
55/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.