fix: resolve Windows compatibility issues in mcp_servers.py

- Fix environment variable name from STATA-MCP-CLIENT to STATA_MCP_CLIENT
- Disable help functionality on Windows systems to prevent crashes
- Disable ado package installation on Windows systems
- Add strict_mode parameter to write_dofile function
- Fix grammar typos in comments

Author: SepineTam

pyproject.toml

@@ -10,7 +10,7 @@ requires-python = ">=3.11"
 license = {text = "Apache-2.0"}
 dependencies = [
     "python-dotenv>=1.1.1",
-    "mcp[cli]>=1.16.0",
+    "mcp[cli]>=1.20.0",
     "pandas>=2.3.0",
     "pexpect>=4.9.0",
 #    "flask>=2.3.0",

src/stata_mcp/core/data_info/_base.py

@@ -9,10 +9,10 @@
 
 import json
 from abc import ABC, abstractmethod
+from os import PathLike
 from pathlib import Path
 from typing import Any, Dict, List
 from urllib.parse import urlparse
-from os import PathLike
 
 import numpy as np
 import pandas as pd

src/stata_mcp/core/data_info/csv.py

@@ -116,8 +116,20 @@ def _read_data(self) -> pd.DataFrame:
                 # Generate default column names
                 self.kwargs['names'] = [f'V{i+1}' for i in range(num_cols)]
 
-            # Read the CSV file
-            df = pd.read_csv(file_path, **self.kwargs)
+            # Read the CSV file with error handling for invalid parameters
+            try:
+                df = pd.read_csv(file_path, **self.kwargs)
+            except TypeError as e:
+                if "unexpected keyword argument" in str(e):
+                    # Filter out problematic parameters and retry with basic ones
+                    basic_kwargs = {k: v for k, v in self.kwargs.items()
+                                    if k in {'sep', 'header', 'encoding', 'names'}}
+                    print(f"Warning: Retrying CSV read with filtered parameters due to: {e}")
+                    df = pd.read_csv(file_path, **basic_kwargs)
+                else:
+                    raise
+            except Exception as e:
+                raise ValueError(f"Error reading CSV file {file_path}: {str(e)}")
 
             return df
 

src/stata_mcp/core/stata/builtin_tools/ado_install/base.py

@@ -34,7 +34,7 @@ def __post_initialization(self):
             ...         self.github_mirror = "https://github.com"  # Fake var
             ...
             ...     def install(self, package: str):
-            ...         return f"Installing {package} with {self.custom_setting}"
+            ...         ...
         """
         pass
 
@@ -56,3 +56,15 @@ def install(self, package: str) -> str: pass
     @abstractmethod
     def check_install(message: str) -> bool:
         ...
+
+    @staticmethod
+    def check_installed_from_msg(msg: str) -> bool:
+        state_with_prompt = msg.split("\n")[0]
+        state_str = state_with_prompt.split(":")[-1].strip()
+        if isinstance(state := eval(state_str), bool):
+            return state
+        else:
+            return False
+
+    def _install_msg_template(self, runner_result: str) -> str:
+        return f"Installation State: {self.check_install(runner_result)}\n" + runner_result

src/stata_mcp/core/stata/builtin_tools/ado_install/github_install.py

@@ -15,7 +15,7 @@ class GITHUB_Install(AdoInstallBase):
     def install(self, package: str) -> str:
         install_command = f"github install {package}{self.REPLACE_MESSAGE}"
         runner_result = self.controller.run(install_command)
-        return f"Installation State: {self.check_install(runner_result)}\n" + runner_result
+        return self._install_msg_template(runner_result)
 
     @property
     def IS_EXIST_GITHUB(self) -> bool:

src/stata_mcp/core/stata/builtin_tools/ado_install/net_install.py

@@ -17,7 +17,7 @@ def install(self, package: str, directory_or_url: str = None) -> str:
 
         install_command = f"net install {package}{self.REPLACE_MESSAGE}{from_message}"
         runner_result = self.controller.run(install_command)
-        return f"Installation State: {self.check_install(runner_result)}\n" + runner_result
+        return self._install_msg_template(runner_result)
 
     @staticmethod
     def check_install(message: str) -> bool:

src/stata_mcp/core/stata/builtin_tools/ado_install/ssc_install.py

@@ -14,7 +14,7 @@ class SSC_Install(AdoInstallBase):
     def install(self, package: str) -> str:
         install_command = f"ssc install {package}{self.REPLACE_MESSAGE}"
         runner_result = self.controller.run(install_command)
-        return f"Installation State: {self.check_install(runner_result)}\n" + runner_result
+        return self._install_msg_template(runner_result)
 
     @staticmethod
     def check_install(message: str) -> bool: