Skip to content

fix(opencode): accept proven Python VCS floors - #2057

Merged
seonghobae merged 1 commit into
mainfrom
codex/fix-vcs-marker-materialization
Sep 9, 2026
Merged

fix(opencode): accept proven Python VCS floors#2057
seonghobae merged 1 commit into
mainfrom
codex/fix-vcs-marker-materialization

Conversation

@seonghobae

Copy link
Copy Markdown
Contributor

The OpenCode base-lock materializer rejected an exact ContextualWisdomLab Git dependency whenever uv export retained a Python lower-bound marker. This blocked contextual-orchestrator even after its mutable tag archive was replaced with the immutable fast-mlsirm commit.

Accept only the normalized python_full_version >= 'major.minor' form when the fixed Python 3.14 coverage image already satisfies it. False floors and all more expressive markers still fail closed, and a regression binds the proof to the pinned coverage image version.

Validation:

  • uv run --group dev pytest -q (2997 passed, 1 skipped, 21 subtests passed)
  • replayed contextual-orchestrator's exported lock through _partition_uv_export; fast-mlsirm was recorded at commit 09f762ded35786dd1078222a4577ff09d649816f
  • git diff --check

Signed-off-by: Seongho Bae <me@seonghobae.me>

Commit-Message-Assisted-by: Codex GPT-6 <codex@openai.com>
@coderabbitai

coderabbitai Bot commented Sep 9, 2026

Copy link
Copy Markdown

Warning

Review limit reached

Next included review available in 9 minutes.

Check out review usage here.

View limit details

Limit details: You’ve used the included review currently available.

You've used all free OSS reviews for now. Wait for the free limit to reset to keep reviewing this public repository.

Learn how review limits work.

Review configuration:

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Advanced

Run ID: ec99ee49-528a-49bc-b015-cfdd5a95b07d

📥 Commits

Reviewing files that changed from the base of the PR and between 7fd571d and 8421fea.

📒 Files selected for processing (3)
  • docs/doctoring/opencode-exact-vcs-dependency-evidence.md
  • scripts/ci/materialize_base_python_requirements.py
  • tests/test_uv_export_isolation_contract.py

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.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@seonghobae

Copy link
Copy Markdown
Contributor Author

@coderabbitai review

@seonghobae

Copy link
Copy Markdown
Contributor Author

@codex review

@seonghobae
seonghobae enabled auto-merge (squash) September 9, 2026 07:10
@chatgpt-codex-connector

Copy link
Copy Markdown

You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard.
To continue using code reviews, add credits to your account and enable them for code reviews in your settings.

@coderabbitai

coderabbitai Bot commented Sep 9, 2026

Copy link
Copy Markdown
⚠️ Action not completed

Review rate limited.

Note: CodeRabbit is an incremental review system and does not re-review already reviewed commits. This command is applicable only when automatic reviews are paused.

@seonghobae

Copy link
Copy Markdown
Contributor Author

@coderabbitai approve

@coderabbitai

coderabbitai Bot commented Sep 9, 2026

Copy link
Copy Markdown
✅ Action performed

Comments resolved. Approval is disabled; enable reviews.request_changes_workflow to allow explicit top-level @coderabbitai resolve or @coderabbitai approve commands.

@opencode-agent
opencode-agent Bot disabled auto-merge September 9, 2026 07:24
@seonghobae
seonghobae enabled auto-merge (squash) September 9, 2026 07:26
@opencode-agent
opencode-agent Bot disabled auto-merge September 9, 2026 07:33
@seonghobae
seonghobae merged commit 88d49f7 into main Sep 9, 2026
47 of 55 checks passed
@seonghobae
seonghobae deleted the codex/fix-vcs-marker-materialization branch September 9, 2026 23:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant