fix: resolve linting errors and test failures after AI-agent CLI updates

- Fix unused variable in generate_llm_txt.py
- Clean up whitespace in docstrings throughout non_interactive.py
- Fix edit command to properly exit with return codes using sys.exit()
- Update tests to correctly mock non-interactive detection
- Fix client edit test to avoid non-interactive mode for external editor test
- Update test assertions to match new simplified help text

All tests now pass (113 passed, 6 skipped) and linting is clean.

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <[email protected]>

Author: niechen

scripts/generate_llm_txt.py

@@ -12,9 +12,10 @@
 from pathlib import Path
 
 # Add src to path so we can import mcpm modules
-sys.path.insert(0, os.path.join(os.path.dirname(__file__), '..', 'src'))
+sys.path.insert(0, os.path.join(os.path.dirname(__file__), "..", "src"))
 
 import click
+
 from mcpm.cli import main as mcpm_cli
 
 # Try to import version, fallback to a default if not available
@@ -27,75 +28,75 @@
 def extract_command_info(cmd, parent_name=""):
     """Extract information from a Click command."""
     info = {
-        'name': cmd.name,
-        'full_name': f"{parent_name} {cmd.name}".strip(),
-        'help': cmd.help or "No description available",
-        'params': [],
-        'subcommands': {}
+        "name": cmd.name,
+        "full_name": f"{parent_name} {cmd.name}".strip(),
+        "help": cmd.help or "No description available",
+        "params": [],
+        "subcommands": {}
     }
-    
+
     # Extract parameters
     for param in cmd.params:
         param_info = {
-            'name': param.name,
-            'opts': getattr(param, 'opts', None) or [param.name],
-            'type': str(param.type),
-            'help': getattr(param, 'help', "") or "",
-            'required': getattr(param, 'required', False),
-            'is_flag': isinstance(param, click.Option) and param.is_flag,
-            'default': getattr(param, 'default', None)
+            "name": param.name,
+            "opts": getattr(param, "opts", None) or [param.name],
+            "type": str(param.type),
+            "help": getattr(param, "help", "") or "",
+            "required": getattr(param, "required", False),
+            "is_flag": isinstance(param, click.Option) and param.is_flag,
+            "default": getattr(param, "default", None)
         }
-        info['params'].append(param_info)
-    
+        info["params"].append(param_info)
+
     # Extract subcommands if this is a group
     if isinstance(cmd, click.Group):
         for subcommand_name, subcommand in cmd.commands.items():
