fabrica-e-commerce 0.1.6 → 0.1.8

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "fabrica-e-commerce",
-  "version": "0.1.6",
+  "version": "0.1.8",
   "description": "Orange themed CMD launcher for deploying Fabrica e-commerce stores with Supabase and Vercel.",
   "type": "module",
   "bin": {

package/src/deploy.js

@@ -4,7 +4,7 @@ import crypto from 'node:crypto';
 import { HARDCODED_ENV, REQUIRED_ENV_KEYS, STORE_REPO } from './config.js';
 import { buildsDir, saveProject } from './store.js';
 import { runCommand, runCommandCapture } from './system.js';
-import { kv, section, spinner } from './ui.js';
+import { dimOrange, kv, section, spinner } from './ui.js';
 import { ask, choose } from './prompt.js';
 
 const VERCEL_ENVIRONMENTS = ['production', 'preview', 'development'];
@@ -60,6 +60,32 @@ async function setEnvEverywhere(project, env) {
   }
 }
 
+// Vercel's GitHub App integration indexes newly created/forked repos on its
+// own delay, separate from GitHub itself being aware of the repo. Right after
+// a fork, `vercel git connect` can fail simply because Vercel hasn't synced
+// yet — not because anything is actually wrong. Retry with backoff before
+// giving up, then fall back to a direct deploy with clear next steps.
+async function connectGithubRepo(project, repoUrl) {
+  const spin = spinner(`Connecting Vercel project to ${repoUrl}`);
+  const attempts = 6;
+  let lastError = '';
+  for (let attempt = 1; attempt <= attempts; attempt += 1) {
+    const connect = await runCommandCapture('npx', ['--yes', 'vercel@latest', 'git', 'connect', repoUrl, '--yes'], { cwd: project.target });
+    if (connect.code === 0) {
+      spin.succeed('Vercel project connected to GitHub repo (future pushes auto-deploy)');
+      return true;
+    }
+    lastError = (connect.stderr || connect.stdout || '').trim();
+    if (attempt < attempts) await new Promise((resolve) => setTimeout(resolve, attempt * 3000));
+  }
+  spin.fail('Could not auto-connect Git — continuing with a direct deploy');
+  console.log(dimOrange('  This usually means one of two things:'));
+  console.log(dimOrange(`  1) Vercel hasn't finished indexing the new fork yet (run "fabrica list" in a minute and re-link manually with: npx vercel git connect ${repoUrl})`));
+  console.log(dimOrange('  2) The "Vercel" GitHub App is set to "Only select repositories" and was never granted access to the new fork — fix at https://github.com/settings/installations'));
+  if (lastError) console.log(dimOrange(`  Last error: ${lastError}`));
+  return false;
+}
+
 export async function deployToVercel(project, env, githubRepo) {
   section('Vercel deployment');
   await ensureVercelLogin();
@@ -67,10 +93,7 @@ export async function deployToVercel(project, env, githubRepo) {
   await runCommand('npx', ['vercel@latest', 'link', '--yes', '--project', project.projectName], { cwd: project.target });
 
   if (githubRepo?.repoUrl) {
-    const spin = spinner(`Connecting Vercel project to ${githubRepo.repoUrl}`);
-    const connect = await runCommandCapture('npx', ['--yes', 'vercel@latest', 'git', 'connect', githubRepo.repoUrl, '--yes'], { cwd: project.target });
-    if (connect.code === 0) spin.succeed('Vercel project connected to GitHub repo (future pushes auto-deploy)');
-    else spin.fail('Could not auto-connect Git — continuing with a direct deploy');
+    await connectGithubRepo(project, githubRepo.repoUrl);
   }
 
   await setEnvEverywhere(project, env);

package/src/ui.js

@@ -16,7 +16,17 @@ export function banner() {
 }
 export function section(title) { console.log('\n' + orange(`◆ ${title}`)); }
 export function kv(key, value) { console.log(`${dimOrange('>')} ${bold(key)} ${dimOrange('→')} ${value}`); }
-export function spinner(text) { process.stdout.write(dimOrange('> ') + text); return { succeed(msg) { process.stdout.write(`\r${orange('✓')} ${msg}\n`); }, fail(msg) { process.stdout.write(`\r${orange('✗')} ${msg}\n`); } }; }
+// Clear the rest of the line (\x1b[K) before writing the final message so a
+// shorter success/fail message never leaves leftover characters from the
+// longer spinner text trailing after it (e.g. "✓ Git foundGit").
+const CLEAR_LINE = '\x1b[K';
+export function spinner(text) {
+  process.stdout.write(dimOrange('> ') + text);
+  return {
+    succeed(msg) { process.stdout.write(`\r${CLEAR_LINE}${orange('✓')} ${msg}\n`); },
+    fail(msg) { process.stdout.write(`\r${CLEAR_LINE}${orange('✗')} ${msg}\n`); }
+  };
+}
 export function help() {
   banner();
   console.log(`