Skip to content

fix: resolve #286#618

Open
Shaidyk wants to merge 1 commit into
arakoodev:tsfrom
Shaidyk:fix/issue-286
Open

fix: resolve #286#618
Shaidyk wants to merge 1 commit into
arakoodev:tsfrom
Shaidyk:fix/issue-286

Conversation

@Shaidyk

@Shaidyk Shaidyk commented Jun 15, 2026

Copy link
Copy Markdown

Resolves #286.

Закрыл оставшийся scope баунти #286 по замечанию ревьюера. Удалил legacy-класс OpenAI (arakoodev/src/ai/src/lib/openai/openai.ts) и его экспорт из ai/src/index.ts. Чтобы Router стал полноценной заменой (litellm-паритет), расширил его: метод Router.embedding() + embeddings в openai-адаптере (buildEmbeddingRequest/parseEmbeddingResponse, supportsEmbeddings); проброс function-calling через completion() (functions/function_call в CompletionRequest, functionCall в CompletionResult/NormalizedResponse); Router.zodSchemaResponse() (zod→function-def→parse, на уже существующих зависимостях zod/zod-to-json-schema). Мигрировал все 7 оставшихся примеров (9 файлов) на Router, добавил по router.jsonnet каждому. Дополнил router.test.ts тестами на embeddings, function-calling и zodSchemaResponse. Ни одного упоминания класса OpenAI или subpath @arakoodev/edgechains.js/openai в исходниках не осталось (проверено grep).

/claim #286

@github-actions

github-actions Bot commented Jun 15, 2026

Copy link
Copy Markdown

CLA Assistant Lite bot All contributors have signed the CLA ✍️ ✅

@Shaidyk

Shaidyk commented Jun 18, 2026

Copy link
Copy Markdown
Author

I have read the CLA Document and I hereby sign the CLA

@Shaidyk

Shaidyk commented Jun 18, 2026

Copy link
Copy Markdown
Author

I have read the Arakoo CLA Document and I hereby sign the CLA

@Shaidyk

Shaidyk commented Jun 18, 2026

Copy link
Copy Markdown
Author

recheck

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

BOUNTY: Convert the endpoints to a smart router like litellm does in python

1 participant