fix: correct Docker image tag format in deploy workflow

[bejranonda]

• Replace invalid {{branch}} template variable with static 'sha-' prefix
• Fixes 'invalid reference format' error in build-and-push job
• Docker tags will now be generated as 'sha-' instead of '-'
• Resolves GitHub Actions run #19506604388 failure

Summary by CodeRabbit

• Chores

  • Updated container image tagging to use commit SHA as the prefix instead of branch name.

• Style

  • Refined workflow documentation formatting.

[coderabbitai]

Walkthrough

Two GitHub Actions workflow files modified: formatting adjustments to auto-pr.yml with no functional impact, and docker/metadata-action tag prefix configuration changes in deploy.yml from branch-based to SHA-based prefixes in two locations.

Changes

Cohort / File(s)	Summary
Workflow formatting .github/workflows/auto-pr.yml	Cosmetic reformatting of PR auto-comment block including indentation and bullet/list structure adjustments to "This pull request…", "Configuration", and "Next Steps" sections. No functional changes.
Docker metadata configuration .github/workflows/deploy.yml	Modified docker/metadata-action tag prefix in two locations from prefix={{branch}}- to prefix=sha-, changing the generated image tag naming strategy.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~8 minutes

• auto-pr.yml: Pure cosmetic changes—verify intended formatting without logic concerns
• deploy.yml: Confirm SHA-based prefix change is intentional and won't break image tagging workflows downstream; verify both occurrences are consistent

Poem

🐰 Indents align like whisker rows,
SHA prefixes bloom where branches flowed,
Workflows dance with tidier prose,
Docker tags on paths freshly sowed! 🏷️

Pre-merge checks and finishing touches

✅ Passed checks (3 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	The title accurately describes the main change: correcting Docker image tag format in the deploy workflow by replacing an invalid template variable with a valid prefix.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✨ Finishing touches

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Post copyable unit tests in a comment
• Commit unit tests in branch claude/fix-ci-pipeline-errors-01Y3i4ec9xoycG77FFJ7devx

---

📜 Recent review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 3f28768 and f0e8c7f.

📒 Files selected for processing (2)

• .github/workflows/auto-pr.yml (1 hunks)
• .github/workflows/deploy.yml (2 hunks)

⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (1)

• GitHub Check: backend-tests

🔇 Additional comments (2)

.github/workflows/deploy.yml (1)

43-43: Docker tag prefix fix looks solid.

The replacement of the invalid prefix={{branch}}- template variable with prefix=sha- correctly resolves the "invalid reference format" error. Docker tags cannot start with a hyphen, and this change ensures tags follow the naming convention (e.g., sha-<commit>). Both backend and frontend metadata actions are updated consistently.

Also applies to: 64-64

.github/workflows/auto-pr.yml (1)

149-163: Comment formatting improvements are clean.

The restructuring of the PR comment block improves readability with better indentation and list formatting. The template literal syntax and actual comment content remain unchanged, so no functional impact on the workflow or PR creation behavior.

---

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