The parameter-plumbing section is the sharpest diagnosis in here, because it names exactly why "stateless" gets weaponized as a defense. Passing includeRawDiagnostics through three methods that don't use it isn't statelessness, it's state with the ownership deliberately erased, so nobody has to admit a concept exists. The DeploymentRequest record fix isn't "adding complexity back," it's just naming the thing that was always there and forcing it to validate itself once instead of trusting every caller to assemble it correctly by hand.
The AI-generated JavaDoc bit is the funniest and truest part, and it generalizes past Java entirely, documentation that restates a method signature in prose ("calculates the total by calculating the total") gives the illusion that a design review happened without touching the actual problem, which is that there was never an architectural boundary to document in the first place. Nine thousand lines of grammatically correct filler is strictly worse than four thousand lines of undocumented mess, because it now looks reviewed.
"Static mocking exists, so do chainsaws" is the line I'll be stealing. The FraudAnalyzer interface example makes the actual point clearly, needing bytecode tricks to fake a collaborator isn't a testing sophistication problem, it's the codebase confessing that a dependency got hidden instead of designed, and the fix is never more mocking machinery, it's an interface.
The parameter-plumbing section is the sharpest diagnosis in here, because it names exactly why "stateless" gets weaponized as a defense. Passing includeRawDiagnostics through three methods that don't use it isn't statelessness, it's state with the ownership deliberately erased, so nobody has to admit a concept exists. The DeploymentRequest record fix isn't "adding complexity back," it's just naming the thing that was always there and forcing it to validate itself once instead of trusting every caller to assemble it correctly by hand.
The AI-generated JavaDoc bit is the funniest and truest part, and it generalizes past Java entirely, documentation that restates a method signature in prose ("calculates the total by calculating the total") gives the illusion that a design review happened without touching the actual problem, which is that there was never an architectural boundary to document in the first place. Nine thousand lines of grammatically correct filler is strictly worse than four thousand lines of undocumented mess, because it now looks reviewed.
"Static mocking exists, so do chainsaws" is the line I'll be stealing. The FraudAnalyzer interface example makes the actual point clearly, needing bytecode tricks to fake a collaborator isn't a testing sophistication problem, it's the codebase confessing that a dependency got hidden instead of designed, and the fix is never more mocking machinery, it's an interface.