Skip to content

feat: validate reviewed Goal Run configuration - #8

Merged
JJasonSun merged 1 commit into
mainfrom
feat/goal-run-configuration
Oct 27, 2026
Merged

JJasonSun merged 1 commit into
mainfrom
feat/goal-run-configuration

Conversation

@JJasonSun

Copy link
Copy Markdown
Collaborator

Summary

Add Prepare CALL-E Goal Run to choose a published Goal by title, version, and ID, show its input/result fields, and check mapped values against a saved reviewed interface. Invalid values or changed publications stop the example before creation.

saved business event → map fields → prepare reviewed Goal
                                      ├─ needs attention → field errors
                                      └─ ready → existing Run tool → check accepted version and result

Ship an importable workflow and plugin version 0.4.0. Existing creation and lookup tools keep their original-request replay and accepted-ID recovery behavior.

Evidence

  • Before: Goal Run configuration required manually entering an ID and JSON without checking the published field definitions.
    After: The selector and preparation node expose required fields, types, defaults, result definitions, and publication changes; 261 tests pass, including the execution tool's actual output shape.
  • After: Packaged 0.4.0 tested in isolated Dify CE 1.17.1: exact example import, first-review gate, preview, a real published Goal call to the official automated test hotline (confirmed=yes), unchanged-request replay returning the same Run ID, and recovery by ID. Missing, wrongly typed, and unknown fields stop before the execution node. Controlled API cases additionally verify changed fields, publication during acceptance, and saved-request recovery after publication.

Merge Danger

Door: two-way

Reverting removes the new helper and example. No data migration or deployment is included.

Blast Radius: Dify

Applies to workflows adopting the new preparation node. The example retains accepted IDs when the API accepts a different published version and requires review before downstream action.

@JJasonSun
JJasonSun merged commit 90f1ffa into main Oct 27, 2026
2 checks passed
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.

1 participant