@apifuse/provider-sdk 2.2.0-beta.4 → 2.2.0-beta.41

package/dist/cli/create.js

@@ -1,11 +1,12 @@
 import { spawn } from "node:child_process";
 import { existsSync, readFileSync } from "node:fs";
-import { mkdir, readFile, writeFile } from "node:fs/promises";
+import { mkdir, readFile, rm, symlink, writeFile } from "node:fs/promises";
 import { dirname, relative, resolve } from "node:path";
 import { fileURLToPath } from "node:url";
 import { cancel, intro, isCancel, note, outro, select, text } from "@clack/prompts";
 import { z } from "zod";
 import packageJson from "../../package.json";
+import { buildPromptAssetManifest, buildPromptAssetPlanEntries, PROMPT_ASSET_MANIFEST_PATH, } from "./prompt-assets.js";
 export const PROVIDER_NAME_REGEX = /^[a-z][a-z0-9]*(?:-[a-z0-9]+)*$/;
 export const CATEGORY_OPTIONS = [
     "developer-tools",
@@ -283,10 +284,15 @@ export async function buildProviderCreatePlan(options, cwd) {
     if (options.sdkSpecifier?.startsWith("workspace:") && !resolvedWorkspaceRoot) {
         throw new Error("workspace:* is only valid inside the APIFuse monorepo because public Provider SDK scaffolds must install from npm or an explicit tarball/file specifier.");
     }
+    // Exact pin, not a caret range: the prompt-asset freshness gate requires
+    // manifest.sdkVersion === installed SDK version, and the manifest records
+    // this CLI's own version. A caret range would let `bun install` resolve a
+    // newer SDK than the create binary, making a fresh scaffold fail its own
+    // gate. `sync-assets` after an explicit SDK upgrade re-records the version.
     const sdkSpecifier = options.sdkSpecifier ??
         (options.preset === "monorepo" && resolvedWorkspaceRoot
             ? "workspace:*"
-            : `^${packageJson.version}`);
+            : packageJson.version);
     const relativeProviderRoot = relative(cwd, providerRoot) || options.name;
     const nextDevCommand = `cd ${relativeProviderRoot} && bun run dev`;
     const packageName = options.preset === "monorepo"
@@ -373,6 +379,12 @@ export async function buildProviderCreatePlan(options, cwd) {
                 sdkSpecifier,
             }),
         },
+        {
+            path: resolve(providerRoot, "provider.json"),
+            content: await renderTemplate("provider.json.tpl", {
+                PROVIDER_ID: options.name,
+            }),
+        },
         {
             path: resolve(providerRoot, "Dockerfile"),
             content: await renderTemplate("Dockerfile.tpl", {}),
@@ -405,39 +417,21 @@ export async function buildProviderCreatePlan(options, cwd) {
                 PROVIDER_ID: options.name,
             }),
         },
-        {
-            path: resolve(providerRoot, "AGENTS.md"),
-            content: await renderTemplate("AGENTS.md.tpl", {}),
-        },
-        {
-            path: resolve(providerRoot, "CLAUDE.md"),
-            content: await renderTemplate("CLAUDE.md.tpl", {}),
-        },
-        {
-            path: resolve(providerRoot, "skills", "normalization-standards", "SKILL.md"),
-            content: await renderTemplate("skills/normalization-standards/SKILL.md.tpl", {}),
-        },
-        {
-            path: resolve(providerRoot, "skills", "upstream-contract-verification", "SKILL.md"),
-            content: await renderTemplate("skills/upstream-contract-verification/SKILL.md.tpl", {}),
-        },
-        {
-            path: resolve(providerRoot, "skills", "fixtures-and-recording", "SKILL.md"),
-            content: await renderTemplate("skills/fixtures-and-recording/SKILL.md.tpl", {}),
-        },
-        {
-            path: resolve(providerRoot, "skills", "pagination-and-counts", "SKILL.md"),
-            content: await renderTemplate("skills/pagination-and-counts/SKILL.md.tpl", {}),
-        },
-        {
-            path: resolve(providerRoot, "skills", "health-checks-and-fail-closed", "SKILL.md"),
-            content: await renderTemplate("skills/health-checks-and-fail-closed/SKILL.md.tpl", {}),
-        },
-        {
-            path: resolve(providerRoot, "skills", "upstream-notes", "README.md"),
-            content: await renderTemplate("skills/upstream-notes/README.md.tpl", {}),
-        },
     ];
