Skip to content

fix: remove hashFiles() from job-level if conditions in security.yml#928

Open
Xaxxoo wants to merge 1 commit into
rinafcode:mainfrom
Xaxxoo:fix/security-yml-hashfiles
Open

fix: remove hashFiles() from job-level if conditions in security.yml#928
Xaxxoo wants to merge 1 commit into
rinafcode:mainfrom
Xaxxoo:fix/security-yml-hashfiles

Conversation

@Xaxxoo

@Xaxxoo Xaxxoo commented Jun 28, 2026

Copy link
Copy Markdown

Summary

  • hashFiles() is not supported in job-level if: expressions in GitHub Actions — it causes an "Invalid workflow file" parse error that blocks all PR workflows
  • Removed if: hashFiles('package.json', 'pnpm-lock.yaml') != '' from the dependency-scan job
  • Removed if: hashFiles('Dockerfile') != '' from the container-scan job

Root Cause

hashFiles() requires a checked-out workspace and is only valid within step-level contexts. Using it in a job-level if: triggers the parse error: Unrecognized function: 'hashFiles'.

Test plan

  • Confirm security workflow runs pass (no "Invalid workflow file" error) after merge
  • Verify dependency-scan and container-scan jobs execute as expected

@drips-wave

drips-wave Bot commented Jun 28, 2026

Copy link
Copy Markdown

Hey @Xaxxoo! 👋 It looks like this PR isn't linked to any issue.

If this PR is for one of the issues assigned to you as part of a Wave, please link it to ensure your contribution is tracked properly. You can do this by adding a keyword to the PR description (e.g., Closes #123), or by clicking a button below:

Issue Title
#877 Add role-based field visibility enforcement at the serialization layer Link to this issue
#875 Add API versioning enforcement middleware to reject requests to deprecated versions Link to this issue
#858 Add feature flag state change audit log for compliance traceability Link to this issue
#876 Add content reporting escalation workflow to ModerationModule Link to this issue

ℹ️ Learn more about linking PRs to issues

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.

1 participant