Remove z.ai provider (extracted to req_llm_zai)#438
Open
Nickcom4 wants to merge 2 commits intoagentjido:mainfrom
Open
Remove z.ai provider (extracted to req_llm_zai)#438Nickcom4 wants to merge 2 commits intoagentjido:mainfrom
Nickcom4 wants to merge 2 commits intoagentjido:mainfrom
Conversation
Z.AI provider has been extracted to a standalone package: https://github.com/agentjido/req_llm_zai Removed files: - lib/req_llm/providers/zai.ex - lib/req_llm/providers/zai_coder.ex - lib/req_llm/providers/zai_coding_plan.ex - test/providers/zai_test.exs - guides/zai.md - guides/zai_coder.md Updated mix.exs docs config to remove z.ai guide references.
Contributor
|
Blocking note for release: we can’t ship this with a Git dependency, since ReqLLM itself is published on Hex and downstream users should install via Hex only. We need req_llm_zai published to Hex before merging this extraction. Once it’s published, I can add a ZAI key and run a quick live validation pass. |
Contributor
Author
|
Published to Hex: https://hex.pm/packages/req_llm_zai/0.1.0 Docs at: https://hexdocs.pm/req_llm_zai/0.1.0 Ready for your live validation pass whenever you get a chance. |
Contributor
|
Cool - happy to accept the transfer to the org - let's |
Contributor
|
@Nickcom4 if you want to initiate the github transfer, should be ready for that - and then lets discuss how to transition hex keys so I can deploy - thx |
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.
Summary
req_llm_zaizai,zai_coder, andzai_coding_planprovider modules from coreFiles Removed
lib/req_llm/providers/zai.ex(531 lines)lib/req_llm/providers/zai_coder.ex(532 lines)lib/req_llm/providers/zai_coding_plan.ex(101 lines)test/providers/zai_test.exs(58 lines)test/coverage/zai/comprehensive_test.exstest/coverage/zai_coder/comprehensive_test.exsguides/zai.md(43 lines)guides/zai_coder.md(47 lines)Remaining z.ai references (intentionally kept)
lib/req_llm/provider/generated/valid_providers.ex- auto-generated from LLMDB model catalog, z.ai models still exist theregoogle_vertexandveniceproviders/tests - these reference z.ai-org models hosted on those platforms, not the z.ai provider itselfMigration Path
Users add
{:req_llm_zai, "~> 0.1.0"}to their deps. The new package auto-registers all three providers on application start viaReqLLM.Providers.register/1. No code changes needed beyond adding the dependency.Test plan
req_llm_zaicompiles cleanly withreq_llm ~> 1.6as dependency