fix: rewrite openenv.yaml grader fields to Python callable path format

Platform reads openenv.yaml directly and requires:
grader: payops_env.server.graders:EASY001Grader

instead of an inline YAML dict. Created server/graders.py with 30
grader classes (one per task) using a factory pattern. All 30 tasks
in openenv.yaml now have string-format grader path references.

Verified: openenv validate passes, all 30 tasks have valid grader paths.

_rewrite_tasks_yaml.py

@@ -0,0 +1,67 @@
+"""
+Rewrite the tasks section of openenv.yaml to use Python callable grader paths.
+The platform expects: grader: payops_env.server.graders:EASY001Grader
+instead of an inline YAML dict.
+"""
+import sys
+sys.path.insert(0, ".")
+from payops_env.tasks import TASKS
+
+with open("openenv.yaml") as f:
+    content = f.read()
+
+# Build new tasks block — one entry per task with string grader path
+lines = []
+lines.append("# tasks: grader field points to a Python callable (module:Class format)")
+lines.append("tasks:")
+for t in TASKS:
+    class_name = t.task_id.replace("-", "") + "Grader"
+    lines.append(f"  - id: {t.task_id}")
+    lines.append(f"    name: \"{t.task_id}\"")
+    lines.append(f"    difficulty: {t.difficulty}")
+    lines.append(f"    grader: payops_env.server.graders:{class_name}")
+
+new_tasks_block = "\n".join(lines)
+
+# Verify counts
+assert new_tasks_block.count("  - id:") == 30, "Expected 30 task entries"
+assert new_tasks_block.count("    grader: payops_env.server.graders:") == 30, "Expected 30 grader paths"
+print("Block verification: 30 tasks, 30 string grader paths - OK")
+
+# Replace in openenv.yaml: from "# tasks:" comment to just before "# ── Budget"
+old_start = "# tasks: flat list — platform iterates this directly for grader check"
+budget_marker = "\n# ── Budget"
+
+if old_start not in content:
+    print("ERROR: Could not find tasks section start marker")
+    sys.exit(1)
+
+tasks_start_pos = content.index(old_start)
+budget_start_pos = content.index(budget_marker)
+
+new_content = content[:tasks_start_pos] + new_tasks_block + "\n" + content[budget_start_pos:]
+
+with open("openenv.yaml", "w") as f:
+    f.write(new_content)
+
+print("openenv.yaml rewritten successfully")
+
+# Parse and verify
+import yaml
+with open("openenv.yaml") as f:
+    d = yaml.safe_load(f.read())
+
+tasks = d.get("tasks", [])
+print(f"Parsed tasks: type={type(tasks).__name__}, count={len(tasks)}")
+assert isinstance(tasks, list) and len(tasks) == 30
+
+grader_strings = [t for t in tasks if isinstance(t.get("grader"), str)]
+print(f"Tasks with string grader: {len(grader_strings)}/30")
+assert len(grader_strings) == 30, "All graders must be strings"
+
+# Spot-check format
+g = tasks[0]["grader"]
+assert "payops_env.server.graders:" in g, f"Bad grader format: {g}"
+assert g.endswith("Grader"), f"Grader should end with Grader: {g}"
+print(f"First task grader: {g}")
+print("VERIFICATION PASSED")

openenv.yaml

@@ -126,451 +126,128 @@ task_difficulty_weights:
   hard: 1.5
   critical: 2.0
 
-# tasks: flat list — platform iterates this directly for grader check
+# tasks: grader field points to a Python callable (module:Class format)
 tasks:
-  - task_id: EASY-001
+  - id: EASY-001
+    name: "EASY-001"
     difficulty: easy
-    grader:
-      enabled: true
-      type: action_match
-      correct_action: approve
-      partial_credit: {}
-      requires_investigation: []
-      regulatory_action: false
-      key_flags: []
-  - task_id: EASY-002
+    grader: payops_env.server.graders:EASY001Grader
+  - id: EASY-002
+    name: "EASY-002"
     difficulty: easy
-    grader:
-      enabled: true
-      type: action_match
-      correct_action: reject
-      partial_credit:
-        escalate: 0.4
-      requires_investigation: []
-      regulatory_action: false
-      key_flags:
-        - sanctioned_country
-        - unknown_sender
-  - task_id: EASY-003
+    grader: payops_env.server.graders:EASY002Grader
+  - id: EASY-003
+    name: "EASY-003"
     difficulty: easy
