Skip to content

An unquoted colon broke every workflow run since the launch commit - #47

Merged
aj-dev-smith merged 1 commit into
mainfrom
pages-yaml-fix
Aug 15, 2026
Merged

aj-dev-smith merged 1 commit into
mainfrom
pages-yaml-fix

Conversation

@aj-dev-smith

Copy link
Copy Markdown
Owner

What this changes

One step name in demo-pages.yml. It was enable Pages if it is not already (build_type: workflow) — a colon-space inside a plain YAML scalar, which fails the parse of the whole file. GitHub records an instant zero-job failure run on every push while a broken workflow file is in the tree, which is why four demo to pages runs failed in zero seconds, including three on pushes that predate the push trigger. The parenthetical is gone; all three workflow files now yaml-parse locally.

This merge touches the workflow file, so its own push starts the first real deploy — the thing PR #46 was for.

Does it add a spatial prior?

No. One YAML scalar.

Numbers

python3 -c "yaml.safe_load(...)": demo-pages.yml BROKEN → OK; demo-aws.yml OK; ci.yml OK. The class of error is the repo's own "assert on every string edit" lesson, applied to CI config: a workflow edit should be yaml-parsed locally before pushing, and wasn't.

Checklist

  • Changes are in src/, not in canalisation.html — neither touched
  • Ran node build.js and committed the regenerated canalisation.html — n/a
  • node test/smoke.mjs passes — unaffected
  • Read docs/TUNING.md if I changed a constant — no constants
  • Added a note to docs/JOURNAL.md if I tried something that did not work — the failure and its tell (zero-second runs on trigger-less pushes) are documented in the commit and here

🤖 Generated with Claude Code

https://claude.ai/code/session_01A4dBsLoHroiPssTXX1L1E6


Generated by Claude Code

The configure-pages step was named 'enable Pages if it is not already
(build_type: workflow)' — a colon-space inside a plain YAML scalar,
which fails the whole file's parse. GitHub then records an instant
zero-job failure run on EVERY push while a broken workflow file is in
the tree, which is why four 'demo to pages' runs failed in zero seconds
including three on pushes that predate the push trigger. The name loses
its parenthetical; all three workflow files now yaml-parse locally,
which should have been checked before the launch commit and is exactly
the class of error 'assert on every string edit' exists for.

This merge touches the workflow file, so its own push starts the first
real deploy.

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01A4dBsLoHroiPssTXX1L1E6
@aj-dev-smith
aj-dev-smith merged commit 85a76c6 into main Aug 15, 2026
4 checks passed
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