From fab5b81027498e9a49d506d6a429209d3859e5c8 Mon Sep 17 00:00:00 2001 From: wenjiefan Date: Tue, 14 Jul 2026 12:56:58 +0200 Subject: [PATCH] style-review: fix example finding id to match references[0].path The al-style-review worked example set findings[0].id to a non-existent knowledge file (apply-approved-label-suffixes.md) while references[0].path points to the real file (label-suffix-approved-list.md). The DO output contract requires id to equal references[0].path for citation-based findings. Align id to the existing file. --- microsoft/skills/review/al-style-review.md | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/microsoft/skills/review/al-style-review.md b/microsoft/skills/review/al-style-review.md index 4c00c45..74dac1f 100644 --- a/microsoft/skills/review/al-style-review.md +++ b/microsoft/skills/review/al-style-review.md @@ -89,7 +89,7 @@ Output conforms to the DO output contract. A populated example: }, "findings": [ { - "id": "microsoft/knowledge/style/apply-approved-label-suffixes.md", + "id": "microsoft/knowledge/style/label-suffix-approved-list.md", "severity": "info", "message": "A Label named Text000 has no approved suffix (Msg/Err/Qst/Tok/Lbl/Txt). Per the referenced CodeCop AA0074 guidance, every Label and TextConst carries a suffix indicating its consuming call.", "location": {