-    grader:
-      enabled: true
-      type: action_match
-      correct_action: approve
-      partial_credit: {}
-      requires_investigation: []
-      regulatory_action: false
-      key_flags: []
-  - task_id: EASY-004
+    grader: payops_env.server.graders:EASY003Grader
+  - id: EASY-004
+    name: "EASY-004"
     difficulty: easy
-    grader:
-      enabled: true
-      type: action_match
-      correct_action: flag
-      partial_credit:
-        escalate: 0.6
-        hold: 0.5
-      requires_investigation: []
-      regulatory_action: false
-      key_flags:
-        - velocity_breach
-  - task_id: EASY-005
+    grader: payops_env.server.graders:EASY004Grader
+  - id: EASY-005
+    name: "EASY-005"
     difficulty: easy
-    grader:
-      enabled: true
-      type: action_match
-      correct_action: approve
-      partial_credit: {}
-      requires_investigation: []
-      regulatory_action: false
-      key_flags: []
-  - task_id: EASY-006
+    grader: payops_env.server.graders:EASY005Grader
+  - id: EASY-006
+    name: "EASY-006"
     difficulty: easy
-    grader:
-      enabled: true
-      type: action_match
-      correct_action: flag
-      partial_credit:
-        hold: 0.6
-      requires_investigation: []
-      regulatory_action: false
-      key_flags:
-        - duplicate_transaction
-  - task_id: MED-001
+    grader: payops_env.server.graders:EASY006Grader
+  - id: MED-001
+    name: "MED-001"
     difficulty: medium
-    grader:
-      enabled: true
-      type: action_match
-      correct_action: escalate
-      partial_credit:
-        flag: 0.5
-        hold: 0.4
-      requires_investigation:
-        - inspect
-      regulatory_action: false
-      key_flags:
-        - cross_border
-        - high_value
-  - task_id: MED-002
+    grader: payops_env.server.graders:MED001Grader
+  - id: MED-002
+    name: "MED-002"
     difficulty: medium
-    grader:
-      enabled: true
-      type: action_match
-      correct_action: hold
-      partial_credit:
-        escalate: 0.6
-        flag: 0.4
-      requires_investigation:
-        - verify_kyc
-      regulatory_action: false
-      key_flags:
-        - kyc_expiry_90d
-  - task_id: MED-003
+    grader: payops_env.server.graders:MED002Grader
+  - id: MED-003
+    name: "MED-003"
     difficulty: medium
-    grader:
-      enabled: true
-      type: action_match
-      correct_action: flag
-      partial_credit:
-        hold: 0.5
-        escalate: 0.3
-      requires_investigation: []
-      regulatory_action: false
-      key_flags:
-        - amount_spike
-        - pattern_deviation
-  - task_id: MED-004
+    grader: payops_env.server.graders:MED003Grader
+  - id: MED-004
+    name: "MED-004"
     difficulty: medium
-    grader:
-      enabled: true
-      type: action_match
-      correct_action: flag
-      partial_credit:
-        escalate: 0.5
-        hold: 0.4
-      requires_investigation: []
-      regulatory_action: false
-      key_flags:
-        - crypto_exchange
-  - task_id: MED-005
+    grader: payops_env.server.graders:MED004Grader
+  - id: MED-005
+    name: "MED-005"
     difficulty: medium
-    grader:
-      enabled: true
-      type: action_match
-      correct_action: hold
-      partial_credit:
-        flag: 0.5
-        escalate: 0.4
-      requires_investigation:
-        - verify_kyc
-      regulatory_action: false
-      key_flags:
-        - kyc_expired
-  - task_id: MED-006
+    grader: payops_env.server.graders:MED005Grader
+  - id: MED-006
+    name: "MED-006"
     difficulty: medium
-    grader:
-      enabled: true
-      type: action_match
-      correct_action: escalate
-      partial_credit:
-        flag: 0.4
-        hold: 0.5
-      requires_investigation:
-        - request_docs
-      regulatory_action: false
-      key_flags:
-        - first_time_payee
-        - large_first_transfer
-  - task_id: MED-007
+    grader: payops_env.server.graders:MED006Grader
+  - id: MED-007
+    name: "MED-007"
     difficulty: medium
