{
"event": "PreToolUse",
"tool_name": "Bash",
"tool_input": {
"command": "python3 quality\/phpmetrics_check.py --refresh 2>&1",
"timeout": 120000,
"description": "Test W13.x Cohesion via PHPMetrics"
}
}
{
"tool_response": {
"stdout": "PHPMetrics Analysis: 291 classes\nClasses with warnings: 45\nTotal warnings: 56\n\nWarnings:\n - W13.6: Controller\\Api\\TaskController has high overall complexity (WMC=65). Class is doing too much - consider splitting.\n - W13.6: Controller\\ContentPipelineController has high overall complexity (WMC=54). Class is doing too much - consider splitting.\n - W13.5: Controller\\ChatController has high method complexity (CCN=24). Consider extracting methods or simplifying logic.\n - W13.6: Controller\\ChatController has high overall complexity (WMC=75). Class is doing too much - consider splitting.\n - W13.5: Controller\\CodeAnalysisController has high method complexity (CCN=17). Consider extracting methods or simplifying logic.\n - W13.6: Controller\\ContentController has high overall complexity (WMC=70). Class is doing too much - consider splitting.\n - W13.5: UseCases\\Chat\\StreamingChatMessageUseCase has high method complexity (CCN=31). Consider extracting methods or simplifying logic.\n - W13.6: UseCases\\Chat\\StreamingChatMessageUseCase has high overall complexity (WMC=108). Class is doing too much - consider splitting.\n - W13.5: UseCases\\Chat\\SendChatMessageUseCase has high method complexity (CCN=21). Consider extracting methods or simplifying logic.\n - W13.6: UseCases\\Chat\\SendChatMessageUseCase has high overall complexity (WMC=71). Class is doing too much - consider splitting.\n - W13.1: Framework\\CsrfService has low cohesion (LCOM=5). Consider splitting into focused classes.\n - W13.1: Framework\\Controller has low cohesion (LCOM=11). Consider splitting into focused classes.\n - W13.2: Framework\\Controller has high afferent coupling (26 dependents). Changes here affect many classes - consider interface.\n - W13.4: Framework\\Controller is very stable (I=0.10) with many dependents. Changes are risky - ensure good test coverage.\n - W13.6: Framework\\Controller has high overall complexity (WMC=52). Class is doing too much - consider splitting.\n - W13.3: ServiceProvider\\ChatServiceProvider has high efferent coupling (22 dependencies). Consider dependency injection or facade pattern.\n - W13.3: ServiceProvider\\InfrastructureServiceProvider has high efferent coupling (30 dependencies). Consider dependency injection or facade pattern.\n - W13.3: ServiceProvider\\ContentServiceProvider has high efferent coupling (24 dependencies). Consider dependency injection or facade pattern.\n - W13.4: Domain\\Repository\\TaskRepositoryInterface is very stable (I=0.09) with many dependents. Changes are risky - ensure good test coverage.\n - W13.4: Domain\\Repository\\ChatMessageRepositoryInterface is very stable (I=0.13) with many dependents. Changes are risky - ensure good test coverage.\n ... and 36 more",
"stderr": "",
"interrupted": false,
"isImage": false
}
}