From 44634bb529966e987d9d3914a491dacb6afdaede Mon Sep 17 00:00:00 2001 From: Jatin K Malik Date: Wed, 28 Jan 2026 23:27:42 -0800 Subject: [PATCH] fix: escape commit message in Detect Changes step (#90) The commit message was being directly inserted into the bash comparison, which broke when the message contained special characters like parentheses. Fix by assigning to a variable first before the comparison. --- .github/workflows/unified-pipeline.yml | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/.github/workflows/unified-pipeline.yml b/.github/workflows/unified-pipeline.yml index 07cebb8..d472ffc 100644 --- a/.github/workflows/unified-pipeline.yml +++ b/.github/workflows/unified-pipeline.yml @@ -41,10 +41,11 @@ jobs: id: deploy_check run: | # Deploy if web files changed OR python files changed (for version updates) + COMMIT_MESSAGE="${{ github.event.head_commit.message }}" if [[ "${{ steps.filter.outputs.web }}" == "true" ]] || \ [[ "${{ steps.filter.outputs.python }}" == "true" ]] || \ [[ "${{ github.event_name }}" == "workflow_dispatch" ]] || \ - [[ "${{ github.event.head_commit.message }}" == *"[web]"* ]]; then + [[ "$COMMIT_MESSAGE" == *"[web]"* ]]; then echo "deploy=true" >> $GITHUB_OUTPUT else echo "deploy=false" >> $GITHUB_OUTPUT