Simplify guardian review hook input

Rename the PermissionRequest advisory field to guardian_review, drop the redundant source field, and share one hook-side guardian review type with protocol risk and authorization enums.

Co-authored-by: Codex <noreply@openai.com>
This commit is contained in:
Abhinav Vedmala
2026-04-10 16:54:56 -07:00
parent a05fe33cc9
commit b9098cb968
8 changed files with 153 additions and 309 deletions

View File

@@ -2,20 +2,7 @@
"$schema": "http://json-schema.org/draft-07/schema#",
"additionalProperties": false,
"definitions": {
"NullableString": {
"type": [
"string",
"null"
]
},
"PermissionRequestApprovalReviewDecisionWire": {
"enum": [
"allow",
"deny"
],
"type": "string"
},
"PermissionRequestApprovalReviewRiskLevelWire": {
"GuardianRiskLevel": {
"enum": [
"low",
"medium",
@@ -24,17 +11,7 @@
],
"type": "string"
},
"PermissionRequestApprovalReviewStatusWire": {
"enum": [
"approved",
"denied",
"aborted",
"failed",
"timed_out"
],
"type": "string"
},
"PermissionRequestApprovalReviewUserAuthorizationWire": {
"GuardianUserAuthorization": {
"enum": [
"unknown",
"low",
@@ -43,13 +20,19 @@
],
"type": "string"
},
"PermissionRequestApprovalReviewWire": {
"NullableString": {
"type": [
"string",
"null"
]
},
"PermissionRequestGuardianReview": {
"additionalProperties": false,
"properties": {
"decision": {
"anyOf": [
{
"$ref": "#/definitions/PermissionRequestApprovalReviewDecisionWire"
"$ref": "#/definitions/PermissionRequestGuardianReviewDecision"
},
{
"type": "null"
@@ -65,24 +48,20 @@
"risk_level": {
"anyOf": [
{
"$ref": "#/definitions/PermissionRequestApprovalReviewRiskLevelWire"
"$ref": "#/definitions/GuardianRiskLevel"
},
{
"type": "null"
}
]
},
"source": {
"const": "guardian",
"type": "string"
},
"status": {
"$ref": "#/definitions/PermissionRequestApprovalReviewStatusWire"
"$ref": "#/definitions/PermissionRequestGuardianReviewStatus"
},
"user_authorization": {
"anyOf": [
{
"$ref": "#/definitions/PermissionRequestApprovalReviewUserAuthorizationWire"
"$ref": "#/definitions/GuardianUserAuthorization"
},
{
"type": "null"
@@ -94,12 +73,28 @@
"decision",
"rationale",
"risk_level",
"source",
"status",
"user_authorization"
],
"type": "object"
},
"PermissionRequestGuardianReviewDecision": {
"enum": [
"allow",
"deny"
],
"type": "string"
},
"PermissionRequestGuardianReviewStatus": {
"enum": [
"approved",
"denied",
"aborted",
"failed",
"timed_out"
],
"type": "string"
},
"PermissionRequestToolInput": {
"additionalProperties": false,
"properties": {
@@ -114,19 +109,19 @@
}
},
"properties": {
"approval_review": {
"cwd": {
"type": "string"
},
"guardian_review": {
"anyOf": [
{
"$ref": "#/definitions/PermissionRequestApprovalReviewWire"
"$ref": "#/definitions/PermissionRequestGuardianReview"
},
{
"type": "null"
}
]
},
"cwd": {
"type": "string"
},
"hook_event_name": {
"const": "PermissionRequest",
"type": "string"
@@ -163,8 +158,8 @@
}
},
"required": [
"approval_review",
"cwd",
"guardian_review",
"hook_event_name",
"model",
"permission_mode",