feat(skill-detect): intent-to-skill injection + skill install CLI (#231, #233) - #290
Conversation
…ill subcommand Agentflare-Agent: 1 Agentflare-Branch: skill-detect-worktree
Agentflare-Agent: claude-code_2-1-216_agent Agentflare-Branch: skill-detect-worktree
…t injection hooks (#231) Agentflare-Agent: claude-code_2-1-216_agent Agentflare-Branch: skill-detect-worktree
…stry tests (#233) Agentflare-Agent: claude-code_2-1-216_agent Agentflare-Branch: skill-detect-worktree
…ll injection, clippy cleanup (#231) Agentflare-Agent: claude-code_2-1-216_agent Agentflare-Branch: skill-detect-worktree
|
Warning Review limit reached
Next review available in: 27 minutes Enable usage-based reviews in Billing to review now. Otherwise, wait until the next included review is available. How can I continue?After more reviews become available, a review can be triggered using the To avoid repeated limits, reduce automatic review volume by pausing incremental auto-reviews earlier, using label-based review opt-in, excluding WIP or generated PR titles, or requesting reviews manually when the PR is ready. If your team needs uninterrupted high-volume reviews, an organization admin can enable usage-based reviews. How do review limits work?CodeRabbit enforces per-developer PR review limits for each organization. Most developers receive the normal plan review availability. For paid Pro and Pro+ PR reviews, CodeRabbit uses adaptive limits for sustained high-volume activity. When a developer's recent PR review activity reaches the 95th percentile or higher among CodeRabbit users, additional reviews become available more gradually as earlier reviews age out of the rolling window. Please refer docs for additional details. Review details⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: CHILL Plan: Pro Plus Run ID: ⛔ Files ignored due to path filters (1)
📒 Files selected for processing (10)
✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
Brings in storage-guards (feat/storage-guards), the search.rs num_results clamp, and the audit-log TOCTOU fix. No conflicts outside main.rs's mod declaration ordering (shim_install vs skill_detect). Agentflare-Agent: claude-code_2-1-216_agent Agentflare-Branch: item-231-233-skill-detect
serde_yml has an unsound RUSTSEC-2025-0068 advisory (segfault risk in its Serializer) and its upstream project is archived; cargo-deny flags the 0.0.13 this crate's unconstrained "0.0" requirement resolved to. Switched to serde_yaml_ng, a maintained drop-in fork, for frontmatter parsing. Also applies the cargo fmt fix CI flagged on types.rs. Agentflare-Agent: claude-code_2-1-216_agent Agentflare-Branch: item-231-233-skill-detect
Summary
agentflare skill install/search/list/remove/registryCLI, backed by theskillcrate (git clone + SKILL.md discovery). Home resolution now goes throughcrate::paths::home()for consistency/testability instead of rawdirs::home_dir().skill_detect::classify) +skill_detectMCP tool + session-start and per-turn hook injection that map a prompt to aTaskTypeand rank matching skills fromskill_registry. Ranking blends BM25 with cosine similarity from the existing MiniLM ONNX pipeline (memory::engine::embed_query/embed_doc, same 0.4/0.6 weighting asmemory::search::search_hybrid); degrades to BM25-only when thesemanticfeature/model isn't available.Test plan
cargo test --bin agentflare skill— 27 tests, incl. newrerank_with_embeddingsunit tests and an end-to-end local-sourceskill installtest that actually copies a fixtureSKILL.mdinto the install dircargo check --features semantic— semantic path compilescargo clippy --bin agentflare -- -D warnings— clean on every file this PR touchesrustfmt --edition 2024 --check— clean on every file this PR touches