Skip to content

ci: repair scheduled drift guards (apidocs pnpm, coreutils push)#2159

Merged
chaliy merged 1 commit into
mainfrom
claude/friendly-einstein-9477v4
Jul 13, 2026
Merged

ci: repair scheduled drift guards (apidocs pnpm, coreutils push)#2159
chaliy merged 1 commit into
mainfrom
claude/friendly-einstein-9477v4

Conversation

@chaliy

@chaliy chaliy commented Jul 13, 2026

Copy link
Copy Markdown
Contributor

What changed

Fixes two weekly scheduled drift-guard workflows that have been failing on main, so both go green again and resume doing their job (detecting API/upstream drift).

  • API Reference Drift — the typescript job now pins pnpm version: 10.33.0 in its Setup pnpm step, matching js.yml / publish-js.yml.
  • Coreutils Args Drift — the open-pr job now pushes the automation branch with plain --force instead of --force-with-lease.

Why

Both guards were red on 2026-07-13 (Monday schedule):

  • apidocs-drift typescript job died at Setup pnpm with Error: No pnpm version is specified. pnpm/action-setup@v6 no longer ships a default version; it reads packageManager from the root package.json, which this repo doesn't have (the field lives in crates/bashkit-js/package.json). Every other workflow already pins version: 10.33.0; this one was the sole exception, so it never reached the actual drift check.
  • coreutils-args-drift open-pr job failed at git push:
    ! [rejected]  chore/coreutils-args-drift -> chore/coreutils-args-drift (stale info)
    error: failed to push some refs
    
    --force-with-lease needs a remote-tracking ref to lease against. The fresh main checkout has none for the drift branch, so once the branch exists from a prior week's run the lease check aborts with stale info. The branch is owned exclusively by this automation and fully regenerated each run, so plain --force is correct here.

Before / After

  • Before: apidocs-driftError: No pnpm version is specified before any drift check; coreutils-args-driftfailed to push some refs ... (stale info) after successfully regenerating.
  • After: apidocs-drift typescript reaches the regenerate/diff step; coreutils-args-drift open-pr pushes and opens/updates the drift PR. Both runs conclude successfully.

Risk

  • Low
  • CI-only YAML changes. Worst case a re-dispatch surfaces genuine drift (the intended behavior), not a workflow error.

Checklist

  • Tests added or updated — N/A (CI workflow config only)
  • Backward compatibility considered — N/A (internal automation)

Both weekly drift workflows failed on main:

- apidocs-drift typescript job: pnpm/action-setup@v6 no longer defaults a
  version and there is no root package.json for it to read packageManager
  from, so setup errored before any drift check. Pin version: 10.33.0 to
  match js.yml / publish-js.yml.

- coreutils-args-drift open-pr job: git push --force-with-lease aborted
  with "stale info" because the drift branch already existed remotely but
  the fresh main checkout has no remote-tracking ref to lease against.
  Use plain --force on this automation-owned, fully-regenerated branch.
@cloudflare-workers-and-pages

Copy link
Copy Markdown

Deploying with  Cloudflare Workers  Cloudflare Workers

The latest updates on your project. Learn more about integrating Git with Workers.

Status Name Latest Commit Preview URL Updated (UTC)
✅ Deployment successful!
View logs
bashkit 180fd50 Commit Preview URL

Branch Preview URL
Jul 13 2026, 09:16 AM

@chaliy chaliy merged commit 2f2526c into main Jul 13, 2026
19 checks passed
@chaliy chaliy deleted the claude/friendly-einstein-9477v4 branch July 13, 2026 09:28
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.

1 participant