Skip to content

Add Claude Skills tooling and settings#372

Closed
zemaj wants to merge 1 commit into
mainfrom
feature/skills-support
Closed

Add Claude Skills tooling and settings#372
zemaj wants to merge 1 commit into
mainfrom
feature/skills-support

Conversation

@zemaj

@zemaj zemaj commented Oct 29, 2025

Copy link
Copy Markdown

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
Copy Markdown

@zemaj

zemaj commented Jun 3, 2026

Copy link
Copy Markdown
Author

Codex PR triage, 2026-06-03:

Closing this as stale/superseded by current main skills infrastructure. The branch is now dirty across core/TUI/Cargo paths, while current main already has the newer skills loader/model/render/system docs and settings wiring. If provider-advertised skill-container behavior is still missing, please open a fresh focused PR for that gap.

@zemaj zemaj closed this Jun 3, 2026
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.

1 participant