From 949104c560ad64d6a1f11da38e5bf9da95bf9121 Mon Sep 17 00:00:00 2001 From: aarroyo Date: Sat, 15 Aug 2026 17:58:19 -0500 Subject: [PATCH] fix(core-domain): drop the import GT-693 left dead in the OPA evaluator spec CodeQL caught this on PR #509 and it is a real finding in my own code, not noise: `GT-693` deleted the test that pinned the 27 unmapped policies, which was the only consumer of `CONTEXT_AWARE_VIOLATION_PREFIXES` in this spec. The import stayed behind. Removed rather than resolved-as-acceptable. The constant is still exported and still used in production as the fallback for a pre-GT-693 `policy.wasm`; what was dead is this file's reference to it. opa-evaluator spec 19/19. Co-Authored-By: Claude Opus 5 --- .../src/application/validators/evaluators/opa-evaluator.spec.ts | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/packages/core-domain/src/application/validators/evaluators/opa-evaluator.spec.ts b/src/packages/core-domain/src/application/validators/evaluators/opa-evaluator.spec.ts index 307d6854..1e8d14ed 100644 --- a/src/packages/core-domain/src/application/validators/evaluators/opa-evaluator.spec.ts +++ b/src/packages/core-domain/src/application/validators/evaluators/opa-evaluator.spec.ts @@ -1,4 +1,4 @@ -import { OpaEvaluator, CONTEXT_AWARE_VIOLATION_PREFIXES, violationBelongsToRule } from './opa-evaluator'; +import { OpaEvaluator, violationBelongsToRule } from './opa-evaluator'; import { createMockFileSystem, createMockLogger } from '../../../test/mocks'; import { NormalizedRule } from '../../../domain/models/normalized-rule'; import { WorkspaceEvaluationContext } from './evaluator.interface';