Skip to content

Fix Xanitizer workflow errors and add DevSkim analysis#1

Merged
CGFixIT merged 1 commit into
mainfrom
claude/quirky-wozniak-1zohvp
Jun 16, 2026
Merged

Fix Xanitizer workflow errors and add DevSkim analysis#1
CGFixIT merged 1 commit into
mainfrom
claude/quirky-wozniak-1zohvp

Conversation

@CGFixIT

@CGFixIT CGFixIT commented Jun 16, 2026

Copy link
Copy Markdown
Owner

Summary

  • Fix xanitizer.yml: Replace invalid exists() expression (not a GitHub Actions function) with hashFiles() in both conditional build steps; fix sarif_file glob to use '.' (directory) since upload-sarif does not expand shell glob patterns
  • Add devskim.yml: Microsoft DevSkim static analysis workflow tailored for this documentation-only repository — scans markdown/YAML/text for security anti-patterns and hardcoded secrets, outputs SARIF to GitHub Code Scanning

Root causes fixed in xanitizer.yml

Issue Before After
Invalid expression function if: exists('pom.xml') if: hashFiles('pom.xml') != ''
Unsupported glob in upload-sarif sarif_file: '*-Findings-List.sarif' sarif_file: '.'

Test plan

  • Confirm xanitizer.yml workflow run completes without expression evaluation errors
  • Confirm devskim.yml runs and uploads findings to the Security → Code Scanning tab
  • Verify no false-positive findings block the PR

https://claude.ai/code/session_01MtaSEMdzQA4Ttpa7nvtzaX


Generated by Claude Code

- Replace invalid exists() with hashFiles() in conditional build steps;
  GitHub Actions does not support an exists() expression function
- Fix upload-sarif sarif_file to use '.' (directory) instead of a glob
  pattern, which upload-sarif does not expand
- Add devskim.yml: Microsoft DevSkim static analysis suited for this
  documentation-only repository, outputs SARIF to Code Scanning

https://claude.ai/code/session_01MtaSEMdzQA4Ttpa7nvtzaX
@CGFixIT CGFixIT marked this pull request as ready for review June 16, 2026 19:22
@CGFixIT CGFixIT merged commit 33647ed into main Jun 16, 2026
0 of 2 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