Skip to content

1.2.0b3 - #17

Merged
dan-snelson merged 1 commit into
developmentfrom
BR-16
Aug 18, 2026
Merged

dan-snelson merged 1 commit into
developmentfrom
BR-16

Conversation

@dan-snelson

Copy link
Copy Markdown
Contributor

Addresses Bug Report #16
Copilot AI lite review requested due to automatic review settings August 18, 2026 16:13
@dan-snelson

Copy link
Copy Markdown
Contributor Author

┌──── ○○○ ────┐
│ Semgrep CLI │
└─────────────┘

Scanning 16 files (only git-tracked) with 1978 Code rules:
            
  CODE RULES
                                                                                                                        
  Language      Rules   Files          Origin      Rules                                                                
 ─────────────────────────────        ───────────────────                                                               
  <multilang>      78      16          Pro rules    1577                                                                
  yaml             12       6          Community     401                                                                
  json              5       1                                                                                           
                                                                                                                        
                    
  SUPPLY CHAIN RULES
                                                       
  💎 Run `semgrep ci` to find dependency
     vulnerabilities and advanced cross-file findings. 
                                                       
          
  PROGRESS
   
  ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━ 100% 0:00:00                                                                                                                        
                
                
┌──────────────┐
│ Scan Summary │
└──────────────┘
✅ Scan completed successfully.
 • Findings: 0 (0 blocking)
 • Rules run: 95
 • Targets scanned: 16
 • Parsed lines: ~99.6%
 • Scan was limited to files tracked by git
 • For a detailed list of skipped files and lines, run semgrep with the --verbose flag
Ran 95 rules on 16 files: 0 findings.

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

Updates SYM-Lite to 1.2.0b3 and fixes Installomator label availability validation so labels defined in multiline/continued top-level case $label in alias arms are detected (Bug Report #16). This keeps valid labels from being incorrectly filtered out during pre-flight in both interactive and silent modes.

Changes:

  • Extend getAvailableInstallomatorLabels() to accumulate continued alias-arm fragments until the terminating ) and then emit a normalized pipe-delimited alias arm.
  • Update version strings and release notes in SYM-Lite.zsh, README.md, and CHANGELOG.md.
  • Update the security scan workflow to add a Zsh regression-test step and tighten pass/fail gating to require both syntax + 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 Implements multiline alias-arm parsing for top-level Installomator label case arms; bumps script version/history.
README.md Documents the updated validation behavior and bumps displayed version.
CHANGELOG.md Adds 1.2.0b3 entry describing the multiline alias-arm validation fix.
.github/workflows/security-scan.yml Adds a Zsh regression test step and updates summary/fail logic to treat tests as part of validation.

💡 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


{
echo "### Zsh Syntax"
echo "### Zsh Validation"
@dan-snelson
dan-snelson merged commit fa282e4 into development Aug 18, 2026
7 of 9 checks passed
@dan-snelson
dan-snelson deleted the BR-16 branch August 20, 2026 09:47
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