Skip to content

fix(ci): unblock repair-smoke-test workflow YAML parse failure#139

Merged
accesswatch merged 3 commits into
mainfrom
fix/repair-smoke-test-yaml
May 26, 2026
Merged

fix(ci): unblock repair-smoke-test workflow YAML parse failure#139
accesswatch merged 3 commits into
mainfrom
fix/repair-smoke-test-yaml

Conversation

@accesswatch
Copy link
Copy Markdown
Collaborator

fix(ci): unblock repair-smoke-test workflow YAML parse failure

The Python heredoc inside the Write minimal install summary fixture
step had its body at column 0, which terminated the surrounding YAML
literal block scalar (run: |) and caused every run of this workflow
to fail at parse time with "workflow file issue" (74/74 historical
runs failed, 0 jobs ever scheduled).

Replace the heredoc with an equivalent python3 -c one-liner so all
shell content stays within the block scalar indentation.

Validated locally with PyYAML.

The Python heredoc inside the `Write minimal install summary fixture`
step had its body at column 0, which terminated the surrounding YAML
literal block scalar (`run: |`) and caused every run of this workflow
to fail at parse time with "workflow file issue" (74/74 historical
runs failed, 0 jobs ever scheduled).

Replace the heredoc with an equivalent `python3 -c` one-liner so all
shell content stays within the block scalar indentation.

Validated locally with PyYAML.
Copilot AI review requested due to automatic review settings May 26, 2026 02:05
@accesswatch accesswatch requested a review from taylorarndt as a code owner May 26, 2026 02:05
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR fixes a GitHub Actions workflow parse failure in repair-smoke-test.yml by removing a Python heredoc whose unindented body prematurely terminated the YAML block scalar, preventing the workflow from ever being scheduled.

Changes:

  • Replaced the inline Python heredoc with a python3 -c one-liner to keep all script content within YAML block-scalar indentation.
  • Preserved the same JSON “minimal install summary” fixture content used by the shell smoke test.

print(f"Written fixture to: {out}")
PYEOF
OUT="$FIXTURE_DIR/smoke-test-summary.json"
python3 -c "import json, os; d=os.environ['RUNNER_TEMP']; json.dump({'schemaVersion':'1.0','operation':'install','scope':'project','targetDir':d,'installed':True,'destinations':{'claude':{'agentsDir':os.path.join(d,'.claude','agents')},'copilot':{},'copilotCli':{},'codex':{},'gemini':{},'mcp':{'serverDir':os.path.join(d,'mcp-server')}},'notes':[],'issues':[],'issueCount':0,'lastRepairRun':None,'version':'4.6.0','elapsedSeconds':0}, open(os.path.join(d,'smoke-test-summary.json'),'w'), indent=2)"
@accesswatch accesswatch merged commit 23df14b into main May 26, 2026
8 checks passed
@accesswatch accesswatch deleted the fix/repair-smoke-test-yaml branch May 26, 2026 02:13
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.

2 participants