Skip to content

fix(ci): repair parse-dead workflow — K9-SVC step at job-level indent - #20

Merged
hyperpolymath merged 1 commit into
mainfrom
fix/k9-svc-step-indent
Aug 3, 2026
Merged

hyperpolymath merged 1 commit into
mainfrom
fix/k9-svc-step-indent

Conversation

@hyperpolymath

Copy link
Copy Markdown
Owner

The fault

A sweep appended a K9-SVC Validation step at two-space indentation — the level of a job key under jobs: — instead of the six spaces that would place it inside a job's steps: list:

jobs:
  dispatch:
    steps:
      - name: Confirm
        run: echo ...

  - name: K9-SVC Validation     # <-- col 2. Should be col 6.
    run: |
      ...

YAML hits a sequence item where it expects a block mapping, and the whole file fails to parse.

Actions rejects a workflow that doesn't parse before allocating a runner, so this workflow has produced no check run and no log since the step was added — it has not been running at all.

Recognising this class

  • the run is listed by file path instead of workflow name
  • gh run view --log-failed returns "log not found"
  • gh pr checks shows nothing — a parse-rejected workflow creates no check run

Only gh run list --json conclusion reveals it.

The fix

Re-indent the step and its body by four spaces so it sits inside the job's steps: list. Nothing else changes — no action pins, no permissions, no logic.

Verified: the file parses, jobs is a non-empty mapping, and every job has steps or uses.

Estate context

Measured 2026-07-27 across all workflow files in hyper-repos/, meta-repos/ and repos/: this fault affects 49 files across 45 repositories, and 100% of them fail to parse — not a sample. The 12 files where the same step is correctly indented are what established the intended shape.

Mostly instant-sync.yml (forge propagation), plus boj-build.yml, casket-pages.yml, release.yml, cflite_pr.yml and one codeql.yml.

A separate group of 6 boj-build.yml files carries additional independent faults (a permissions: block injected at column 0 mid-file, and a mangled curl payload) and is being repaired individually rather than by this sweep.

Provenance

Built with git plumbing directly against origin/HEAD, so no local working tree was involved and no unrelated local changes are included — several of these repositories have sweep debris in their working copies.

🤖 Generated with Claude Code

A sweep appended a `K9-SVC Validation` step at two-space indentation —
the level of a job key under `jobs:` — instead of the six spaces that
would place it inside a job's `steps:` list. YAML then hits a sequence
item where it expects a block mapping, and the whole file fails to
parse.

Actions rejects a workflow that does not parse *before* allocating a
runner, so this workflow has produced no check run and no log since the
step was added. It has not been running at all.

The signature is worth recognising: the run is listed by FILE PATH
rather than workflow name, `gh run view --log-failed` returns "log not
found", and `gh pr checks` shows nothing, because a parse-rejected
workflow creates no check run. Only `gh run list --json conclusion`
reveals it.

Fix: re-indent the step and its body by four spaces so it sits inside
the job's `steps:` list. Nothing else is changed — no action pins, no
permissions, no logic.

Estate-wide measurement (2026-07-27): this fault affects 49 workflow
files across 45 repositories, and every single one of them fails to
parse — 100%, not a sample. Compare the 12 files where the same step is
correctly indented, which is how the intended shape was determined.
Affected workflows are mostly instant-sync.yml (forge propagation),
plus boj-build.yml, casket-pages.yml, release.yml, cflite_pr.yml and
one codeql.yml.

Built with git plumbing directly against origin/HEAD, so no local
working tree was involved and no unrelated local changes are included.

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
@gitar-bot

gitar-bot Bot commented Aug 3, 2026 •

Copy link
Copy Markdown

Note

Automatic reviews are paused because your trial's included automatic processing has been used for this period. Upgrade now, or comment "Gitar review" to run a review anytime.
Learn more

Code Review ✅ Approved

Corrects YAML indentation for the K9-SVC Validation step in the parse-dead workflow to resolve workflow parsing failures. No issues found.

Auto-approved and auto-merge armed: No blocking issues found.
Please see Auto-approve Docs for details on setting custom approval criteria. — merges when pipeline and required approvals pass.

Options

Display: compact → Showing less information.

Comment with these commands to change the behavior for this request:

Compact
gitar display:verbose         

Important

Your trial ends in 7 days — upgrade now to keep code review, CI analysis, auto-apply, custom automations, and more.

Was this helpful? React with 👍 / 👎 | Gitar

@gitar-bot

gitar-bot Bot commented Aug 3, 2026

Copy link
Copy Markdown

⚠️ Gitar auto-approved this PR but could not enable auto-merge: auto-merge is disabled for this repository — enable "Allow auto-merge" in the repository settings.

@gitar-bot gitar-bot Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Gitar has auto-approved this PR and enabled auto-merge (configure)

@gitar-bot gitar-bot Bot added the gitar-approved Added by Gitar label Aug 3, 2026
@hyperpolymath
hyperpolymath marked this pull request as ready for review August 3, 2026 13:03
@hyperpolymath
hyperpolymath merged commit 73b88a3 into main Aug 3, 2026
17 of 18 checks passed
@hyperpolymath
hyperpolymath deleted the fix/k9-svc-step-indent branch August 3, 2026 13:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

gitar-approved Added by Gitar

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant