fix(plugins): fallback install entrypoints for legacy manifests

liuxiaopai-ai · liuxiaopai-ai · commit ce0b3ae7e35f · 2026-03-03T02:52:20.000+08:00
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -50,6 +50,7 @@ Docs: https://docs.openclaw.ai
 - Web UI/config form: support SecretInput string-or-secret-ref unions in map `additionalProperties`, so provider API key fields stay editable instead of being marked unsupported. (#31866) Thanks @ningding97.
 - Slack/Bolt startup compatibility: remove invalid `message.channels` and `message.groups` event registrations so Slack providers no longer crash on startup with Bolt 4.6+; channel/group traffic continues through the unified `message` handler (`channel_type`). (#32033) Thanks @mahopan.
 - Telegram: guard duplicate-token checks and gateway startup token normalization when account tokens are missing, preventing `token.trim()` crashes during status/start flows. (#31973) Thanks @ningding97.
+- Plugins/install compatibility: allow installs to fall back to default plugin entry files (`./index.ts`, `./index.js`, `./dist/index.js`) when `package.json` is missing `openclaw.extensions` but a valid `openclaw.plugin.json` is present, restoring compatibility for older plugin package shapes. (#32019) Thanks @liuxiaopai-ai.
 - Skills/sherpa-onnx-tts: run the `sherpa-onnx-tts` bin under ESM (replace CommonJS `require` imports) and add regression coverage to prevent `require is not defined in ES module scope` startup crashes. (#31965) Thanks @bmendonca3.
 - Browser/default profile selection: default `browser.defaultProfile` behavior now prefers `openclaw` (managed standalone CDP) when no explicit default is configured, while still auto-provisioning the `chrome` relay profile for explicit opt-in use. (#32031) Fixes #31907. Thanks @liuxiaopai-ai.
 - Doctor/local memory provider checks: stop false-positive local-provider warnings when `provider=local` and no explicit `modelPath` is set by honoring default local model fallback while still warning when gateway probe reports local embeddings not ready. (#32014) Fixes #31998. Thanks @adhishthite.
diff --git a/src/plugins/install.test.ts b/src/plugins/install.test.ts
@@ -428,6 +428,40 @@ describe("installPluginFromArchive", () => {
     expect(result.error).toContain("openclaw.extensions");
   });
 
+  it("uses legacy entry fallback when openclaw.extensions is missing but plugin manifest exists", async () => {
+    const { pluginDir, extensionsDir } = setupPluginInstallDirs();
+    fs.writeFileSync(
+      path.join(pluginDir, "package.json"),
+      JSON.stringify({
+        name: "@openclaw/legacy-entry-fallback",
+        version: "0.0.1",
+      }),
+      "utf-8",
+    );
+    fs.writeFileSync(
+      path.join(pluginDir, "openclaw.plugin.json"),
+      JSON.stringify({
+        id: "legacy-entry-fallback",
+        configSchema: { type: "object", properties: {} },
+      }),
+      "utf-8",
+    );
+    fs.writeFileSync(path.join(pluginDir, "index.ts"), "export {};\n", "utf-8");
+
+    const result = await installPluginFromDir({
+      dirPath: pluginDir,
+      extensionsDir,
+    });
+
+    expect(result.ok).toBe(true);
+    if (!result.ok) {
+      return;
+    }
+    expect(result.pluginId).toBe("legacy-entry-fallback");
+    expect(result.extensions).toEqual(["./index.ts"]);
+    expect(fs.existsSync(path.join(result.targetDir, "index.ts"))).toBe(true);
+  });
+
   it("warns when plugin contains dangerous code patterns", async () => {
     const { pluginDir, extensionsDir } = setupPluginInstallDirs();
 
diff --git a/src/plugins/install.ts b/src/plugins/install.ts
@@ -43,6 +43,8 @@ type PackageManifest = {
   dependencies?: Record<string, string>;
 } & Partial<Record<typeof MANIFEST_KEY, { extensions?: string[] }>>;
 
+const LEGACY_EXTENSION_FALLBACK_ENTRIES = ["./index.ts", "./index.js", "./dist/index.js"] as const;
+
 export type InstallPluginResult =
   | {
       ok: true;
@@ -81,16 +83,41 @@ function validatePluginId(pluginId: string): string | null {
   return null;
 }
 
-async function ensureOpenClawExtensions(manifest: PackageManifest) {
-  const extensions = manifest[MANIFEST_KEY]?.extensions;
+async function resolveLegacyExtensionsFallback(packageDir: string): Promise<string[]> {
+  const manifestResult = loadPluginManifest(packageDir);
+  if (!manifestResult.ok) {
+    return [];
+  }
+  const resolvedEntries = await Promise.all(
+    LEGACY_EXTENSION_FALLBACK_ENTRIES.map(async (entry) => ({
+      entry,
+      exists: await fileExists(path.join(packageDir, entry)),
+    })),
+  );
+  return resolvedEntries.filter((item) => item.exists).map((item) => item.entry);
+}
+
+async function ensureOpenClawExtensions(params: {
+  manifest: PackageManifest;
+  packageDir: string;
+}): Promise<string[]> {
+  const extensions = params.manifest[MANIFEST_KEY]?.extensions;
+  if (Array.isArray(extensions)) {
+    const list = extensions.map((e) => (typeof e === "string" ? e.trim() : "")).filter(Boolean);
+    if (list.length > 0) {
+      return list;
+    }
+  }
+
+  const fallback = await resolveLegacyExtensionsFallback(params.packageDir);
+  if (fallback.length > 0) {
+    return fallback;
+  }
+
   if (!Array.isArray(extensions)) {
     throw new Error("package.json missing openclaw.extensions");
   }
-  const list = extensions.map((e) => (typeof e === "string" ? e.trim() : "")).filter(Boolean);
-  if (list.length === 0) {
-    throw new Error("package.json openclaw.extensions is empty");
-  }
-  return list;
+  throw new Error("package.json openclaw.extensions is empty");
 }
 
 function buildFileInstallResult(pluginId: string, targetFile: string): InstallPluginResult {
@@ -148,7 +175,10 @@ async function installPluginFromPackageDir(params: {
 
   let extensions: string[];
   try {
-    extensions = await ensureOpenClawExtensions(manifest);
+    extensions = await ensureOpenClawExtensions({
+      manifest,
+      packageDir: params.packageDir,
+    });
   } catch (err) {
     return { ok: false, error: String(err) };
   }
