fix(openai-adapters): convert request timeout to milliseconds#12453
fix(openai-adapters): convert request timeout to milliseconds#12453kgarg2468 wants to merge 1 commit into
Conversation
|
I have read the CLA Document and I hereby sign the CLA You can retrigger this bot by commenting recheck in this Pull Request. Posted by the CLA Assistant Lite bot. |
|
bot? |
|
Not a bot account, but I did use an AI coding assistant to prepare this small timeout-units fix. I also checked the cancelled Windows VSIX run: the VSIX package and artifact upload both completed, and the cancellation happened later during post-job cache cleanup, so I do not see a code change needed for that job. Happy to adjust anything else reviewers want. |
Summary
requestOptions.timeoutfrom Continue's documented seconds to the OpenAI SDK's milliseconds before constructing the client.0instead of treating it as unset.Fixes #12450.
Testing
npm test -- --run src/apis/OpenAI.test.ts src/test/openai-adapter.vitest.tsnpm run build -- --pretty falseSummary by cubic
Convert
requestOptions.timeoutinopenai-adaptersfrom seconds to milliseconds for theopenaiSDK to fix incorrect request timeouts. Preserves the SDK default when unset and respects an explicit0.openaiclient.0(no timeout) and add unit tests for conversion, omitted, and zero cases.Written for commit a7c5fef. Summary will update on new commits. Review in cubic