Skip to content

fix: report partial script compilation failures on stderr - #300

Merged
swananan merged 1 commit into
mainfrom
fix/report-partial-script-failures
Sep 6, 2026
Merged

fix: report partial script compilation failures on stderr#300
swananan merged 1 commit into
mainfrom
fix/report-partial-script-failures

Conversation

@swananan

@swananan swananan commented Sep 6, 2026

Copy link
Copy Markdown
Owner

A script containing both valid and invalid trace targets could keep tracing the valid targets while hiding the failed targets when logging and status output were disabled.

Report partial compilation failures on stderr independently of those settings. Keep the successful probe configurations and existing attachment and exit-code behavior. Complete failures continue through the existing error path.

Rebased onto main at f28b207; one commit, two files. git range-diff confirms the original patch is unchanged.

Validation on f418dcf:

  • Formatting check and full workspace Clippy with CI warning settings passed.
  • All 955 non-e2e tests passed locally, with the same test-name list as GitHub Test Suite.
  • Full standard e2e passed locally and in GitHub: 348 tests, including the new --no-log --no-status regression.
  • A temporary base/head CLI matrix passed 12 scenarios (24 invocations): missing targets in either order, body compilation failures, all-success/all-failure cases, and syntax errors in live and dry-run modes. Partial runs gained stderr diagnostics while valid tracing, dry-run stdout, and exit behavior remained consistent.
  • All nine applicable GitHub checks passed, including three full container topologies and host-PID smoke. Local container runs were omitted because this change does not alter container behavior.

@swananan
swananan marked this pull request as ready for review September 6, 2026 04:59
@swananan
swananan force-pushed the fix/bitfield-integer-promotion branch from 957f16e to 543e64d Compare September 6, 2026 09:32
Base automatically changed from fix/bitfield-integer-promotion to main September 6, 2026 11:23
@swananan
swananan force-pushed the fix/report-partial-script-failures branch from 7b5d007 to f418dcf Compare September 6, 2026 11:30
@swananan
swananan merged commit 9b0c982 into main Sep 6, 2026
9 checks passed
@swananan
swananan deleted the fix/report-partial-script-failures branch September 6, 2026 12: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