Skip to content

chore(hitl): update hitl in both js and python#813

Merged
Eugene Yurtsev (eyurtsev) merged 1 commit into
mainfrom
hitl-updates
Oct 11, 2025
Merged

chore(hitl): update hitl in both js and python#813
Eugene Yurtsev (eyurtsev) merged 1 commit into
mainfrom
hitl-updates

Conversation

@eyurtsev

@eyurtsev Eugene Yurtsev (eyurtsev) commented Oct 10, 2025

Copy link
Copy Markdown
Collaborator
  • Update js and python to the new HITL schema.
  • Do not merge until js / python implementations are merged and out

@github-actions github-actions Bot added langchain For docs changes to LangChain oss labels Oct 10, 2025
@github-actions

Copy link
Copy Markdown
Contributor

Preview ID generated: preview-hitlup-1760062057-3c54633

@eyurtsev
Eugene Yurtsev (eyurtsev) marked this pull request as ready for review October 11, 2025 02:02
Copilot AI review requested due to automatic review settings October 11, 2025 02:02
@eyurtsev
Eugene Yurtsev (eyurtsev) merged commit e8b987e into main Oct 11, 2025
12 checks passed
@eyurtsev
Eugene Yurtsev (eyurtsev) deleted the hitl-updates branch October 11, 2025 02:02

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This PR updates the Human-in-the-Loop (HITL) documentation to reflect a new schema that changes the terminology and structure of the HITL implementation. The update affects both JavaScript and Python examples to maintain consistency across languages.

Key changes:

  • Updates terminology from "response types" to "decision types" and from "accept/respond" to "approve/reject"
  • Changes the data structure from simple arrays to objects with decisions wrapper and new field names
  • Updates configuration syntax for allowed decisions
Comments suppressed due to low confidence (1)

src/oss/langchain/human-in-the-loop.mdx:1

  • The "UI integration" section was removed but contained valuable information about integration with LangChain UI applications. This removal may leave users without guidance on how to integrate HITL with UI components.
---

interrupt_on={
"write_file": True, # All actions (accept, edit, respond) allowed
"execute_sql": {"allow_accept": True, "allow_respond": True}, # No editing allowed
"write_file": True, # All decisions (approve, edit, reject) allowed

Copilot AI Oct 11, 2025

Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The configuration structure is inconsistent. Line 53 uses True for all decisions allowed, but line 54 uses an object with allowed_decisions array. Consider using consistent structure, such as {"allowed_decisions": ["approve", "edit", "reject"]} for the write_file configuration.

Suggested change
"write_file": True, # All decisions (approve, edit, reject) allowed
"write_file": {"allowed_decisions": ["approve", "edit", "reject"]}, # All decisions (approve, edit, reject) allowed

Copilot uses AI. Check for mistakes.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

langchain For docs changes to LangChain oss

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants