Protokoll #6562

{
    "event": "PreToolUse",
    "tool_name": "mcp__mcp-contracts__contracts_update",
    "tool_input": {
        "id": 11,
        "yaml_content": "architecture_gate_contract:\n  meta:\n    name: architecture_gate_contract\n    version: 1.1.0\n    status: binding\n    scope: every_new_php_file\n\n  purpose:\n    description: >\n      Defines mandatory analysis and validation steps\n      before and after creating any new file.\n\n  hard_constraints:\n    description: >\n      Pre-creation blocking rules. These are enforced by architecture_guard.py\n      and will block file creation if violated.\n    applies_pre_creation: true\n    rules:\n      - id: H1\n        name: strict_types_required\n        pattern: \"declare(strict_types=1)\"\n        must_match: true\n        applies_to: all\n        block: true\n\n      - id: H2\n        name: domain_no_infrastructure\n        pattern: \"use Infrastructure\\\\\"\n        must_match: false\n        applies_to: \"\/Domain\/\"\n        block: true\n\n      - id: H3\n        name: db_factory_only\n        pattern: \"DatabaseFactory::\"\n        must_match: false\n        applies_to_not: \"\/Factory\/\"\n        block: true\n\n      - id: H4\n        name: no_new_repository_in_controller\n        pattern: \"new \\\\w+Repository\\\\(\"\n        must_match: false\n        applies_to: \"\/Controller\/\"\n        block: true\n\n      - id: H5\n        name: no_new_infrastructure_in_controller\n        pattern: \"new Infrastructure\\\\\"\n        must_match: false\n        applies_to: \"\/Controller\/\"\n        block: true\n\n    allowlist:\n      - \"\/Factory\/\"\n      - \"\/Bootstrap\/\"\n      - \"\/tests\/\"\n      - \"\/Test\/\"\n\n  phase_1_pre_creation_analysis:\n    A1_purpose_definition:\n      rule: exactly_one_purpose\n      check: purpose_must_be_expressible_in_one_sentence\n\n    A2_layer_assignment:\n      rule: exactly_one_layer\n      allowed_layers:\n        - domain\n        - application\n        - infrastructure\n        - presentation\n\n    A3_responsibility_boundary:\n      rule: explicit_exclusions_required\n      check: non_responsibilities_must_be_named\n\n    A4_dependency_analysis:\n      rule: dependencies_must_be_known_upfront\n      required:\n        - required_dependencies_listed\n        - forbidden_dependencies_listed\n\n    A5_abstraction_level:\n      rule: single_abstraction_level\n      allowed:\n        - policy\n        - detail\n\n    A6_data_representation:\n      rule: explicit_decision_required\n      allowed:\n        - objects\n        - value_objects\n        - arrays\n\n    A7_instantiation_source:\n      rule: instantiation_must_be_defined\n      allowed:\n        - container\n        - factory\n        - explicit_creator\n\n    A8_extension_strategy:\n      rule: extension_without_modification\n      check: open_closed_principle_addressed\n\n    A9_test_strategy:\n      rule: isolated_testability_required\n      check: test_approach_defined\n\n  phase_2_post_creation_validation:\n    B1_srp:\n      rule: single_responsibility\n      violation_if: more_than_one_reason_to_change\n\n    B2_dry:\n      rule: no_logic_duplication\n      violation_if: duplicated_behavior_detected\n\n    B3_kiss:\n      rule: minimal_complexity\n      violation_if: unnecessary_abstractions_present\n\n    B4_solid:\n      rule: solid_principles_respected\n      checks:\n        - no_new_of_dependencies_in_business_code\n        - no_static_infrastructure_access\n\n    B5_dip:\n      rule: dependency_direction_enforced\n      violation_if: domain_depends_on_infrastructure\n\n    B6_crud_discipline:\n      rule: responsibility_separation\n      mapping:\n        controller: orchestration_only\n        use_case: decision_logic\n        repository: persistence_only\n\n    B7_mvc_consistency:\n      rule: mvc_roles_respected\n      violation_if:\n        - business_logic_in_controller\n        - decision_logic_in_view\n\n    B8_oop_integrity:\n      rule: behavior_and_state_coupled\n      violation_if: anemic_objects_without_reason\n\n    B9_yagni:\n      rule: no_future_speculation\n      violation_if: unused_extension_points\n\n    B10_type_consistency:\n      rule: strict_typing_required\n      checks:\n        - strict_types_enabled\n        - parameters_typed\n        - return_types_defined\n\n    B11_psr_conformance:\n      rule: psr_compliance_required\n      checks:\n        - namespace_matches_path\n        - class_name_matches_file\n\n    B12_dependency_visibility:\n      rule: explicit_dependencies_only\n      check: all_dependencies_in_constructor\n\n    B13_removability:\n      rule: safe_deletion_possible\n      check: no_hidden_side_effects\n\n    B14_readability:\n      rule: fast_comprehension\n      check: purpose_and_boundary_understandable_in_minutes\n\n  warning_indicators:\n    description: >\n      Post-creation warnings. These are enforced by architecture_validator.py\n      and will warn but not block.\n    applies_post_creation: true\n    rules:\n      - id: W1\n        name: class_size\n        check: line_count\n        threshold: 300\n        warn: true\n\n      - id: W2\n        name: constructor_complexity\n        check: parameter_count\n        threshold: 5\n        warn: true\n\n      - id: W3\n        name: sql_in_controller\n        check: sql_keywords\n        applies_to: \"\/Controller\/\"\n        warn: true\n\n      - id: W4\n        name: missing_phpdoc\n        check: public_methods_without_doc\n        warn: true",
        "new_version": "1.1",
        "change_description": "Added hard_constraints section with 5 blocking rules (H1-H5), allowlist for exceptions, and warning_indicators section (W1-W4) based on supervision review"
    }
}

