Skip to content

feat: add composite action analysis to security scan - #1

Merged
SamFleming-TylerTech merged 1 commit into
tyler-technologies-oss:mainfrom
SamFleming-TylerTech:feat/composite-action-analysis
Apr 9, 2026
Merged

feat: add composite action analysis to security scan#1
SamFleming-TylerTech merged 1 commit into
tyler-technologies-oss:mainfrom
SamFleming-TylerTech:feat/composite-action-analysis

Conversation

@SamFleming-TylerTech

Copy link
Copy Markdown
Contributor

Summary

  • Adds a new "Analyze composite action changes" step to the security-scan diff-summary job
  • Parses action.yml/action.yaml diffs to detect: uses: reference changes, new run: steps, using: field changes, and unpinned references
  • Appends a "Composite Action Analysis" section to the PR comment when findings exist

Test plan

  • Fork an action with a composite action.yml, modify it, and verify the scan detects changes
  • Verify the PR comment renders correctly with composite findings
  • Verify no composite section appears when action.yml is unchanged

Comment thread .github/workflows/security-scan.yml Fixed
Comment thread .github/workflows/security-scan.yml Dismissed
Comment thread .github/workflows/security-scan.yml Dismissed
Comment thread .github/workflows/security-scan.yml Dismissed
Comment thread .github/workflows/sync-tags.yml Dismissed
Comment thread .github/workflows/sync-upstream.yml Dismissed
Extract all inline shell/JS from reusable workflows into standalone
scripts under scripts/, add composite action change detection to the
security scan workflow, and move all GHA expressions from run blocks
to env blocks.

- Extract 18 scripts from 3 workflows (1333 -> 516 YAML lines)
- Add composite action analysis: SHA-changed refs, run block content
  diffing, unpinned reference detection, using-field changes
- Add security alerts for composite action changes in sync PRs
- Add tag drift, mutation, and deletion detection scripts
- Add divergence issue creation for failed ff-only merges
- Parameterize fetch-tags.sh for both upstream and fork usage

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@SamFleming-TylerTech
SamFleming-TylerTech merged commit 63f8cbb into tyler-technologies-oss:main Apr 9, 2026
3 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