Skip to content

Adopt upstream clitest binary from Core - #826

Merged
jviotti merged 2 commits into
mainfrom
new-core-clitest
Aug 17, 2026
Merged

Adopt upstream clitest binary from Core#826
jviotti merged 2 commits into
mainfrom
new-core-clitest

Conversation

@jviotti

@jviotti jviotti commented Aug 17, 2026

Copy link
Copy Markdown
Member

No description provided.

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

augmentcode Bot commented Aug 17, 2026

Copy link
Copy Markdown
🤖 Augment PR Summary

Summary: This PR replaces the repository-local Python CLI test interpreter with Core's upstream clitest binary.

Changes:

  • Updates the vendored Core revision and enables its test and stacktrace language components.
  • Registers JSON Schema CTest scripts directly against sourcemeta_core_clitest.
  • Removes the Python interpreter dependency, its source file, and Python from the Alpine test image.
  • Adds Core's C++ CLI-test runner, TAP reporting, DSL parser, checks, process execution, and filesystem commands.
  • Adds Core Unicode validation, text replacement/join helpers, and binary-safe file reads used by the runner.
  • Removes direct Snap workflow invocations of the deleted Python runner.

Technical Notes: The new runner provides isolated temporary sandboxes, explicit environment handling, deterministic stream observations, and static checks for vacuous scripts.

🤖 Was this summary useful? React with 👍 or 👎

@cubic-dev-ai cubic-dev-ai 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.

All reported issues were addressed across 16 files

Tip: cubic can generate docs of your entire codebase and keep them up to date. Try it here.

Re-trigger cubic

Comment thread .github/workflows/test.yml
Comment thread test/CMakeLists.txt
Signed-off-by: Juan Cruz Viotti <jv@jviotti.com>
@jviotti
jviotti merged commit 6a9b01c into main Aug 17, 2026
16 checks passed
@jviotti
jviotti deleted the new-core-clitest branch August 17, 2026 18:27
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