feat(docs): add thinking in langgraph #792

eyurtsev · 2025-10-08T19:06:21Z

Adding thinking in langgraph

github-actions · 2025-10-08T19:07:46Z

Preview ID generated: preview-wayofl-1759950434-e82b2d5

Copilot

Pull Request Overview

This PR introduces a comprehensive tutorial document titled "Thinking in LangGraph" that guides developers through the thought process and methodology for building agents with LangGraph. The tutorial uses a customer support email agent as a practical example to demonstrate the framework's concepts and best practices.

Key changes:

New tutorial document covering the 5-step process for building LangGraph agents
Integration of the new document into the documentation navigation structure

Reviewed Changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 3 comments.

File	Description
src/oss/langgraph/thinking-in-langgraph.mdx	New comprehensive tutorial document explaining LangGraph agent development methodology with practical examples
src/docs.json	Added navigation entry for the new thinking-in-langgraph tutorial in the LangGraph section

Copilot · 2025-10-09T16:00:07Z

src/oss/langgraph/thinking-in-langgraph.mdx

+
+llm = ChatOpenAI(model="gpt-4")
+


API key configuration is exposed in the code example. Consider showing environment variable usage or configuration patterns instead of direct instantiation to avoid exposing credentials.

Suggested change

llm = ChatOpenAI(model="gpt-4")

import os

# Read API key from environment variable for security

openai_api_key = os.environ.get("OPENAI_API_KEY")

if not openai_api_key:

raise RuntimeError("OPENAI_API_KEY environment variable not set")

llm = ChatOpenAI(model="gpt-4", api_key=openai_api_key)

Copilot · 2025-10-09T16:00:07Z

src/oss/langgraph/thinking-in-langgraph.mdx

