Skip to content

fix(commands): honor PILOT_HOME for user command paths#345

Open
MicroMilo wants to merge 1 commit into
OpenBMB:mainfrom
MicroMilo:codex/fix-command-list-pilot-home
Open

fix(commands): honor PILOT_HOME for user command paths#345
MicroMilo wants to merge 1 commit into
OpenBMB:mainfrom
MicroMilo:codex/fix-command-list-pilot-home

Conversation

@MicroMilo

Copy link
Copy Markdown

Summary

  • resolve user command and skill roots from PILOT_HOME in the commands route
  • apply the same active-home roots when listing, loading, and executing user commands/skills
  • add a regression test with separate HOME and PILOT_HOME directories

Fixes #344.

Root cause

/api/commands/list imported resolvePilotHome, but still scanned user command and skill directories from os.homedir()/.pilotdeck. That made the command menu diverge from SkillManager and /api/skills/* when PILOT_HOME was set.

The same path assumption also affected the follow-up load/execute path checks, so this PR keeps those handlers on the same active-home roots.

Validation

  • npm --workspace ui exec vitest run server/routes/commands.test.js
  • npm exec -- tsc --noEmit -p tsconfig.json

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.

Command list ignores PILOT_HOME when discovering user skill commands

1 participant