Function check_unused_code

apps/lobehub-compatibility/test_tool_parser.py:291–334

Check for unused code in the parser.

()

def check_unused_code():
    """Check for unused code in the parser."""
    print("\n" + "=" * 60)
    print("CODE ANALYSIS - Potential Issues")
    print("=" * 60)

    issues = []

    # Issue 1: Unused variables
    unused_vars = [
        "self.current_tool_name_sent",
        "self.prev_tool_call_arr",
        "self.current_tool_id",
        "self.streamed_args_for_tool",
        "self.buffer",
    ]
    issues.append(
        f"⚠️  Unused instance variables (defined but never used in main logic):\n   {', '.join(unused_vars)}"
    )

    # Issue 2: Unused method
    issues.append("⚠️  `_ensure_tool_id_valid` method is defined but never called")

    # Issue 3: Unused regex
    issues.append("⚠️  `partial_tool_regex` is defined but never used")

    # Issue 4: server_name handling
    issues.append(
        "⚠️  `_resolve_tool_name` checks for 'default' server_name,\n   but chat_template.jinja uses 'my_mcp_server'"
    )

    for issue in issues:
        print(f"\n{issue}")

    print("\n" + "=" * 60)
    print("RECOMMENDATIONS")
    print("=" * 60)
    print("""
1. Remove unused variables and methods to clean up the code
2. Either use `partial_tool_regex` or remove it
3. Update `_resolve_tool_name` to handle 'my_mcp_server' correctly
4. The streaming implementation looks correct with the state machine approach
5. The main `extract_tool_calls` and `extract_tool_calls_streaming` logic appears sound
""")


def main():