Skip to content

Ensure LM Studio requests expose apply_patch tool#110

Merged
hackall360 merged 1 commit intoLMStudiofrom
codex/fix-lmstudio-file-handling-issues
Sep 28, 2025
Merged

Ensure LM Studio requests expose apply_patch tool#110
hackall360 merged 1 commit intoLMStudiofrom
codex/fix-lmstudio-file-handling-issues

Conversation

@hackall360
Copy link
Owner

Summary

  • map LM Studio model slugs to a ModelFamily that prefers the function variant of the apply_patch tool and cover it with unit tests
  • confirm LM Studio tool assembly emits the apply_patch entry for both Responses and Chat Completions payloads
  • extend the LM Studio exec integration test to assert the captured request advertises the apply_patch function tool

Testing

  • just fmt
  • just fix -p codex-core
  • just fix -p codex-exec
  • cargo test -p codex-core lmstudio_models_use_function_apply_patch_tool -- --nocapture
  • cargo test -p codex-core chat_completions_tools_include_apply_patch_for_lmstudio -- --nocapture
  • cargo test -p codex-exec suite::lmstudio::exec_enables_apply_patch_tool_for_lmstudio -- --nocapture

https://chatgpt.com/codex/tasks/task_b_68d80b5c8674832f9f0d86aad1b4e918

@hackall360 hackall360 merged commit 4767865 into LMStudio Sep 28, 2025
3 of 6 checks passed
@hackall360 hackall360 deleted the codex/fix-lmstudio-file-handling-issues branch September 28, 2025 06:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant