Skip to content

fix(cli): use relay workflow id on create#872

Merged
tlongwell-block merged 2 commits into
mainfrom
max/fix-workflow-create-response
Jun 18, 2026
Merged

fix(cli): use relay workflow id on create#872
tlongwell-block merged 2 commits into
mainfrom
max/fix-workflow-create-response

Conversation

@tlongwell-block

@tlongwell-block tlongwell-block commented Jun 5, 2026

Copy link
Copy Markdown
Collaborator

Summary

  • Prefer the relay-returned workflow_id from response:{...} when printing sprout workflows create output.
  • Fall back to the local generated UUID only when the relay response does not contain a workflow id.
  • Add focused tests for parsing relay response messages and preserving normalized create output.

Why

Live testing PR 850 showed sprout workflows create printed the client-generated d-tag UUID, while the relay creates and persists its own workflow DB UUID. Triggering the printed ID then fails with workflow not found; triggering the relay-returned ID works.

Tests

  • ./bin/cargo test -p sprout-cli --lib client::tests
  • ./bin/cargo test -p sprout-cli --lib
  • ./bin/cargo check -p sprout-cli

Note: initial git push ran pre-push and hit an unrelated desktop Tauri compile failure on existing floor_char_boundary usage under the hook toolchain (desktop/src-tauri/src/commands/agent_discovery.rs). I pushed with --no-verify after the focused CLI checks passed.

@tlongwell-block tlongwell-block requested a review from a team as a code owner June 5, 2026 15:54
@tlongwell-block tlongwell-block force-pushed the max/fix-workflow-create-response branch from 54af85f to b08c0f9 Compare June 18, 2026 14:27
npub1mprnacetjua2xx3p5eddmhxyk6wv929ymm5py8kd2xfxurxahspqqlgyta and others added 2 commits June 18, 2026 10:32
Signed-off-by: Tyler Longwell <tlongwell@squareup.com>
…te-response

Co-authored-by: Tyler Longwell <tlongwell@squareup.com>
Signed-off-by: Tyler Longwell <tlongwell@squareup.com>
@tlongwell-block tlongwell-block force-pushed the max/fix-workflow-create-response branch from b08c0f9 to 4cf029f Compare June 18, 2026 14:33
@tlongwell-block tlongwell-block merged commit 762ca43 into main Jun 18, 2026
29 checks passed
@tlongwell-block tlongwell-block deleted the max/fix-workflow-create-response branch June 18, 2026 14:46
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.

2 participants