insta 0.0.82 → 0.1.0

package/dist/deploy-archive.js

@@ -1,3 +1,4 @@
+import { setTimeout as delay } from 'node:timers/promises';
 import { handleApproval } from './util.js';
 export function archiveBuildSpec(hasDockerfile) {
     return hasDockerfile ? { type: 'dockerfile' } : { type: 'nixpacks' };
@@ -79,69 +80,77 @@ export async function deployArchive(api, projectId, ref, branch, opts, now = Dat
     const operationId = started.body?.operationId;
     if (typeof operationId !== 'string' || !operationId)
         throw new Error('the platform accepted the deploy but returned no operation id — re-run the deploy');
-    log(`build logs: insta build-logs ${operationId}`);
+    log(`build logs: insta build logs ${operationId}`);
     if (started.body?.resumed === true)
         log('resuming the deploy this archive already started');
     const deadline = now() + DEPLOY_DEADLINE_MS;
     const overdue = () => new Error(`the deploy did not finish within ${Math.round(DEPLOY_DEADLINE_MS / 60000)} minutes — check \`insta status\` or re-run`);
-    let last = '';
-    for (;;) {
-        // The deadline bounds the wall clock, not the number of answers: it is checked before each poll,
-        // and each poll is itself bounded by what remains, so a stalled endpoint cannot hold the CLI
-        // past it, and an answer that would arrive after it is not waited for.
-        const remaining = deadline - now();
-        if (remaining <= 0)
-            throw overdue();
-        const res = await api.rawRequest('GET', `/projects/${projectId}/archive-deploys/${encodeURIComponent(operationId)}`, undefined, {
-            signal: AbortSignal.timeout(Math.min(remaining, POLL_REQUEST_TIMEOUT_MS)),
-        }).catch((e) => {
-            if (!isAbort(e))
-                throw e;
-            throw remaining <= POLL_REQUEST_TIMEOUT_MS ? overdue() : new Error(`the platform did not answer a status poll within ${POLL_REQUEST_TIMEOUT_MS / 1000}s — check \`insta status\` or re-run`);
-        });
-        const state = res.body?.state;
-        await watchLogs?.(operationId, state === 'failed' || state === 'live', Math.max(0, deadline - now()));
-        // A failed operation is an ANSWER, not a transport error: the poll worked, and the sentence
-        // it carries (usually the gateway's own, e.g. "no Dockerfile at ./api") is the one to show.
-        if (state === 'failed') {
-            // `||` would let a non-string through and the CLI would print "[object Object]" for the one
-            // sentence that explains the failure. Only a non-empty string is a message.
-            const error = res.body?.error;
-            return { failed: typeof error === 'string' && error ? error : 'the deploy failed' };
+    const logController = new AbortController();
+    const logTask = watchLogs ? (async () => {
+        while (!logController.signal.aborted && now() < deadline) {
+            await watchLogs(operationId, false, Math.max(0, deadline - now()), logController.signal);
+            await delay(Math.min(POLL_MS, Math.max(0, deadline - now())), undefined, { signal: logController.signal });
         }
-        if (state === 'live') {
-            const image = res.body?.imageRef;
-            const url = res.body?.url;
-            if (typeof image !== 'string' || !image || typeof url !== 'string' || !url) {
-                throw new Error('the deploy finished but the platform returned no image or URL for it — check `insta status`');
+    })().catch(() => {
+        if (!logController.signal.aborted)
+            log(`Could not follow build logs. Retry with: insta build logs ${operationId}`);
+    }) : undefined;
+    let last = '';
+    try {
+        for (;;) {
+            const remaining = deadline - now();
+            if (remaining <= 0)
+                throw overdue();
+            const res = await api.rawRequest('GET', `/projects/${projectId}/archive-deploys/${encodeURIComponent(operationId)}`, undefined, {
+                signal: AbortSignal.timeout(Math.min(remaining, POLL_REQUEST_TIMEOUT_MS)),
+            }).catch((e) => {
+                if (!isAbort(e))
+                    throw e;
+                throw remaining <= POLL_REQUEST_TIMEOUT_MS ? overdue() : new Error(`the platform did not answer a status poll within ${POLL_REQUEST_TIMEOUT_MS / 1000}s — check \`insta status\` or re-run`);
+            });
+            const state = res.body?.state;
+            if (state === 'failed' || state === 'live') {
+                logController.abort();
+                await logTask;
+                await watchLogs?.(operationId, true, Math.max(0, deadline - now()));
             }
-            // Optional strings, validated as such. String() would have coerced a protocol error into a
-            // plausible-looking branch or group and reported a target the deploy never named. An omitted
-            // field falls back to what was requested; a field of the wrong type is a broken contract.
-            const optionalString = (field, v) => {
-                if (v === undefined || v === null)
-                    return undefined;
-                if (typeof v !== 'string')
-                    throw new Error(`the platform returned a non-string ${field} for the deploy — upgrade with \`insta upgrade\``);
-                return v;
-            };
-            return {
-                image, url,
-                branch: optionalString('branch', res.body.branch) ?? branch,
-                group: optionalString('group', res.body.group) ?? opts.group ?? '',
-                machineId: optionalString('machineId', res.body.machineId),
-            };
-        }
-        // Only the platform's own in-flight states keep the loop going. An absent or unknown state
-        // would otherwise spend the whole deadline looking like a slow build.
-        if (state !== 'queued' && state !== 'building' && state !== 'deploying') {
-            throw new Error(`the platform reported an unknown deploy state (${JSON.stringify(state)}) — upgrade with \`insta upgrade\``);
-        }
-        if (state !== last) {
-            log(state === 'deploying' ? 'image built, deploying it' : `${state}…`);
-            last = state;
+            if (state === 'failed') {
+                const error = res.body?.error;
+                return { failed: typeof error === 'string' && error ? error : 'the deploy failed' };
+            }
+            if (state === 'live') {
+                const image = res.body?.imageRef;
+                const url = res.body?.url;
+                if (typeof image !== 'string' || !image || typeof url !== 'string' || !url) {
+                    throw new Error('the deploy finished but the platform returned no image or URL for it — check `insta status`');
+                }
+                const optionalString = (field, v) => {
+                    if (v === undefined || v === null)
+                        return undefined;
+                    if (typeof v !== 'string')
+                        throw new Error(`the platform returned a non-string ${field} for the deploy — upgrade with \`insta upgrade\``);
+                    return v;
+                };
+                return {
+                    image, url,
+                    branch: optionalString('branch', res.body.branch) ?? branch,
+                    group: optionalString('group', res.body.group) ?? opts.group ?? '',
+                    machineId: optionalString('machineId', res.body.machineId),
+                };
+            }
+            if (state !== 'queued' && state !== 'building' && state !== 'deploying') {
+                throw new Error(`the platform reported an unknown deploy state (${JSON.stringify(state)}) — upgrade with \`insta upgrade\``);
+            }
+            if (state !== last) {
+                log(state === 'deploying' ? 'image built, deploying it' : `${state}…`);
+                last = state;
+            }
+            await wait(POLL_MS);
         }
-        await wait(POLL_MS);
+    }
+    finally {
+        logController.abort();
+        await logTask;
     }
 }
 //# sourceMappingURL=deploy-archive.js.map

package/dist/ensure-skills.js

@@ -27,7 +27,7 @@ const GITIGNORE_COMMENT = '# InstaCloud: agent skills installed by `npx skills a
 // existing AI_AGENT (e.g. running inside another agent) rather than clobbering it.
 const defaultRunner = (cmdIn, argsIn, inherit = false) => new Promise((resolve) => {
     // resolveSpawnable: on Windows `npx` is a .cmd shim spawn() refuses without a shell —
-    // re-enter npm's CLI script via node instead (same treatment as `insta setup agent`).
+    // re-enter npm's CLI script via node instead (same treatment as `insta agent setup`).
     const { cmd, args } = resolveSpawnable(cmdIn, argsIn);
     const env = { ...process.env, AI_AGENT: process.env.AI_AGENT || 'insta' };
     // npx exports its flags as npm_config_* to children; npm_config_package would pin the inner