+     └─────────────┘
+
+```
+


The Mermaid diagram appears to duplicate the ASCII diagram above it (lines 35-72). The node names in the Mermaid diagram don't match the ASCII version (e.g., 'write_response' vs 'Draft Reply'). Consider removing one diagram or ensuring they show complementary information with consistent naming.

src/oss/langgraph/thinking-in-langgraph.mdx

src/docs.json

Copilot

Pull Request Overview

Copilot reviewed 2 out of 2 changed files in this pull request and generated 2 comments.

Copilot · 2025-10-09T19:15:01Z

src/oss/langgraph/thinking-in-langgraph.mdx

+
+llm = ChatOpenAI(model="gpt-4")


Hard-coded model configuration without API key management. Consider using environment variables or configuration files for API credentials and model settings.

Suggested change

llm = ChatOpenAI(model="gpt-4")

import os

llm = ChatOpenAI(model=os.environ.get("OPENAI_MODEL", "gpt-4"))

src/oss/langgraph/thinking-in-langgraph.mdx

github-actions · 2025-10-09T19:17:14Z

Preview ID generated: preview-wayofl-1760037402-d2562fa

lnhsingh · 2025-10-09T21:30:52Z

src/oss/langgraph/thinking-in-langgraph.mdx

@@ -0,0 +1,568 @@
+---
+title: Thinking in LangGraph


Suggested change

title: Thinking in LangGraph

title: LangGraph design patterns

What about something like this?

I like thinking in langgraph more (I think it's clearer that it's more of a conceptual walk through than design patterns). We can poll more folks?

github-actions · 2025-10-11T15:38:07Z

Preview ID generated: preview-wayofl-1760197058-0726af2

Copilot

Pull Request Overview

Copilot reviewed 2 out of 2 changed files in this pull request and generated 2 comments.

Comments suppressed due to low confidence (1)

src/oss/langgraph/thinking-in-langgraph.mdx:1

The navigation path references 'oss/python/langgraph/thinking-in-langgraph' but the actual file is located at 'oss/langgraph/thinking-in-langgraph.mdx'. This path mismatch will result in a broken navigation link.

---

Copilot · 2025-10-12T02:18:49Z

src/oss/langgraph/thinking-in-langgraph.mdx

+from langchain_openai import ChatOpenAI
+from langchain_core.messages import HumanMessage
+
+llm = ChatOpenAI(model="gpt-4")


API key configuration is exposed in the code example. Consider using environment variables or configuration patterns to avoid exposing API keys in documentation.

Copilot · 2025-10-12T02:18:49Z

src/oss/langgraph/thinking-in-langgraph.mdx

+## Step 1: Map out your workflow as discrete steps
+
+Start by identifying the distinct steps in your process. Each step will become a **node** (a function that does one specific thing). Then sketch how these steps connect to each other.
+


Two different diagram formats (Mermaid and ASCII) show the same workflow. This duplication could lead to inconsistencies if one is updated but not the other. Consider using only the Mermaid diagram for consistency and maintainability.

github-actions · 2025-10-12T02:19:17Z

Preview ID generated: preview-wayofl-1760235528-c17017c

github-actions · 2025-10-12T13:07:27Z

Preview ID generated: preview-wayofl-1760274417-6afd996

Copilot

Pull Request Overview

Copilot reviewed 2 out of 2 changed files in this pull request and generated 2 comments.

Copilot · 2025-10-12T13:09:02Z

src/oss/langgraph/thinking-in-langgraph.mdx

+    from langgraph.pregel import RetryPolicy
+    from langchain_openai import ChatOpenAI
+    from langchain_core.messages import HumanMessage
+


The code contains a hardcoded LLM initialization without an API key parameter, but API keys should not be present in documentation examples. Consider adding a comment about environment variable configuration or use a placeholder pattern.

Suggested change

# Assumes OPENAI_API_KEY is set in your environment variables

Copilot · 2025-10-12T13:09:02Z

src/oss/langgraph/thinking-in-langgraph.mdx

+    from langgraph.graph import StateGraph, START, END
+    from langgraph.types import interrupt, Command
+    from langgraph.pregel import RetryPolicy
+    from langchain_openai import ChatOpenAI


The import suggests OpenAI API usage but there's no guidance about API key management. Documentation should include a note about setting up environment variables like OPENAI_API_KEY for security.

github-actions · 2025-10-12T13:09:43Z

Preview ID generated: preview-wayofl-1760274552-e155104

github-actions · 2025-10-12T13:13:44Z

Preview ID generated: preview-wayofl-1760274792-764d063

Copilot

Pull Request Overview

Copilot reviewed 2 out of 2 changed files in this pull request and generated 3 comments.

src/oss/langgraph/thinking-in-langgraph.mdx

Copilot · 2025-10-14T01:46:08Z

src/oss/langgraph/thinking-in-langgraph.mdx

+
+```python
+from langgraph.checkpoint.memory import MemorySaver
+from langgraph.pregel import RetryPolicy


The import from langgraph.pregel import RetryPolicy is incorrect. RetryPolicy should be imported from langgraph.types not langgraph.pregel.

Suggested change

from langgraph.pregel import RetryPolicy

from langgraph.types import RetryPolicy

Copilot · 2025-10-14T01:46:08Z

src/oss/langgraph/thinking-in-langgraph.mdx

