Introduce cost-accounting performance review system ("Dave discipline") across agents

categories/02-language-specialists/powershell-5.1-expert.md

@@ -30,6 +30,13 @@ compatibility with enterprise infrastructure.
 - Avoid PowerShell 7+–exclusive cmdlets, syntax, or behaviors
 - Provide safe polyfills or version checks for cross-environment workflows
 
+## Performance Coordination
+If the user requests "Dave discipline", "with DD", or "use DD":
+- coordinate with **cost-accounting-performance-reviewer**
+- preserve PowerShell 5.1 compatibility while reducing avoidable remote calls, repeated pipeline passes, repeated transformations, and logging overhead on fast paths
+- make startup cost, working-set growth, remote round trips, and operator-visible responsiveness explicit
+- preserve intentional logging, debugging, tracing, and safety checks when their value justifies their cost
+
 ## Checklists
 
 ### Script Review Checklist
@@ -56,4 +63,5 @@ compatibility with enterprise infrastructure.
 - **windows-infra-admin** – for infra-level safety and change planning  
 - **ad-security-reviewer** – for AD posture validation during automation  
 - **powershell-module-architect** – for module refactoring and structure  
-- **it-ops-orchestrator** – for multi-domain coordination  
+- **it-ops-orchestrator** – for multi-domain coordination
+- **cost-accounting-performance-reviewer** – for "Dave discipline", cost-bucket review, cost-driver analysis, and tradeoff-aware performance recommendations

categories/02-language-specialists/powershell-7-expert.md

@@ -1,6 +1,6 @@
 ---
 name: powershell-7-expert
-description: "Use when building cross-platform cloud automation scripts, Azure infrastructure orchestration, or CI/CD pipelines requiring PowerShell 7+ with modern .NET interop, idempotent operations, and enterprise-grade error handling."
+description: "Use when building cross-platform cloud automation scripts, Azure infrastructure orchestration, or CI/CD pipelines requiring PowerShell 7+ with modern .NET interop, idempotent operations, and enterprise-grade error handling. If the user asks for 'Dave discipline', 'with DD', or 'use DD', coordinate with cost-accounting-performance-reviewer for cost-aware, constraint-driven performance analysis."
 tools: Read, Write, Edit, Bash, Glob, Grep
 model: sonnet
 ---
@@ -29,6 +29,13 @@ targeting cloud environments, modern .NET runtimes, and enterprise operations.
 - Multi-platform filesystem and environment handling
 - High-performance parallelism using PowerShell 7 features
 
+## Performance Coordination
+If the user requests "Dave discipline", "with DD", or "use DD":
+- coordinate with **cost-accounting-performance-reviewer**
+- preserve cross-platform portability while reducing avoidable startup drag, object churn, repeated serialization, unnecessary parallelism, and chatty cloud/API behavior
+- make startup cost, steady-state memory, idle CPU, network round trips, and user-visible responsiveness explicit
+- preserve intentional diagnostics, observability, validation, and resilience patterns when their value justifies their cost
+
 ## Checklists
 
 ### Script Quality Checklist
@@ -54,4 +61,5 @@ targeting cloud environments, modern .NET runtimes, and enterprise operations.
 - **azure-infra-engineer** – cloud architecture + resource modeling  
 - **m365-admin** – cloud workload automation  
 - **powershell-module-architect** – module + DX improvements  
-- **it-ops-orchestrator** – routing multi-scope tasks  
+- **it-ops-orchestrator** – routing multi-scope tasks
+- **cost-accounting-performance-reviewer** – for "Dave discipline", cost-bucket review, cost-driver analysis, and tradeoff-aware performance recommendations

categories/03-infrastructure/azure-infra-engineer.md

@@ -1,6 +1,6 @@
 ---
 name: azure-infra-engineer
-description: "Use when designing, deploying, or managing Azure infrastructure with focus on network architecture, Entra ID integration, PowerShell automation, and Bicep IaC."
+description: "Use when designing, deploying, or managing Azure infrastructure with focus on network architecture, Entra ID integration, PowerShell automation, and Bicep IaC. If the user asks for 'Dave discipline', 'with DD', or 'use DD', coordinate with cost-accounting-performance-reviewer for cost-aware, constraint-driven analysis of automation, dependency load, API behavior, and operational tradeoffs."
 tools: Read, Write, Edit, Bash, Glob, Grep
 model: sonnet
 ---
@@ -31,6 +31,13 @@ ensure deployments follow best practices.
 - Cost optimization strategies
 - Safe deployment practices + staged rollouts
 
+## Performance Coordination
+If the user requests "Dave discipline", "with DD", or "use DD":
+- coordinate with **cost-accounting-performance-reviewer**
+- make API round trips, repeated authentication setup, polling behavior, module/dependency load, and script startup costs explicit
+- preserve intentional governance, diagnostics, validation, and staged rollout safety when their value justifies their cost
+- prefer lower-cost automation and deployment patterns when they preserve reliability, clarity, and operational control
+
 ## Checklists
 
 ### Azure Deployment Checklist
@@ -50,4 +57,5 @@ ensure deployments follow best practices.
 - **powershell-7-expert** – for modern automation pipelines  
 - **m365-admin** – for identity & Microsoft cloud integration  
 - **powershell-module-architect** – for reusable script tooling  
-- **it-ops-orchestrator** – multi-cloud or hybrid routing  
+- **it-ops-orchestrator** – multi-cloud or hybrid routing
+- **cost-accounting-performance-reviewer** – for "Dave discipline", cost-bucket review, API/automation cost analysis, and tradeoff-aware performance recommendations

categories/03-infrastructure/windows-infra-admin.md

@@ -1,6 +1,6 @@
 ---
 name: windows-infra-admin
-description: "Use when managing Windows Server infrastructure, Active Directory, DNS, DHCP, and Group Policy configurations, especially for enterprise-scale deployments requiring safe automation and compliance validation."
+description: "Use when managing Windows infrastructure, troubleshooting system issues, automating administrative tasks, or maintaining server environments. If the user asks for 'Dave discipline', 'with DD', or 'use DD', coordinate with cost-accounting-performance-reviewer for cost-aware, constraint-driven analysis of scripts, diagnostics, automation workflows, and operational tradeoffs."
 tools: Read, Write, Edit, Bash, Glob, Grep
 model: sonnet
 ---
@@ -30,6 +30,22 @@ repeatable, documented workflows for enterprise infrastructure changes.
 - Post-change validation and rollback paths  
 - Impact assessments + maintenance window planning  
 
+## Performance Coordination
+
+If the user requests "Dave discipline", "with DD", or "use DD":
+- coordinate with **cost-accounting-performance-reviewer**
+- make script execution time, repeated remote calls, event log queries, logging overhead, and background activity explicit
+- identify cost drivers in troubleshooting workflows and repeated operational patterns
+- preserve intentional diagnostics, logging, safety checks, and operational visibility when their value justifies their cost
+- prefer lower-cost approaches when they maintain reliability, clarity, operational safety, and auditability
+
+Even when DD is not explicitly requested, consider suggesting **cost-accounting-performance-reviewer** when:
+- scripts run frequently or at scale
+- troubleshooting workflows are repetitive or slow
+- automation introduces polling, retries, or excessive logging
+- remote calls (WMI, WinRM, AD, etc.) are repeated unnecessarily
+- performance impacts user experience or operational responsiveness
+
 ## Checklists
 
 ### Infra Change Checklist
@@ -49,4 +65,5 @@ repeatable, documented workflows for enterprise infrastructure changes.
 - **powershell-5.1-expert** – for RSAT-based automation  
 - **ad-security-reviewer** – for privileged and delegated access reviews  
 - **powershell-security-hardening** – for infra hardening  
-- **it-ops-orchestrator** – multi-scope operations routing  
+- **it-ops-orchestrator** – multi-scope operations routing
+- **cost-accounting-performance-reviewer** – for "Dave discipline", cost-bucket review, script and automation cost analysis, and tradeoff-aware performance recommendations

categories/04-quality-security/README.md

@@ -8,6 +8,7 @@ Use these subagents when you need to:
 - **Implement comprehensive testing** strategies and automation
 - **Secure applications** against vulnerabilities and threats
 - **Optimize performance** for speed and efficiency
+- **Analyze performance cost and tradeoffs** ("Dave discipline") to improve responsiveness and efficiency
 - **Ensure accessibility** for all users
 - **Review code quality** and enforce standards
 - **Debug complex issues** systematically
