fix: mirror bedrock runtime dep in root package

steipete · steipete · commit d74a12264aa5 · 2026-04-02T19:26:56.000+01:00
diff --git a/extensions/amazon-bedrock/package.json b/extensions/amazon-bedrock/package.json
@@ -13,6 +13,11 @@
     },
     "extensions": [
       "./index.ts"
-    ]
+    ],
+    "releaseChecks": {
+      "rootDependencyMirrorAllowlist": [
+        "@aws-sdk/client-bedrock"
+      ]
+    }
   }
 }
diff --git a/package.json b/package.json
@@ -1221,6 +1221,7 @@
   "dependencies": {
     "@agentclientprotocol/sdk": "0.17.1",
     "@anthropic-ai/vertex-sdk": "^0.14.4",
+    "@aws-sdk/client-bedrock": "3.1020.0",
     "@clack/prompts": "^1.1.0",
     "@homebridge/ciao": "^1.3.6",
     "@line/bot-sdk": "^10.6.0",
diff --git a/pnpm-lock.yaml b/pnpm-lock.yaml
diff --git a/src/plugins/contracts/plugin-sdk-package-contract-guardrails.test.ts b/src/plugins/contracts/plugin-sdk-package-contract-guardrails.test.ts
@@ -82,6 +82,28 @@ function readMatrixPackageJson(): {
   };
 }
 
+function readAmazonBedrockPackageJson(): {
+  dependencies?: Record<string, string>;
+  optionalDependencies?: Record<string, string>;
+  openclaw?: {
+    releaseChecks?: {
+      rootDependencyMirrorAllowlist?: unknown;
+    };
+  };
+} {
+  return JSON.parse(
+    readFileSync(resolve(REPO_ROOT, "extensions/amazon-bedrock/package.json"), "utf8"),
+  ) as {
+    dependencies?: Record<string, string>;
+    optionalDependencies?: Record<string, string>;
+    openclaw?: {
+      releaseChecks?: {
+        rootDependencyMirrorAllowlist?: unknown;
+      };
+    };
+  };
+}
+
 function collectRuntimeDependencySpecs(packageJson: {
   dependencies?: Record<string, string>;
   optionalDependencies?: Record<string, string>;
@@ -328,6 +350,21 @@ describe("plugin-sdk package contract guardrails", () => {
     }
   });
 
+  it("mirrors Bedrock runtime deps needed by the bundled host graph", () => {
+    const rootRuntimeDeps = collectRuntimeDependencySpecs(readRootPackageJson());
+    const bedrockPackageJson = readAmazonBedrockPackageJson();
+    const bedrockRuntimeDeps = collectRuntimeDependencySpecs(bedrockPackageJson);
+    const allowlist = bedrockPackageJson.openclaw?.releaseChecks?.rootDependencyMirrorAllowlist;
+
+    expect(Array.isArray(allowlist)).toBe(true);
+    const bedrockRootMirrorAllowlist = allowlist as string[];
+    expect(bedrockRootMirrorAllowlist).toEqual(expect.arrayContaining(["@aws-sdk/client-bedrock"]));
+
+    for (const dep of bedrockRootMirrorAllowlist) {
+      expect(rootRuntimeDeps.get(dep)).toBe(bedrockRuntimeDeps.get(dep));
+    }
+  });
+
   it("resolves matrix crypto WASM from the root runtime surface", () => {
     const rootRequire = createRootPackageRequire();
     // Normalize filesystem separators so the package assertion stays portable.
@@ -347,12 +384,15 @@ describe("plugin-sdk package contract guardrails", () => {
       const archivePath = packOpenClawToTempDir(packDir);
       const packedPackageJson = await readPackedRootPackageJson(archivePath);
       const matrixPackageJson = readMatrixPackageJson();
+      const bedrockPackageJson = readAmazonBedrockPackageJson();
 
       expect(packedPackageJson.dependencies?.["@matrix-org/matrix-sdk-crypto-wasm"]).toBe(
         matrixPackageJson.dependencies?.["@matrix-org/matrix-sdk-crypto-wasm"],
       );
+      expect(packedPackageJson.dependencies?.["@aws-sdk/client-bedrock"]).toBe(
+        bedrockPackageJson.dependencies?.["@aws-sdk/client-bedrock"],
+      );
       expect(packedPackageJson.dependencies?.["@openclaw/plugin-package-contract"]).toBeUndefined();
-      expect(packedPackageJson.dependencies?.["@aws-sdk/client-bedrock"]).toBeUndefined();
     } finally {
       rmSync(tempRoot, { recursive: true, force: true });
     }

Original file line number	Diff line number	Diff line change
`@@ -13,6 +13,11 @@`
`13`	`13`	`},`
`14`	`14`	`"extensions": [`
`15`	`15`	`"./index.ts"`
`16`		`- ]`
	`16`	`+ ],`
	`17`	`+ "releaseChecks": {`
	`18`	`+ "rootDependencyMirrorAllowlist": [`
	`19`	`+ "@aws-sdk/client-bedrock"`
	`20`	`+ ]`
	`21`	`+ }`
`17`	`22`	`}`
`18`	`23`	`}`