Skip to content

1.2.0 - #18

Merged
dan-snelson merged 6 commits into
mainfrom
development
Aug 20, 2026
Merged

dan-snelson merged 6 commits into
mainfrom
development

Conversation

@dan-snelson

Copy link
Copy Markdown
Contributor

1.2.0 - 19-Aug-2026

* 1.2.0b1

Addresses FR #14; thanks for the suggestion, @jeffmw777)

* 1.2.0b2

Updated 'codex' Validation Path
Addresses Bug Report #16
Copilot AI lite review requested due to automatic review settings August 19, 2026 16:53

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

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 under case $label in.
  • Add selectionDialogDefaultChecked to 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.

Comment on lines +272 to +276
- name: Run Zsh Regression Tests
id: zsh_tests
if: steps.zsh_syntax.outputs.status == '0'
run: zsh tests/installomator-label-parser.zsh

Comment thread README.md Outdated
Post-Copilot review of PR #18
Copilot AI review requested due to automatic review settings August 19, 2026 17:13

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

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 .gitignore is set to ignore tests/). 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

Post-Copilot review of PR #18
Copilot AI review requested due to automatic review settings August 19, 2026 20:13

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

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}"

@dan-snelson
dan-snelson merged commit 23859cc into main Aug 20, 2026
18 of 20 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.

2 participants