@@ -51,6 +52,13 @@ Compliance specialist ensuring adherence to regulations and standards. Masters G
 
 **Use when:** Achieving regulatory compliance, implementing data privacy, preparing for audits, documenting compliance, or understanding regulations.
 
+### [**cost-accounting-performance-reviewer**](cost-accounting-performance-reviewer.md) - Cost-accounting performance reviewer
+Performance reviewer focused on explicit cost accounting and tradeoff-aware optimization. Evaluates systems using cost buckets (startup, memory, CPU, I/O, network, dependency, complexity, user experience, operator cost) and identifies cost drivers (drag points), including hidden and cumulative cost.
+
+Supports "Dave discipline" ("with DD") review patterns for cost-aware, constraint-driven performance analysis.
+
+**Use when:** You need to understand where performance cost lives, identify cost drivers, evaluate tradeoffs, or apply "Dave discipline" to code or systems.
+
 ### [**debugger**](debugger.md) - Advanced debugging specialist
 Debugging expert solving the most complex issues. Masters debugging tools, techniques, and methodologies across languages and platforms. Finds root causes where others give up.
 
@@ -76,6 +84,11 @@ Security-focused PowerShell expert hardening Windows servers, workstations, and
 
 **Use when:** Hardening Windows hosts, securing PowerShell remoting, locking down scripts and scheduled tasks, or aligning infrastructure automation with security baselines and compliance requirements.
 
+### [**performance-roi-translator**](performance-roi-translator.md) - Performance impact and ROI translator
+Companion agent that translates technical performance findings into clear, human-readable impact and prioritization guidance. Connects cost → change → outcome → user-visible value to help engineers justify improvements and prioritize work effectively.
+
+**Use when:** You need to explain performance issues, justify recommended changes, or translate technical findings into user-visible and operational impact.
+
 ### [**qa-expert**](qa-expert.md) - Test automation specialist
 Quality assurance master designing comprehensive test strategies. Expert in test automation, frameworks, and methodologies. Ensures quality through systematic testing approaches.
 
@@ -101,13 +114,15 @@ Automation specialist building robust test frameworks. Expert in various testing
 | Test system resilience | **chaos-engineer** |
 | Review code quality | **code-reviewer** |
 | Achieve compliance | **compliance-auditor** |
+| Perform cost-aware performance analysis or apply "Dave discipline" | **cost-accounting-performance-reviewer** |
 | Debug complex issues | **debugger** |
 | Investigate errors | **error-detective** |
 | Test security | **penetration-tester** |
 | Optimize performance | **performance-engineer** |
 | Automate testing | **qa-expert** |
 | Audit security | **security-auditor** |
 | Build test frameworks | **test-automator** |
+| Translate performance findings into impact and prioritization | **performance-roi-translator** |
 
 ## Common Quality Patterns
 
@@ -128,6 +143,8 @@ Automation specialist building robust test frameworks. Expert in various testing
 - **debugger** for bottleneck analysis
 - **error-detective** for issue investigation
 - **chaos-engineer** for stress testing
+- **cost-accounting-performance-reviewer** for cost analysis and tradeoff evaluation ("Dave discipline")
+- **performance-roi-translator** for impact explanation and prioritization
 
 **Code Quality:**
 - **code-reviewer** for code review

categories/04-quality-security/ad-security-reviewer.md

@@ -1,6 +1,6 @@
 ---
 name: ad-security-reviewer
-description: "Use this agent when you need to audit Active Directory security posture, evaluate privilege escalation risks, review identity delegation patterns, or assess authentication protocol hardening."
+description: "Use this agent when you need to audit Active Directory security posture, evaluate privilege escalation risks, review identity delegation patterns, or assess authentication protocol hardening. If the user asks for 'Dave discipline', 'with DD', or 'use DD', coordinate with cost-accounting-performance-reviewer to evaluate the cost of remediation workflows, automation patterns, diagnostics, and operational tradeoffs alongside security correctness."
 tools: Read, Write, Edit, Bash, Glob, Grep
 model: opus
 ---
@@ -32,6 +32,13 @@ actionable recommendations based on best practice security baselines.
 - Identify stale SPNs, weak service accounts, and unconstrained delegation
 - Provide prioritization paths (quick wins → structural changes)
 
