feat: SSH agent auth for compute nodes + fix config path#180
feat: SSH agent auth for compute nodes + fix config path#180liuyixin-louis wants to merge 2 commits intoOpenLAIR:mainfrom
Conversation
- Add "SSH Agent" as a third authentication option alongside SSH Key and Password, supporting 1Password, ssh-agent, and other key agents - Fall back to SSH agent in execSsh/execRsync instead of throwing an error when no explicit key or password is configured - Move config directory from ~/.openclaw to ~/.dr-claw for consistency with the rest of the app's config paths - Update NodeForm UI with SSH Agent button and explanatory text - Fix configured check to not require explicit key/password Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Review notesTested CI is clean and the logic changes (SSH agent fallback in Missing migration:
|
On first load, if ~/.dr-claw/compute-node.json doesn't exist but ~/.openclaw/compute-node.json does, copy it over. Same for community-tools.json. Addresses review feedback on PR OpenLAIR#180. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
|
Thanks for the review! Addressed the migration concern in 87a2f7b: Migration logic added:
Both log when migration happens. Tested three scenarios: legacy-only → migrates, new-path-exists → no-op, neither-exists → empty default. Re: the minor notes — yes, |
Summary
execSsh()/execRsync()to fall back to SSH agent instead of throwing "No authentication method configured" when no explicit key or password is set~/.openclawto~/.dr-clawfor consistency with the rest of the appNodeFormUI with SSH Agent button and explanatory hint textTest plan
~/.dr-claw/compute-node.jsonused instead of~/.openclaw/~/.dr-claw/