Skip to content

Release v0.0.51 - #296

Merged
queil merged 10 commits into
mainfrom
release-v0.0.51
Sep 24, 2026
Merged

queil merged 10 commits into
mainfrom
release-v0.0.51

Conversation

@queil

@queil queil commented Sep 24, 2026

Copy link
Copy Markdown
Contributor

No description provided.

@strix-security

strix-security Bot commented Sep 24, 2026 •

Copy link
Copy Markdown

Strix Security Review

Warning

This pull request has 12 commits after the last Strix review (40eb277). Strix has not reviewed these changes.
Automatic review on push is off for this repository. To review the latest changes, tag @strix-security in a comment, or turn on re-review on push.

No security issues found.

Review summary

Reviewed PR #296 (Release v0.0.51) end to end. The diff adds a cmd | line reifier, a REPL kill-ring and $name argv-splice completion, an az -o table dashes-separator skip in from table, and a typed Proc.CommandFailure exception so a bare command statement's nonzero exit renders as a quiet exit-code status in the REPL. All new command execution continues to use ProcessStartInfo with ArgumentList (no shell interpolation), error messages follow the existing command-line-echo pattern, the kill-ring string operations are bounds-checked, and the new exception catch is scoped to bare command statements only while value/reifier positions keep the loud error path. No injection, secret exposure, access-control, or denial-of-service issues were identified in the changed code.

Updated for 40eb277.


Reviewed by Strix
Re-run review · Configure security review settings

@queil
queil force-pushed the release-v0.0.51 branch 3 times, most recently from 188c5eb to 7d88183 Compare September 24, 2026 13:44
@queil
queil changed the base branch from release-v0.0.50 to main September 24, 2026 17:32
@queil
queil merged commit 2af6079 into main Sep 24, 2026
3 checks passed
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