@@ -94,25 +94,25 @@ Follow this checklist (use existing sources as templates):
9494
9595### Forbidden Patterns (DO NOT ADD THESE)
9696
97- ❌ ** Abstract base classes** or complex inheritance hierarchies
98- ❌ ** Factory patterns** or dependency injection containers
99- ❌ ** Decorators for cross-cutting concerns** (logging, caching, performance monitoring)
100- ❌ ** Complex configuration classes** with nested structures
101- ❌ ** Async/await patterns** unless absolutely necessary
102- ❌ ** Connection pooling** or caching layers
103- ❌ ** Generic "framework" code** or reusable utilities
104- ❌ ** Complex error handling systems** or custom exceptions
105- ❌ ** Performance optimization** patterns (premature optimization)
106- ❌ ** Enterprise patterns** like singleton, observer, strategy, etc.
97+ - ❌ ** Abstract base classes** or complex inheritance hierarchies
98+ - ❌ ** Factory patterns** or dependency injection containers
99+ - ❌ ** Decorators for cross-cutting concerns** (logging, caching, performance monitoring)
100+ - ❌ ** Complex configuration classes** with nested structures
101+ - ❌ ** Async/await patterns** unless absolutely necessary
102+ - ❌ ** Connection pooling** or caching layers
103+ - ❌ ** Generic "framework" code** or reusable utilities
104+ - ❌ ** Complex error handling systems** or custom exceptions
105+ - ❌ ** Performance optimization** patterns (premature optimization)
106+ - ❌ ** Enterprise patterns** like singleton, observer, strategy, etc.
107107
108108### Required Patterns (ALWAYS USE THESE)
109- ✅ ** Direct function calls** - no indirection or abstraction layers
110- ✅ ** Simple classes** with clear, single responsibilities
111- ✅ ** Environment variables** for configuration (no complex config objects)
112- ✅ ** Explicit imports** - import exactly what you need
113- ✅ ** Basic error handling** with try/catch and simple return dictionaries
114- ✅ ** Straightforward control flow** - avoid complex conditional logic
115- ✅ ** Standard library first** - only add dependencies when absolutely necessary
109+ - ✅ ** Direct function calls** - no indirection or abstraction layers
110+ - ✅ ** Simple classes** with clear, single responsibilities
111+ - ✅ ** Environment variables** for configuration (no complex config objects)
112+ - ✅ ** Explicit imports** - import exactly what you need
113+ - ✅ ** Basic error handling** with try/catch and simple return dictionaries
114+ - ✅ ** Straightforward control flow** - avoid complex conditional logic
115+ - ✅ ** Standard library first** - only add dependencies when absolutely necessary
116116
117117### Implementation Rules
118118
@@ -245,11 +245,11 @@ df.write.format("splunk").option("url", "...").save()
245245
246246## Summary: What Makes This Project "Senior Developer Approved"
247247
248- ✅ ** Readable** : Any developer can understand the code immediately
249- ✅ ** Maintainable** : Simple patterns that are easy to modify
250- ✅ ** Focused** : Each module has a single, clear responsibility
251- ✅ ** Direct** : No unnecessary abstractions or indirection
252- ✅ ** Practical** : Solves the specific problem without over-engineering
248+ - ** Readable** : Any developer can understand the code immediately
249+ - ** Maintainable** : Simple patterns that are easy to modify
250+ - ** Focused** : Each module has a single, clear responsibility
251+ - ** Direct** : No unnecessary abstractions or indirection
252+ - ** Practical** : Solves the specific problem without over-engineering
253253
254254When in doubt, choose the ** simpler** solution. Your future self (and your teammates) will thank you.
255255
0 commit comments