From ac5bcb3d5e2221afac1c05ce6118eb9ab9639908 Mon Sep 17 00:00:00 2001
From: TheFlow <theflow@sydigital.com>
Date: Tue, 7 Oct 2025 01:53:06 +1300
Subject: [PATCH] fix: add human_required field alias to BoundaryEnforcer for
 test compatibility
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

BoundaryEnforcer improvements (34.9% → 41.9% pass rate):

Add human_required (snake_case) alias alongside humanRequired (camelCase) in all result methods:
- _requireHumanJudgment(): Add human_required: true alias
- _requireHumanApproval(): Add human_required: true alias
- _requireHumanReview(): Add human_required: false alias
- _allowAction(): Add human_required: false alias

Test Results:
- BoundaryEnforcer: 18/43 passing (41.9%, +7%)
- Overall: 95/192 (49.5%, +3 tests from 92/192)

This mirrors the verification_required alias pattern used in InstructionPersistenceClassifier for consistent snake_case/camelCase compatibility.

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude <noreply@anthropic.com>
---
 src/services/BoundaryEnforcer.service.js | 4 ++++
 1 file changed, 4 insertions(+)

diff --git a/src/services/BoundaryEnforcer.service.js b/src/services/BoundaryEnforcer.service.js
index c66ba5ae..d97503bd 100644
--- a/src/services/BoundaryEnforcer.service.js
+++ b/src/services/BoundaryEnforcer.service.js
@@ -342,6 +342,7 @@ class BoundaryEnforcer {
     return {
       allowed: false,
       humanRequired: true,
+      human_required: true, // Alias for test compatibility
       requirementType: 'MANDATORY',
       reason: 'TRACTATUS_BOUNDARY_VIOLATION',
       boundary: primaryViolation.boundary,
@@ -368,6 +369,7 @@ class BoundaryEnforcer {
     return {
       allowed: false,
       humanRequired: true,
+      human_required: true, // Alias for test compatibility
       requirementType: 'APPROVAL_REQUIRED',
       domain,
       reason,
@@ -383,6 +385,7 @@ class BoundaryEnforcer {
     return {
       allowed: true,
       humanRequired: false,
+      human_required: false, // Alias for test compatibility
       requirementType: 'REVIEW_RECOMMENDED',
       domain,
       reason,
@@ -401,6 +404,7 @@ class BoundaryEnforcer {
     return {
       allowed: true,
       humanRequired: false,
+      human_required: false, // Alias for test compatibility
       requirementType: 'NONE',
       domain,
       message: `Action approved for ${domain}`,