Skip to content

Add skills feature with code quality improvements#22

Merged
HartBrook merged 2 commits intomainfrom
feature/skills
Jan 27, 2026
Merged

Add skills feature with code quality improvements#22
HartBrook merged 2 commits intomainfrom
feature/skills

Conversation

@HartBrook
Copy link
Copy Markdown
Owner

@HartBrook HartBrook commented Jan 26, 2026

Summary

  • Add skills management feature implementing the Agent Skills standard (agentskills.io) with Claude Code extensions
  • Include starter skills: code-review, security-audit, test-gen
  • Improve code quality across the codebase based on code review findings

Changes

Skills Feature

  • New internal/skills package for skill parsing, registry, and syncing
  • CLI command stag skills for listing and managing skills
  • Skills sync from team repos during stag sync
  • Support for skill installation to ~/.claude/skills/ or .claudeskills/
  • Hierarchical precedence: project > personal > team > starter

Code Quality Improvements

  • Error handling: Fixed swallowed error in YAML marshal (claude.go)
  • Context propagation: GitHub client methods now properly use context for cancellation/timeouts
  • Code deduplication: Extracted syncDirectoryContents helper to reduce repeated sync logic
  • Consistent logging: Changed raw stderr writes to log.Printf for library code
  • Better errors: Added directory paths to error messages for easier debugging
  • Configurable threshold: Token warning threshold now uses config instead of hardcoded value

Test plan

  • All existing tests pass (go test ./...)
  • Skills parsing handles valid and invalid frontmatter
  • Registry correctly handles precedence across sources
  • Integration tests verify end-to-end sync behavior
  • Manual testing of stag skills command
  • Manual testing of skills sync during stag sync

@HartBrook HartBrook merged commit 231d1f0 into main Jan 27, 2026
4 checks passed
HartBrook added a commit that referenced this pull request Jan 28, 2026
Add entries for path-scoped rules (#20), multi-source sync (#21),
skills feature (#22), and integration test framework (#16).
@HartBrook HartBrook mentioned this pull request Jan 28, 2026
3 tasks
HartBrook added a commit that referenced this pull request Jan 28, 2026
Add entries for path-scoped rules (#20), multi-source sync (#21),
skills feature (#22), and integration test framework (#16).
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