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

[PR #1084] [MERGED] Fix: Handle direct routing from classify_intent to human_review #1283

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

📋 Pull Request Information

Original PR: https://github.com/langchain-ai/docs/pull/1084
Author: @MHKone
Created: 10/22/2025
Status: ✅ Merged
Merged: 10/22/2025
Merged by: @lnhsingh

Base: main ← Head: patch-1

📝 Commits (4)

  • 9d126cc Refactor human review state handling in EmailAgent, error syntax
  • 5f2e7a4 Update src/oss/langgraph/thinking-in-langgraph.mdx
  • b297382 Update src/oss/langgraph/thinking-in-langgraph.mdx
  • 5941197 Merge branch 'main' into patch-1

📊 Changes

1 file changed (+6 additions, -6 deletions)

View changed files

📝 src/oss/langgraph/thinking-in-langgraph.mdx (+6 -6)

📄 Description

Overview

Fix code error caused by direct routing from classify_intent node to human_review node when intent is 'billing' or 'critical', bypassing the draft_response node.

Type of change

Type: Fix typo/bug/link/formatting

Problem

The current workflow has a routing error:

  • When classify_intent detects 'billing' or 'critical' intent, it routes directly to human_review
  • This bypasses the draft_response node, causing a code error
  • The human_review node likely expects data/structure that should be created by draft_response

Solution

  • Updated routing logic to ensure draft_response runs before human_review for all intents

Checklist

  • I have read the contributing guidelines
  • I have tested my changes locally using docs dev
  • All code examples have been tested and work correctly
  • I have used root relative paths for internal links
  • I have updated navigation in src/docs.json if needed
  • I have gotten approval from the relevant reviewers

Additional notes

This fix ensures the workflow executes correctly for all intent types without breaking the expected data flow between nodes.

🔄 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/1084 **Author:** [@MHKone](https://github.com/MHKone) **Created:** 10/22/2025 **Status:** ✅ Merged **Merged:** 10/22/2025 **Merged by:** [@lnhsingh](https://github.com/lnhsingh) **Base:** `main` ← **Head:** `patch-1` --- ### 📝 Commits (4) - [`9d126cc`](https://github.com/langchain-ai/docs/commit/9d126cc35039ce4d4d9ca4e5f39ba86bdf105e84) Refactor human review state handling in EmailAgent, error syntax - [`5f2e7a4`](https://github.com/langchain-ai/docs/commit/5f2e7a4d99bc54b706b7b5cbb1218e45ffb471c5) Update src/oss/langgraph/thinking-in-langgraph.mdx - [`b297382`](https://github.com/langchain-ai/docs/commit/b29738220fdcdd4f8358429673e7cb9f41dcef81) Update src/oss/langgraph/thinking-in-langgraph.mdx - [`5941197`](https://github.com/langchain-ai/docs/commit/5941197b9603d975c9d9db8cc45e02747a461c78) Merge branch 'main' into patch-1 ### 📊 Changes **1 file changed** (+6 additions, -6 deletions) <details> <summary>View changed files</summary> 📝 `src/oss/langgraph/thinking-in-langgraph.mdx` (+6 -6) </details> ### 📄 Description ## Overview Fix code error caused by direct routing from `classify_intent` node to `human_review` node when intent is 'billing' or 'critical', bypassing the `draft_response` node. ## Type of change **Type:** Fix typo/bug/link/formatting ## Problem The current workflow has a routing error: - When `classify_intent` detects 'billing' or 'critical' intent, it routes directly to `human_review` - This bypasses the `draft_response` node, causing a code error - The `human_review` node likely expects data/structure that should be created by `draft_response` ## Solution - Updated routing logic to ensure `draft_response` runs before `human_review` for all intents - ## Checklist - [x] I have read the contributing guidelines - [x] I have tested my changes locally using `docs dev` - [x] All code examples have been tested and work correctly - [x] I have used **root relative** paths for internal links - [ ] I have updated navigation in `src/docs.json` if needed - [ ] I have gotten approval from the relevant reviewers ## Additional notes This fix ensures the workflow executes correctly for all intent types without breaking the expected data flow between nodes. --- <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:22:04 -05:00
yindo closed this issue 2026-02-17 17:22:04 -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#1283
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?