diff --git a/.github/prompts/duplicate-issue-check.md b/.github/prompts/duplicate-issue-check.md index 97305886f1f..c2006943fa5 100644 --- a/.github/prompts/duplicate-issue-check.md +++ b/.github/prompts/duplicate-issue-check.md @@ -48,4 +48,3 @@ Return only JSON: - `duplicate_of`: the issue number of the earlier report, or `null` - `confidence`: 0.0 to 1.0 - `evidence`: one sentence naming the shared root cause and symptom, or why nothing matched -- `considered`: the issue numbers you actually read diff --git a/.github/prompts/duplicate-issue-check.schema.json b/.github/prompts/duplicate-issue-check.schema.json index 1ae62e05aec..3064e15de8b 100644 --- a/.github/prompts/duplicate-issue-check.schema.json +++ b/.github/prompts/duplicate-issue-check.schema.json @@ -1,7 +1,7 @@ { "type": "object", "additionalProperties": false, - "required": ["duplicate_of", "confidence", "evidence", "considered"], + "required": ["duplicate_of", "confidence", "evidence"], "properties": { "duplicate_of": { "type": ["integer", "null"], @@ -15,10 +15,6 @@ "evidence": { "type": "string", "description": "One sentence naming the shared root cause and symptom, or why nothing matched." - }, - "considered": { - "type": "array", - "items": { "type": "integer" } } } } diff --git a/scripts/flag-duplicate-issue.test.ts b/scripts/flag-duplicate-issue.test.ts index 4f857e29e70..81785c668e8 100644 --- a/scripts/flag-duplicate-issue.test.ts +++ b/scripts/flag-duplicate-issue.test.ts @@ -31,10 +31,15 @@ const config: FlagConfig = { repo: "BerriAI/litellm", issueNumber: 35, dryRun: f describe("parseVerdict", () => { test("accepts the schema's shape, with a null duplicate_of", () => { - const parsed = parseVerdict('{"duplicate_of": null, "confidence": 0.9, "evidence": "Nothing matches.", "considered": [1]}'); + const parsed = parseVerdict('{"duplicate_of": null, "confidence": 0.9, "evidence": "Nothing matches."}'); expect(parsed).toEqual({ kind: "verdict", verdict: { duplicate_of: null, confidence: 0.9, evidence: "Nothing matches." } }); }); + test("keeps only the three fields the flag step uses, whatever else Codex sends", () => { + const parsed = parseVerdict('{"duplicate_of": 12, "confidence": 0.99, "evidence": "Same traceback.", "considered": [12, 34]}'); + expect(parsed).toEqual({ kind: "verdict", verdict: { duplicate_of: 12, confidence: 0.99, evidence: "Same traceback." } }); + }); + test("rejects non-JSON, a non-object, a non-integer target, a missing confidence and empty evidence", () => { expect(parseVerdict("not json").kind).toBe("skip"); expect(parseVerdict('"just a string"').kind).toBe("skip");