insta 0.0.14 → 0.0.16

package/dist/commands/project.js

@@ -1,18 +1,24 @@
-import { createInterface } from 'node:readline/promises';
+import { homedir } from 'node:os';
 import { ApiClient, requireProject } from '../api.js';
 import { writeProject } from '../config.js';
 import { info, die, printJson, handleApproval, renderNextActions } from '../util.js';
 import { installObserve } from '../observe/install.js';
 import { installSkills } from '../ensure-skills.js';
-// Interactive name prompt (stderr, so piped stdout stays clean). Enter accepts the default.
-async function promptName(question, def) {
-    const rl = createInterface({ input: process.stdin, output: process.stderr });
-    try {
-        return (await rl.question(`${question} [${def}]: `)).trim();
-    }
-    finally {
-        rl.close();
-    }
+// Generic directory names that make a useless project name ("projects", "~", "tmp", …). When the
+// cwd basename is one of these we auto-generate a friendly name instead of using it.
+const GENERIC_DIRS = new Set([
+    'projects', 'project', 'home', 'tmp', 'temp', 'desktop', 'documents', 'downloads',
+    'src', 'source', 'code', 'dev', 'work', 'workspace', 'repos', 'repo', 'git',
+    'app', 'apps', 'users', 'user', 'bin', 'new', 'test', 'tests',
+]);
+const ADJ = ['swift', 'brave', 'calm', 'bright', 'bold', 'quiet', 'warm', 'keen', 'wise',
+    'lucky', 'sunny', 'cosmic', 'gentle', 'rapid', 'vivid', 'amber', 'crisp', 'noble'];
+const NOUN = ['otter', 'falcon', 'maple', 'river', 'harbor', 'meadow', 'comet', 'cedar', 'lark',
+    'delta', 'summit', 'ember', 'willow', 'pixel', 'forge', 'harbor', 'atlas', 'quartz'];
+/** A friendly auto-generated name like `swift-meadow-482` (Vercel/Render style). */
+export function generateProjectName(rand = Math.random) {
+    const pick = (a) => a[Math.floor(rand() * a.length)];
+    return `${pick(ADJ)}-${pick(NOUN)}-${100 + Math.floor(rand() * 900)}`;
 }
 // Best-effort: wire the credential-audit hook into the project (no-op if assets aren't built).
 function tryInstallObserve() {
@@ -35,20 +41,23 @@ async function resolveOrg(api, given) {
 export function slugifyName(raw) {
     return raw.toLowerCase().replace(/[^a-z0-9]+/g, '-').replace(/^-+|-+$/g, '').slice(0, 40);
 }
-/** Name resolution so the recommended one-liner (`insta project create`, no arg) is paste-and-run:
- *  explicit arg wins; else prompt with the cwd basename as default (TTY); else use the basename. */
-export async function resolveProjectName(nameArg, cwd = process.cwd(), prompt) {
-    const fromDir = slugifyName(cwd.split('/').filter(Boolean).pop() ?? 'app') || 'app';
+/** Name resolution so `insta project create` (no arg) NEVER blocks on a prompt — the whole point
+ *  of a paste-and-run one-liner. Explicit arg wins; else use the cwd basename when it's a real
+ *  project-dir name; else auto-generate a friendly name (e.g. in ~/projects, where the basename
+ *  "projects" is useless). You can always pass a name or rename later. */
+export async function resolveProjectName(nameArg, cwd = process.cwd(), generate = generateProjectName) {
     if (nameArg)
         return slugifyName(nameArg);
-    if (prompt && process.stdin.isTTY)
-        return slugifyName((await prompt('project name', fromDir)) || fromDir) || fromDir;
-    return fromDir;
+    const base = slugifyName(cwd.split('/').filter(Boolean).pop() ?? '');
+    const home = slugifyName(homedir().split('/').filter(Boolean).pop() ?? '');
+    if (base && base !== home && !GENERIC_DIRS.has(base))
+        return base;
+    return generate();
 }
 export async function projectCreate(name, opts) {
     const api = await ApiClient.load();
     const orgId = await resolveOrg(api, opts.org);
-    const resolved = await resolveProjectName(name, process.cwd(), promptName);
+    const resolved = await resolveProjectName(name, process.cwd());
     const out = await api.request('POST', `/orgs/${orgId}/projects`, { name: resolved });
     await writeProject({ projectId: out.project.id, orgId, branch: out.defaultBranch.name });
     info(`created project ${out.project.id} (${resolved})`);

package/dist/commands/setup.js

@@ -71,11 +71,14 @@ export function summarizeInstall(output) {
         : `${count} agent${count === 1 ? '' : 's'}`;
     return `✓ Agent skills — ${list}`;
 }
-// Capture stdout+stderr silently (don't stream) so we can print our own clean summary. We keep
-// the child's stdin inherited in case the tool ever needs a TTY, but with -y it shouldn't.
+// Capture stdout+stderr silently (don't stream) so we can print our own clean summary.
+// stdin is 'ignore', NOT 'inherit': under the canonical `curl … | sh` install, stdin is the
+// piped install script itself — a child that inherits it (npx/skills reads for keypresses even
+// with -y) consumes the rest of the script, so the shell never runs the trailing "Get started"
+// guidance. Ignoring stdin keeps the installer's own output intact. (-y means no prompt anyway.)
 const defaultRunner = (cmd, args) => new Promise((resolve) => {
     const env = { ...process.env, AI_AGENT: process.env.AI_AGENT || 'insta', FORCE_COLOR: '0' };
-    const p = spawn(cmd, args, { stdio: ['inherit', 'pipe', 'pipe'], env });
+    const p = spawn(cmd, args, { stdio: ['ignore', 'pipe', 'pipe'], env });
     let output = '';
     const grab = (chunk) => { output += chunk.toString(); };
     p.stdout?.on('data', grab);
@@ -90,7 +93,7 @@ export async function setupAgent(opts, run = defaultRunner) {
     if (!opts.yes && !process.stdout.isTTY) {
         info('non-interactive shell — assuming -y');
     }
-    info('setting up coding-agent skills … (~20s)');
+    info('setting up coding-agent skills …');
     const res = await run('npx', SETUP_ARGS);
     if (!res.ok) {
         info('  skill install failed — install manually with:');

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "insta",
-  "version": "0.0.14",
+  "version": "0.0.16",
   "type": "module",
   "description": "InstaCloud CLI — a thin client of the platform control-plane API.",
   "keywords": [