{
    "tool_response": [
        {
            "type": "text",
            "text": "{\n  \"success\": true,\n  \"contract\": {\n    \"id\": 11,\n    \"uuid\": \"00be3e7a-e6a5-4151-b973-39d36b2a0303\",\n    \"name\": \"architecture-gate-contract\",\n    \"version\": \"1.1\",\n    \"status\": \"active\",\n    \"yaml_content\": \"architecture_gate_contract:\\n  meta:\\n    name: architecture_gate_contract\\n    version: 1.1.0\\n    status: binding\\n    scope: every_new_php_file\\n\\n  purpose:\\n    description: >\\n      Defines mandatory analysis and validation steps\\n      before and after creating any new file.\\n\\n  hard_constraints:\\n    description: >\\n      Pre-creation blocking rules. These are enforced by architecture_guard.py\\n      and will block file creation if violated.\\n    applies_pre_creation: true\\n    rules:\\n      - id: H1\\n        name: strict_types_required\\n        pattern: \\\"declare(strict_types=1)\\\"\\n        must_match: true\\n        applies_to: all\\n        block: true\\n\\n      - id: H2\\n        name: domain_no_infrastructure\\n        pattern: \\\"use Infrastructure\\\\\\\\\\\"\\n        must_match: false\\n        applies_to: \\\"\/Domain\/\\\"\\n        block: true\\n\\n      - id: H3\\n        name: db_factory_only\\n        pattern: \\\"DatabaseFactory::\\\"\\n        must_match: false\\n        applies_to_not: \\\"\/Factory\/\\\"\\n        block: true\\n\\n      - id: H4\\n        name: no_new_repository_in_controller\\n        pattern: \\\"new \\\\\\\\w+Repository\\\\\\\\(\\\"\\n        must_match: false\\n        applies_to: \\\"\/Controller\/\\\"\\n        block: true\\n\\n      - id: H5\\n        name: no_new_infrastructure_in_controller\\n        pattern: \\\"new Infrastructure\\\\\\\\\\\"\\n        must_match: false\\n        applies_to: \\\"\/Controller\/\\\"\\n        block: true\\n\\n    allowlist:\\n      - \\\"\/Factory\/\\\"\\n      - \\\"\/Bootstrap\/\\\"\\n      - \\\"\/tests\/\\\"\\n      - \\\"\/Test\/\\\"\\n\\n  phase_1_pre_creation_analysis:\\n    A1_purpose_definition:\\n      rule: exactly_one_purpose\\n      check: purpose_must_be_expressible_in_one_sentence\\n\\n    A2_layer_assignment:\\n      rule: exactly_one_layer\\n      allowed_layers:\\n        - domain\\n        - application\\n        - infrastructure\\n        - presentation\\n\\n    A3_responsibility_boundary:\\n      rule: explicit_exclusions_required\\n      check: non_responsibilities_must_be_named\\n\\n    A4_dependency_analysis:\\n      rule: dependencies_must_be_known_upfront\\n      required:\\n        - required_dependencies_listed\\n        - forbidden_dependencies_listed\\n\\n    A5_abstraction_level:\\n      rule: single_abstraction_level\\n      allowed:\\n        - policy\\n        - detail\\n\\n    A6_data_representation:\\n      rule: explicit_decision_required\\n      allowed:\\n        - objects\\n        - value_objects\\n        - arrays\\n\\n    A7_instantiation_source:\\n      rule: instantiation_must_be_defined\\n      allowed:\\n        - container\\n        - factory\\n        - explicit_creator\\n\\n    A8_extension_strategy:\\n      rule: extension_without_modification\\n      check: open_closed_principle_addressed\\n\\n    A9_test_strategy:\\n      rule: isolated_testability_required\\n      check: test_approach_defined\\n\\n  phase_2_post_creation_validation:\\n    B1_srp:\\n      rule: single_responsibility\\n      violation_if: more_than_one_reason_to_change\\n\\n    B2_dry:\\n      rule: no_logic_duplication\\n      violation_if: duplicated_behavior_detected\\n\\n    B3_kiss:\\n      rule: minimal_complexity\\n      violation_if: unnecessary_abstractions_present\\n\\n    B4_solid:\\n      rule: solid_principles_respected\\n      checks:\\n        - no_new_of_dependencies_in_business_code\\n        - no_static_infrastructure_access\\n\\n    B5_dip:\\n      rule: dependency_direction_enforced\\n      violation_if: domain_depends_on_infrastructure\\n\\n    B6_crud_discipline:\\n      rule: responsibility_separation\\n      mapping:\\n        controller: orchestration_only\\n        use_case: decision_logic\\n        repository: persistence_only\\n\\n    B7_mvc_consistency:\\n      rule: mvc_roles_respected\\n      violation_if:\\n        - business_logic_in_controller\\n        - decision_logic_in_view\\n\\n    B8_oop_integrity:\\n      rule: behavior_and_state_coupled\\n      violation_if: anemic_objects_without_reason\\n\\n    B9_yagni:\\n      rule: no_future_speculation\\n      violation_if: unused_extension_points\\n\\n    B10_type_consistency:\\n      rule: strict_typing_required\\n      checks:\\n        - strict_types_enabled\\n        - parameters_typed\\n        - return_types_defined\\n\\n    B11_psr_conformance:\\n      rule: psr_compliance_required\\n      checks:\\n        - namespace_matches_path\\n        - class_name_matches_file\\n\\n    B12_dependency_visibility:\\n      rule: explicit_dependencies_only\\n      check: all_dependencies_in_constructor\\n\\n    B13_removability:\\n      rule: safe_deletion_possible\\n      check: no_hidden_side_effects\\n\\n    B14_readability:\\n      rule: fast_comprehension\\n      check: purpose_and_boundary_understandable_in_minutes\\n\\n  warning_indicators:\\n    description: >\\n      Post-creation warnings. These are enforced by architecture_validator.py\\n      and will warn but not block.\\n    applies_post_creation: true\\n    rules:\\n      - id: W1\\n        name: class_size\\n        check: line_count\\n        threshold: 300\\n        warn: true\\n\\n      - id: W2\\n        name: constructor_complexity\\n        check: parameter_count\\n        threshold: 5\\n        warn: true\\n\\n      - id: W3\\n        name: sql_in_controller\\n        check: sql_keywords\\n        applies_to: \\\"\/Controller\/\\\"\\n        warn: true\\n\\n      - id: W4\\n        name: missing_phpdoc\\n        check: public_methods_without_doc\\n        warn: true\",\n    \"scope_description\": \"Mandatory analysis and validation for every new PHP file\",\n    \"created_at\": \"2025-12-23T06:09:03\",\n    \"created_by\": \"supervision\",\n    \"updated_at\": \"2025-12-23T06:17:12\"\n  },\n  \"message\": \"Contract updated to v1.1\"\n}"
        }
    ]
}