diff --git a/eng/skill-validator/src/Services/AgentRunner.cs b/eng/skill-validator/src/Services/AgentRunner.cs index 38c24656..4a3513a5 100644 --- a/eng/skill-validator/src/Services/AgentRunner.cs +++ b/eng/skill-validator/src/Services/AgentRunner.cs @@ -148,7 +148,7 @@ internal static SessionConfig BuildSessionConfig( var result = CheckPermission(request, workDir, skillPath); return Task.FromResult(new PermissionRequestResult { - Kind = result ? "approved" : "denied-by-rules", + Kind = result ? PermissionRequestResultKind.Approved : PermissionRequestResultKind.DeniedByRules, }); }, }; diff --git a/eng/skill-validator/src/Services/Judge.cs b/eng/skill-validator/src/Services/Judge.cs index 46a630b0..c31ebe70 100644 --- a/eng/skill-validator/src/Services/Judge.cs +++ b/eng/skill-validator/src/Services/Judge.cs @@ -50,7 +50,7 @@ private static async Task JudgeRunOnce( var result = AgentRunner.CheckPermission(request, options.WorkDir, options.SkillPath); return Task.FromResult(new PermissionRequestResult { - Kind = result ? "approved" : "denied-by-rules", + Kind = result ? PermissionRequestResultKind.Approved : PermissionRequestResultKind.DeniedByRules, }); }, }); diff --git a/eng/skill-validator/src/Services/OverfittingJudge.cs b/eng/skill-validator/src/Services/OverfittingJudge.cs index a6796a08..5186f360 100644 --- a/eng/skill-validator/src/Services/OverfittingJudge.cs +++ b/eng/skill-validator/src/Services/OverfittingJudge.cs @@ -59,7 +59,7 @@ private static async Task AnalyzeOnce(SkillInfo skill, Overfi // No tools needed — all content is passed inline OnPermissionRequest = (_, _) => Task.FromResult(new PermissionRequestResult { - Kind = "denied-by-rules", + Kind = PermissionRequestResultKind.DeniedByRules, }), }); @@ -120,7 +120,7 @@ public static async Task GenerateFix(SkillInfo skill, OverfittingResult result, InfiniteSessions = new InfiniteSessionConfig { Enabled = false }, OnPermissionRequest = (_, _) => Task.FromResult(new PermissionRequestResult { - Kind = "denied-by-rules", + Kind = PermissionRequestResultKind.DeniedByRules, }), }); diff --git a/eng/skill-validator/src/Services/PairwiseJudge.cs b/eng/skill-validator/src/Services/PairwiseJudge.cs index a7dcab25..51e76aaa 100644 --- a/eng/skill-validator/src/Services/PairwiseJudge.cs +++ b/eng/skill-validator/src/Services/PairwiseJudge.cs @@ -85,7 +85,7 @@ private static async Task JudgeCall( var result = AgentRunner.CheckPermission(request, options.WorkDir, options.SkillPath); return Task.FromResult(new PermissionRequestResult { - Kind = result ? "approved" : "denied-by-rules", + Kind = result ? PermissionRequestResultKind.Approved : PermissionRequestResultKind.DeniedByRules, }); }, }); diff --git a/eng/skill-validator/src/SkillValidator.csproj b/eng/skill-validator/src/SkillValidator.csproj index 925c30c8..bd363ea6 100644 --- a/eng/skill-validator/src/SkillValidator.csproj +++ b/eng/skill-validator/src/SkillValidator.csproj @@ -35,7 +35,7 @@ - +