+
+```typescript
+import { MemorySaver } from "@langchain/langgraph";
+import { RetryPolicy } from "@langchain/langgraph";


The RetryPolicy import appears twice in this code block. Line 902 has a duplicate import that should be removed since it's already imported on line 312.

Suggested change

import { RetryPolicy } from "@langchain/langgraph";

github-actions · 2025-10-14T01:46:49Z

Preview ID generated: preview-wayofl-1760406375-52ff3fe

github-actions · 2025-10-14T02:21:24Z

Preview ID generated: preview-wayofl-1760408456-482b16a

Adding thinking in langgraph --------- Co-authored-by: Hunter Lovell <hunter@hntrl.io>

x

b58466e

github-actions bot added langgraph For docs changes to LangGraph oss labels Oct 8, 2025

eyurtsev marked this pull request as ready for review October 9, 2025 15:59

eyurtsev requested review from katmayb and lnhsingh as code owners October 9, 2025 15:59

Copilot AI review requested due to automatic review settings October 9, 2025 15:59

Copilot AI reviewed Oct 9, 2025

View reviewed changes

eyurtsev commented Oct 9, 2025

View reviewed changes

src/docs.json Outdated Show resolved Hide resolved

Apply suggestion from @eyurtsev

fafe6e6

Copilot AI review requested due to automatic review settings October 9, 2025 19:14

Copilot AI reviewed Oct 9, 2025

View reviewed changes

lnhsingh reviewed Oct 9, 2025

View reviewed changes

eyurtsev added 2 commits October 11, 2025 11:07

x

e5fe895

x

f4de864

eyurtsev added 2 commits October 11, 2025 21:47

x

28fd755

more visual changes

665fe88

Copilot AI review requested due to automatic review settings October 12, 2025 02:18

Copilot AI reviewed Oct 12, 2025

View reviewed changes

update

b691547

x

3ae582e

Copilot AI review requested due to automatic review settings October 12, 2025 13:08

Copilot AI reviewed Oct 12, 2025

View reviewed changes

x

67af4ad

update

329549a

Copilot AI review requested due to automatic review settings October 14, 2025 01:45

Copilot AI reviewed Oct 14, 2025

View reviewed changes

eyurtsev changed the title ~~feat(docs): first pass at the WAY OF LANGGRAPH~~ feat(docs): add thinking in langgraph Oct 14, 2025

chore(oss/langgraph): update js translations

11bccb8

eyurtsev merged commit f84c5c1 into main Oct 14, 2025
11 checks passed

eyurtsev deleted the way_of_langgraph branch October 14, 2025 16:32

mdrxy pushed a commit that referenced this pull request Oct 14, 2025

feat(docs): add thinking in langgraph (#792)

8624fc3

Adding thinking in langgraph --------- Co-authored-by: Hunter Lovell <hunter@hntrl.io>

-llm = ChatOpenAI(model="gpt-4")
+import os
+# Read API key from environment variable for security
+openai_api_key = os.environ.get("OPENAI_API_KEY")
+if not openai_api_key:
+    raise RuntimeError("OPENAI_API_KEY environment variable not set")
+llm = ChatOpenAI(model="gpt-4", api_key=openai_api_key)

	title: Thinking in LangGraph
	title: LangGraph design patterns

		## Step 1: Map out your workflow as discrete steps

		Start by identifying the distinct steps in your process. Each step will become a node (a function that does one specific thing). Then sketch how these steps connect to each other.



	# Assumes OPENAI_API_KEY is set in your environment variables

	from langgraph.pregel import RetryPolicy
	from langgraph.types import RetryPolicy

		└─────────────┘

		```

feat(docs): add thinking in langgraph #792

feat(docs): add thinking in langgraph #792

Uh oh!

Conversation

eyurtsev commented Oct 8, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

github-actions bot commented Oct 8, 2025

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull Request Overview

Reviewed Changes

Uh oh!

Copilot AI Oct 9, 2025

Choose a reason for hiding this comment

Uh oh!

Copilot AI Oct 9, 2025

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull Request Overview

Uh oh!

Copilot AI Oct 9, 2025

Choose a reason for hiding this comment

Uh oh!

Uh oh!

github-actions bot commented Oct 9, 2025

Uh oh!

lnhsingh Oct 9, 2025

Choose a reason for hiding this comment

Uh oh!

eyurtsev Oct 12, 2025

Choose a reason for hiding this comment

Uh oh!

github-actions bot commented Oct 11, 2025

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull Request Overview

Uh oh!

Copilot AI Oct 12, 2025

Choose a reason for hiding this comment

Uh oh!

Copilot AI Oct 12, 2025

Choose a reason for hiding this comment

Uh oh!

github-actions bot commented Oct 12, 2025

Uh oh!

github-actions bot commented Oct 12, 2025

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull Request Overview

Uh oh!

Copilot AI Oct 12, 2025

Choose a reason for hiding this comment

Uh oh!

Copilot AI Oct 12, 2025

Choose a reason for hiding this comment

Uh oh!

github-actions bot commented Oct 12, 2025

Uh oh!

github-actions bot commented Oct 12, 2025

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull Request Overview

Uh oh!

Uh oh!

Copilot AI Oct 14, 2025

Choose a reason for hiding this comment

Uh oh!

Copilot AI Oct 14, 2025

Choose a reason for hiding this comment

Uh oh!

github-actions bot commented Oct 14, 2025

eyurtsev commented Oct 8, 2025 •

edited

Loading