Skip to content

Conversation

@stesie
Copy link
Contributor

@stesie stesie commented Aug 14, 2025

PR Checklist

Please check if your PR fulfills the following requirements:

  • The commit message follows our guidelines
  • Tests for the changes have been added (for bug fixes / features)
  • Docs have been added / updated (for bug fixes / features)

PR Type

What kind of change does this PR introduce?

  • Bugfix
  • Feature
  • Code style update (formatting, local variables)
  • Refactoring (no functional changes, no api changes)
  • Build related changes
  • CI related changes
  • Documentation content changes
  • Other... Please describe:

What is the current behavior?

Issue Number: N/A

So far it's a bit cumbersome if you want to capture the command output to a variable. The obvious work-around being redirecting stdout to a well-known temporary file, then using FORMULA type step in combination with inputFile and issue another command to delete the temporary file.

What is the new behavior?

You can specify resultVariable prop on the COMMAND type step, this will run the command with output capturing.

A subsequent FORMULA type step can the process the result further, e.g. split by newlines etc. to prepare entities for a select prompt etc.pp

Does this PR introduce a breaking change?

  • Yes
  • No

Other information

@stesie stesie requested a review from MarcScheib as a code owner August 14, 2025 18:14
@MarcScheib
Copy link
Contributor

Tested locally with an example use-case. Works really nice.

@MarcScheib MarcScheib merged commit eca1175 into CycriLabs:develop Aug 15, 2025
1 check 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.

2 participants