Skip to content

fix Python snippet for HITL middleware#565

Merged
Sydney Runkle (sydney-runkle) merged 4 commits into
mainfrom
sr/fix-middleware
Sep 17, 2025
Merged

fix Python snippet for HITL middleware#565
Sydney Runkle (sydney-runkle) merged 4 commits into
mainfrom
sr/fix-middleware

Conversation

@sydney-runkle

Copy link
Copy Markdown
Collaborator

No description provided.

Copilot AI review requested due to automatic review settings September 17, 2025 18:45
@github-actions github-actions Bot added langchain For docs changes to LangChain oss labels Sep 17, 2025

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 fixes the Python code snippet for Human-in-the-Loop (HITL) middleware configuration by updating the tool configuration parameters to match the current API specification.

  • Updates tool configuration dictionary keys from old API (require_approval, description) to new API (allow_approve, allow_edit, allow_respond)
  • Removes the read_data tool configuration example
  • Maintains the same tools (write_file and execute_sql) but with corrected parameter structure

@github-actions

Copy link
Copy Markdown
Contributor

Preview ID generated: preview-srfixm-1758134777-d4168b8

@sydney-runkle Sydney Runkle (sydney-runkle) changed the title fix: fix Python snippet for HITL middleware fix Python snippet for HITL middleware Sep 17, 2025
@github-actions

Copy link
Copy Markdown
Contributor

Preview ID generated: preview-srfixm-1758135538-e5fdeb8

Copilot AI review requested due to automatic review settings September 17, 2025 19:09

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

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

@github-actions

Copy link
Copy Markdown
Contributor

Preview ID generated: preview-srfixm-1758136262-7ec5dfd

@langchain-ai LangChain (langchain-ai) deleted a comment from Copilot AI Sep 17, 2025
@langchain-ai LangChain (langchain-ai) deleted a comment from Copilot AI Sep 17, 2025
@sydney-runkle
Sydney Runkle (sydney-runkle) merged commit f8325f5 into main Sep 17, 2025
12 checks passed
@sydney-runkle
Sydney Runkle (sydney-runkle) deleted the sr/fix-middleware branch September 17, 2025 19:14
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.

3 participants