langchain-ai/docs
1
0
Fork 0
mirror of https://github.com/langchain-ai/docs.git synced 2026-07-22 04:15:25 -04:00
Code Issues 490 Packages Projects Releases Wiki Activity

[PR #415] [CLOSED] Brody/port callout install tab changes #729

New Issue
Closed
opened 2026-02-17 17:21:00 -05:00 by yindo · 0 comments
yindo commented 2026-02-17 17:21:00 -05:00
Owner
Copy Link
Copy Source

📋 Pull Request Information

Original PR: https://github.com/langchain-ai/docs/pull/415
Author: @beklapko
Created: 9/8/2025
Status: ❌ Closed

Base: main ← Head: brody/port-callout-install-tab-changes

📝 Commits (7)

  • 39a075f Remove callouts first
  • 3ac6d48 Fix install blocks
  • da8e507 Remove script
  • 018e4e4 Fix remaining install blocks,
  • c1686ce Fix tip callout
  • 790a1a0 Fix tags, fix broken migration content
  • 38a9f24 Merge branch 'main' into brody/port-callout-install-tab-changes

📊 Changes

234 files changed (+1146 additions, -1447 deletions)

View changed files

📝 src/oss/javascript/integrations/callbacks/datadog_tracer.mdx (+0 -3)
📝 src/oss/javascript/integrations/callbacks/upstash_ratelimit_callback.mdx (+0 -3)
📝 src/oss/javascript/integrations/chat/alibaba_tongyi.mdx (+0 -3)
📝 src/oss/javascript/integrations/chat/anthropic.mdx (+10 -13)
📝 src/oss/javascript/integrations/chat/arcjet.mdx (+21 -16)
📝 src/oss/javascript/integrations/chat/azure.mdx (+20 -15)
📝 src/oss/javascript/integrations/chat/baidu_qianfan.mdx (+0 -3)
📝 src/oss/javascript/integrations/chat/baidu_wenxin.mdx (+0 -3)
📝 src/oss/javascript/integrations/chat/bedrock.mdx (+23 -16)
📝 src/oss/javascript/integrations/chat/bedrock_converse.mdx (+10 -8)
📝 src/oss/javascript/integrations/chat/cerebras.mdx (+10 -9)
📝 src/oss/javascript/integrations/chat/cloudflare_workersai.mdx (+10 -8)
📝 src/oss/javascript/integrations/chat/cohere.mdx (+10 -9)
📝 src/oss/javascript/integrations/chat/deep_infra.mdx (+0 -3)
📝 src/oss/javascript/integrations/chat/fireworks.mdx (+10 -9)
📝 src/oss/javascript/integrations/chat/friendli.mdx (+0 -3)
📝 src/oss/javascript/integrations/chat/google_generativeai.mdx (+11 -11)
📝 src/oss/javascript/integrations/chat/google_vertex_ai.mdx (+10 -14)
📝 src/oss/javascript/integrations/chat/groq.mdx (+10 -9)
📝 src/oss/javascript/integrations/chat/ibm.mdx (+10 -8)

...and 80 more files

📄 Description

Summary

There are three main changes in this PR but they're spread across a lot of docs.

  1. This tooltip was removed (the callout linked to a page that's in the old docs but not the new docs, and didn't seem critical to me to have right now).
import IntegrationInstallTooltip from '/snippets/javascript-integrations/integration-install-tooltip.mdx';

<IntegrationInstallTooltip/>
  1. Installation code blocks like this didn't migrate so well.
<Npm2Yarn>
  @arcjet/redact
</Npm2Yarn>

For now, I've replaced these with code groups (below is just an example, each code group is different based on the library getting installed)

<CodeGroup>
```bash npm
npm install @arcjet/redact
```
```bash yarn
yarn add @arcjet/redact
```
```bash pnpm
pnpm add @arcjet/redact
```
</CodeGroup>

There is probably a smarter/better long term way to do this with snippets but this was doable in the short term to fix the missing example/broken rendering.

  1. Some tooltips and snippets got put in these {=mdx} blocks:
```{=mdx}
<Warning>
**Compatibility**

This feature is currently in beta.
</Warning>
```

So I removed the opening and closing "tags" which fixes the rendering issues.

Unresolved issue

This snippet shows up in several places:

<ChatModelTabs customVarName="llm" />

It was enclosed with {=mdx} and the triple backticks which caused it to be rendered like this in prod:

Screenshot 2025-09-08 at 1 32 06 PM

I removed the enclosing {=mdx} so it doesn't get rendered at all, but this doesn't fix the issue entirely. That snippet, from what I can tell, should render out something like this section in the old JS docs: https://js.langchain.com/docs/integrations/retrievers/exa/#use-within-a-chain.

It'd take more time and research to see if we can actually recreate that functionality here or not, which this PR does not address.

Testing

I spot checked a few instances of each of these changes locally but I did not verify every change in every file.

🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.

## 📋 Pull Request Information **Original PR:** https://github.com/langchain-ai/docs/pull/415 **Author:** [@beklapko](https://github.com/beklapko) **Created:** 9/8/2025 **Status:** ❌ Closed **Base:** `main` ← **Head:** `brody/port-callout-install-tab-changes` --- ### 📝 Commits (7) - [`39a075f`](https://github.com/langchain-ai/docs/commit/39a075f73d71250bde8b92e862d269c783b762d2) Remove callouts first - [`3ac6d48`](https://github.com/langchain-ai/docs/commit/3ac6d480214b0759832c411e476e7998336afbbf) Fix install blocks - [`da8e507`](https://github.com/langchain-ai/docs/commit/da8e5071d8ae6507d74158bf81cb4c07bd655b6a) Remove script - [`018e4e4`](https://github.com/langchain-ai/docs/commit/018e4e4e5a727ffc29a7351410de7b73b8604432) Fix remaining install blocks, - [`c1686ce`](https://github.com/langchain-ai/docs/commit/c1686ce5210bf4ee75daf90ced98e4e8c0da4d10) Fix tip callout - [`790a1a0`](https://github.com/langchain-ai/docs/commit/790a1a0846a271fea8405c2a2029263140153543) Fix tags, fix broken migration content - [`38a9f24`](https://github.com/langchain-ai/docs/commit/38a9f24df41b9991a048a6b91cecc85f54620d2d) Merge branch 'main' into brody/port-callout-install-tab-changes ### 📊 Changes **234 files changed** (+1146 additions, -1447 deletions) <details> <summary>View changed files</summary> 📝 `src/oss/javascript/integrations/callbacks/datadog_tracer.mdx` (+0 -3) 📝 `src/oss/javascript/integrations/callbacks/upstash_ratelimit_callback.mdx` (+0 -3) 📝 `src/oss/javascript/integrations/chat/alibaba_tongyi.mdx` (+0 -3) 📝 `src/oss/javascript/integrations/chat/anthropic.mdx` (+10 -13) 📝 `src/oss/javascript/integrations/chat/arcjet.mdx` (+21 -16) 📝 `src/oss/javascript/integrations/chat/azure.mdx` (+20 -15) 📝 `src/oss/javascript/integrations/chat/baidu_qianfan.mdx` (+0 -3) 📝 `src/oss/javascript/integrations/chat/baidu_wenxin.mdx` (+0 -3) 📝 `src/oss/javascript/integrations/chat/bedrock.mdx` (+23 -16) 📝 `src/oss/javascript/integrations/chat/bedrock_converse.mdx` (+10 -8) 📝 `src/oss/javascript/integrations/chat/cerebras.mdx` (+10 -9) 📝 `src/oss/javascript/integrations/chat/cloudflare_workersai.mdx` (+10 -8) 📝 `src/oss/javascript/integrations/chat/cohere.mdx` (+10 -9) 📝 `src/oss/javascript/integrations/chat/deep_infra.mdx` (+0 -3) 📝 `src/oss/javascript/integrations/chat/fireworks.mdx` (+10 -9) 📝 `src/oss/javascript/integrations/chat/friendli.mdx` (+0 -3) 📝 `src/oss/javascript/integrations/chat/google_generativeai.mdx` (+11 -11) 📝 `src/oss/javascript/integrations/chat/google_vertex_ai.mdx` (+10 -14) 📝 `src/oss/javascript/integrations/chat/groq.mdx` (+10 -9) 📝 `src/oss/javascript/integrations/chat/ibm.mdx` (+10 -8) _...and 80 more files_ </details> ### 📄 Description ## Summary There are three main changes in this PR but they're spread across a lot of docs. 1. This tooltip was removed (the callout linked to a page that's in the old docs but not the new docs, and didn't seem critical to me to have right now). ``` import IntegrationInstallTooltip from '/snippets/javascript-integrations/integration-install-tooltip.mdx'; <IntegrationInstallTooltip/> ``` 2. Installation code blocks like this didn't migrate so well. ``` <Npm2Yarn> @arcjet/redact </Npm2Yarn> ``` For now, I've replaced these with code groups (below is just an example, each code group is different based on the library getting installed) ```` <CodeGroup> ```bash npm npm install @arcjet/redact ``` ```bash yarn yarn add @arcjet/redact ``` ```bash pnpm pnpm add @arcjet/redact ``` </CodeGroup> ```` There is probably a smarter/better long term way to do this with snippets but this was doable in the short term to fix the missing example/broken rendering. 3. Some [tooltips](https://github.com/langchain-ai/docs/pull/415/files#diff-cedeee983cdeb7a6d86f7e209be9d3719baa1107f1f7bd21c5756c903333f0daL369) and snippets got put in these `{=mdx}` blocks: ```` ```{=mdx} <Warning> **Compatibility** This feature is currently in beta. </Warning> ``` ```` So I removed the opening and closing "tags" which fixes the rendering issues. ## Unresolved issue This snippet shows up in several places: ``` <ChatModelTabs customVarName="llm" /> ``` It was enclosed with `{=mdx}` and the triple backticks which caused it to be rendered like this in prod: <img width="821" height="230" alt="Screenshot 2025-09-08 at 1 32 06 PM" src="https://github.com/user-attachments/assets/4ee51da1-3a55-48ef-9d18-f231797d6900" /> I removed the enclosing `{=mdx}` so it doesn't get rendered at all, but this doesn't fix the issue entirely. That snippet, from what I can tell, should render out something like this section in the old JS docs: https://js.langchain.com/docs/integrations/retrievers/exa/#use-within-a-chain. It'd take more time and research to see if we can actually recreate that functionality here or not, which this PR does not address. ## Testing I spot checked a few instances of each of these changes locally but I did not verify every change in every file. --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
yindo added the pull-request label 2026-02-17 17:21:00 -05:00
yindo closed this issue 2026-02-17 17:21:00 -05:00
Sign in to join this conversation.
No Branch/Tag Specified
Branches Tags
Labels
Clear labels
bug
deepagents
docs-infra
external
external
help wanted
integration
internal
internal
javascript
js/ts
langchain
langgraph
langsmith
needs-title
pull-request
Mirrored from GitHub Pull Request
 python
reference
No Label pull-request
Milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
yindo
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: langchain-ai/docs#729
Delete Branch "%!s()"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?