Skip to content

fix: remove GitHub Actions expression syntax from project_context documentation#45

Merged
bkrem merged 6 commits intomasterfrom
devin/1761581318-fix-always-function-error
Oct 27, 2025
Merged

fix: remove GitHub Actions expression syntax from project_context documentation#45
bkrem merged 6 commits intomasterfrom
devin/1761581318-fix-always-function-error

Conversation

@devin-ai-integration
Copy link
Contributor

fix: remove GitHub Actions expression syntax from project_context documentation

Summary

Fixes the workflow failure caused by GitHub Actions attempting to parse ${{ always() }} expressions within the project_context documentation text. The workflow was failing with error: (Line: 36, Col: 28): Unrecognized function: 'always()'. Located at position 1 within expression: always()

Changed two instances in the documentation from if: ${{ always() }} to if: always() to prevent GitHub Actions from treating the documentation text as executable expressions.

Review & Testing Checklist for Human

  • Critical: Test the workflow by opening a test PR to verify it no longer fails with the "Unrecognized function: 'always()'" error
  • Verify the documentation is still accurate and helpful (the if: always() syntax is correct for GitHub Actions conditionals, just without the expression wrapper)
  • Confirm no other instances of ${{ }} in the project_context could cause similar parsing issues

Test Plan

  1. Open a test PR against the master branch in this repository
  2. Verify the Claude Auto Review workflow runs successfully without the "Unrecognized function: 'always()'" error
  3. Optionally, comment @claude review on the test PR to trigger the manual review flow and verify that also works

Notes

  • This is a minimal fix that only changes documentation text, not any functional code
  • YAML syntax has been validated locally
  • Smart friend consultation confirmed this is the correct approach for escaping GitHub Actions expression syntax in multi-line string inputs
  • The documentation is still accurate - always() is indeed the function used in if: conditionals

Link to Devin run: https://app.devin.ai/sessions/c0853e6178a942dca52032753749f834
Requested by: Ben Kremer (ben@reown.com), @bkrem

…umentation

Co-Authored-By: Ben Kremer <ben@reown.com>
@devin-ai-integration
Copy link
Contributor Author

🤖 Devin AI Engineer

I'll be helping with this pull request! Here's what you should know:

✅ I will automatically:

  • Address comments on this PR. Add '(aside)' to your comment to have me ignore it.
  • Look at CI failures and help fix them

Note: I can only respond to comments from users who have write access to this repository.

⚙️ Control Options:

  • Disable automatic comment and CI monitoring

Copy link
Member

Choose a reason for hiding this comment

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

remove this section

@bkrem
Copy link
Member

bkrem commented Oct 27, 2025

undo the commit you added in 40c97a9

@bkrem bkrem merged commit d198081 into master Oct 27, 2025
@bkrem bkrem deleted the devin/1761581318-fix-always-function-error branch October 27, 2025 16:27
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