Skip to content

docs: Update OCI provider to link to samples directory#2943

Merged
Mason Daugherty (mdrxy) merged 2 commits intolangchain-ai:mainfrom
fede-kamel:fix-samples-links
Mar 5, 2026
Merged

docs: Update OCI provider to link to samples directory#2943
Mason Daugherty (mdrxy) merged 2 commits intolangchain-ai:mainfrom
fede-kamel:fix-samples-links

Conversation

@fede-kamel
Copy link
Copy Markdown
Contributor

Summary

Update the OCI provider documentation to link to samples/ instead of tutorials/. This aligns with the samples directory being added to langchain-oracle (see oracle/langchain-oracle#152).

Changes

  • Rename "Tutorials" section to "Samples"
  • Update all GitHub links from /tutorials/ to /samples/
  • No content changes, just link updates

Related

@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Mar 5, 2026

❌ Import check failed

This PR contains imports from langchain_core that should be imported from langchain instead.

Detailed issues
Analyzing diff for import issues...
 Found 10 import issues:

File: src/oss/python/integrations/providers/oci-tutorials/building-agents.mdx
Line: 37
Issue: Import from langchain.tools instead of langchain_core.tools
Current:   from langchain_core.tools import tool
Suggested: from langchain.tools import tool
--------------------------------------------------------------------------------
File: src/oss/python/integrations/providers/oci-tutorials/building-agents.mdx
Line: 76
Issue: Import from langchain.messages instead of langchain_core.messages
Current:   from langchain_core.messages import HumanMessage
Suggested: from langchain.messages import HumanMessage
--------------------------------------------------------------------------------
File: src/oss/python/integrations/providers/oci-tutorials/getting-started.mdx
Line: 191
Issue: Import from langchain.messages instead of langchain_core.messages
Current:   from langchain_core.messages import HumanMessage, AIMessage, SystemMessage
Suggested: from langchain.messages import HumanMessage, AIMessage, SystemMessage
--------------------------------------------------------------------------------
File: src/oss/python/integrations/providers/oci-tutorials/tool-calling.mdx
Line: 13
Issue: Import from langchain.messages instead of langchain_core.messages
Current:   from langchain_core.messages import HumanMessage, ToolMessage
Suggested: from langchain.messages import HumanMessage, ToolMessage
--------------------------------------------------------------------------------
File: src/oss/python/integrations/providers/oci-tutorials/tool-calling.mdx
Line: 14
Issue: Import from langchain.tools instead of langchain_core.tools
Current:   from langchain_core.tools import tool
Suggested: from langchain.tools import tool
--------------------------------------------------------------------------------
File: src/oss/python/integrations/providers/oci-tutorials/vision-multimodal.mdx
Line: 105
Issue: Import from langchain.messages instead of langchain_core.messages
Current:   from langchain_core.messages import HumanMessage
Suggested: from langchain.messages import HumanMessage
--------------------------------------------------------------------------------
File: src/oss/python/integrations/providers/oci-tutorials/vision-multimodal.mdx
Line: 140
Issue: Import from langchain.messages instead of langchain_core.messages
Current:   from langchain_core.messages import HumanMessage
Suggested: from langchain.messages import HumanMessage
--------------------------------------------------------------------------------
File: src/oss/python/integrations/providers/oci-tutorials/vision-multimodal.mdx
Line: 168
Issue: Import from langchain.messages instead of langchain_core.messages
Current:   from langchain_core.messages import HumanMessage
Suggested: from langchain.messages import HumanMessage
--------------------------------------------------------------------------------
File: src/oss/python/integrations/providers/oci-tutorials/vision-multimodal.mdx
Line: 214
Issue: Import from langchain.messages instead of langchain_core.messages
Current:   from langchain_core.messages import HumanMessage
Suggested: from langchain.messages import HumanMessage
--------------------------------------------------------------------------------
File: src/oss/python/integrations/providers/oci-tutorials/vision-multimodal.mdx
Line: 248
Issue: Import from langchain.messages instead of langchain_core.messages
Current:   from langchain_core.messages import HumanMessage
Suggested: from langchain.messages import HumanMessage
--------------------------------------------------------------------------------

 Found 10 import issues that need to be fixed.

Why this is a problem

The langchain package re-exports many modules and classes from langchain_core. When possible, imports should use langchain instead of langchain_core for:

  • Better user experience (single import source)
  • Consistency across documentation
  • Reduced cognitive load for users

How to fix

Replace the imports as suggested above. For example:

  • from langchain_core.messages import HumanMessage
  • from langchain.messages import HumanMessage

🤖 Automated check

This check is based on the latest analysis of langchain re-exports from langchain_core.

@github-actions github-actions bot added the external User is not a member of langchain-ai label Mar 5, 2026
@fede-kamel Federico Kamelhar (fede-kamel) marked this pull request as draft March 5, 2026 18:52
@fede-kamel Federico Kamelhar (fede-kamel) marked this pull request as ready for review March 5, 2026 19:02
@mdrxy Mason Daugherty (mdrxy) merged commit 2766f28 into langchain-ai:main Mar 5, 2026
13 checks passed
@fede-kamel Federico Kamelhar (fede-kamel) deleted the fix-samples-links branch March 5, 2026 19:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

external User is not a member of langchain-ai langchain For docs changes to LangChain oss python For content related to the Python version of LangChain projects

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants