Cursor rule

1---
2description: Common mistakes to avoid and best practices to follow
3globs: ["src/**/*.py"]
4alwaysApply: false
5---
6 
7# Common Mistakes to Avoid
8 
9## ❌ DON'T
10 
11- Use synchronous blocking calls in async functions
12- Omit type hints or use `Any` unnecessarily
13- Skip tests for new features
14- Forget `confirm=True` safety checks on mutating operations
15- Expose sensitive data in logs
16- Commit without running pre-commit hooks
17- Mix sync and async code improperly
18- Return untyped dicts when Pydantic models are appropriate
19- Ignore project guidelines (`CLAUDE.md`, `AGENTS.md`) when editing command/agent assets or MCP tool instructions
20 
21## ✅ DO
22 
23- Use async/await consistently
24- Provide comprehensive type hints
25- Write tests first (TDD)
26- Implement dry-run mode for dangerous operations
27- Mask passwords and keys in logs
28- Run full test suite before committing
29- Use `async with` for resource management
30- Define Pydantic models for complex data structures
31- Follow project guidelines in `CLAUDE.md` and `AGENTS.md` for consistent development practices
32 
33## Quick Commands
34 
35```bash
36# Development
37uv run mcp dev src/main.py      # Run with MCP Inspector
38 
39# Testing
40pytest                          # Run all tests
41pytest -m unit                  # Unit tests only
42pytest -m integration           # Integration tests only
43pytest --cov=src --cov-report=html  # With coverage report
44 
45# Code Quality
46black src/ tests/               # Format
47isort src/ tests/               # Sort imports
48ruff check src/ tests/ --fix   # Lint and fix
49mypy src/                       # Type check
50bandit -r src/                  # Security scan
51 
52# Pre-commit
53pre-commit run --all-files      # Run all hooks
54```
55 
56## Key Resources
57 
58- **Project Documentation**: README.md, API.md, CONTRIBUTING.md, SECURITY.md
59- **MCP Specification**: https://modelcontextprotocol.io/
60- **FastMCP Docs**: https://gofastmcp.com/
61- **UniFi API**: Official UniFi Controller API documentation
62- **Pydantic**: https://docs.pydantic.dev/
63 
64## Remember
65 
66- **Safety First**: Require `confirm=True` for all network modifications
67- **Test Everything**: 80%+ coverage is not optional
68- **Type Everything**: Type hints improve code quality and catch bugs
69- **Document Well**: Future you (and other developers) will thank you
70- **Async Always**: This is an async-first project - respect the pattern
71- **Security Matters**: Never expose credentials or sensitive data
72