-    grader:
-      enabled: true
-      type: action_match
-      correct_action: hold
-      partial_credit:
-        escalate: 0.6
-        flag: 0.4
-      requires_investigation:
-        - contact_sender
-        - request_docs
-      regulatory_action: false
-      key_flags:
-        - bank_detail_change
-        - first_time_payee
-  - task_id: MED-008
+    grader: payops_env.server.graders:MED007Grader
+  - id: MED-008
+    name: "MED-008"
     difficulty: medium
-    grader:
-      enabled: true
-      type: action_match
-      correct_action: flag
-      partial_credit:
-        hold: 0.6
-        escalate: 0.4
-      requires_investigation:
-        - verify_kyc
-      regulatory_action: false
-      key_flags:
-        - thin_credit_file
-        - new_account
-  - task_id: HARD-001
+    grader: payops_env.server.graders:MED008Grader
+  - id: HARD-001
+    name: "HARD-001"
     difficulty: hard
-    grader:
-      enabled: true
-      type: action_match
-      correct_action: escalate
-      partial_credit:
-        flag: 0.6
-        hold: 0.5
-        reject: 0.3
-      requires_investigation:
-        - inspect
-      regulatory_action: false
-      key_flags:
-        - solicitor_mule_pattern
-  - task_id: HARD-002
+    grader: payops_env.server.graders:HARD001Grader
+  - id: HARD-002
+    name: "HARD-002"
     difficulty: hard
-    grader:
-      enabled: true
-      type: action_match
-      correct_action: reject
-      partial_credit:
-        escalate: 0.6
-        hold: 0.5
-        flag: 0.3
-      requires_investigation:
-        - contact_sender
-      regulatory_action: false
-      key_flags:
-        - app_scam_indicator
-        - mule_account_pattern
-  - task_id: HARD-003
+    grader: payops_env.server.graders:HARD002Grader
+  - id: HARD-003
+    name: "HARD-003"
     difficulty: hard
-    grader:
-      enabled: true
-      type: action_match
-      correct_action: reject
-      partial_credit:
-        escalate: 0.5
-        flag: 0.2
-      requires_investigation:
-        - inspect
-      regulatory_action: true
-      key_flags:
-        - structuring_pattern
-        - ctr_threshold_avoidance
-  - task_id: HARD-004
+    grader: payops_env.server.graders:HARD003Grader
+  - id: HARD-004
+    name: "HARD-004"
     difficulty: hard
-    grader:
-      enabled: true
-      type: action_match
-      correct_action: approve
-      partial_credit:
-        escalate: 0.5
-        hold: 0.4
-        flag: 0.3
-      requires_investigation:
-        - inspect
-      regulatory_action: false
-      key_flags:
-        - fx_settlement
-  - task_id: HARD-005
+    grader: payops_env.server.graders:HARD004Grader
+  - id: HARD-005
+    name: "HARD-005"
     difficulty: hard
-    grader:
-      enabled: true
-      type: action_match
-      correct_action: escalate
-      partial_credit:
-        flag: 0.5
-        hold: 0.4
-        reject: 0.3
-      requires_investigation:
-        - contact_sender
-        - inspect
-      regulatory_action: false
-      key_flags:
-        - internal_to_personal
-        - unusual_beneficiary
-  - task_id: HARD-006
+    grader: payops_env.server.graders:HARD005Grader
+  - id: HARD-006
+    name: "HARD-006"
     difficulty: hard
-    grader:
-      enabled: true
-      type: action_match
-      correct_action: flag
-      partial_credit:
-        hold: 0.6
-        escalate: 0.5
-      requires_investigation:
-        - inspect
-      regulatory_action: false
-      key_flags:
-        - dormant_receiver
-        - sudden_activity
-  - task_id: HARD-007
+    grader: payops_env.server.graders:HARD006Grader
+  - id: HARD-007
+    name: "HARD-007"
     difficulty: hard