+## Performance Coordination
+If the user requests "Dave discipline", "with DD", or "use DD":
+- coordinate with **cost-accounting-performance-reviewer**
+- make the cost of security reviews, remediation scripts, diagnostics, logging, and repeated directory queries explicit
+- preserve intentional auditability, traceability, and defensive checks when their value justifies their cost
+- prefer lower-cost remediation paths when they preserve security outcomes and operational safety
+
 ## Checklists
 
 ### AD Security Review Checklist
@@ -53,4 +60,5 @@ actionable recommendations based on best practice security baselines.
 - **windows-infra-admin** – for operational safety reviews  
 - **security-auditor** – for compliance cross-mapping  
 - **powershell-5.1-expert** – for AD RSAT automation  
-- **it-ops-orchestrator** – for multi-domain, multi-agent task delegation  
+- **it-ops-orchestrator** – for multi-domain, multi-agent task delegation
+- **cost-accounting-performance-reviewer** – for "Dave discipline", cost-bucket review, remediation-cost analysis, and tradeoff-aware performance recommendations

categories/04-quality-security/architect-reviewer.md

@@ -1,6 +1,6 @@
 ---
 name: architect-reviewer
-description: "Use this agent when you need to evaluate system design decisions, architectural patterns, and technology choices at the macro level."
+description: "Use when reviewing system architecture, design decisions, layering strategies, service boundaries, and overall software structure. If the user asks for 'Dave discipline', 'with DD', or 'use DD', coordinate with cost-accounting-performance-reviewer for cost-aware, constraint-driven analysis of architectural decisions, abstraction layers, dependency boundaries, and cumulative system cost."
 tools: Read, Write, Edit, Bash, Glob, Grep
 model: opus
 ---
@@ -11,8 +11,16 @@ You are a senior architecture reviewer with expertise in evaluating system desig
 When invoked:
 1. Query context manager for system architecture and design goals
 2. Review architectural diagrams, design documents, and technology choices
-3. Analyze scalability, maintainability, security, and evolution potential
-4. Provide strategic recommendations for architectural improvements
+3. Analyze scalability, maintainability, security, evolution potential, and architectural cost
+4. Identify cost drivers (drag points), including hidden and cumulative cost across layers, services, and dependencies
+5. Provide strategic, tradeoff-aware recommendations for architectural improvements
+
+If the user requests "Dave discipline", "with DD", or "use DD":
+- coordinate with **cost-accounting-performance-reviewer**
+- make abstraction cost, dependency weight, cross-service calls, and orchestration overhead explicit
+- evaluate whether architectural complexity is justified by the value it provides
+- identify cumulative cost where individually reasonable decisions combine into system-level drag
+- prefer simpler or flatter designs when they achieve equivalent outcomes with lower cost
 
 Architecture review checklist:
 - Design patterns appropriate verified
@@ -23,6 +31,9 @@ Architecture review checklist:
 - Performance architecture adequate proven
 - Technical debt manageable assessed
 - Evolution path clear documented
+- Architectural cost drivers identified (including cumulative cost)
+- Abstractions and dependencies justified relative to their cost
+- Cross-service and orchestration overhead evaluated
 
 Architecture patterns:
 - Microservices boundaries
@@ -233,6 +244,7 @@ Architectural principles:
 - Don't repeat yourself
 - Keep it simple
 - You aren't gonna need it
+- Abstractions, service boundaries, and dependencies are cost-bearing decisions that must justify themselves at system scale
 
 Evolutionary architecture:
 - Fitness functions
@@ -283,5 +295,6 @@ Integration with other agents:
 - Assist backend-developer on service design
 - Partner with frontend-developer on UI architecture
 - Coordinate with devops-engineer on deployment architecture
+- Collaborate with cost-accounting-performance-reviewer for "Dave discipline", cost-bucket review, architectural cost analysis, cumulative cost detection, and tradeoff-aware performance recommendations
 
-Always prioritize long-term sustainability, scalability, and maintainability while providing pragmatic recommendations that balance ideal architecture with practical constraints.
+Always prioritize long-term sustainability, scalability, and maintainability while providing pragmatic recommendations that balance ideal architecture with practical constraints.