aui-agent-builder 0.4.2 → 0.4.4

package/dist/commands/agents.js

@@ -1131,11 +1131,12 @@ export async function agents(options = {}) {
         const config = getConfig();
         // ─── Resolve & validate kind + bundle_mode up-front ──────────────
         //
+        // Agent creation is ALWAYS bundle mode now. Records-mode creation has
+        // been retired (the `--records-mode` flag was removed and every records
+        // creation branch below is commented out — see the notes there). So the
+        // only thing left to resolve here is `kind` (regular vs template).
+        //
         // Rules (matches the integration guide §3 — "Working with templates"):
-        //   - `--template` + `--records-mode`       → REJECTED (templates are
-        //                                              blob-mode-only)
-        //   - `--bundle-mode` + `--records-mode`    → REJECTED (mutually
-        //                                              exclusive)
         //   - `--template` + `--network-id`         → REJECTED (templates
         //                                              are NETWORK_CATEGORY-
         //                                              scoped; they don't
@@ -1147,30 +1148,40 @@ export async function agents(options = {}) {
         //                                              category for templates"
         //                                              fallback)
         //
-        // Defaults when nothing is passed:
-        //   - kind        → "regular"
-        //   - bundle_mode → ENVIRONMENT-DRIVEN (resolved just below where
-        //                   `bundleMode` is computed):
-        //                     • production / eu-production → false (records
-        //                       mode — the legacy per-entity flow still used
-        //                       in prod).
-        //                     • staging / custom           → true (bundle
-        //                       mode). `custom` is the staging-v3 backend,
-        //                       mapped to staging, so it shares the default.
-        //                   Records-mode regulars auto-snapshot a v1.0 draft
-        //                   from agent-scope; bundle-mode regulars are
-        //                   provisioned end-to-end via the Apollo endpoint
-        //                   (version cloned from a template). Both then
-        //                   auto-import locally. Override either way with
-        //                   `--bundle-mode` / `--records-mode`.
-        if (opts.bundleMode && opts.recordsMode) {
-            renderView(_jsxs(Box, { flexDirection: "column", paddingX: 1, children: [_jsx(StatusLine, { kind: "error", label: "--bundle-mode and --records-mode are mutually exclusive." }), _jsx(Text, { color: "gray", children: "  Pick one — or pass neither to use the default (records-mode)." })] }));
-            return;
-        }
-        if (opts.template && opts.recordsMode) {
-            renderView(_jsxs(Box, { flexDirection: "column", paddingX: 1, children: [_jsx(StatusLine, { kind: "error", label: "--template forces bundle_mode=true; --records-mode is incompatible." }), _jsx(Text, { color: "gray", children: "  Templates are blob-mode-only by design (see integration guide §3)." })] }));
-            return;
-        }
+        // ─── RECORDS-MODE CREATION RETIRED — moved to bundle-mode creation ──
+        // The two validation gates below used to reject `--records-mode` paired
+        // with `--bundle-mode` / `--template`. The `--records-mode` flag no
+        // longer exists (creation is bundle-only), so these gates are dead and
+        // are kept commented for historical reference only.
+        //
+        // if (opts.bundleMode && opts.recordsMode) {
+        //   renderView(
+        //     <Box flexDirection="column" paddingX={1}>
+        //       <StatusLine
+        //         kind="error"
+        //         label="--bundle-mode and --records-mode are mutually exclusive."
+        //       />
+        //       <Text color="gray">
+        //         {"  Pick one — or pass neither to use the default (records-mode)."}
+        //       </Text>
+        //     </Box>,
+        //   );
+        //   return;
+        // }
+        // if (opts.template && opts.recordsMode) {
+        //   renderView(
+        //     <Box flexDirection="column" paddingX={1}>
+        //       <StatusLine
+        //         kind="error"
+        //         label="--template forces bundle_mode=true; --records-mode is incompatible."
+        //       />
+        //       <Text color="gray">
+        //         {"  Templates are blob-mode-only by design (see integration guide §3)."}
+        //       </Text>
+        //     </Box>,
+        //   );
+        //   return;
+        // }
         if (opts.template && opts.networkId) {
             renderView(_jsxs(Box, { flexDirection: "column", paddingX: 1, children: [_jsx(StatusLine, { kind: "error", label: "--template is incompatible with --network-id." }), _jsx(Text, { color: "gray", children: "  Templates are scoped to a NETWORK_CATEGORY, not a NETWORK." })] }));
             return;
@@ -1189,30 +1200,21 @@ export async function agents(options = {}) {
         const kind = opts.template ? "template" : "regular";
         // ─── Resolve bundle_mode ──────────────────────────────────────────
         //
-        // Precedence (highest first):
-        //   1. Templates                 → always bundle-mode (forced).
-        //   2. Explicit --bundle-mode    → bundle-mode.
-        //   3. Explicit --records-mode   → records-mode.
-        //   4. Otherwise (no flag)       → derived from the signed-in
-        //                                  environment (see below).
+        // RECORDS-MODE CREATION RETIRED — moved to bundle-mode creation.
+        // Every agent (regular or template, every environment, interactive or
+        // non-interactive) is now created in bundle mode, provisioned end-to-end
+        // via the Apollo endpoint. There is no `bundleMode` variable anymore —
+        // the create paths below all go through `createBundledAgentViaApollo`.
+        // The old environment/flag-driven resolution is kept commented for
+        // historical reference only:
         //
-        // Environment-driven default for regulars when neither flag is passed:
-        //   - production / eu-production → records-mode (bundle_mode=false).
-        //     Prod still runs on the legacy per-entity records flow.
-        //   - staging / custom          → bundle-mode (bundle_mode=true).
-        //     NOTE: `custom` is the staging-v3 backend — it's mapped to
-        //     staging, so it gets the same bundle-mode default as staging.
-        //
-        // Override either way with --bundle-mode / --records-mode explicitly.
-        const env = config.environment ?? "staging";
-        const envDefaultsToBundle = !(env === "production" || env === "eu-production");
-        const bundleMode = opts.template
-            ? true
-            : opts.bundleMode === true
-                ? true
-                : opts.recordsMode === true
-                    ? false
-                    : envDefaultsToBundle;
+        // const bundleMode = opts.template
+        //   ? true
+        //   : opts.bundleMode === true
+        //     ? true
+        //     : opts.recordsMode === true
+        //       ? false
+        //       : true;
         const client = new AUIClient({
             baseUrl: config.apiUrl,
             authToken: config.authToken,
@@ -1300,63 +1302,64 @@ export async function agents(options = {}) {
             });
             return;
         }
-        // Direct single-command: aui agents --create --name X --network-id Y
-        if (agentName && networkId) {
-            if (!categoryId) {
-                const networks = (await client.networks.list()).data;
-                const net = networks.find((n) => (n._id === networkId || n.id === networkId));
-                categoryId = net
-                    ? (typeof net.category === "string" ? net.category : net.category?._id || "")
-                    : "";
-            }
-            const result = await createAgentOnBackend(agentName, networkId, categoryId || "", {
-                kind,
-                bundleMode,
-            });
-            // Records-mode parity with the interactive/fall-through path below:
-            // a brand-new agent ALWAYS gets its v1.0 draft seeded (works
-            // non-interactively too) and is auto-imported when appropriate
-            // (`shouldAutoImport` — same gate bundle-mode uses). `runDraftFlow`
-            // no-ops the draft for bundle-mode (nothing to snapshot yet).
-            await runPostCreateFlow(result, bundleMode, opts.dir, agentName, opts.full);
-            return;
-        }
+        // ─── RECORDS-MODE CREATION RETIRED — moved to bundle-mode creation ──
+        // Direct single-command `aui agents --create --name X --network-id Y`
+        // created the agent inside a caller-supplied existing network — a
+        // records-mode concept. Bundle-mode provisioning creates its OWN network
+        // server-side, so an external `--network-id` is meaningless. This branch
+        // is commented out (kept for historical reference). `--name X` now flows
+        // through the bundle path below / the interactive flow.
+        //
+        // // Direct single-command: aui agents --create --name X --network-id Y
+        // if (agentName && networkId) {
+        //   if (!categoryId) {
+        //     const networks = (await client.networks.list()).data;
+        //     const net = networks.find((n) => (n._id === networkId || n.id === networkId));
+        //     categoryId = net
+        //       ? (typeof net.category === "string" ? net.category : net.category?._id || "")
+        //       : "";
+        //   }
+        //   const result = await createAgentOnBackend(agentName, networkId, categoryId || "", {
+        //     kind,
+        //     bundleMode,
+        //   });
+        //   await runPostCreateFlow(result, bundleMode, opts.dir, agentName, opts.full);
+        //   return;
+        // }
         // Direct single-command: aui agents --create --name X --category Y (new network)
         if (agentName && categoryId && !networkId) {
             // Bundle-mode: hand the whole flow (network → agent → version →
-            // publish → activate) to the Apollo provisioning endpoint. The
-            // legacy per-step path below is records-mode only.
-            if (bundleMode) {
-                const created = await createBundledAgentViaApollo({
-                    name: agentName,
-                    networkCategoryId: categoryId,
-                    organizationId: client.getOrganizationId(),
-                    accountId: client.getAccountId(),
-                });
-                await autoImportAndEnter(created, opts.dir, agentName);
-                return;
-            }
-            const netSpinner = render(_jsx(Spinner, { label: "Creating agent..." }));
-            try {
-                const resp = await client.networks.create({ name: agentName, category: categoryId });
-                networkId = resp.data._id || resp.data.id;
-                netSpinner.unmount();
-                logView(_jsx(StatusLine, { kind: "success", label: `Network created: ${agentName} (${networkId})` }));
-            }
-            catch (error) {
-                netSpinner.unmount();
-                renderView(_jsx(ErrorDisplay, { error: error, message: "Failed to create agent." }));
-                return;
-            }
-            const result = await createAgentOnBackend(agentName, networkId, categoryId, {
-                kind,
-                bundleMode,
+            // publish → activate) to the Apollo provisioning endpoint.
+            const created = await createBundledAgentViaApollo({
+                name: agentName,
+                networkCategoryId: categoryId,
+                organizationId: client.getOrganizationId(),
+                accountId: client.getAccountId(),
             });
-            // Records-mode parity with the interactive/fall-through path below:
-            // seed the v1.0 draft (non-interactive included) and auto-import it
-            // when appropriate. See note on the --network-id branch above.
-            await runPostCreateFlow(result, bundleMode, opts.dir, agentName, opts.full);
+            await autoImportAndEnter(created, opts.dir, agentName);
             return;
+            // ─── RECORDS-MODE CREATION RETIRED — moved to bundle-mode creation ──
+            // The legacy per-step records path (network create → createAgentOnBackend
+            // → seed v1.0 draft → auto-import) is commented out below; bundle-mode
+            // provisioning above replaces it.
+            //
+            // const netSpinner = render(<Spinner label="Creating agent..." />);
+            // try {
+            //   const resp = await client.networks.create({ name: agentName, category: categoryId });
+            //   networkId = resp.data._id || resp.data.id;
+            //   netSpinner.unmount();
+            //   logView(<StatusLine kind="success" label={`Network created: ${agentName} (${networkId})`} />);
+            // } catch (error: unknown) {
+            //   netSpinner.unmount();
+            //   renderView(<ErrorDisplay error={error} message="Failed to create agent." />);
+            //   return;
+            // }
+            // const result = await createAgentOnBackend(agentName, networkId, categoryId, {
+            //   kind,
+            //   bundleMode,
+            // });
+            // await runPostCreateFlow(result, bundleMode, opts.dir, agentName, opts.full);
+            // return;
         }
         // ─── Interactive flow with step indicators ───
         logView(_jsxs(Box, { flexDirection: "column", paddingX: 1, children: [_jsxs(Box, { children: [_jsx(Text, { color: "cyan", bold: true, children: "\u25C6 " }), _jsx(Text, { bold: true, children: "Create a New Agent" })] }), !categoryId && (_jsxs(_Fragment, { children: [_jsx(Box, { children: _jsx(Text, { color: "gray", children: "  Using default category (General). To use a specific one:" }) }), _jsxs(Box, { children: [_jsx(Text, { color: "gray", children: "  aui agents --create --category " }), _jsx(Text, { color: "gray", italic: true, children: "<name or key>" })] })] })), categoryId && (_jsxs(Box, { children: [_jsx(Text, { color: "gray", children: "  Category: " }), _jsx(Text, { children: opts.category })] }))] }));
@@ -1471,48 +1474,49 @@ export async function agents(options = {}) {
         }
         // Bundle-mode: the Apollo provisioning endpoint runs the whole flow
         // (network → agent → version-from-template → publish → activate) in a
-        // single call, so we skip the legacy per-step path (network create →
-        // createAgentOnBackend → draft/full flow) entirely.
-        if (bundleMode) {
-            const created = await createBundledAgentViaApollo({
-                name: agentName,
-                networkCategoryId: categoryId || "",
-                organizationId: client.getOrganizationId(),
-                accountId: client.getAccountId(),
-            });
-            await autoImportAndEnter(created, opts.dir, agentName);
-            return;
-        }
-        // Create network
-        const netSpinner = render(_jsx(Spinner, { label: `Creating "${agentName}"...` }));
-        try {
-            const resp = await client.networks.create({ name: agentName, category: categoryId || "" });
-            networkId = resp.data._id || resp.data.id;
-            netSpinner.unmount();
-            if (process.env.AUI_DEBUG) {
-                console.log(`[debug] Step 1 - network created: ${networkId} (name: ${agentName}, category: ${categoryId})`);
-            }
-        }
-        catch (error) {
-            netSpinner.unmount();
-            let detail = "";
-            if (error && typeof error === "object" && "body" in error) {
-                const body = error.body;
-                const msg = body?.msg || body?.errors?.errors?.[0]?.msg || body?.error?.detail;
-                if (msg)
-                    detail = `: ${msg}`;
-            }
-            else if (error instanceof Error) {
-                detail = `: ${error.message}`;
-            }
-            renderView(_jsx(ErrorDisplay, { error: error, message: `Failed to create agent${detail}` }));
-            return;
-        }
-        const result = await createAgentOnBackend(agentName, networkId, categoryId || "", {
-            kind,
-            bundleMode,
+        // single call. This is the only creation path now.
+        const created = await createBundledAgentViaApollo({
+            name: agentName,
+            networkCategoryId: categoryId || "",
+            organizationId: client.getOrganizationId(),
+            accountId: client.getAccountId(),
         });
-        await runPostCreateFlow(result, bundleMode, opts.dir, agentName, opts.full);
+        await autoImportAndEnter(created, opts.dir, agentName);
+        return;
+        // ─── RECORDS-MODE CREATION RETIRED — moved to bundle-mode creation ──
+        // The legacy interactive records path (create network → createAgentOnBackend
+        // → seed v1.0 draft / runPostCreateFlow) is commented out below; the
+        // bundle-mode provisioning above replaces it.
+        //
+        // // Create network
+        // const netSpinner = render(<Spinner label={`Creating "${agentName}"...`} />);
+        // try {
+        //   const resp = await client.networks.create({ name: agentName!, category: categoryId || "" });
+        //   networkId = resp.data._id || resp.data.id;
+        //   netSpinner.unmount();
+        //   if (process.env.AUI_DEBUG) {
+        //     console.log(`[debug] Step 1 - network created: ${networkId} (name: ${agentName}, category: ${categoryId})`);
+        //   }
+        // } catch (error: unknown) {
+        //   netSpinner.unmount();
+        //   let detail = "";
+        //   if (error && typeof error === "object" && "body" in error) {
+        //     const body = (error as any).body;
+        //     const msg = body?.msg || body?.errors?.errors?.[0]?.msg || body?.error?.detail;
+        //     if (msg) detail = `: ${msg}`;
+        //   } else if (error instanceof Error) {
+        //     detail = `: ${error.message}`;
+        //   }
+        //   renderView(<ErrorDisplay error={error} message={`Failed to create agent${detail}`} />);
+        //   return;
+        // }
+        //
+        // const result = await createAgentOnBackend(agentName!, networkId!, categoryId || "", {
+        //   kind,
+        //   bundleMode,
+        // });
+        //
+        // await runPostCreateFlow(result, bundleMode, opts.dir, agentName, opts.full);
     }
 }
 /**