Skip to content

Harden CLI installer command arguments#154

Merged
Daliys merged 1 commit into
developmentfrom
daliys/issue-65-cli-installer-arglist
Jul 9, 2026
Merged

Harden CLI installer command arguments#154
Daliys merged 1 commit into
developmentfrom
daliys/issue-65-cli-installer-arglist

Conversation

@Daliys

@Daliys Daliys commented Jul 7, 2026

Copy link
Copy Markdown
Member

Summary

  • Replace CLI installer shell command strings with executable plus quoted argument construction.
  • Keep Windows .cmd/.bat shims working through an escaped cmd.exe invocation.
  • Add editor tests for metacharacter preservation and batch shim escaping.

Validation

  • PYTHONDONTWRITEBYTECODE=1 scripts/prepush-validate.sh --static-only
  • git push pre-push validation passed; live smoke was skipped because the Unity server returned HTTP 401 Unauthorized.

Issue: #65

@Daliys Daliys marked this pull request as ready for review July 7, 2026 23:53
@Daliys Daliys merged commit 8b3c33a into development Jul 9, 2026
5 checks passed
@Daliys Daliys deleted the daliys/issue-65-cli-installer-arglist branch July 9, 2026 08:20
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