-    grader:
-      enabled: true
-      type: action_match
-      correct_action: reject
-      partial_credit:
-        hold: 0.5
-        escalate: 0.4
-        flag: 0.3
-      requires_investigation:
-        - contact_sender
-        - inspect
-      regulatory_action: false
-      key_flags:
-        - sim_swap_detected
-        - new_withdrawal_address
-  - task_id: HARD-008
+    grader: payops_env.server.graders:HARD007Grader
+  - id: HARD-008
+    name: "HARD-008"
     difficulty: hard
-    grader:
-      enabled: true
-      type: action_match
-      correct_action: reject
-      partial_credit:
-        hold: 0.5
-        escalate: 0.4
-        flag: 0.3
-      requires_investigation:
-        - contact_sender
-        - inspect
-      regulatory_action: false
-      key_flags:
-        - romance_scam_indicator
-        - escalating_transfers
-  - task_id: HARD-009
+    grader: payops_env.server.graders:HARD008Grader
+  - id: HARD-009
+    name: "HARD-009"
     difficulty: hard
-    grader:
-      enabled: true
-      type: action_match
-      correct_action: escalate
-      partial_credit:
-        hold: 0.5
-        flag: 0.4
-        reject: 0.3
-      requires_investigation:
-        - inspect
-        - verify_kyc
-      regulatory_action: false
-      key_flags:
-        - new_account_10d
-        - perfect_score_anomaly
-  - task_id: HARD-010
+    grader: payops_env.server.graders:HARD009Grader
+  - id: HARD-010
+    name: "HARD-010"
     difficulty: hard
-    grader:
-      enabled: true
-      type: action_match
-      correct_action: reject
-      partial_credit:
-        hold: 0.5
-        escalate: 0.4
-        flag: 0.3
-      requires_investigation:
-        - contact_sender
-        - inspect
-      regulatory_action: false
-      key_flags:
-        - direct_deposit_changed
-        - payroll_diversion_indicator
-  - task_id: CRIT-001
+    grader: payops_env.server.graders:HARD010Grader
+  - id: CRIT-001
+    name: "CRIT-001"
     difficulty: critical
-    grader:
-      enabled: true
-      type: action_match
-      correct_action: approve
-      partial_credit:
-        escalate: 0.5
-        hold: 0.4
-        flag: 0.3
-      requires_investigation:
-        - inspect
-        - request_docs
-      regulatory_action: false
-      key_flags:
-        - first_time_payee
-        - new_counterparty
-  - task_id: CRIT-002
+    grader: payops_env.server.graders:CRIT001Grader
+  - id: CRIT-002
+    name: "CRIT-002"
     difficulty: critical
-    grader:
-      enabled: true
-      type: action_match
-      correct_action: reject
-      partial_credit:
-        escalate: 0.4
-        flag: 0.2
-      requires_investigation:
-        - inspect
-      regulatory_action: true
-      key_flags:
-        - fraud_ring_indicator
-        - coordinated_transfers
-  - task_id: CRIT-003
+    grader: payops_env.server.graders:CRIT002Grader
+  - id: CRIT-003
+    name: "CRIT-003"
     difficulty: critical
-    grader:
-      enabled: true
-      type: action_match
-      correct_action: escalate
-      partial_credit:
-        flag: 0.4
-        hold: 0.35
-        reject: 0.3
-      requires_investigation:
-        - inspect
-        - request_docs
-      regulatory_action: true
-      key_flags:
-        - invoice_mismatch
-        - trade_finance
-  - task_id: CRIT-004
+    grader: payops_env.server.graders:CRIT003Grader
+  - id: CRIT-004
+    name: "CRIT-004"
     difficulty: critical
-    grader:
-      enabled: true
-      type: action_match
-      correct_action: reject
-      partial_credit:
-        escalate: 0.5
-        hold: 0.4
-      requires_investigation:
-        - contact_sender
-        - inspect
-      regulatory_action: false
-      key_flags:
-        - geo_impossible_login
-        - account_takeover_indicator
-  - task_id: CRIT-005
+    grader: payops_env.server.graders:CRIT004Grader
+  - id: CRIT-005
+    name: "CRIT-005"
     difficulty: critical
