fix(setup): Windows backslash paths in the agents summary; suffix names CLI too

Merged
TO
3mo ago
+20-5

Windows user screenshot: ✓ Agents set up — ready to use InstaCloud (skill + MCP…) — nameless, because parseInstalledAgents only matched forward-slash paths and the skills tool prints C:\Users\…\.claude\skills\insta on Windows. The install itself succeeded; only the summary line degraded.

  • Path match is now separator-agnostic ([\\/]skills[\\/]), pinned by a Windows-backslash test (3 paths → 3 named agents).
  • Tony's copy: the ready-line names all three installs — (CLI + skill + MCP; restart any open tools) (the CLI is part of what's ready whether it self-installed this run or was already durable).

Gate: typecheck clean, 482/482 tests. Ships as 0.0.45.

🤖 Generated with Claude Code


Summary by cubic

Fixes Windows path parsing in the setup summary and updates the ready message to include CLI. Previously, Windows runs showed a nameless "Agents —" because the parser only matched "/skills/..."; now it is separator-agnostic and names agents, and the ready line reads "(CLI + skill + MCP; restart any open tools)".

  • Parser now matches both separators via /→\s*(\S+)[\/]skills[\/][A-Za-z0-9_-]+/; covered by a new test that maps three backslash paths to three named agents.
  • Update any assertions or docs expecting "(skill + MCP; ...)" to "(CLI + skill + MCP; ...)". No install flow changes; only parsing and copy.