Skip to content

Add --skip-verification flag to pyAnalyzeLaurel#1097

Open
joehendrix wants to merge 1 commit intomainfrom
jhx/pyspec_to_laurel_only
Open

Add --skip-verification flag to pyAnalyzeLaurel#1097
joehendrix wants to merge 1 commit intomainfrom
jhx/pyspec_to_laurel_only

Conversation

@joehendrix
Copy link
Copy Markdown
Contributor

@joehendrix joehendrix commented May 1, 2026

This adds a --skip-verification flag to the pyAnalyzeLaurel command that bypasses all the steps after converting to Core translation. This flag enables fast detection of type-checker and translation bugs without that overhead of a full verification pass.

  • Adds a --skip-verification boolean flag to the pyAnalyzeLaurel command
  • When set, runs the full Python→Laurel→Core translation pipeline (including all 11 Laurel lowering passes) but skips SMT verification
  • Reports translation diagnostics via the existing RESULT/DETAIL output format
  • SARIF output is written (with empty VC results) when --sarif is also set, so the flag is a drop-in addition to existing invocations
  • SMT-only flags (--check-mode, --entry-point, --solver, --timeout) are silently ignored

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Runs the Python→Laurel→Core translation pipeline without SMT
verification. Intended for fast detection of type-checker and
translation bugs without the overhead and unpredictability of
SMT solving.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@joehendrix joehendrix marked this pull request as ready for review May 2, 2026 01:32
@joehendrix joehendrix requested a review from a team May 2, 2026 01:32
Comment thread StrataMain.lean
Comment on lines +656 to +657
-- When --skip-verification is set, report translation diagnostics and exit
-- without running SMT verification (stages 3-4).
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Can we please have at least one test that shows that this works and what this looks like?

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.

3 participants