-    grader:
-      enabled: true
-      type: action_match
-      correct_action: reject
-      partial_credit:
-        escalate: 0.5
-        hold: 0.4
-      requires_investigation:
-        - inspect
-        - request_docs
-        - verify_kyc
-      regulatory_action: true
-      key_flags:
-        - shell_company_indicator
-        - sanctions_adjacent
-  - task_id: CRIT-006
+    grader: payops_env.server.graders:CRIT005Grader
+  - id: CRIT-006
+    name: "CRIT-006"
     difficulty: critical
-    grader:
-      enabled: true
-      type: action_match
-      correct_action: escalate
-      partial_credit:
-        hold: 0.5
-        reject: 0.4
-      requires_investigation:
-        - inspect
-        - request_docs
-      regulatory_action: true
-      key_flags:
-        - fincen_311_alert
-        - correspondent_risk
+    grader: payops_env.server.graders:CRIT006Grader
 
 # ── Budget ───────────────────────────────────────────────────────────────────
 budget:

server/graders.py

@@ -0,0 +1,80 @@
+"""
+Grader callables for PayOps OpenEnv tasks.
+
+Each class is referenced from openenv.yaml as:
+    payops_env.server.graders:EASY001Grader
+
+The platform reads the grader field from openenv.yaml, imports the class,
+and uses it to grade agent actions.  Each grader is a callable that accepts
+an action string and returns a float reward in [0.0, 1.0].
+"""
+from __future__ import annotations
+
+
+class _BaseGrader:
+    """Base grader: correct action → 1.0, partial credit if configured, else 0.0."""
+
+    task_id: str = ""
+
+    def grade(self, action: str, **kwargs) -> float:
+        from payops_env.tasks import TASKS_BY_ID  # lazy import avoids circular deps
+
+        t = TASKS_BY_ID.get(self.task_id)
+        if t is None:
+            return 0.0
+        if action == t.correct_action:
+            return 1.0
+        return float(
+            dict(getattr(t, "partial_credit_actions", {})).get(action, 0.0)
+        )
+
+    def __call__(self, action: str, **kwargs) -> float:
+        return self.grade(action, **kwargs)
+
+
+def _make(task_id: str) -> type:
+    """Factory: create a named grader class for the given task_id."""
+    return type(
+        task_id.replace("-", "") + "Grader",
+        (_BaseGrader,),
+        {"task_id": task_id},
+    )
+
+
+# ── Easy ─────────────────────────────────────────────────────────────────────
+EASY001Grader = _make("EASY-001")  # approve
+EASY002Grader = _make("EASY-002")  # reject
+EASY003Grader = _make("EASY-003")  # approve
+EASY004Grader = _make("EASY-004")  # flag
+EASY005Grader = _make("EASY-005")  # approve
+EASY006Grader = _make("EASY-006")  # flag
+
+# ── Medium ────────────────────────────────────────────────────────────────────
+MED001Grader = _make("MED-001")   # escalate
+MED002Grader = _make("MED-002")   # hold
+MED003Grader = _make("MED-003")   # flag
+MED004Grader = _make("MED-004")   # flag
+MED005Grader = _make("MED-005")   # hold
+MED006Grader = _make("MED-006")   # escalate
+MED007Grader = _make("MED-007")   # hold
+MED008Grader = _make("MED-008")   # flag
+
+# ── Hard ──────────────────────────────────────────────────────────────────────
+HARD001Grader = _make("HARD-001")  # escalate
+HARD002Grader = _make("HARD-002")  # reject
+HARD003Grader = _make("HARD-003")  # reject
+HARD004Grader = _make("HARD-004")  # approve
+HARD005Grader = _make("HARD-005")  # escalate
+HARD006Grader = _make("HARD-006")  # flag
+HARD007Grader = _make("HARD-007")  # reject
+HARD008Grader = _make("HARD-008")  # reject
+HARD009Grader = _make("HARD-009")  # escalate
+HARD010Grader = _make("HARD-010")  # reject
+
+# ── Critical ──────────────────────────────────────────────────────────────────
+CRIT001Grader = _make("CRIT-001")  # approve
+CRIT002Grader = _make("CRIT-002")  # reject
+CRIT003Grader = _make("CRIT-003")  # escalate
+CRIT004Grader = _make("CRIT-004")  # reject
+CRIT005Grader = _make("CRIT-005")  # reject
+CRIT006Grader = _make("CRIT-006")  # escalate