fix(cli): derive drift command from exact npm spec

ooiuuii · ooiuuii · commit 623e3dfcd619 · 2026-06-21T19:32:23.000+08:00
diff --git a/src/plugins/plugin-version-drift.test.ts b/src/plugins/plugin-version-drift.test.ts
@@ -348,6 +348,19 @@ describe("resolvePluginVersionDriftUpdateCommand", () => {
     ).toBe("openclaw plugins update @openclaw/brave-plugin@2026.6.10-beta.1");
   });
 
+  it("prefers the parsed exact npm spec package over inconsistent drift metadata", () => {
+    expect(
+      resolvePluginVersionDriftUpdateCommand({
+        pluginId: "brave",
+        installedVersion: "2026.6.9",
+        gatewayVersion: "2026.6.10-beta.1",
+        source: "npm",
+        packageName: "@openclaw/other-plugin",
+        spec: "@openclaw/brave-plugin@2026.6.9",
+      }),
+    ).toBe("openclaw plugins update @openclaw/brave-plugin@2026.6.10-beta.1");
+  });
+
   it("keeps plugin-id updates for floating npm install records", () => {
     expect(
       resolvePluginVersionDriftUpdateCommand({
diff --git a/src/plugins/plugin-version-drift.ts b/src/plugins/plugin-version-drift.ts
@@ -31,7 +31,7 @@ function resolveExactNpmPinPackageName(entry: PluginVersionDriftEntry): string |
   if (parsed?.selectorKind !== "exact-version") {
     return undefined;
   }
-  return entry.packageName ?? parsed.name;
+  return parsed.name;
 }
 
 /** Exact npm pins need a package@version target; id-only updates preserve the old pin. */

Original file line number	Diff line number	Diff line change
`@@ -31,7 +31,7 @@ function resolveExactNpmPinPackageName(entry: PluginVersionDriftEntry): string \|`
`31`	`31`	`if (parsed?.selectorKind !== "exact-version") {`
`32`	`32`	`return undefined;`
`33`	`33`	`}`
`34`		`- return entry.packageName ?? parsed.name;`
	`34`	`+ return parsed.name;`
`35`	`35`	`}`
`36`	`36`
`37`	`37`	`/** Exact npm pins need a package@version target; id-only updates preserve the old pin. */`