Orphaned think/reasoning tags appearing in user-facing responses

Fixes on record

Merged to main. A stamp counts the issues a fix is linked to.

• #4311 fix: strip orphaned think/reasoning tags from user-facing responses

  merged to main

  files: run_agent.py, tests/test_run_agent.py

  Show the diff

  • run_agent.py

    @@ -1389,6 +1389,7 @@ def _strip_think_blocks(self, content: str) -> str:
             content = re.sub(r'<thinking>.*?</thinking>', '', content, flags=re.DOTALL | re.IGNORECASE)
             content = re.sub(r'<reasoning>.*?</reasoning>', '', content, flags=re.DOTALL)
             content = re.sub(r'<REASONING_SCRATCHPAD>.*?</REASONING_SCRATCHPAD>', '', content, flags=re.DOTALL)
    +        content = re.sub(r'</?(?:think|thinking|reasoning|REASONING_SCRATCHPAD)>\s*', '', content, flags=re.IGNORECASE)
             return content
     
         def _looks_like_codex_intermediate_ack(

  • tests/test_run_agent.py

    @@ -230,6 +230,27 @@ def test_multiline_block_removed(self, agent):
             assert "line1" not in result
             assert "visible" in result
     
    +    def test_orphaned_closing_think_tag(self, agent):
    +        result = agent._strip_think_blocks("some reasoning</think>actual answer")
    +        assert "</think>" not in result
    +        assert "actual answer" in result
    +
    +    def test_orphaned_closing_thinking_tag(self, agent):
    +        result = agent._strip_think_blocks("reasoning</thinking>answer")
    +        assert "</thinking>" not in result
    +        assert "answer" in result
    +
    +    def test_orphaned_opening_think_tag(self, agent):
    +        result = agent._strip_think_blocks("<think>orphaned reasoning without close")
    +        assert "<think>" not in result
    +
    +    def test_mixed_orphaned_and_paired_tags(self, agent):
    +        text = "stray</think><think>paired reasoning</think> visible"
    +        result = agent._strip_think_blocks(text)
    +        assert "</think>" not in result
    +        assert "<think>" not in result
    +        assert "visible" in result
    +
     
     class TestExtractReasoning:
         def test_reasoning_field(self, agent):