Skip to content

Replace the stdout extraction idiom with an EXTRACT command - #817

Merged
jviotti merged 1 commit into
mainfrom
extract-stdout
Aug 12, 2026
Merged

Replace the stdout extraction idiom with an EXTRACT command#817
jviotti merged 1 commit into
mainfrom
extract-stdout

Conversation

@jviotti

@jviotti jviotti commented Aug 12, 2026

Copy link
Copy Markdown
Member

Signed-off-by: Juan Cruz Viotti jv@jviotti.com

Review in cubic

Signed-off-by: Juan Cruz Viotti <jv@jviotti.com>
@augmentcode

augmentcode Bot commented Aug 12, 2026

Copy link
Copy Markdown
🤖 Augment PR Summary

Summary: This PR replaces the multi-step stdout extraction idiom in CLI tests with a dedicated DSL command.

Changes:

  • Introduces EXTRACT STDOUT FROM <observation> INTO <destination> in scripts/clitest.py.
  • Removes the obsolete KEEP LINES MATCHING command from the interpreter and DSL reference.
  • Extracts only 1>-prefixed observation records and restores their original content lines.
  • Registers extracted files as produced artifacts and observations as consumed inputs in the static test checker.
  • Migrates bundle, compile, validate, and JSON test-report fixtures to the new command.

Technical Notes: Observation files encode stdout as 1> records and bare 1> records for empty lines; the new command handles both forms and writes normalized LF-terminated output.

🤖 Was this summary useful? React with 👍 or 👎

@augmentcode augmentcode Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Review completed. No suggestions at this time.

Comment augment review to trigger a new review at any time.

@jviotti
jviotti merged commit e32d995 into main Aug 12, 2026
16 checks passed
@jviotti
jviotti deleted the extract-stdout branch August 12, 2026 16:21
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