Repository navigation
ci(noema): stage self-hosted reviewer quality supplier - #2656
Draft
seonghobae wants to merge 1 commit into
Draft
seonghobae wants to merge 1 commit into
seonghobae wants to merge 1 commit into
Conversation
Contributor
|
Important Draft PR not reviewedDraft PRs are not automatically reviewed by default.
To automatically review draft PRs, update your CodeRabbit configuration: reviews:
auto_review:
drafts: true
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
This branch has not been deployed
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.
Scope
Stage a fixed-caller, workflow_call-only self-hosted supplier for ContextualWisdomLab/noema reviewer-ci. Two additive source/test files; no consumer is activated.
Preserves the original exact PR-head checkout, Python 3.11, 100% line/branch and docstring gates, wheel/sdist/editable checks, authenticated/scanned image preparation and real no-network CodeGraph Docker smoke. Adds explicit Node setup, platform/tool preflight, a private virtual environment and per-job temporary/cache paths. No inherited secrets, privileged-pool or hosted fallback is introduced.
Verification
Required activation gates
Draft/source preparation only. The existing noema Hosted caller remains unchanged. A protected immutable supplier revision and actual self-hosted product execution must precede caller adoption.
This supplier admits same-repository PRs and main pushes only. The old caller also covers forks. Before replacement, explicitly design and verify an isolated fork-validation path and required-check behavior; an unconditional skipped-success fork job is not an acceptable migration.
Job-local directories and cleanup are hygiene, not a security boundary: PR code sharing a guest can mutate environment/state. Activation requires independently verified single-job disposable guests without host mounts/sockets, ambient operational credentials or production-network reachability, plus external timeout and whole-guest teardown. The guest Docker daemon must belong solely to that disposable guest. Labels and this preflight do not prove these properties.
No runner registration, credential issuance, ACL/network changes, required-gate weakening, merge or live rollout is included.