Skip to content

Support multiple --skills-dir and --files-dir arguments#5

Merged
juhgiyo merged 3 commits intomainfrom
claude/multi-dirs-and-files
Mar 9, 2026
Merged

Support multiple --skills-dir and --files-dir arguments#5
juhgiyo merged 3 commits intomainfrom
claude/multi-dirs-and-files

Conversation

@juhgiyo
Copy link
Copy Markdown
Contributor

@juhgiyo juhgiyo commented Mar 9, 2026

Summary

  • Change SkillsDir stringSkillsDirs []string to accept multiple --skills-dir arguments
  • Add FilesDirs []string to accept multiple --files-dir arguments for project files
  • Skills symlink deduplication via os.Lstat check (first-wins, matching --roles-dir behavior)
  • Each files directory passed as --add-dir to Codex CLI

Test plan

  • Verify go build compiles cleanly
  • Test with single --skills-dir and --files-dir (backward compatible)
  • Test with multiple --skills-dir arguments
  • Test with multiple --files-dir arguments

🤖 Generated with Claude Code

juhgiyo and others added 3 commits March 9, 2026 10:30
- Change SkillsDir string → SkillsDirs []string (accumulate via append)
- Add FilesDirs []string for project files directories
- Skills symlink loop deduplicates with os.Lstat check
- Each files dir passed as --add-dir to Codex

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@juhgiyo juhgiyo merged commit 02a421d into main Mar 9, 2026
1 check passed
@juhgiyo juhgiyo deleted the claude/multi-dirs-and-files branch March 9, 2026 17:48
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