[TS] Route thrown exceptions into catch blocks - #463
Open
CaelmBleidd wants to merge 11 commits into
Open
CaelmBleidd wants to merge 11 commits into
CaelmBleidd wants to merge 11 commits into
Conversation
Implement supported string equality cases and preserve explicit unsupported outcomes for unbacked symbolic witnesses. Reuse shared Node replay tests.
CaelmBleidd
force-pushed
the
caelmbleidd/ts-422-string-value-equality
branch
from
October 3, 2026 05:21
7927e93 to
127ecb5
Compare
CaelmBleidd
force-pushed
the
caelmbleidd/ts-427-catch-routing
branch
from
October 3, 2026 05:28
e0bc56a to
2d655b8
Compare
Keep catch witness assertions and use the shared Node process harness.
CaelmBleidd
marked this pull request as ready for review
October 3, 2026 06:44
CaelmBleidd
force-pushed
the
caelmbleidd/ts-422-string-value-equality
branch
from
October 4, 2026 19:51
e5527a3 to
7217c2a
Compare
Base automatically changed from
caelmbleidd/ts-422-string-value-equality
to
main
October 5, 2026 12:37
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Current stacked revision
127ecb5df35366c550a32725cf4e6af25208f673. Head:253e55a72435c5c0aa7cf996b36f1ee745d15c83. The rebased issue-specific implementation remains linear; the final commit only replaces duplicate test-side Node process handling with the shared helper.2d655b84.Closes #427.
Change
undefinedexception.7a2cda3bfd9175ba11a60e47ff4a32e6e3e209e2from [TS] Preserve exceptional CFG edges for try/catch jacodb#393, which supplies exceptional CFG edges andCaughtExceptionRef.Verification
ExceptionsandTsCatchRoutingTestpassed with a clean local composite build of that exact JacoDB commit.TsCatchRoutingTestuses ordinaryanalyzeWithOutcome, requiresEXHAUSTEDwith no unsupported paths, checks both conditional branches, catch bindings, nested catch and rethrow, and exceptions from a callee. Generated witnesses replay in Node.js.:usvm-ts:test: 1105 tests, 0 failures/errors, 143 skipped. The final rethrow regression was then run separately and passed.:usvm-ts:detektMainand:usvm-ts:detektTestpass with zero findings. Full CI run 37080042969 passed all six jobs on heade0bc56ad09df3af083772057992e5feb683c2a17.my-reviewhygiene pass on2d655b84found duplicate Node process handling;253e55a7removes it without changing catch-specific script assertions. The repository pre-commit hook invokes a missingcheckLicensetask, so this follow-up commit bypassed that hook after the focused test, Detekt, andgit diff --checkpassed.Stack
127ecb5df35366c550a32725cf4e6af25208f673).7a2cda3bfd9175ba11a60e47ff4a32e6e3e209e2).finallyis tracked separately from [TS] Route thrown exceptions into catch blocks #427.