IEP-1728: Fix shell expansion issue

[alirana01]

Fix the shell expansion problem in workflow

Summary by CodeRabbit

• Chores
  • Enhanced GitHub Actions workflow for improved Jira integration. Refined pull request title parsing with explicit permissions configuration and better error handling when issue keys are not detected in PR titles.

[coderabbitai]

Caution

Review failed

Pull request was closed or merged during review

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: c991ceac-f244-45d5-b511-a96710cfc865

📥 Commits

Reviewing files that changed from the base of the PR and between 0eb9e44 and 1c43bf2.

📒 Files selected for processing (1)

• .github/workflows/jira-pr-comment.yml

---

📝 Walkthrough

Walkthrough

The GitHub Actions workflow for Jira PR commenting is updated to introduce an explicit PR_TITLE environment variable and refactor the Jira issue key extraction logic to use this variable instead of direct title references. Error handling is added for cases where no issue key is found.

Changes

Cohort / File(s)	Summary
Jira PR Comment Workflow .github/workflows/jira-pr-comment.yml	Adds explicit permissions configuration, introduces PR_TITLE environment variable, refactors issue key extraction to use the new variable, and adds warning output when no Jira issue key is detected in the PR title.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Possibly related PRs

• IEP-1707: workflow to sync the pr link to jira issue as a comment #1399: Directly implements the same workflow modifications to .github/workflows/jira-pr-comment.yml, introducing PR_TITLE variable handling and updating Jira issue key extraction logic with no-key handling.

Poem

🐰 A workflow hops with precision bright,
PR_TITLE now captures each title's might,
When Jira keys hide, we warn with care,
GitHub Actions dance through the air! ✨

🚥 Pre-merge checks | ✅ 2 | ❌ 1

❌ Failed checks (1 warning)

Check name	Status	Explanation	Resolution
Title check	⚠️ Warning	The title mentions 'shell expansion issue' but the actual changes involve GitHub Actions workflow permissions and environment variables for Jira PR comments—unrelated to shell expansion.	Update the title to accurately reflect the main change, such as 'Add GitHub Actions permissions and Jira issue key environment variable to workflow' or similar.

✅ Passed checks (2 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing Touches

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Commit unit tests in branch IEP-1728

📝 Coding Plan

• Generate coding plan for human review comments

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

Tip

You can validate your CodeRabbit configuration file in your editor.

If your editor has YAML language server, you can enable auto-completion and validation by adding # yaml-language-server: $schema=https://coderabbit.ai/integrations/schema.v2.json at the top of your CodeRabbit configuration file.