Repository navigation
Conversation
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.
Current integration: isolated F2 observer and Linux measurement
Head
e700770adds separate-process F2 product inspection with explicit stage framing, bounded stdout/stderr collection, validated stage shapes, independent holder-instance samples and a recorded trusted research build.F2_DISTremains refused;F2_BUILDnow enables isolated inspection. F4 remains an unapplied proposal artifact, not a production change. This is a follow-up to the historical measurements below, not a claim that their checks certify this head.b483cd587c62329243366e64aaaac477edd36adc: actual exit 1, collection true, expectations false, no incomplete waits. Direct prepare/guarded participation passed at dumpable=1, but incomplete-visibility lsof warnings and the Python fixture's unsupported executable/arguments prevented a passing control. Dumpable=0 prepare returned EACCES and inspection refused with the participation blocker; without a passing control, refusalCounted remains false. No warning suppression or identity-rule relaxation was used.stall:prepare: actual exit 1, both inspectors incomplete at the stage bound, open stage prepare; both later exited 4 without signals. Holders exited 0. This measures an injected finite stall, not a real procfs hang or cancellation guarantee.git apply --checkagainst the pinned unpatched source.claude-opus-5-5) and Codex CLI 0.163.0 (default selection, actual model not independently reported). Source-checked fixes address stderr EOF, malformed result evaluation, decimal instance ticks, manifest scope and evidence wording. These pre-correction reviews do not certify this head; repository CI/current-head automatic reviews are separate gates.Keep Draft. #126/#127 and both #137 readiness gates remain open; no merge/release/publication is requested.
Historical integration record (through 56dc50d)
Findings and research scope
Holding native coordination during slow inspection delays other writers that need the same coordinator. The synthetic macOS measurements show that injected guard delays increase a cooperating waiter's delay. A separate
lsof +fgexperiment excludes never-locked and failed-acquirer descriptors, but cannot distinguish a former holder that released its lock while retaining its descriptor. Under injected omission of the actual holder, that former holder remains a wrong eligible target. The sticky bit is therefore refusal-side evidence only, not positive ownership evidence.This Draft records those results and adds unpublished, hardened research fixtures under
experimental/126-guard-liveness/. It does not change production TypeScript, eligibility, platform support, dependencies, JSON, workflows or releases. Refs #126 and #127; neither issue is closed by this PR.Implementation and open gates
F2_DISTis refused: a synchronous syscall would prevent its timer and cleanup from running. Separate-process inspection, native candidate controls, and actual native-access evidence remain required./prochang was reproduced. No actual Codex process or Windows recovery was tested. Default macOS recovery policy and Draft fix: prototype Linux descriptor-level flock attribution #137 are unchanged by this PR.Validation
cfa2eba9e3809afa060d85c15ff487f811c925e4; source archive SHA-256cac2171ba9424d229a204b32a140321795fad000b6bef9fbbc709192a154ec4a, raw result SHA-256656c1d08d816303dfac870a87e426f4ace0f6972698a46f491a81a4c3b1900fc. Runner exit 0, failures/incomplete 0. Stage/result directories were 0700 and result 0600. Owned remote scratch was removed after confirming holder exits and preserving evidence. No dependency installation, privilege or host-policy change was needed. This does not run fix: prototype Linux descriptor-level flock attribution #137 or measure deployed native Codex dumpability.e782e396df3f5feebb556bfe2f371d101176e03f, with input SHA-25670db601c2d878f86621c2f153758b89ea1fbe2973ee60f7cee6deda1ff0fa6da. Claude Code 2.1.296 requested and returnedclaude-opus-5-5; Codex CLI 0.162.1 used an isolated default selection, actual model unconfirmed. Both exited zero with complete responses and did not execute tests. Subsequent corrections were checked by the integrating Codex; those earlier CLI responses do not certify the final commit.All 39 checks passed at
e5cf6dd7af5f61542a842527a216fa5eec918e61, including repository CI and all five CodeQL language analyses. A cooperating read-only review of that integration requested actual final-script validation and correction of an incomplete-count claim. Both are addressed: one F1/M2 nominal run each and the five documented fault cases returned the expected exit statuses and report fields. The ignored-EOF case records incomplete 1, then child exit 3. The raw result hashes and revision are recorded in evidence.md.Previous head
cfa2eba9e3809afa060d85c15ff487f811c925e4passed all 39 checks, including CI and CodeQL. It narrows theincomplete: 0statement to nominal runs, documents the 11-test command and adds missing original result hashes. The cooperating review's two publication-blocking candidates are addressed by actual script execution and the corrected scope. Barrier contention can invalidate a scenario; successful finite runs are not a guarantee for every schedule. No automatic review comments were visible when those results were checked.Historical head
56dc50d5f610d3356228aacee743cbd22a6706ebrecords the approved Linux measurement and corrects the fdinfo ownership table. Only documentation and stale preparation labels in script comments/docstrings changed. Diff check, type check, lint, F2 syntax and package smoke passed. All 39 checks also passed for this commit, including CI and CodeQL. No merge or publication was performed.Keep this PR Draft until the final review and research scope are assessed. Production adoption, merging and package publication are separate decisions.