fix(workflow): repair sync-large-assets — heredoc-in-YAML parse bug#851
Open
DeveloperAlly wants to merge 2 commits intomainfrom
Open
fix(workflow): repair sync-large-assets — heredoc-in-YAML parse bug#851DeveloperAlly wants to merge 2 commits intomainfrom
DeveloperAlly wants to merge 2 commits intomainfrom
Conversation
The workflow has never successfully run since it was added. Root cause: bash heredocs (<<EOF, <<MANIFEST) inside a YAML | block put content at column 0, which breaks GitHub Actions' YAML parser. Every run failed at parse time with 0s duration. Fixes: - Replace all heredocs with printf/echo (the actual parse fix) - Lower threshold from 20MB to 1MB (catches all 10 binary files, not just 1) - Add weekly cron schedule (Sunday midnight UTC) - Add paths filter so push trigger only fires on asset changes - Explicit branch list (docs-v2, docs-v2-dev) replaces wildcard + if condition - Move input resolution to a separate step to avoid || '' expression pattern Ref: #849 Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Contributor
📊 V2 Browser Sweep Results❌ Test report not found. The test may have failed before report generation.
|
Adds a boolean dry_run input to workflow_dispatch. When enabled, the workflow runs all selection and staging logic but skips the push — reports what would be synced via --stat output instead. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
sync-large-assets.ymlworkflow which has never successfully run since it was added<<EOF,<<MANIFEST) inside a YAML|block put content at column 0, which breaks GitHub Actions' YAML parser — every run failed at parse time with 0s durationpaths:filter so push trigger only fires on actual asset changesifcondition with explicit branch listContext
Ref: #849 — this PR fixes the automation that was supposed to be syncing large assets to the
docs-v2-assetsbranch. Once merged tomain, theschedule(cron) andworkflow_dispatch(manual) triggers will work. Push triggers ondocs-v2anddocs-v2-devalready use the workflow from their respective branches —docs-v2-devhas this fix, anddocs-v2will pick it up on next merge.Test plan
workflow_dispatchin Actions UI to confirm it parses and runsdocs-v2-assetsbranch receives synced files🤖 Generated with Claude Code