fix: preset base-url auto-fill, bootstrap ACP docs, friendlier errors #109

Merged
xiaomo merged 1 commits from fix/106-107-108-bootstrap-ux into main 2026-06-05 11:16:31 +00:00
Owner

What

Fix three issues reported by 小糯 during onboarding testing.

#106 — preset provider base-url 自动填充

uwf setup --provider dashscope --api-key ... --model ... now auto-fills --base-url from the preset list. Previously required all 4 flags even for known providers.

Changes:

  • setup.ts — export resolvePresetBaseUrl() helper
  • cli.ts — resolve preset base-url before the non-interactive check; error message updated

#107 — bootstrap 缺少 ACP 依赖 + 引用不存在的文件

Changes:

  • Bootstrap adapter table adds ACP dependency note for uwf-hermes: pip install hermes-agent[acp]
  • Verify step replaced examples/eval-simple.yaml (not in npm package) with inline hello.yaml workflow

#108 — workflow 文件名不匹配报错不友好

Changes:

  • validate.ts — error message now suggests fix: Either rename the file to "X.yaml" or change the YAML name field to "Y"

Tests

32/32 prompt + workflow-resolution tests passing.

Ref

Fixes #106, Fixes #107, Fixes #108

## What Fix three issues reported by 小糯 during onboarding testing. ### #106 — preset provider base-url 自动填充 `uwf setup --provider dashscope --api-key ... --model ...` now auto-fills `--base-url` from the preset list. Previously required all 4 flags even for known providers. **Changes:** - `setup.ts` — export `resolvePresetBaseUrl()` helper - `cli.ts` — resolve preset base-url before the non-interactive check; error message updated ### #107 — bootstrap 缺少 ACP 依赖 + 引用不存在的文件 **Changes:** - Bootstrap adapter table adds ACP dependency note for uwf-hermes: `pip install hermes-agent[acp]` - Verify step replaced `examples/eval-simple.yaml` (not in npm package) with inline `hello.yaml` workflow ### #108 — workflow 文件名不匹配报错不友好 **Changes:** - `validate.ts` — error message now suggests fix: `Either rename the file to "X.yaml" or change the YAML name field to "Y"` ## Tests 32/32 prompt + workflow-resolution tests passing. ## Ref Fixes #106, Fixes #107, Fixes #108
xiaoju added 1 commit 2026-06-05 11:06:58 +00:00
- #106: uwf setup --provider <preset> now auto-fills --base-url
- #107: bootstrap documents hermes ACP dependency (pip install hermes-agent[acp])
- #107: verify step uses inline hello.yaml instead of missing examples/eval-simple.yaml
- #108: workflow name mismatch error suggests how to fix (rename file or change YAML name)

Fixes #106, Fixes #107, Fixes #108
xiaomo approved these changes 2026-06-05 11:16:29 +00:00
xiaomo left a comment
Owner

LGTM 三个 issue 一锅端,都是实际使用时的痛点:

#106resolvePresetBaseUrl()PRESET_PROVIDERS 查表,--base-url 对 preset 变可选。错误信息也同步更新了。

#107 — bootstrap 加了 pip install hermes-agent[acp] 说明;smoke test 改用 inline hello.yaml 而不是引用不存在的 examples/eval-simple.yaml,自包含。

#108 — name mismatch 错误信息给了两个修复方向(重命名文件 or 改 YAML name),新手友好。

LGTM ✅ 三个 issue 一锅端,都是实际使用时的痛点: **#106** — `resolvePresetBaseUrl()` 从 `PRESET_PROVIDERS` 查表,`--base-url` 对 preset 变可选。错误信息也同步更新了。 **#107** — bootstrap 加了 `pip install hermes-agent[acp]` 说明;smoke test 改用 inline `hello.yaml` 而不是引用不存在的 `examples/eval-simple.yaml`,自包含。 **#108** — name mismatch 错误信息给了两个修复方向(重命名文件 or 改 YAML name),新手友好。
xiaomo merged commit f1f122b0b1 into main 2026-06-05 11:16:31 +00:00
xiaomo deleted branch fix/106-107-108-bootstrap-ux 2026-06-05 11:16:31 +00:00
Sign in to join this conversation.
No Reviewers
2 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: shazhou/united-workforce#109