Skip to content

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

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

fix(ci): repair parse-dead workflow — K9-SVC step at job-level indent#94
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 the indentation of the K9-SVC Validation step in the CI workflow to place it properly inside the job's steps list, resolving a YAML parse failure that prevented the workflow from running. 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

@sonarqubecloud

sonarqubecloud Bot commented Aug 3, 2026

Copy link
Copy Markdown

Quality Gate Failed Quality Gate failed

Failed conditions
C Security Rating on New Code (required ≥ A)

See analysis details on SonarQube Cloud

Catch issues before they fail your Quality Gate with our IDE extension SonarQube for IDE

@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 added the gitar-approved Added by Gitar label Aug 3, 2026

@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)

@hyperpolymath
hyperpolymath marked this pull request as ready for review August 3, 2026 13:03
@hyperpolymath
hyperpolymath merged commit 300f5bb into main Aug 3, 2026
24 of 30 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