Repository navigation
refactor(integrations): organize framework packages - #6017
Merged
ZaynJarvis merged 8 commits intoOct 15, 2026
Merged
Conversation
This was referenced Oct 14, 2026
Open
Move framework packages, tests, and examples into integrations/frameworks. Remove the legacy LangChain forwarding namespace and document canonical imports. Add an SDK-based adapter guide and update CI and release paths. Signed-off-by: Hao Zhe <haozhe4547@gmail.com>
Signed-off-by: Hao Zhe <haozhe4547@gmail.com>
Signed-off-by: Hao Zhe <haozhe4547@gmail.com>
Signed-off-by: Hao Zhe <haozhe4547@gmail.com>
Signed-off-by: Hao Zhe <haozhe4547@gmail.com>
Signed-off-by: Hao Zhe <haozhe4547@gmail.com>
Signed-off-by: Hao Zhe <haozhe4547@gmail.com>
ehz0ah
force-pushed
the
refactor/framework-integrations
branch
from
October 14, 2026 11:56
7e481f0 to
59c3387
Compare
Signed-off-by: Hao Zhe <haozhe4547@gmail.com>
ZaynJarvis
approved these changes
Oct 15, 2026
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.
Description
Give maintained framework adapters one home under
integrations/frameworks/, with each package owning its source, tests, examples, and documentation. Add a practical SDK-based guide for future framework integrations.Previously these packages lived under
examples/, and LangChain tests were spread across the repository. The new layout makes the implementation and maintenance boundary clear without changing adapter runtime behavior.Human Involvement
Related Issue
Follow-up to #5578. Preserves the standalone package boundary established in #3685 and #3711.
Type of Change
Changes Made
langchain-openviking(including LangGraph) andopenviking-agent-frameworkintointegrations/frameworks/. Move their tests and runnable examples alongside the packages.langchain-openvikingand replaceopenviking.integrations.langchainimports withlangchain_openviking. No compatibility shim remains.license-files. Raise LangChain's build-only setuptools minimum to 77 for this metadata support; runtime dependencies are unchanged.All 17 runtime Python source files and existing runnable examples are byte-identical to the base. The only deleted tests are the two tests for the intentionally removed import aliases. Existing example licenses are retained in their new locations. Root server manifests, lockfile, and Docker configuration are unchanged.
Testing
Local validation on macOS:
AGPL-3.0metadata is preservedTwo legacy-alias tests were removed and two source/layout boundary tests added. The rebase preserves main's #5878 health regression cases and its newer timeout API changes; the relocated runtime sources remain byte-identical to current main. Existing streaming, tool capture, restoration, and commit-policy coverage remains intact. Both adapters' updated wheels were installed and smoke-tested outside the checkout. The three new installed server-wheel guards are exercised by GitHub CI; no full native server wheel was built locally. No live provider calls were rerun for this migration.
Checklist
Additional Notes
Public imports remain
langchain_openvikingandopenviking_agent_framework. LangChain and LangGraph remain one distribution with the optional[langgraph]extra. MAF PyPI publication is a separate follow-up; its README includes checkout installation instructions.PRs based on the former layout must move any newly added LangChain tests into
integrations/frameworks/langchain/tests/when rebasing. The guard keeps such tests from silently falling out of CI.Maintainer confirmation before merge
examples/. The existingAGPL-3.0identifier is preserved; selectingAGPL-3.0-onlyorAGPL-3.0-or-laterrequires confirmation of the intended variant.Rebased onto
ae1e45e00after #5878 and #6048 landed. All 70 integration test functions (and their three helpers) survive unchanged. Removed the redundant MAF layout steps, made the layout check name explicit, and made URI contract imports fail normally if the adapter installation is broken.