Skip to content

Manual Execution #139

Merged
raphael-goetz merged 8 commits into#96-distinguish-local-remote-executionfrom
#138-manual-exec
Apr 5, 2026
Merged

Manual Execution #139
raphael-goetz merged 8 commits into#96-distinguish-local-remote-executionfrom
#138-manual-exec

Conversation

@raphael-goetz
Copy link
Copy Markdown
Member

Resolves: ##138

Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Adds a new “manual” CLI for executing validation flows (including remote node execution via NATS), factors shared test-case loading/types into a new tests-core crate, and updates flow fixtures/executor logic to support definition_source-based remote detection.

Changes:

  • Add crates/manual binary to execute a single flow case from a JSON file, with optional remote execution via NATS.
  • Introduce crates/tests-core to share flow test-case structs and JSON loading helpers; update crates/tests to use it.
  • Add definition_source: "taurus" to sample flows and adjust executor remote detection behavior for empty definition_source.

Reviewed changes

Copilot reviewed 11 out of 12 changed files in this pull request and generated 7 comments.

Show a summary per file
File Description
flows/03_for_each.json Adds definition_source to node functions in the flow fixture.
flows/02_return_flow_input.json Adds definition_source to node functions in the flow fixture.
flows/01_return_object.json Adds definition_source to node functions in the flow fixture.
crates/tests/src/main.rs Switches to shared tests-core types/helpers and refactors test runner structure.
crates/tests/Cargo.toml Adds tests-core dependency.
crates/tests-core/src/lib.rs New shared crate for test-case models + JSON loading helpers.
crates/tests-core/Cargo.toml New crate manifest for tests-core.
crates/manual/src/main.rs New CLI to run a selected flow/input and print the execution result.
crates/manual/Cargo.toml New crate manifest for manual.
crates/core/src/context/executor.rs Tweaks remote-node detection when definition_source is empty.
Cargo.toml Adds manual and tests-core to the workspace and workspace dependencies.
Cargo.lock Locks new dependencies introduced by the added crates (e.g., clap).

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

raphael-goetz and others added 2 commits April 5, 2026 14:12
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Signed-off-by: Raphael Götz <52959657+raphael-goetz@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Signed-off-by: Raphael Götz <52959657+raphael-goetz@users.noreply.github.com>
@raphael-goetz raphael-goetz merged commit 92292f7 into #96-distinguish-local-remote-execution Apr 5, 2026
1 check passed
@raphael-goetz raphael-goetz linked an issue Apr 5, 2026 that may be closed by this pull request
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.

Add Manual Execution

2 participants