1.2.0 - #18
1.2.0#18
Conversation
There was a problem hiding this comment.
Pull request overview
Release bump to 1.2.0 for SYM-Lite, adding a more robust Installomator label parser (to handle multiline alias arms), introducing a configurable default-checked behavior for interactive selection items, and aligning docs/CI metadata with the release.
Changes:
- Update
getAvailableInstallomatorLabels()parsing to support continued (multiline) top-level alias arms undercase $label in. - Add
selectionDialogDefaultCheckedto pre-check selectable items in interactive mode while keeping already-installed items disabled/unchecked. - Update release documentation/versioning (README + CHANGELOG) and extend the security workflow to include Zsh regression tests.
Reviewed changes
Copilot reviewed 4 out of 4 changed files in this pull request and generated 2 comments.
| File | Description |
|---|---|
| SYM-Lite.zsh | Bumps script version to 1.2.0; adds interactive default-checked toggle; fixes Installomator multiline alias parsing. |
| README.md | Updates version and documents multiline alias validation + selectionDialogDefaultChecked. |
| CHANGELOG.md | Adds 1.2.0 release notes matching PR scope. |
| .github/workflows/security-scan.yml | Adds a Zsh regression test step and tightens the pass/fail gate to include tests. |
💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
| - name: Run Zsh Regression Tests | ||
| id: zsh_tests | ||
| if: steps.zsh_syntax.outputs.status == '0' | ||
| run: zsh tests/installomator-label-parser.zsh | ||
|
|
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 4 out of 5 changed files in this pull request and generated no new comments.
Suppressed comments (1)
.github/workflows/security-scan.yml:276
- This workflow now runs
zsh tests/installomator-label-parser.zsh, but the PR does not include that script (and.gitignoreis set to ignoretests/). As-is, CI will fail with “no such file” rather than validating the new parser behavior. Commit the regression test script (and ensure it’s not ignored) or remove/guard this step.
- name: Run Zsh Regression Tests
id: zsh_tests
if: steps.zsh_syntax.outputs.status == '0'
run: zsh tests/installomator-label-parser.zsh
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 5 out of 5 changed files in this pull request and generated no new comments.
Suppressed comments (1)
.github/workflows/security-scan.yml:290
- In the no-target early-exit branch, the step summary heading still says "Zsh Syntax" even though the job now reports overall "Zsh Validation" (syntax + regression tests). This makes the summary inconsistent depending on whether any files are found.
if [[ "${target_count}" == "0" ]]; then
{
echo "### Zsh Syntax"
echo
echo "✅ No tracked zsh files matched the scan patterns."
} >> "${GITHUB_STEP_SUMMARY}"
1.2.0 - 19-Aug-2026
selectionDialogDefaultCheckedto configure default selection for interactive-mode items (while keeping already-installed items disabled and unchecked; thanks for FR [Feature]: Option to set toggles to all on by default, or to control each one granularly. #14, @jeffmw777!)codexValidation Path