@timekast/factory 1.12.0 → 1.14.2

package/dist/commands/doctor.js

@@ -18,9 +18,11 @@
  */
 import { existsSync, readFileSync, statSync } from 'node:fs';
 import path from 'node:path';
+import { CLIError } from '../lib/cli-error.js';
 import { collectClaudePaths } from '../lib/claude-paths.js';
 import { CLAUDE_MD_FILE } from '../lib/constants.js';
 import { hasLockfile, normalizeThenHash, readLockfile } from '../lib/lockfile.js';
+import { runLauncherPreflight, } from '../lib/preflight.js';
 import { detectRepo } from '../lib/repo-detection.js';
 import { BETA_BANNER } from './status.js';
 /** Matches a flat always-on rule file shipped by the kit (`.claude/rules/<name>.md`). */
@@ -44,6 +46,13 @@ export const A1_WARNING = 'Aviso de seguridad (A1): el boilerplate `src/` y sus
     'dependencia) NO llega por este canal. El canal de parches de `src/` es EPIC 2 ' +
     '(update agentico con merge inteligente), aún no disponible. Mantén `src/` y tus ' +
     'dependencias al día por tu cuenta mientras tanto.';
+/** Parse `doctor`'s argv slice into flags. Unknown flags are ignored. */
+export function parseDoctorFlags(argv) {
+    return {
+        launcher: argv.includes('--launcher'),
+        json: argv.includes('--json'),
+    };
+}
 /**
  * Concatenate the body of the reserved "on-demand reference" section of CLAUDE.md
  * — the lines under a heading matching `ON_DEMAND_HEADING_RE`, until the next
@@ -220,8 +229,87 @@ function renderDoctor(result) {
 /**
  * Run `factory doctor`. Always exits 0 — a missing lockfile is advisory, not a
  * failure (DIST-006 §8). The A1 notice is emitted on every run.
+ *
+ * Synchronous: the standard doctor report has no network/async seam. The
+ * additive `--launcher` flow is async (it spawns `gh`) and lives in its own
+ * entrypoint, {@link runLauncherDoctor}.
  */
 export function runDoctor(deps = {}) {
     const result = computeDoctor(deps);
     renderDoctor(result);
 }
+// ---------------------------------------------------------------------------
+// `factory doctor --launcher` (LTUI-002) — env preflight, any-platform/any-shell
+// ---------------------------------------------------------------------------
+/** The glyph per launcher-check status. */
+const STATUS_GLYPH = {
+    ok: '✅',
+    fail: '❌',
+    skipped: '—',
+    warn: '🟡',
+};
+/**
+ * Render the launcher preflight report. Each check on its own line with a glyph,
+ * label and (for fail/skip) the actionable message. Closes with a one-line
+ * verdict. Exported so tests can assert the rendered text.
+ */
+export function renderLauncherDoctor(checks) {
+    for (const check of checks) {
+        const glyph = STATUS_GLYPH[check.status];
+        if (check.status === 'ok') {
+            console.log(`${glyph} ${check.label} — ${check.message}`);
+        }
+        else {
+            // fail / skipped: surface the actionable message (multi-line indented).
+            const lines = check.message.split('\n');
+            console.log(`${glyph} ${check.label}: ${lines[0]}`);
+            for (const extra of lines.slice(1)) {
+                if (extra.trim())
+                    console.log(`     ${extra}`);
+            }
+        }
+    }
+    const hasFail = checks.some((c) => c.status === 'fail');
+    const hasWarn = checks.some((c) => c.status === 'warn');
+    console.log('');
+    if (hasFail) {
+        console.log('Entorno incompleto: corrige los checks marcados ❌ y vuelve a intentar.');
+    }
+    else if (hasWarn) {
+        console.log('Entorno listo. Revisa los avisos 🟡 (no bloquean el launcher).');
+    }
+    else {
+        console.log('Entorno listo para usar el launcher.');
+    }
+}
+/**
+ * Run `factory doctor --launcher`: the cross-platform environment preflight
+ * (LTUI-002). Renders the four checks in order (Node ≥ 22 → gh → auth → org)
+ * with ✅ / ❌, then exits 0 if all pass or throws a `CLIError` (exit 1) if any
+ * fails — so a CI step or the TUI (LTUI-001) can branch on the exit code.
+ *
+ * In `--json` mode (TGUI-003) the `LauncherCheck[]` is written as a JSON array
+ * to stdout and NOTHING else (the human renderer is suppressed so stdout stays
+ * pure-parseable JSON). It ALWAYS exits 0 — a failed check is a `status: 'fail'`
+ * element the GUI reads, never a non-zero exit; the exit-1 contract is preserved
+ * only for the interactive / CI path (no `--json`). `runLauncherPreflight` is
+ * non-fail-fast (returns the full report instead of throwing), so the JSON path
+ * is fail-open by construction.
+ *
+ * `deps` is the same injectable seam as `runLauncherPreflight` (tests drive the
+ * Node version + the gh steps without spawning sub-processes).
+ */
+export async function runLauncherDoctor(deps = {}, json = false) {
+    const checks = await runLauncherPreflight(deps);
+    if (json) {
+        process.stdout.write(`${JSON.stringify(checks)}\n`);
+        return;
+    }
+    renderLauncherDoctor(checks);
+    if (checks.some((c) => c.status === 'fail')) {
+        // Non-zero exit without a duplicate error dump: the per-check ❌ lines above
+        // already told the user what to fix. The empty message keeps the top-level
+        // handler from printing a second "✖ ..." block.
+        throw new CLIError('', 1);
+    }
+}

