fix: pass explicit argv to CLI entrypoint - #801
Conversation
|
Important Review skippedAuto reviews are disabled on base/target branches other than the default branch. Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Plus Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
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 |
|
Exact-head review completed at 1ed5148. This is the clean two-file CLI boundary stacked directly on the current #765 HEAD a4e4f68; no #789 drift is included. Local proof: 9 tests passed in tests/test_cli_auth.py, Ruff, compileall, and git diff --check passed. Auto-merge is being armed; protected Checks and independent approval remain required. |
…effort-contract-rebased' into repair/pr801-latest-parent
|
Exact-head RCA: Strix run 32436050151 / job 96637367737 materialized only one changed Python file, so unchanged local imports in contextual_orchestrator/main.py were absent from the temporary scan tree. The resulting ImportError report is scanner-context false positive; the current package contains those modules. The trusted central fix is stacked as ContextualWisdomLab/.github#1192 (parent #1191), which enumerates contextual_orchestrator/*.py from the exact PR head while preserving changed-file attribution. Do not weaken the Strix gate or rerun unchanged. |
3bbe137
into
fix/auto-reasoning-effort-contract-rebased
|
Exact stack recheck:
Decision: |
Why
The LineageWeave bootstrap had to replace
sys.argvbefore importing the CLI entrypoint. That is a monkey patch and makes the authenticated server invocation implicit.Change
argvparameter tocontextual_orchestrator.__main__.mainThis PR is stacked on #765 because that is the current upstream base carrying the orchestrator auto-discovery and bounded multimodal body options consumed by LineageWeave.
Verification
uv run --with pytest pytest tests/test_cli_auth.py -qExact-head reconciliation (2026-08-21)
39072a654261c3570496849bb4da1e2c340e2fbc; current headeb9ec5f4e3f8ecbcf96cb132f58a212981ff0a6d.0.2..venv/bin/python -m pytest -q tests/test_cli_auth.py tests/test_discover_models_cli.py— 15 passed..venv/bin/python -m pytest tests -q— 1521 passed in 555.46s.git diff --checkandcodegraph statuspassed.