[PR #211] [MERGED] Adds web entrypoint #579

Closed
opened 2026-02-15 19:15:06 -05:00 by yindo · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/langchain-ai/langgraphjs/pull/211
Author: @jacoblee93
Created: 6/11/2024
Status: Merged
Merged: 6/13/2024
Merged by: @jacoblee93

Base: mainHead: jacob/web


📝 Commits (8)

📊 Changes

82 files changed (+2780 additions, -112 deletions)

View changed files

.github/workflows/test-exports.yml (+173 -0)
📝 docs/mkdocs.yml (+1 -1)
environment_tests/docker-compose.yml (+103 -0)
environment_tests/scripts/docker-ci-entrypoint.sh (+29 -0)
environment_tests/test-exports-cf/.gitignore (+1 -0)
environment_tests/test-exports-cf/README.md (+3 -0)
environment_tests/test-exports-cf/package.json (+25 -0)
environment_tests/test-exports-cf/src/entrypoints.js (+2 -0)
environment_tests/test-exports-cf/src/index.int.test.ts (+26 -0)
environment_tests/test-exports-cf/src/index.ts (+76 -0)
environment_tests/test-exports-cf/src/index.unit.test.ts (+21 -0)
environment_tests/test-exports-cf/tsconfig.json (+97 -0)
environment_tests/test-exports-cf/wrangler.toml (+3 -0)
environment_tests/test-exports-cjs/package.json (+30 -0)
environment_tests/test-exports-cjs/src/entrypoints.js (+2 -0)
environment_tests/test-exports-cjs/src/import.js (+60 -0)
environment_tests/test-exports-cjs/src/index.mjs (+53 -0)
environment_tests/test-exports-cjs/src/index.ts (+62 -0)
environment_tests/test-exports-cjs/src/require.js (+61 -0)
environment_tests/test-exports-cjs/tsconfig.json (+16 -0)

...and 62 more files

📄 Description

Creates separate entrypoint for LangGraph classes that does not include async local storage.

I'm not sure why the initialize call was so nested - I think may have been unnecessary, tracing seems fine.

We will want to document this carefully and also update prebuilt components to explicitly pass config around to avoid the need for a prebuilt/web entrypoint.

CC @nfcampos @andrewnguonly @dqbd

Fixes #208 and #81


🔄 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/langgraphjs/pull/211 **Author:** [@jacoblee93](https://github.com/jacoblee93) **Created:** 6/11/2024 **Status:** ✅ Merged **Merged:** 6/13/2024 **Merged by:** [@jacoblee93](https://github.com/jacoblee93) **Base:** `main` ← **Head:** `jacob/web` --- ### 📝 Commits (8) - [`5db2492`](https://github.com/langchain-ai/langgraphjs/commit/5db2492b70d50252532036093d4aac8c16fb3895) Adds web entrypoint - [`db840ff`](https://github.com/langchain-ai/langgraphjs/commit/db840ffc531e6cb14491930967229c2598d58ffe) Update prebuilt to pass config - [`dc7b8a0`](https://github.com/langchain-ai/langgraphjs/commit/dc7b8a0fd16b82687f1d2c653bc3426a39396fa6) Add env tests - [`825c45c`](https://github.com/langchain-ai/langgraphjs/commit/825c45cb7319a62ba8cddfa651449293e735668f) Add to CI - [`4ea98a4`](https://github.com/langchain-ai/langgraphjs/commit/4ea98a424baaab7c67195b77d3aea634bde87c8f) Fix - [`9c96c57`](https://github.com/langchain-ai/langgraphjs/commit/9c96c57267faa1405e92c20af6b6b22792ee6261) Update CI condition - [`714892b`](https://github.com/langchain-ai/langgraphjs/commit/714892bec20561665ff3e13ed96894a5af950fe1) Fix docs build - [`8b8b029`](https://github.com/langchain-ai/langgraphjs/commit/8b8b0291a95fb943e40aca4c6f2ed108f784b943) Adds tracing unit test ### 📊 Changes **82 files changed** (+2780 additions, -112 deletions) <details> <summary>View changed files</summary> ➕ `.github/workflows/test-exports.yml` (+173 -0) 📝 `docs/mkdocs.yml` (+1 -1) ➕ `environment_tests/docker-compose.yml` (+103 -0) ➕ `environment_tests/scripts/docker-ci-entrypoint.sh` (+29 -0) ➕ `environment_tests/test-exports-cf/.gitignore` (+1 -0) ➕ `environment_tests/test-exports-cf/README.md` (+3 -0) ➕ `environment_tests/test-exports-cf/package.json` (+25 -0) ➕ `environment_tests/test-exports-cf/src/entrypoints.js` (+2 -0) ➕ `environment_tests/test-exports-cf/src/index.int.test.ts` (+26 -0) ➕ `environment_tests/test-exports-cf/src/index.ts` (+76 -0) ➕ `environment_tests/test-exports-cf/src/index.unit.test.ts` (+21 -0) ➕ `environment_tests/test-exports-cf/tsconfig.json` (+97 -0) ➕ `environment_tests/test-exports-cf/wrangler.toml` (+3 -0) ➕ `environment_tests/test-exports-cjs/package.json` (+30 -0) ➕ `environment_tests/test-exports-cjs/src/entrypoints.js` (+2 -0) ➕ `environment_tests/test-exports-cjs/src/import.js` (+60 -0) ➕ `environment_tests/test-exports-cjs/src/index.mjs` (+53 -0) ➕ `environment_tests/test-exports-cjs/src/index.ts` (+62 -0) ➕ `environment_tests/test-exports-cjs/src/require.js` (+61 -0) ➕ `environment_tests/test-exports-cjs/tsconfig.json` (+16 -0) _...and 62 more files_ </details> ### 📄 Description Creates separate entrypoint for LangGraph classes that does not include async local storage. I'm not sure why the initialize call was so nested - I think may have been unnecessary, tracing seems fine. We will want to document this carefully and also update prebuilt components to explicitly pass config around to avoid the need for a `prebuilt/web` entrypoint. CC @nfcampos @andrewnguonly @dqbd Fixes #208 and #81 --- <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-15 19:15:06 -05:00
yindo closed this issue 2026-02-15 19:15:06 -05:00
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: langchain-ai/langgraphjs#579