Repository navigation
Clarify reviewer defaults and qualify native Lab boundaries - #103
Merged
Merged
Conversation
Select two fresh independent reviewers, scoped Notion authoring and project context bootstrap. Record the dedicated official Nightly cutover, native remote review flow and its remaining limits. Keep on-demand worker sandboxing as qualification work, preferring in-VM control without an extra coordinator. Co-authored-by: Codex <noreply@openai.com> Co-authored-by: Claude Opus 5.5 <noreply@anthropic.com>
Label sibling VMs as qualification, keep only AgentOps pinned, preserve weaker shared-environment limits and record actual pending-request/worktree gaps. Restore bundle links and include the worker guest-root and untested-route limits. Co-authored-by: Codex <noreply@openai.com>
gustavonline
marked this pull request as ready for review
October 28, 2026 10:55
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. |
This was referenced Oct 28, 2026
Closed
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.
Factory's portable guidance did not yet express the selected two-reviewer default, project context bootstrap or the current Lab's actual isolation limits. This exports the corresponding Notion originals at the recorded revision and updates README, architecture and qualification guidance.
Validation
Exact reviewed candidate:
bfe9d987d28fa60b59f4f0f76290e1a53972caa5.Both new independent Claude Opus 5.5 High, Fast off source reviews ACCEPT, no blockers, after the previous pair's findings were repaired. Native T3 thread/run model selections were read back separately by the operator; the reviewers' environment/settings observations alone were not used as proof.
Both reviewers independently verified the 96-member packet manifest, complete Git bundle, exact diffs, source/export correspondence, unchanged inputs, all 11 supporting-file hashes and preserved portable links. Each ran the existing checker (9 skills, 0 errors), all 16 tests and staging/link validation on Z13. No scripts, tests or workflows changed.
The four #88 decision packets are synthetic source-behavior rehearsals, not four real PR demonstrations. The reviewers accepted them for the scoped method change. No screenshots were invented or counted as code execution evidence.
Native observations and limits
The Factory-only Notion create/read/update/readback/trash and excluded-page 404 checks ran on T3 0.1.2 before the 3796 cutover, not as a second test on Nightly. The later native author → two reviewers → repair → two new reviewers flow completed on official Nightly 3796. An approval needed operator detection/steering; a stale approval remained unanswered. Remote
isolation:worktreewas refused. A fresh read-only worker probe was denied access to the lead.Private operator installation records additionally support the official archive checksum, migration counts, native pin/auto-settle setting, worker recovery copies, expected recovered environment and completion wakes. Those operator observations were not reproduced by the source reviewers. Two idle crash cycles and scratch restoration do not qualify reboot, active-write recovery or full native restore/resume.
Guest sudo/Docker makes controls inside that guest advisory. Connector flag efficacy under account OAuth and worker WebSearch/WebFetch/RemoteTrigger/GitHub-git routes remain unqualified. A diagnostic gVisor component preflight is not a model/T3 recipe, VM-equivalence result or benchmark.
No unattended intake, automatic provisioning, new backend, host upgrade or other-factory adoption is enabled by this PR. #32, #34 and #80 remain open. Version 0.1.26 is a portable distribution; installation is recorded separately.
Canonical source: the Notion Factory Skills library snapshot
d8cca3c0a7a04b5f9f77962ae47109283bafa1234f5f61a1298185be4d91cd7f; all required supporting files were exported deliberately. YAML original 0.3 is unchanged, SHA25698f48a4287d9b4fd14c39780adc70c5932af2185fd3546b22a5fab97f8f7647f, code-block edit 28 October 08:09Z. Git remains a reviewed export, not a second editable original.Delivery follows the owner's explicit instruction to complete this scoped work and retains ordinary branch protection. Credits in the final merge must preserve the human author and actual Codex / Claude contributions.