Repository navigation
revert: revert the untested Renovate batch - #796
Conversation
The batch moved packageManager to pnpm@10.34.6 while CI still installs 10.34.5. GitHub merged it before those jobs finished.
GitHub merged the batch as soon as the pull request opened. Renovate will merge on a later run, after the checks are green. pnpm stays at the version pinned in the workflow.
Codex Review SummaryThis comment shows the latest Codex review activity on this pull request.
ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings. |
|
The revert restored a lockfile that resolved TypeScript 5.9.3 while package.json requires ~5.8.3.
|
@greptileai Please re-review the current head 26fc375. The lockfile mismatch from the 4/5 review is fixed, its thread is resolved, and npm ci now passes. |
Reverts the Renovate session's merged changes from #784, #792, #795, and #796 while preserving unrelated commits. Merged current main to resolve the conflict with #780. Its sandbox Dockerfile ignorePaths are retained in the restored renovate.json, so the unrelated sandbox toolchain exclusions survive the rollback. Removes the self-hosted runner and its repair scripts and restores the previous Renovate configuration and repair workflow. Hosted Renovate remains disabled with `enabled: false`, and the restored repair job is paused with an explicit false condition. Keeps the SDK lockfile aligned with its unchanged manifest instead of restoring the pre-session TypeScript mismatch. Regenerated it with `npm install --package-lock-only --ignore-scripts --offline --no-audit --no-fund`; it matches the valid lockfile on main exactly. Validation: - Independently replayed all four reverse patches and compared the full repository tree. Only the automation pause, retained valid SDK lockfile, and preserved sandbox exclusions differ from that inverse. The complete diff against current main contains only the eight rollback files. - SDK `npm ci --ignore-scripts --offline --no-audit --no-fund` and TypeScript build passed. The clean install and TypeScript build passed again after merging main with Node 24.21.0 and a 12 GiB heap. - Workflow syntax validated with actionlint, supplying the existing Depot runner labels and ignoring the deliberate constant-false pause warning. `git diff --check` and the commit hooks passed. All six original review threads have documented dispositions: the SDK lockfile was regenerated, and the repaired workflow plus hosted bot remain disabled. The privileged repair trust boundary must be reviewed before any future re-enablement. GitHub CI and automated reviews must run against the new head. This PR has not been merged.
Pull request #795 moved
packageManagertopnpm@10.34.6while the workflows still install10.34.5. GitHub merged it about twenty seconds after it opened, and the test jobs then failed inpnpm/action-setup.This reverts that commit. Renovate no longer asks GitHub to auto-merge when the pull request opens. It merges on a later run, after the checks are green.
pnpmupdates are disabled because this token cannot update the version pinned in the workflow.