impel-cli 0.15.1 → 0.15.2

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "impel-cli",
-  "version": "0.15.1",
+  "version": "0.15.2",
   "description": "Configure Claude Code and Codex CLI to talk to Impel's gateway, authenticated by an Impel Personal Access Token",
   "type": "module",
   "bin": {

package/src/skills.js

@@ -74,6 +74,23 @@ export function resolveMarketplaceName(marketplace) {
   return typeof name === "string" && name.trim() ? name.trim() : null;
 }
 
+/** Resolve a marketplace name from Claude's local `marketplace list --json`. */
+export function resolveConfiguredMarketplaceName(output, sourceUrl) {
+  let marketplaces;
+  try {
+    marketplaces = JSON.parse(String(output || ""));
+  } catch {
+    return null;
+  }
+  if (!Array.isArray(marketplaces)) return null;
+  const normalizedSource = String(sourceUrl || "").replace(/\/+$/u, "");
+  const marketplace = marketplaces.find((candidate) => (
+    typeof candidate?.url === "string"
+    && candidate.url.replace(/\/+$/u, "") === normalizedSource
+  ));
+  return resolveMarketplaceName(marketplace);
+}
+
 /**
  * Resolve the gateway URL for skill serving. Reuses the CLI's configured gateway
  * (the caller passes `config.gatewayUrl`), then the env/default from config.js,
@@ -317,11 +334,38 @@ export async function syncSkills({
 
     const manifestUrl = marketplaceUrl(gatewayUrl, client);
     const sourceUrl = marketplaceSourceUrl(gatewayUrl, client);
-    const marketplaceName = await fetchMarketplaceName(manifestUrl, fetchImpl);
-    const commands = buildSkillCommands({ client, marketplaceSourceUrl: sourceUrl, marketplaceName });
+    let marketplaceName = await fetchMarketplaceName(manifestUrl, fetchImpl);
 
     logger.log(`Skills: syncing ${SKILL_PLUGIN_NAME} for ${displayLabel}…`);
     const failures = [];
+    const registerCommand = buildSkillCommands({
+      client,
+      marketplaceSourceUrl: sourceUrl,
+      marketplaceName,
+    })[0];
+    const registerResult = await run(spec.bin, registerCommand.args, env);
+    if (registerResult.missing) {
+      failures.push({ phase: registerCommand.phase, reason: `\`${spec.bin}\` disappeared mid-sync` });
+    } else if (!isBenign(registerResult)) {
+      failures.push({
+        phase: registerCommand.phase,
+        reason: firstLine(registerResult.stderr) || `exit ${registerResult.status}`,
+      });
+    }
+
+    // The public manifest fetch is deliberately best-effort and can time out.
+    // Once Claude has registered the marketplace, its local JSON index gives us
+    // the same dynamic name so refreshes can still use PLUGIN@MARKETPLACE.
+    if (!marketplaceName && client === "claude" && !registerResult.missing) {
+      const listed = await run(spec.bin, ["plugin", "marketplace", "list", "--json"], env);
+      if (listed.ok) marketplaceName = resolveConfiguredMarketplaceName(listed.stdout, sourceUrl);
+    }
+
+    const commands = registerResult.missing ? [] : buildSkillCommands({
+      client,
+      marketplaceSourceUrl: sourceUrl,
+      marketplaceName,
+    }).slice(1);
     for (const command of commands) {
       const result = await run(spec.bin, command.args, env);
       if (result.missing) {