fix(workflow-utils): omit --model/--provider when not explicitly set #223

2026-04-28T06:34:38Z

xiaoju commented

2026-04-28 06:34:38 +00:00

What

Stop passing --model auto --provider auto to hermes when not explicitly configured.

Why

Custom providers return 404 for model name "auto". Hermes should use its own config defaults.

Changes

hermes-options.ts — default to undefined instead of "auto"
hermes-agent.ts — conditionally include args only when defined
role-types.ts — type as string | undefined

Fixes #222

## What Stop passing `--model auto --provider auto` to hermes when not explicitly configured. ## Why Custom providers return 404 for model name `"auto"`. Hermes should use its own config defaults. ## Changes - `hermes-options.ts` — default to `undefined` instead of `"auto"` - `hermes-agent.ts` — conditionally include args only when defined - `role-types.ts` — type as `string | undefined` Fixes #222

xiaoju added 1 commit 2026-04-28 06:34:39 +00:00

fix(workflow-utils): omit --model/--provider when not explicitly set 323ba75ebf

Hermes role defaulted model/provider to 'auto', causing 404 on
custom providers. Now defaults to undefined and only passes
--model/--provider args when explicitly provided.

Fixes #222

xiaoju force-pushed fix/222-hermes-role-no-default-model from 323ba75ebf to 1efdc4dcd1

2026-04-28 06:52:19 +00:00

Compare

xiaomo approved these changes 2026-04-28 07:02:24 +00:00

xiaomo left a comment

LGTM ✅

简洁正确，T | null 符合项目规范。让 hermes 用自己的 config defaults 而非强制传 auto。

LGTM ✅ 简洁正确，`T | null` 符合项目规范。让 hermes 用自己的 config defaults 而非强制传 `auto`。

xiaomo merged commit 814d94f9de into main

2026-04-28 07:02:26 +00:00

xiaomo referenced this issue from a commit

2026-04-28 07:02:27 +00:00

Merge pull request 'fix(workflow-utils): omit --model/--provider when not explicitly set' (#223) from fix/222-hermes-role-no-default-model into main

This repo is archived. You cannot comment on pull requests.

2 Participants

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: uncaged/nerve#223