Repository navigation
Specialize thirdshift on Linux and drop native macOS support - #1673
Merged
Merged
Conversation
Document Linux-only support and Lima on macOS
Drop native macOS release and CI jobs
Enforce Linux-only compilation and remove platform fallbacks
JacobStephens2
marked this pull request as ready for review
October 30, 2026 22:15
Preserve main's release recovery tests as unconditional Linux tests and regenerate the manual from the merged guide.
Merged
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
Closes #1665.
The three Tickets fit together: #1666 removes native macOS release/CI configuration while preserving #1663's ARM Linux builds and smoke checks; #1667 rejects non-Linux compilation and makes process identity, process watching, Setup probes, Factory-skills handles, child executable inheritance and Linux scope paths unconditional; #1668 updates the README, manual, ADR 0003 and site to match. Windows users use WSL2, and VM repositories stay on the Linux filesystem.
The comparison with
mainalso includes earlier merged work: #1663's ARM prerequisite, #1619's resumable plan-limit Paused Commands, and release 0.18.8. Both review axes covered those changes. This Spec review removed two obsolete macOS path comments, preserving canonicalization and its symlinked-home regression.Evidence
/private/varand FSEvents despite the Linux-only Spec.After: both comments describe canonical paths and symlinked homes; the reviewer's exact static check passes. No runtime behavior changed in the review correction.
cargo clippy --all-targets -- -D warnings,cargo fmt --check, release-generator freshness, and all 34 Python tooling tests pass.RUST_TEST_THREADS=3 scripts/affected-tests mainpasses: all 61 site tests, all 126 Console browser tests, and the full Rust suite. Rust wall time was 3,072.87 seconds, including a 0.40-second cached compilation check. Local tooling used Python 3.11 and the installed Playwright browser cache.cargo testrun stopped at the unchangedtests/experiments/rechecks.rs:261test with a lock-contention refusal after the test dropped its lock handle. The exact rerun,cargo test --test experiments rechecks::invalid_concurrent_and_empty_batches_leave_the_experiment_unchanged -- --exact --nocapture, passed (7.19s); the same case also passed within the full three-thread rerun. No source change was made for that validation anomaly.b534fbc5, hosted CI passed, including its full Linux test job. The Release run built both static Linux binaries and passed the ARM Linux, Rocky Linux and WSL2 smoke checks. Full ARM CI was skipped under the documented PR opt-in policy.Review coverage
site/docs/index.htmland removed fallback contents. Generated-output exclusions: none. Unread changed files: none under either axis.Unaddressed findings
Standards
None.
Spec
None; the obsolete macOS comment finding was fixed.
Security
None. A single-session guidance-mode review of
git diff 86c8887f557b225573a32552bb3fb0927bc04edf...a665015d762c2bb2d0e58c16bd8a3b251982253e, including supporting code, found no introduced Security findings. The review covered Linux process identity and signaling, executable inheritance, sandbox probing and scope behavior, Factory-skills directory handles, and CI/release trust boundaries. No source fixes or proof-of-concept tests were needed.Security review outcome
No unaddressed introduced findings.
Merge Danger
Door: two-way
Code and workflow changes can be reverted. This release deliberately ends native macOS installation and source-build support; existing macOS users must move to a Linux VM. Native Windows remains unsupported; WSL2 runs the Linux binary.
Blast Radius: Platforms
Linux process supervision, sandbox probing, release publication, CI and installation documentation. Hosted ARM, Rocky Linux and WSL2 checks remain CI responsibilities.
Built with codex codex-cli 0.166.0 · gpt-6.1-sol · xhigh
Tickets