Skip to content

fix(ci): repair YAML syntax in sync-alpha-from-main workflow#1101

Open
Gkrumbach07 wants to merge 1 commit intomainfrom
fix/sync-alpha-yaml-syntax
Open

fix(ci): repair YAML syntax in sync-alpha-from-main workflow#1101
Gkrumbach07 wants to merge 1 commit intomainfrom
fix/sync-alpha-yaml-syntax

Conversation

@Gkrumbach07
Copy link
Copy Markdown
Contributor

Summary

  • Fixes broken sync-alpha-from-main.yml workflow (failed run)
  • Root cause: heredocs (cat <<'MSG') inside YAML run: | blocks produce content at column 0, which YAML parsers interpret as new mapping keys
  • Replaced heredocs with shell string assignments and built the PR body incrementally to keep all content properly indented

Test plan

  • YAML lint passes locally (yaml-lint)
  • Workflow runs successfully on next push to main or via workflow_dispatch

🤖 Generated with Claude Code

@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai bot commented Mar 30, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: ASSERTIVE

Plan: Pro

Run ID: 37b02bf8-fe20-402b-a6d1-f2d0780891fe

📥 Commits

Reviewing files that changed from the base of the PR and between 374ed47 and 80652eb.

📒 Files selected for processing (1)
  • .github/workflows/sync-alpha-from-main.yml

Walkthrough

The workflow's fallback merge path is refactored to replace heredoc-based string construction with direct variable assignment and dynamic string concatenation for building PR messages and bodies. No control-flow changes occur.

Changes

Cohort / File(s) Summary
GitHub Actions Workflow Refactoring
.github/workflows/sync-alpha-from-main.yml
Replaces heredoc syntax for MERGE_MSG and CONFLICT_MSG with single-quoted incremental string assembly. Refactors PR creation to build PR_BODY via printf/concatenation and passes --body "${PR_BODY}" instead of an inline heredoc argument.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~12 minutes

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Title check ✅ Passed The title directly describes the main change: fixing YAML syntax issues in the sync-alpha-from-main workflow, which is the primary objective of the PR.
Description check ✅ Passed The description clearly explains the problem (heredocs breaking YAML parsing), the solution (replacing with string assignments), and provides test verification details, all directly related to the changeset.
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 fix/sync-alpha-yaml-syntax

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

Heredocs (cat <<'MSG') inside YAML run: | blocks produce content at
column 0, which YAML parsers interpret as new mapping keys. Replace
with inline string building to keep content indented within the block.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@Gkrumbach07 Gkrumbach07 force-pushed the fix/sync-alpha-yaml-syntax branch from 374ed47 to 80652eb Compare March 30, 2026 15:57
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