mirror of
https://github.com/hwchase17/langchain.git
synced 2025-06-19 13:23:35 +00:00
Update regex in output parser (#15082)
The regex used to match "Action" and "Action Input" in the output parser has been updated. Previously, the regex did not correctly handle multi-line inputs for "Action Input". The updated code uses the 're.DOTALL' flag to ensure multi-line inputs are correctly captured. <!-- Thank you for contributing to LangChain! Please title your PR "<package>: <description>", where <package> is whichever of langchain, community, core, experimental, etc. is being modified. Replace this entire comment with: - **Description:** a description of the change, - **Issue:** the issue # it fixes if applicable, - **Dependencies:** any dependencies required for this change, - **Twitter handle:** we announce bigger features on Twitter. If your PR gets announced, and you'd like a mention, we'll gladly shout you out! Please make sure your PR is passing linting and testing before submitting. Run `make format`, `make lint` and `make test` from the root of the package you've modified to check this locally. See contribution guidelines for more information on how to write/run tests, lint, etc: https://python.langchain.com/docs/contributing/ If you're adding a new integration, please include: 1. a test for the integration, preferably unit tests that do not rely on network access, 2. an example notebook showing its use. It lives in `docs/docs/integrations` directory. If no one reviews your PR within a few days, please @-mention one of @baskaryan, @eyurtsev, @hwchase17. --> --------- Co-authored-by: Harrison Chase <hw.chase.17@gmail.com>
This commit is contained in:
parent
32e96a471c
commit
dfd7b9edda
@ -22,8 +22,8 @@ class ConvoOutputParser(AgentOutputParser):
|
|||||||
return AgentFinish(
|
return AgentFinish(
|
||||||
{"output": text.split(f"{self.ai_prefix}:")[-1].strip()}, text
|
{"output": text.split(f"{self.ai_prefix}:")[-1].strip()}, text
|
||||||
)
|
)
|
||||||
regex = r"Action: (.*?)[\n]*Action Input: (.*)"
|
regex = r"Action: (.*?)[\n]*Action Input: ([\s\S]*)"
|
||||||
match = re.search(regex, text)
|
match = re.search(regex, text, re.DOTALL)
|
||||||
if not match:
|
if not match:
|
||||||
raise OutputParserException(f"Could not parse LLM output: `{text}`")
|
raise OutputParserException(f"Could not parse LLM output: `{text}`")
|
||||||
action = match.group(1)
|
action = match.group(1)
|
||||||
|
@ -0,0 +1,42 @@
|
|||||||
|
from langchain_core.agents import AgentAction
|
||||||
|
|
||||||
|
from langchain.agents.conversational.output_parser import ConvoOutputParser
|
||||||
|
|
||||||
|
|
||||||
|
def test_normal_output_parsing() -> None:
|
||||||
|
_test_convo_output(
|
||||||
|
"""
|
||||||
|
Action: my_action
|
||||||
|
Action Input: my action input
|
||||||
|
""",
|
||||||
|
"my_action",
|
||||||
|
"my action input",
|
||||||
|
)
|
||||||
|
|
||||||
|
|
||||||
|
def test_multiline_output_parsing() -> None:
|
||||||
|
_test_convo_output(
|
||||||
|
"""
|
||||||
|
Thought: Do I need to use a tool? Yes
|
||||||
|
Action: evaluate_code
|
||||||
|
Action Input: Evaluate Code with the following Python content:
|
||||||
|
```python
|
||||||
|
print("Hello fifty shades of gray mans!"[::-1])
|
||||||
|
```
|
||||||
|
""",
|
||||||
|
"evaluate_code",
|
||||||
|
"""
|
||||||
|
Evaluate Code with the following Python content:
|
||||||
|
```python
|
||||||
|
print("Hello fifty shades of gray mans!"[::-1])
|
||||||
|
```""".lstrip(),
|
||||||
|
)
|
||||||
|
|
||||||
|
|
||||||
|
def _test_convo_output(
|
||||||
|
input: str, expected_tool: str, expected_tool_input: str
|
||||||
|
) -> None:
|
||||||
|
result = ConvoOutputParser().parse(input.strip())
|
||||||
|
assert isinstance(result, AgentAction)
|
||||||
|
assert result.tool == expected_tool
|
||||||
|
assert result.tool_input == expected_tool_input
|
Loading…
Reference in New Issue
Block a user