From a855414bc665dd05da0b203450c8dc02f448e2d5 Mon Sep 17 00:00:00 2001 From: Shakker Date: Sat, 9 May 2026 11:15:21 +0100 Subject: [PATCH] test: fix bundled lint and override guard --- extensions/oc-path/src/oc-path/jsonc/parse.ts | 11 +++++++++-- test/scripts/root-package-overrides.test.ts | 3 ++- 2 files changed, 11 insertions(+), 3 deletions(-) diff --git a/extensions/oc-path/src/oc-path/jsonc/parse.ts b/extensions/oc-path/src/oc-path/jsonc/parse.ts index dd58e68a71e..109b57cbca9 100644 --- a/extensions/oc-path/src/oc-path/jsonc/parse.ts +++ b/extensions/oc-path/src/oc-path/jsonc/parse.ts @@ -1,6 +1,5 @@ import { ParseErrorCode, - type Node as JsoncParserNode, type ParseError, parseTree, printParseErrorCode, @@ -35,6 +34,14 @@ type LineMap = { lineForOffset(offset: number): number; }; +type JsoncParserNode = { + readonly type: "array" | "boolean" | "null" | "number" | "object" | "property" | "string"; + readonly offset: number; + readonly length: number; + readonly value?: unknown; + readonly children?: readonly JsoncParserNode[]; +}; + export function parseJsonc(raw: string): JsoncParseResult { if (raw.trim().length === 0) { return { ast: { kind: "jsonc", raw, root: null }, diagnostics: [] }; @@ -64,7 +71,7 @@ export function parseJsonc(raw: string): JsoncParseResult { allowTrailingComma: true, disallowComments: false, allowEmptyContent: true, - }); + }) as JsoncParserNode | undefined; const lineMap = createLineMap(raw); const diagnostics = errors.map((error) => toDiagnostic(error, lineMap, tree)); let root: JsoncValue | null = null; diff --git a/test/scripts/root-package-overrides.test.ts b/test/scripts/root-package-overrides.test.ts index f2d0830f751..d6649287d83 100644 --- a/test/scripts/root-package-overrides.test.ts +++ b/test/scripts/root-package-overrides.test.ts @@ -19,11 +19,12 @@ describe("root package override guardrails", () => { it("pins the Bedrock runtime below the Windows ARM Node 24 npm resolver failure", () => { const manifest = readRootManifest(); const packageName = "@aws-sdk/client-bedrock-runtime"; + const dependencyVersion = manifest.dependencies?.[packageName]; const npmOverride = manifest.overrides?.[packageName]; const pnpmOverride = manifest.pnpm?.overrides?.["@aws-sdk/client-bedrock-runtime"]; - expect(pnpmOverride).toBe("3.1024.0"); expect(manifest.dependencies).toHaveProperty(packageName); + expect(pnpmOverride).toBe(dependencyVersion); expect(npmOverride).toBe(`$${packageName}`); });