@ductape/cli 0.3.29 → 0.3.31

package/dist/commands/resources.js

@@ -1,4 +1,4 @@
-import { buildCrudParams, listComponentsFromProduct, listResourceTypes, proxyMethodForVerb, resolveResourceType, } from '../lib/resources.js';
+import { buildCrudParams, CONNECT_IDENTIFIER_FIELD, listComponentsFromProduct, listResourceTypes, proxyMethodForVerb, resolveResourceType, } from '../lib/resources.js';
 import { resolveResourceSchema } from '../lib/forms/index.js';
 import { isInteractive, promptTag } from '../lib/forms/prompt.js';
 import { toInteractiveOpts } from '../lib/interactive-opts.js';
@@ -291,7 +291,16 @@ export async function runResourceCrud(typeName, verb, opts, extraArgs) {
         }
     }
     const method = proxyMethodForVerb(crud);
-    const proxyMethod = (module === 'models' || module === 'feature') && method === 'list' ? 'fetchAll' : method;
+    // databases.update() on the SDK is the row/query-level table updater (options: {database, env,
+    // table, where, data, ...}), not the resource-definition updater — calling it with (product, tag,
+    // body) destructures product's string as `options` and fails with "No database connection
+    // established", not a useful error about the definition update. The definition updater is a
+    // separate method, databases.updateDatabase(product, database, data).
+    const proxyMethod = (module === 'models' || module === 'feature') && method === 'list'
+        ? 'fetchAll'
+        : module === 'databases' && method === 'update'
+            ? 'updateDatabase'
+            : method;
     const interactive = toInteractiveOpts(opts);
     let tag = opts.tag ?? extraArgs[0];
     if (!tag && ['get', 'update', 'delete'].includes(crud) && isInteractive(interactive)) {
@@ -313,8 +322,11 @@ export async function runResourceCrud(typeName, verb, opts, extraArgs) {
     }
     let payload = body;
     if (crud === 'connect' && tag) {
+        const existing = payload && typeof payload === 'object' ? payload : {};
+        const identifierField = CONNECT_IDENTIFIER_FIELD[module];
         payload = {
-            ...(payload && typeof payload === 'object' ? payload : {}),
+            ...existing,
+            ...(identifierField ? { [identifierField]: existing[identifierField] ?? tag } : {}),
             tag,
         };
     }

package/dist/lib/api-client.js

@@ -51,7 +51,9 @@ export class ApiClient {
                 endpoint: safeEndpoint(url), httpStatus: res.status,
                 requestId: res.headers.get('x-request-id') ?? res.headers.get('x-correlation-id') ?? undefined,
                 retryable: res.status === 408 || res.status === 429 || res.status >= 500,
-                mutationState: mutating ? 'unknown' : 'not_attempted',
+                mutationState: mutating
+                    ? (res.status >= 400 && res.status < 500 && ![408, 429].includes(res.status) ? 'confirmed_failed' : 'unknown')
+                    : 'not_attempted',
                 contentType: res.headers.get('content-type') ?? undefined,
             });
         }

package/dist/lib/http.js

@@ -33,6 +33,7 @@ export async function parseJsonResponse(res, url, context = {}) {
     }
 }
 function responseError(message, res, url, context, code) {
+    const deterministicClientFailure = res.status >= 400 && res.status < 500 && ![408, 429].includes(res.status);
     return new DuctapeOperationError(message, {
         code,
         operation: context.operation ?? 'http.request',
@@ -40,7 +41,9 @@ function responseError(message, res, url, context, code) {
         httpStatus: res.status,
         requestId: res.headers.get('x-request-id') ?? res.headers.get('x-correlation-id') ?? undefined,
         retryable: res.status === 408 || res.status === 429 || res.status >= 500,
-        mutationState: context.mutationState ?? 'not_attempted',
+        mutationState: deterministicClientFailure && context.mutationState === 'unknown'
+            ? 'confirmed_failed'
+            : (context.mutationState ?? 'not_attempted'),
         contentType: res.headers.get('content-type') ?? undefined,
     });
 }

package/dist/lib/resources.d.ts

@@ -2,6 +2,14 @@ import type { SDKModule } from './proxy/sdk-proxy.js';
 /** CLI resource type → sdk-proxy module */
 export declare const RESOURCE_MODULES: Record<string, SDKModule>;
 export type CrudVerb = 'create' | 'list' | 'get' | 'update' | 'delete' | 'connect';
+/**
+ * `connect` config field the SDK actually reads for the resource identifier, per module.
+ * databases.connect() reads config.database, graph.connect() reads config.graph,
+ * vector.connect() reads config.vector — none of them read config.tag, so `-t <tag>` must be
+ * mapped to the right key or a tag-only connect fails with a misleading "product and env are
+ * required" (the identifier field was silently empty, not actually missing product/env).
+ */
+export declare const CONNECT_IDENTIFIER_FIELD: Partial<Record<SDKModule, string>>;
 export declare function resolveResourceType(name: string): SDKModule;
 export declare function proxyMethodForVerb(verb: CrudVerb): string;
 /** Build params array matching Workbench sdkProxy.ts conventions */

package/dist/lib/resources.js

@@ -39,6 +39,18 @@ export const RESOURCE_MODULES = {
     product: 'product',
     app: 'app',
 };
+/**
+ * `connect` config field the SDK actually reads for the resource identifier, per module.
+ * databases.connect() reads config.database, graph.connect() reads config.graph,
+ * vector.connect() reads config.vector — none of them read config.tag, so `-t <tag>` must be
+ * mapped to the right key or a tag-only connect fails with a misleading "product and env are
+ * required" (the identifier field was silently empty, not actually missing product/env).
+ */
+export const CONNECT_IDENTIFIER_FIELD = {
+    databases: 'database',
+    graph: 'graph',
+    vector: 'vector',
+};
 export function resolveResourceType(name) {
     const mod = RESOURCE_MODULES[name.toLowerCase()];
     if (!mod) {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@ductape/cli",
-  "version": "0.3.29",
+  "version": "0.3.31",
   "description": "Ductape CLI — local platform, login, link projects, and manage resources via the proxy (Workbench-compatible)",
   "type": "module",
   "main": "dist/index.js",