Skip to content

Conversation

@zemaj
Copy link

@zemaj zemaj commented Oct 29, 2025

Summary

  • add skills manifest loader/registry with Anthropic and local discovery
  • expose skills via the new skill tool and enforce allowed tool delegation
  • surface skills management in the settings UI and document authoring guidance

Checkpoint Summary

Status

  • Rust toolchain installed (rustup default stable).
  • cargo check -p code-core and cargo test -p code-core skills::manifest -- --nocapture both succeed.
  • cc crate pinned via [patch.crates-io] to commit 8a45e2b2e99daf9abe45ae404984dc6a65356ded because crates.io v1.2.41 was missing platform modules.

Plan (Skill Tool Wiring)

  1. Introduce Skill tool types and config plumbing
  2. Populate Anthropic payload with enabled skills
  3. Enforce allowed-tools gating stubs
  4. Summarize work

Outstanding TODOs

  1. code-rs/tui/src/chatwidget.rs
    • Verify imports for SkillsSettingsView, SkillDisplay, SkillsSettingsContent, and SkillLoader were added correctly.
    • Ensure no redundant imports remain.
  2. code-rs/tui/src/bottom_pane/skills_settings_view.rs
    • Confirm use ratatui::prelude::Widget; is present so .render(...) calls compile.
  3. code-rs/tui/src/chatwidget/settings_overlay.rs
    • Audit matches between lines ~1920–2100 to be sure SettingsSection::Skills is covered everywhere (render, mut access, notify_close, etc.).
  4. code-rs/tui/src/chatwidget.rs
    • Add/verify SettingsSection::Skills branch in the settings overview match around line ~18040 to avoid non-exhaustive match error.
  5. Re-run cargo check -p code-tui once the above fixes land.

Testing Gaps

  • No automated coverage yet for Anthropic skill stubs or the skill tool routing (browser/agents delegation). Consider targeted integration tests once compilation succeeds.

Quirks / Setup Notes

  • Workspace still carries the [patch.crates-io] override for cc; keep an eye on future dependency updates.
  • TUI build currently fails due to the missing imports/match arms above—fix before proceeding.

Testing

  • ./build-fast.sh

@github-actions
Copy link

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