+    const promptAssetEntries = await buildPromptAssetPlanEntries(renderTemplate);
+    for (const entry of promptAssetEntries) {
+        files.push({
+            path: resolve(providerRoot, entry.path),
+            content: entry.content,
+            ...(entry.kind === "symlink" ? { kind: "symlink" } : {}),
+        });
+    }
+    // Manifest last: writePlan writes in order, so a crash mid-scaffold never
+    // leaves a manifest that claims assets which were not written.
+    files.push({
+        path: resolve(providerRoot, PROMPT_ASSET_MANIFEST_PATH),
+        content: buildPromptAssetManifest(promptAssetEntries, packageJson.version),
+    });
     return {
         displayName: options.displayName,
         files,
@@ -477,6 +471,7 @@ function renderPackageJson(input) {
             check: "apifuse check . && bun run type-check",
             "type-check": "tsc --noEmit",
             "submit-check": "apifuse submit-check . --markdown submission-report.md",
+            "sync-assets": "apifuse sync-assets .",
             test: "apifuse test .",
             record: "apifuse record .",
             start: "bun start.ts",
@@ -696,6 +691,13 @@ function isApifuseInternalWorkspaceRoot(workspaceRoot) {
 async function writePlan(plan) {
     for (const file of plan.files) {
         await mkdir(dirname(file.path), { recursive: true });
+        if (file.kind === "symlink") {
+            // rm operates on the link itself (lstat semantics) and tolerates
+            // directories, so whatever occupies the path is replaced by the link.
+            await rm(file.path, { recursive: true, force: true });
+            await symlink(file.content, file.path);
+            continue;
+        }
         await writeFile(file.path, file.content);
     }
 }
@@ -754,7 +756,10 @@ function printResult(plan, jsonMode, dryRun) {
         },
         validationCommands: plan.validationCommands,
         nextDevCommand: plan.nextDevCommand,
-        files: plan.files.map((file) => relative(plan.providerRoot, file.path) || file.path),
+        files: plan.files.map((file) => {
+            const relativePath = relative(plan.providerRoot, file.path) || file.path;
+            return file.kind === "symlink" ? `${relativePath} -> ${file.content} (symlink)` : relativePath;
+        }),
     };
     if (jsonMode) {
         console.log(JSON.stringify(payload, null, 2));

package/dist/cli/migrate-provider-shape.d.ts

@@ -0,0 +1,52 @@
+/**
+ * Provider authoring shape migration.
+ *
+ * `defineProvider` changed in 2.2.0-beta.37 from returning a finished provider
+ * to returning a builder, splitting authoring into two phases:
+ *
+ *   const buildProvider = defineProvider(<declaration>);
+ *   export type ProviderContext = ProviderContextOf<typeof buildProvider>;
+ *   export default buildProvider({ operations });
+ *
+ * Source written against the single-phase shape still type-checks against an
+ * older pin but default-exports a builder function once the pin moves, so
+ * `loadProviderDefinition` rejects it. Bumping the pin without migrating the
+ * source therefore breaks the module. This transform performs the source half
+ * so the SDK bump fan-out can ship both in one commit.
+ *
+ * The transform is deliberately conservative: it rewrites only the shapes it
+ * can fully account for and reports `skipped` with a reason for anything else,
+ * rather than emitting a partial migration a reviewer would have to audit.
+ */
+/** Every source shape this transform recognizes. */
+export type ProviderShapeKind = 
+/** `export default defineProvider({ ..., operations })` */
+"single-phase-default-export"
+/** `const p = defineProvider({ ..., operations }); export default p;` */
+ | "single-phase-variable-export"
+/** `const p = defineProvider({ ..., operations }); export default { ...p, deployment };` */
+ | "single-phase-variable-spread-export"
+/** Already `const b = defineProvider(...); export default b({ operations })` */
+ | "two-phase";
+export type ProviderShapeMigration = {
+    readonly status: "migrated";
+    readonly kind: ProviderShapeKind;
+    readonly code: string;
+    /** Source text the operations map was supplied as, for reporting. */
+    readonly operationsExpression: string;
+} | {
+    readonly status: "unchanged";
+    readonly kind: "two-phase";
+    readonly code: string;
+} | {
+    readonly status: "skipped";
+    readonly reason: string;
+};
+/**
+ * Migrate one provider `index.ts` to the two-phase authoring shape.
+ *
+ * Returns the rewritten source on success. Callers MUST treat `skipped` as a
+ * hard stop for that provider — a skipped provider needs a human, and pairing
+ * a pin bump with a skipped migration produces an unloadable module.
+ */
+export declare function migrateProviderShape(sourceText: string, fileName?: string): ProviderShapeMigration;