package/dist/commands/new.js

@@ -5,7 +5,8 @@
  * and is non-empty → preflight (gh) → choose profile (full | core) → create the
  * GitHub repo → download the profile tarball → unpack into `./<name>/` (no
  * Factory `.git/`) → `git init` → rename `package.json.name` → write the initial
- * lockfile → inject `factory:update` → initial commit (non-fatal if no git id).
+ * lockfile → inject `factory:update` → initial commit → wire `origin` + push `main`
+ * (the last two non-fatal if no git id / no network; provision re-establishes them).
  *
  * Expected failures throw `CLIError`; the top-level handler prints the message
  * and exits with the carried code.
@@ -18,6 +19,7 @@ import prompts from 'prompts';
 import { CLIError } from '../lib/cli-error.js';
 import { FACTORY_ORG, PROFILES } from '../lib/constants.js';
 import { parseLockfile, writeInitialLockfile } from '../lib/lockfile.js';
+import { printProvisionPointer } from '../lib/provision-pointer.js';
 import { applyPackageJsonEdits, kitLocalScripts } from '../lib/package-json.js';
 import { renderDerivedReadme } from '../lib/readme.js';
 import { runPreflight } from '../lib/preflight.js';
@@ -135,16 +137,37 @@ export async function runNew(name, flags = { core: false, full: false }) {
         writeInitialLockfile(destDir, manifestRaw);
         // Initial commit (H3). Non-fatal: a fresh machine may lack git identity —
         // the project is already usable; warn and let the dev commit by hand (B2).
+        let committed = false;
         try {
             await execa('git', ['add', '-A'], { cwd: destDir });
             await execa('git', ['commit', '-m', 'chore: initial commit from TimeKast Factory'], {
                 cwd: destDir,
             });
+            committed = true;
         }
         catch {
             console.warn('Aviso: no se pudo crear el commit inicial (¿falta `git config --global user.email`/`user.name`?). ' +
                 'El proyecto está listo; commitea a mano cuando configures tu identidad git.');
         }
+        // First push: name the branch `main`, wire the `origin` remote (the repo
+        // `gh repo create` made is empty until now), and push. This is what lets the
+        // DOCS pipeline (discovery/design/backlog) commit + push BEFORE provision —
+        // `provision` later creates + pushes the `develop` branch when infra is set up.
+        // Non-fatal: no network / no auth → the local project is ready and `provision`
+        // re-establishes the substrate. Only attempted when the commit landed.
+        if (committed) {
+            try {
+                await execa('git', ['branch', '-M', 'main'], { cwd: destDir });
+                await execa('git', ['remote', 'add', 'origin', `https://github.com/${FACTORY_ORG}/${validName}.git`], {
+                    cwd: destDir,
+                });
+                await execa('git', ['push', '-u', 'origin', 'main'], { cwd: destDir });
+            }
+            catch {
+                console.warn('Aviso: no se pudo pushear `main` al repo (¿sin red o sin permiso?). ' +
+                    'El repo local está listo; `factory provision` reestablece el remote y el push.');
+            }
+        }
     }
     finally {
         process.removeListener('SIGINT', onSignal);
@@ -154,4 +177,8 @@ export async function runNew(name, flags = { core: false, full: false }) {
     console.log(`\n✔ Proyecto \`${validName}\` listo (perfil \`${profile}\`, cerebro v${version}). ` +
         `Repo: ${FACTORY_ORG}/${validName}`);
     console.log(`  cd ${validName}`);
+    // End-pointer to the next step (PVSKILL-002). Only reached on a successful run —
+    // an earlier throw (CLIError / Ctrl-C) never gets here. `new` has no `--yes` flag,
+    // so this always prints in a normal run (a pure pipe is still informative).
+    printProvisionPointer();
 }