Expose apply_patch for fallback model metadata#62
Open
ctkm-aelf wants to merge 15 commits into
Open
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.
Closes #60
Changed files
codex-rs/models-manager/src/model_info.rsupdates unknown/fallback model metadata so unknown slugs such asgpt-oss:20badvertise the local freeformapply_patchtool, enable parallel tool calls, and include themulti_tool_use.parallelguidance in fallback base instructions.codex-rs/models-manager/src/model_info_tests.rsadds a fallback metadata regression forgpt-oss:20b.codex-rs/core/src/tools/spec_plan_tests.rsadds a spec-planning regression that verifies fallback metadata exposes and registers model-visibleapply_patch.codex-rs/core/tests/suite/prompt_caching.rsadds a request-body regression proving an unknown model request sendsgpt-oss:20b, exposesapply_patch, preservesmulti_tool_use.parallel, and serializesparallel_tool_calls: true.Test results
just fmtincodex-rs.bash -lc 'bash "$CONSENSUS_RND_RUST_GUARD" build'.just test -p codex-models-manager.just test -p codex-core fallback_model_info_exposes_apply_patch_to_model.just test -p codex-core fallback_model_request_exposes_apply_patch_and_parallel_tool_calls.just fix -p codex-models-manager.just fix -p codex-core.bash -lc 'bash "$CONSENSUS_RND_RUST_GUARD" test'failed twice incodex-apply-patch::all suite::scenarios::test_apply_patch_scenarioswithNo such file or directory (os error 2); 90/91 guard tests passed on the rerun.Deviations
HOST_REFACTOR_COMMENT_POLICYwas unset, normalized tonone; refactor self-doc comments were not added.HOST_WORK_LANGUAGEwas unset, so the PR artifact prose is English.HOST_PROTO_POLICYwas unset and no schema/protocol files changed, so no schema regeneration was run.codex-apply-patchscenario fixture lookup failure outside the authorized file scope; no scope expansion was performed.⟦AI:AUTO-LOOP⟧