Repository navigation
ci: give the Windows native rows 180 minutes - #369
Merged
Merged
Conversation
Windows arm64 took 113 minutes on a pull request and was cancelled at its 120-minute limit on main on 2026-11-13, after every step had passed. Windows x64 took 108 minutes in the same run. Both runs had no Rust cache. Co-Authored-By: Claude Fable 5.2 <noreply@anthropic.com>
Codex Review Summary: ✅ CompletedThis 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. |
github-actions
Bot
added
size:S
vouch:trusted
mubeda
added a commit
that referenced
this pull request
Nov 14, 2026
Bump every version source to 0.16.0 and turn the Unreleased CHANGELOG section (PRs #349, #359, #365 and #369) into the v0.16.0 entry. Raise the Claude Code additive-hook ceiling to 2.1.323 after the live PreToolUse guard proof passed on that build, and move the above-ceiling test example to 2.1.324. Give the CI Test lookup contract test a 30-second limit: it runs bash and jq eleven times and timed out against the 5-second default under load. Verified locally: cargo check for the workspace, vp check, vp run typecheck, the full vp run -r --concurrency-limit 1 test graph in all nine packages, release:smoke, the updater regression sets, cargo test -p bibcode-desktop, and the macOS arm64 DMG signature and Finder icon check. CI: the Seeded Packaged Desktop Upgrade workflow from v0.8.0 passed on all seven lanes for this branch (macOS x64 on a rerun after its hosted runner lost its connection). Reviewed by Codex with no findings. Co-Authored-By: Claude Fable 5.2 <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Both Windows rows of
native_desktopget a 180-minute limit instead of 120.Why
On
main(commit 706d63b, CI run 43523947929) "Native desktop - Windows arm64" ran every step successfully and was then cancelled at 120 minutes during the post steps. This happened in two attempts, both without a Rust cache. On the pull request run 43496452314 the same job took 113 minutes and Windows x64 took 108, also without a cache.Windows x64 has not been cancelled yet. It is included because 108 of 120 minutes leaves the same thin margin.
Changes
.github/workflows/ci.yml:jobTimeoutMinutes: 180for the two Windows rows.scripts/ci-platform-contract.test.ts: the contract test requires at least 180 for Windows rows.docs/operations/ci.md: the two lines that state the row limits.Validation
vp test run scripts: 941 passed, 23 skipped.vp check: passed.vp run typecheck: passed with the pinned Rust toolchain 1.98.1.Not covered
🤖 Generated with Claude Code