Skip to content

feat: Install vault-task-management skill during vault setup#401

Merged
rjroy merged 1 commit intomainfrom
feat/374-vault-management-task
Jan 24, 2026
Merged

feat: Install vault-task-management skill during vault setup#401
rjroy merged 1 commit intomainfrom
feat/374-vault-management-task

Conversation

@rjroy
Copy link
Copy Markdown
Owner

@rjroy rjroy commented Jan 24, 2026

Summary

  • Add skill installation step to vault setup process (installs to .claude/skills/)
  • Bundle vault-task-management skill for Obsidian task queries with checkbox syntax
  • Preserve executable permissions on shell scripts during installation
  • Add parallel test execution warning to CLAUDE.md

Closes #374

Test plan

  • All 76 vault-setup tests pass
  • Full pre-commit checks pass
  • Skills are copied with correct directory structure
  • Shell scripts retain executable permissions
  • Existing skills are not overwritten

🤖 Generated with Claude Code

Add skill installation to the vault setup process. Skills are copied
from backend/src/skills/ to .claude/skills/ in the vault, with
executable permissions preserved on shell scripts.

- Add installSkills() function with recursive directory copying
- Add vault-task-management skill (task queries for Obsidian checkboxes)
- Update SetupCompleteMarker to track skillsInstalled
- Bump SETUP_VERSION to 1.3.0
- Add parallel test warning to CLAUDE.md

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
@codecov
Copy link
Copy Markdown

codecov Bot commented Jan 24, 2026

Codecov Report

❌ Patch coverage is 73.04348% with 31 lines in your changes missing coverage. Please review.

Files with missing lines Patch % Lines
backend/src/vault-setup.ts 73.04% 31 Missing ⚠️

📢 Thoughts on this report? Let us know!

@rjroy rjroy merged commit 752a4be into main Jan 24, 2026
1 of 2 checks passed
@rjroy rjroy deleted the feat/374-vault-management-task branch January 24, 2026 22:05
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.

The attached skill needs to be part of vault setup

1 participant