[PR #1357] [MERGED] fix(sdk): code hygiene #1401

Closed
opened 2026-02-15 20:15:37 -05:00 by yindo · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/langchain-ai/langgraphjs/pull/1357
Author: @dqbd
Created: 7/7/2025
Status: Merged
Merged: 7/7/2025
Merged by: @dqbd

Base: mainHead: dqbd/sdk-js-eslint


📝 Commits (2)

📊 Changes

20 files changed (+207 additions, -62 deletions)

View changed files

.changeset/few-spies-bet.md (+5 -0)
libs/sdk/.eslintrc.cjs (+78 -0)
📝 libs/sdk/package.json (+18 -2)
📝 libs/sdk/src/auth/error.ts (+1 -0)
📝 libs/sdk/src/auth/index.ts (+1 -1)
📝 libs/sdk/src/auth/types.ts (+2 -2)
📝 libs/sdk/src/client.ts (+19 -13)
📝 libs/sdk/src/react-ui/client.tsx (+11 -9)
📝 libs/sdk/src/react-ui/index.ts (+1 -0)
📝 libs/sdk/src/react-ui/server/server.ts (+1 -1)
📝 libs/sdk/src/react/debug.tsx (+1 -0)
📝 libs/sdk/src/react/stream.tsx (+35 -26)
📝 libs/sdk/src/schema.ts (+3 -2)
📝 libs/sdk/src/singletons/fetch.ts (+1 -0)
📝 libs/sdk/src/types.messages.ts (+1 -0)
📝 libs/sdk/src/types.stream.ts (+1 -1)
📝 libs/sdk/src/types.ts (+1 -1)
📝 libs/sdk/src/utils/async_caller.ts (+4 -2)
📝 libs/sdk/src/utils/sse.ts (+2 -2)
📝 yarn.lock (+21 -0)

📄 Description

Adds missing optional peerDependency on react-dom and adds ESLint + ESLint React Hooks rule


🔄 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/1357 **Author:** [@dqbd](https://github.com/dqbd) **Created:** 7/7/2025 **Status:** ✅ Merged **Merged:** 7/7/2025 **Merged by:** [@dqbd](https://github.com/dqbd) **Base:** `main` ← **Head:** `dqbd/sdk-js-eslint` --- ### 📝 Commits (2) - [`302fd4e`](https://github.com/langchain-ai/langgraphjs/commit/302fd4ec84f43b67800b0711f39fcf1bbb386187) fix(sdk): code hygiene - [`476e49f`](https://github.com/langchain-ai/langgraphjs/commit/476e49fdb86500704d2955cfa3bc25e594290774) Add changeset ### 📊 Changes **20 files changed** (+207 additions, -62 deletions) <details> <summary>View changed files</summary> ➕ `.changeset/few-spies-bet.md` (+5 -0) ➕ `libs/sdk/.eslintrc.cjs` (+78 -0) 📝 `libs/sdk/package.json` (+18 -2) 📝 `libs/sdk/src/auth/error.ts` (+1 -0) 📝 `libs/sdk/src/auth/index.ts` (+1 -1) 📝 `libs/sdk/src/auth/types.ts` (+2 -2) 📝 `libs/sdk/src/client.ts` (+19 -13) 📝 `libs/sdk/src/react-ui/client.tsx` (+11 -9) 📝 `libs/sdk/src/react-ui/index.ts` (+1 -0) 📝 `libs/sdk/src/react-ui/server/server.ts` (+1 -1) 📝 `libs/sdk/src/react/debug.tsx` (+1 -0) 📝 `libs/sdk/src/react/stream.tsx` (+35 -26) 📝 `libs/sdk/src/schema.ts` (+3 -2) 📝 `libs/sdk/src/singletons/fetch.ts` (+1 -0) 📝 `libs/sdk/src/types.messages.ts` (+1 -0) 📝 `libs/sdk/src/types.stream.ts` (+1 -1) 📝 `libs/sdk/src/types.ts` (+1 -1) 📝 `libs/sdk/src/utils/async_caller.ts` (+4 -2) 📝 `libs/sdk/src/utils/sse.ts` (+2 -2) 📝 `yarn.lock` (+21 -0) </details> ### 📄 Description Adds missing optional `peerDependency` on `react-dom` and adds ESLint + ESLint React Hooks rule --- <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 20:15:37 -05:00
yindo closed this issue 2026-02-15 20:15:37 -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#1401