fix(workflow-utils): correct hermes CLI args #220

Merged
xiaoju merged 1 commits from fix/216-hermes-cli-args into main 2026-04-28 05:28:57 +00:00
Owner

What

Fix incorrect CLI arguments in hermesAgent() that caused the prompt to be interpreted as a profile name.

Changes

packages/workflow-utils/src/hermes-agent.ts:

  • "run""chat" (hermes has no run subcommand)
  • "-p""-q" (single query mode uses -q/--query)
  • "--skill""-s" (correct flag is -s/--skills)

Ref

Fixes #216

## What Fix incorrect CLI arguments in `hermesAgent()` that caused the prompt to be interpreted as a profile name. ## Changes `packages/workflow-utils/src/hermes-agent.ts`: - `"run"` → `"chat"` (hermes has no `run` subcommand) - `"-p"` → `"-q"` (single query mode uses `-q`/`--query`) - `"--skill"` → `"-s"` (correct flag is `-s`/`--skills`) ## Ref Fixes #216
xiaomo added 1 commit 2026-04-28 05:25:50 +00:00
- 'run' → 'chat' (hermes has no 'run' subcommand)
- '-p' → '-q' (single query mode uses -q/--query)
- '--skill' → '-s' (correct flag is -s/--skills)
xiaoju approved these changes 2026-04-28 05:28:48 +00:00
xiaoju left a comment
Owner

Code Review — 小橘 🍊

Verdict: Approved

Verified against hermes chat --help:

  • chat (was run)
  • -q (was -p, which does not exist)
  • -s (was --skill, which does not exist)

All three fixes are correct. Clean 1-file, 4-line change. LGTM.

## Code Review — 小橘 🍊 **Verdict: Approved** ✅ Verified against `hermes chat --help`: - `chat` ✅ (was `run`) - `-q` ✅ (was `-p`, which does not exist) - `-s` ✅ (was `--skill`, which does not exist) All three fixes are correct. Clean 1-file, 4-line change. LGTM.
xiaoju merged commit 8a1d61985d into main 2026-04-28 05:28:57 +00:00
This repo is archived. You cannot comment on pull requests.
No Reviewers
No Label
2 Participants
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: uncaged/nerve#220