Repository navigation
Rewrite the SLIM parser and move to protocol V0.5 with stdin/stdout - #96
Merged
Merged
Conversation
konstantinvlasenko
commented
Oct 30, 2026
konstantinvlasenko
commented
Owner
- Parse instruction lists with a real deserializer instead of turning them into PowerShell code with regexes and Invoke-Expression; lengths may have more than six digits.
- Run page code in its own runspace (global scope), so variables persist without rewriting $x= to $script:x= and pages cannot touch PowerSlim state.
- Symbols are bound as PowerShell variables; values are never pasted into code. FitNesse text rules remain for '...', "$name_more" and te$name.
- Report Slim -- V0.5: add the assign instruction, ABORT_SLIM_SUITE for StopSuite, IGNORE_SCRIPT_TEST / IGNORE_ALL_TESTS.
- Port 1 runs over stdin/stdout (FitNesse default), so SLIM_PORT is no longer needed; TCP still works and stays covered by TestScenario.
- Remote symbol sync sends base64 literals and skips symbols that came from the target agent; decimals stay Double.
added 2 commits
October 30, 2026 00:42
- Parse instruction lists with a real deserializer instead of turning them into PowerShell code with regexes and Invoke-Expression; lengths may have more than six digits. - Run page code in its own runspace (global scope), so variables persist without rewriting $x= to $script:x= and pages cannot touch PowerSlim state. - Symbols are bound as PowerShell variables; values are never pasted into code. FitNesse text rules remain for '...', "$name_more" and te$name. - Report Slim -- V0.5: add the assign instruction, ABORT_SLIM_SUITE for StopSuite, IGNORE_SCRIPT_TEST / IGNORE_ALL_TESTS. - Port 1 runs over stdin/stdout (FitNesse default), so SLIM_PORT is no longer needed; TCP still works and stays covered by TestScenario. - Remote symbol sync sends base64 literals and skips symbols that came from the target agent; decimals stay Double.
Windows PowerShell 5.1 (.NET Framework) writes a BOM to a child's stdin when the console encoding is UTF-8; the runner read it as a bad length and exited. The stdio test now shows the runner's stderr when it gets no answer.
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.