-
Notifications
You must be signed in to change notification settings - Fork 9
[BUG] Windows absolute drive paths (C:\...) are not recognized as local sources #138
Copy link
Copy link
Open
Labels
bugSomething isn't workingSomething isn't working
Description
Bug Description
The asm install command fails to recognize Windows absolute drive paths (e.g., C:\Users\...\skill) as valid local sources, incorrectly treating them as remote GitHub repositories and throwing an "Invalid source format" error.
Steps to Reproduce
- Open terminal on Windows.
- Run
asm install C:\path\to\your\local-skill. - See error:
Error: Invalid source format. Got: "C:\path\to\your\local-skill".
Expected Behavior
The tool should recognize the Windows drive letter prefix and treat it as a local path, resolving it to the absolute disk location and proceeding with the installation.
Actual Behavior
The tool fails at the parsing stage with Error: Invalid source format.
Environment
- OS: Windows 10 (10.0.19045)
- Bun version: 1.3.11
- agent-skill-manager version: 1.19.0
Additional Context
The root cause is in src/installer.ts, where isLocalPath only checks for POSIX-style paths. Adding /^[a-zA-Z]:[/\\]/ to the detection logic resolves the issue. Verification shows that after the fix, isLocalPath("C:\\Users\\...") correctly returns true.
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
bugSomething isn't workingSomething isn't working
Projects
Status
In progress