-            info['subcommands'][subcommand_name] = extract_command_info(
-                subcommand, 
-                info['full_name']
+            info["subcommands"][subcommand_name] = extract_command_info(
+                subcommand,
+                info["full_name"]
             )
-    
+
     return info
 
 
 def format_command_section(cmd_info, level=2):
     """Format a command's information for the LLM.txt file."""
     lines = []
-    
+
     # Command header
     header = "#" * level + f" {cmd_info['full_name']}"
     lines.append(header)
     lines.append("")
-    
+
     # Description
-    lines.append(cmd_info['help'])
+    lines.append(cmd_info["help"])
     lines.append("")
-    
+
     # Parameters
-    if cmd_info['params']:
+    if cmd_info["params"]:
         lines.append("**Parameters:**")
         lines.append("")
-        
+
         # Separate arguments from options
-        args = [p for p in cmd_info['params'] if not p['opts'][0].startswith('-')]
-        opts = [p for p in cmd_info['params'] if p['opts'][0].startswith('-')]
-        
+        args = [p for p in cmd_info["params"] if not p["opts"][0].startswith("-")]
+        opts = [p for p in cmd_info["params"] if p["opts"][0].startswith("-")]
+
         if args:
             for param in args:
-                req = "REQUIRED" if param['required'] else "OPTIONAL"
+                req = "REQUIRED" if param["required"] else "OPTIONAL"
                 lines.append(f"- `{param['name']}` ({req}): {param['help']}")
             lines.append("")
-        
+
         if opts:
             for param in opts:
-                opt_str = ", ".join(f"`{opt}`" for opt in param['opts'])
-                if param['is_flag']:
+                opt_str = ", ".join(f"`{opt}`" for opt in param["opts"])
+                if param["is_flag"]:
                     lines.append(f"- {opt_str}: {param['help']} (flag)")
                 else:
-                    default_str = f" (default: {param['default']})" if param['default'] is not None else ""
+                    default_str = f" (default: {param['default']})" if param["default"] is not None else ""
                     lines.append(f"- {opt_str}: {param['help']}{default_str}")
             lines.append("")
-    
+
     # Examples section
     examples = generate_examples_for_command(cmd_info)
     if examples:
@@ -105,101 +106,100 @@ def format_command_section(cmd_info, level=2):
         lines.extend(examples)
         lines.append("```")
         lines.append("")
-    
+
     # Subcommands
-    for subcmd_info in cmd_info['subcommands'].values():
+    for subcmd_info in cmd_info["subcommands"].values():
         lines.extend(format_command_section(subcmd_info, level + 1))
-    
+
     return lines
 
 
 def generate_examples_for_command(cmd_info):
     """Generate relevant examples for a command based on its name and parameters."""
-    examples = []
-    cmd = cmd_info['full_name']
-    
+    cmd = cmd_info["full_name"]
+
     # Map of command patterns to example sets
     example_map = {
-        'mcpm new': [
-            '# Create a stdio server',
+        "mcpm new": [
+            "# Create a stdio server",
             'mcpm new myserver --type stdio --command "python -m myserver"',
-            '',
-            '# Create a remote server',
+            "",
+            "# Create a remote server",
             'mcpm new apiserver --type remote --url "https://api.example.com"',
-            '',
-            '# Create server with environment variables',
+            "",
+            "# Create server with environment variables",
             'mcpm new myserver --type stdio --command "python server.py" --env "API_KEY=secret,PORT=8080"',
         ],
-        'mcpm edit': [
-            '# Update server name',
+        "mcpm edit": [
+            "# Update server name",
             'mcpm edit myserver --name "new-name"',
-            '',
-            '# Update command and arguments',
+            "",
+            "# Update command and arguments",
             'mcpm edit myserver --command "python -m updated_server" --args "--port 8080"',
-            '',
-            '# Update environment variables',
+            "",
+            "# Update environment variables",
             'mcpm edit myserver --env "API_KEY=new-secret,DEBUG=true"',
         ],
-        'mcpm install': [
-            '# Install a server',
-            'mcpm install sqlite',
-            '',
-            '# Install with environment variables',
-            'ANTHROPIC_API_KEY=sk-ant-... mcpm install claude',
-            '',
-            '# Force installation',
-            'mcpm install filesystem --force',
+        "mcpm install": [
+            "# Install a server",
+            "mcpm install sqlite",
+            "",
+            "# Install with environment variables",
+            "ANTHROPIC_API_KEY=sk-ant-... mcpm install claude",
+            "",
+            "# Force installation",
+            "mcpm install filesystem --force",
         ],
-        'mcpm profile edit': [
-            '# Add server to profile',
-            'mcpm profile edit web-dev --add-server sqlite',
-            '',
-            '# Remove server from profile',
-            'mcpm profile edit web-dev --remove-server old-server',
-            '',
-            '# Set profile servers (replaces all)',
+        "mcpm profile edit": [
+            "# Add server to profile",
+            "mcpm profile edit web-dev --add-server sqlite",
+            "",
+            "# Remove server from profile",
+            "mcpm profile edit web-dev --remove-server old-server",
+            "",
+            "# Set profile servers (replaces all)",
             'mcpm profile edit web-dev --set-servers "sqlite,filesystem,git"',
-            '',
-            '# Rename profile',
-            'mcpm profile edit old-name --name new-name',
+            "",
+            "# Rename profile",
+            "mcpm profile edit old-name --name new-name",
         ],
-        'mcpm client edit': [
-            '# Add server to client',
-            'mcpm client edit cursor --add-server sqlite',
-            '',
-            '# Add profile to client',
-            'mcpm client edit cursor --add-profile web-dev',
-            '',
-            '# Set all servers for client',
+        "mcpm client edit": [
+            "# Add server to client",
+            "mcpm client edit cursor --add-server sqlite",
+            "",
+            "# Add profile to client",
+            "mcpm client edit cursor --add-profile web-dev",
+            "",
+            "# Set all servers for client",
             'mcpm client edit claude-desktop --set-servers "sqlite,filesystem"',
-            '',
-            '# Remove profile from client',
-            'mcpm client edit cursor --remove-profile old-profile',
+            "",
+            "# Remove profile from client",
+            "mcpm client edit cursor --remove-profile old-profile",
         ],
-        'mcpm run': [
-            '# Run a server',
-            'mcpm run sqlite',
-            '',
-            '# Run with HTTP transport',
-            'mcpm run myserver --http --port 8080',
+        "mcpm run": [
+            "# Run a server",
+            "mcpm run sqlite",
+            "",
+            "# Run with HTTP transport",
+            "mcpm run myserver --http --port 8080",
         ],
-        'mcpm profile run': [
-            '# Run all servers in a profile',
-            'mcpm profile run web-dev',
-            '',
-            '# Run profile with custom port',
-            'mcpm profile run web-dev --port 8080 --http',
+        "mcpm profile run": [
+            "# Run all servers in a profile",
+            "mcpm profile run web-dev",
+            "",
+            "# Run profile with custom port",
+            "mcpm profile run web-dev --port 8080 --http",
         ],
     }
-    
+
     # Return examples if we have them for this command
     if cmd in example_map:
         return example_map[cmd]
-    
+
     # Generate basic example if no specific examples
-    if cmd_info['params']:
-        return [f"# Basic usage", f"{cmd} <arguments>"]
-    
+    if cmd_info["params"]:
+        return ["# Basic usage", f"{cmd} <arguments>"]
+
     return []
 
 
@@ -241,15 +241,15 @@ def generate_llm_txt():
         "## Command Reference",
         "",
     ]
-    
+
     # Extract command structure
     cmd_info = extract_command_info(mcpm_cli)
-    
+
     # Format main commands
-    for subcmd_name in sorted(cmd_info['subcommands'].keys()):
-        subcmd_info = cmd_info['subcommands'][subcmd_name]
+    for subcmd_name in sorted(cmd_info["subcommands"].keys()):
+        subcmd_info = cmd_info["subcommands"][subcmd_name]
         lines.extend(format_command_section(subcmd_info))
-    
+
     # Add best practices section
     lines.extend([
         "## Best Practices for AI Agents",
@@ -318,10 +318,10 @@ def generate_llm_txt():
         "",
         "```bash",
         "# Add multiple servers at once",
-        "mcpm profile edit myprofile --add-server \"server1,server2,server3\"",
+        'mcpm profile edit myprofile --add-server "server1,server2,server3"',
         "",
         "# Remove multiple servers",
-        "mcpm client edit cursor --remove-server \"old1,old2\"",
+        'mcpm client edit cursor --remove-server "old1,old2"',
         "```",
         "",
         "### Using Environment Variables for Secrets",
@@ -374,27 +374,27 @@ def generate_llm_txt():
         "- Use `mcpm doctor` to diagnose system issues",
         "",
     ])
-    
+
     return "\n".join(lines)
 
 
 def main():
     """Generate and save the LLM.txt file."""
     content = generate_llm_txt()
-    
+
     # Determine output path
     script_dir = Path(__file__).parent
     project_root = script_dir.parent
     output_path = project_root / "llm.txt"
-    
+
     # Write the file
-    with open(output_path, 'w', encoding='utf-8') as f:
+    with open(output_path, "w", encoding="utf-8") as f:
         f.write(content)
-    
+
     print(f"✅ Generated llm.txt at: {output_path}")
     print(f"📄 File size: {len(content):,} bytes")
     print(f"📝 Lines: {content.count(chr(10)):,}")
 
 
 if __name__ == "__main__":
-    main()
+    main()