create-oke 0.1.6 → 0.2.1

package/README.md

@@ -3,7 +3,7 @@
 Scaffold an [okengine](https://www.npmjs.com/package/okengine) app.
 
 ```bash
-bunx create-oke@latest my-app
+bunx create-oke@latest my-app --template hello
 cd my-app
 oke dev
 ```
@@ -14,21 +14,37 @@ oke dev
 | `:6533` | Console |
 | `:6535` | MCP |
 
-Default template is **Notes** (same tree as [`examples/notes`](https://github.com/omqkhafi/okengine/tree/main/examples/notes)). Pick another with `--template`:
+Default template is **standard** (full recommended file layout, empty scaffolding).
+Pick another with `--template`, or start from a teaching example with `--from-example`:
 
 ```bash
-bunx create-oke@latest my-links --template linkly
-bunx create-oke@latest my-shop --template provisions
-bunx create-oke@latest my-sky --template skyport
+bunx create-oke@latest my-app --template hello
+bunx create-oke@latest my-app --template minimal
+bunx create-oke@latest my-app --template full
+bunx create-oke@latest my-notes --from-example notes
 ```
 
-| Template | Adds |
+| Template | Purpose |
+|---|---|
+| `hello` | Fastest possible "it works" — one flow, no Store |
+| `minimal` | Smallest shape you'd actually ship — Store + 1–2 flows |
+| `standard` | Full recommended file layout, empty scaffolding (default) |
+| `full` | Every element present and wired, no business logic |
+
+`--from-example` copies a teaching example (including its business logic and
+comments) — most new projects want `--template` instead.
+
+| Example | Adds |
 |---|---|
 | `notes` | `oke` · `flow` · `http` · `store.sql` · typed client |
 | `linkly` | `signal` · `clock` · `gate` |
 | `provisions` | durable flows · `vault` · `channel` · plugins |
 | `skyport` | `ai` · multi-tenancy · distributed topology |
 
+With no arguments in a real terminal, create-oke asks for a project name and
+template interactively. Non-TTY / CI / any explicit flag stays fully scriptable
+— zero prompts.
+
 ## Registries
 
 | Registry | Package |

package/{templates → examples}/linkly/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@oke/example-linkly",
-  "version": "0.1.6",
+  "version": "0.2.1",
   "private": true,
   "type": "module",
   "dependencies": {

package/{templates → examples}/notes/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@oke/example-notes",
-  "version": "0.1.6",
+  "version": "0.2.1",
   "private": true,
   "type": "module",
   "dependencies": {

package/{templates → examples}/provisions/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@oke/example-provisions",
-  "version": "0.1.6",
+  "version": "0.2.1",
   "private": true,
   "type": "module",
   "dependencies": {

package/{templates → examples}/skyport/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@oke/example-skyport",
-  "version": "0.1.6",
+  "version": "0.2.1",
   "private": true,
   "type": "module",
   "dependencies": {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "create-oke",
-  "version": "0.1.6",
+  "version": "0.2.1",
   "description": "Scaffold an okengine app — bunx create-oke@latest <name>",
   "license": "MIT",
   "type": "module",
@@ -18,7 +18,8 @@
   },
   "files": [
     "src",
-    "templates"
+    "templates",
+    "examples"
   ],
   "scripts": {
     "prepack": "bun ./src/sync-templates.ts",
@@ -29,6 +30,9 @@
   "engines": {
     "bun": ">=1.3"
   },
+  "dependencies": {
+    "@clack/prompts": "^1.7.0"
+  },
   "devDependencies": {
     "@types/bun": "latest",
     "typescript": "^7.0.2"

package/src/cli.test.ts

@@ -1,14 +1,32 @@
 /**
- * Unit tests for create-oke argument parsing and transforms.
+ * Unit tests for create-oke argument parsing, transforms, and non-TTY behavior.
  */
 
 import { describe, expect, test } from "bun:test";
-import { mkdtempSync, readFileSync, rmSync } from "node:fs";
+import { mkdtempSync, readFileSync, readdirSync, rmSync } from "node:fs";
 import { tmpdir } from "node:os";
 import { join } from "node:path";
-import { formatCdPath, parseArgs } from "./cli.ts";
+import {
+  FROM_EXAMPLE_CHOICE,
+  formatCdPath,
+  nextStepsText,
+  parseArgs,
+  run,
+  scaffoldArgsFromAnswers,
+  scaffoldArgsFromCli,
+  shouldPrompt,
+  sourceFromArgs,
+  type InteractiveAnswers,
+} from "./cli.ts";
 import { listTemplateFiles, scaffold } from "./scaffold.ts";
-import { resolveTemplateDir, TEMPLATES } from "./templates.ts";
+import {
+  DEFAULT_TEMPLATE,
+  EXAMPLES,
+  TEMPLATES,
+  packageRoot,
+  resolveExampleDir,
+  resolveTemplateDir,
+} from "./templates.ts";
 import {
   sanitizeProjectName,
   shouldSkipTemplatePath,
@@ -16,22 +34,98 @@ import {
 } from "./transform.ts";
 
 describe("parseArgs", () => {
-  test("defaults template to notes", () => {
+  test("defaults template to standard", () => {
     const a = parseArgs(["my-app"]);
     expect(a.name).toBe("my-app");
-    expect(a.template).toBe("notes");
+    expect(a.template).toBe("standard");
+    expect(a.templateExplicit).toBe(false);
+    expect(a.fromExample).toBeUndefined();
   });
 
   test("accepts --template and -t", () => {
-    expect(parseArgs(["x", "--template", "linkly"]).template).toBe("linkly");
-    expect(parseArgs(["x", "-t", "skyport"]).template).toBe("skyport");
-    expect(parseArgs(["x", "--template=provisions"]).template).toBe(
-      "provisions",
+    expect(parseArgs(["x", "--template", "hello"]).template).toBe("hello");
+    expect(parseArgs(["x", "-t", "full"]).template).toBe("full");
+    expect(parseArgs(["x", "--template=minimal"]).template).toBe("minimal");
+    expect(parseArgs(["x", "--template", "hello"]).templateExplicit).toBe(true);
+  });
+
+  test("accepts --from-example", () => {
+    expect(parseArgs(["x", "--from-example", "notes"]).fromExample).toBe(
+      "notes",
+    );
+    expect(parseArgs(["x", "--from-example=skyport"]).fromExample).toBe(
+      "skyport",
     );
   });
 
-  test("rejects unknown template", () => {
+  test("rejects unknown template / example / both", () => {
     expect(() => parseArgs(["x", "--template", "nope"])).toThrow(/template/);
+    expect(() => parseArgs(["x", "--from-example", "nope"])).toThrow(/example/);
+    expect(() =>
+      parseArgs(["x", "--template", "hello", "--from-example", "notes"]),
+    ).toThrow(/either/);
+  });
+});
+
+describe("shouldPrompt", () => {
+  test("TTY + no args → interactive", () => {
+    expect(shouldPrompt(parseArgs([]), true)).toBe(true);
+  });
+
+  test("non-TTY or any explicit input → no prompts", () => {
+    expect(shouldPrompt(parseArgs([]), false)).toBe(false);
+    expect(shouldPrompt(parseArgs(["my-app"]), true)).toBe(false);
+    expect(shouldPrompt(parseArgs(["--template", "hello"]), true)).toBe(false);
+    expect(shouldPrompt(parseArgs(["--from-example", "notes"]), true)).toBe(
+      false,
+    );
+    expect(shouldPrompt(parseArgs(["--help"]), true)).toBe(false);
+  });
+});
+
+describe("sourceFromArgs", () => {
+  test("prefers example when set", () => {
+    expect(sourceFromArgs(parseArgs(["x", "--from-example", "linkly"]))).toEqual(
+      { kind: "example", id: "linkly" },
+    );
+    expect(sourceFromArgs(parseArgs(["x"]))).toEqual({
+      kind: "template",
+      id: DEFAULT_TEMPLATE,
+    });
+  });
+});
+
+describe("scaffoldArgsFromAnswers ≡ flag-driven", () => {
+  test("each --template path matches interactive choice", () => {
+    for (const id of TEMPLATES) {
+      const answers: InteractiveAnswers = { name: "x", choice: id };
+      const fromAnswers = scaffoldArgsFromAnswers(answers);
+      const fromFlags = scaffoldArgsFromCli(
+        parseArgs(["x", "--template", id]),
+      );
+      expect(fromAnswers).toEqual(fromFlags);
+      expect(fromAnswers.source).toEqual({ kind: "template", id });
+    }
+  });
+
+  test("each --from-example path matches interactive from-example choice", () => {
+    for (const id of EXAMPLES) {
+      const answers: InteractiveAnswers = {
+        name: "x",
+        choice: FROM_EXAMPLE_CHOICE,
+        example: id,
+      };
+      const fromAnswers = scaffoldArgsFromAnswers(answers);
+      const fromFlags = scaffoldArgsFromCli(
+        parseArgs(["x", "--from-example", id]),
+      );
+      expect(fromAnswers).toEqual(fromFlags);
+      expect(fromAnswers.source).toEqual({ kind: "example", id });
+    }
+  });
+
+  test("8 paths cover every template and every example", () => {
+    expect(TEMPLATES.length + EXAMPLES.length).toBe(8);
   });
 });
 
@@ -39,7 +133,7 @@ describe("transformPackageJson", () => {
   test("rewrites name and okengine file:../.. to installable ref", () => {
     const next = transformPackageJson(
       {
-        name: "@oke/example-notes",
+        name: "@oke/template-hello",
         private: true,
         dependencies: {
           okengine: "file:../..",
@@ -81,7 +175,7 @@ describe("formatCdPath", () => {
 });
 
 describe("scaffold structure", () => {
-  test("each template produces the example tree (minus skips)", () => {
+  test("each clean template produces its source tree (minus skips)", () => {
     for (const id of TEMPLATES) {
       const dir = mkdtempSync(join(tmpdir(), `create-oke-${id}-`));
       try {
@@ -90,7 +184,7 @@ describe("scaffold structure", () => {
         const result = scaffold({
           targetDir: join(dir, id),
           name: `app-${id}`,
-          template: id,
+          source: { kind: "template", id },
         });
         expect([...result.files].sort()).toEqual(expected);
         const pkg = JSON.parse(
@@ -98,13 +192,147 @@ describe("scaffold structure", () => {
         ) as { name: string; dependencies: { okengine: string } };
         expect(pkg.name).toBe(`app-${id}`);
         expect(pkg.dependencies.okengine).not.toBe("file:../..");
-        expect(
-          pkg.dependencies.okengine.startsWith("file:") ||
-            /^\d+\.\d+\.\d+/.test(pkg.dependencies.okengine),
-        ).toBe(true);
       } finally {
         rmSync(dir, { recursive: true, force: true });
       }
     }
   });
+
+  test("hello has exactly one flow and no Store", () => {
+    const dir = mkdtempSync(join(tmpdir(), "create-oke-hello-assert-"));
+    try {
+      const result = scaffold({
+        targetDir: join(dir, "hello"),
+        name: "hello-app",
+        source: { kind: "template", id: "hello" },
+      });
+      const flowFiles = result.files.filter(
+        (f) => f.startsWith("src/flows/") && f.endsWith(".ts"),
+      );
+      expect(flowFiles).toEqual(["src/flows/hello/index.ts"]);
+      const flowSrc = readFileSync(
+        join(result.targetDir, "src/flows/hello/index.ts"),
+        "utf8",
+      );
+      expect(flowSrc).toMatch(/export const hello/);
+      expect(flowSrc).not.toMatch(/\bstore\b/);
+      expect(result.files.some((f) => f.includes("core.ts"))).toBe(false);
+      expect(result.files.some((f) => f.includes("schema.ts"))).toBe(false);
+      const all = result.files
+        .filter((f) => f.endsWith(".ts"))
+        .map((f) => readFileSync(join(result.targetDir, f), "utf8"))
+        .join("\n");
+      expect(all).not.toMatch(/store\.sql/);
+      expect(all).not.toMatch(/\bbookings\b|\borders\b|\blinks\b|\bnotes\b/);
+    } finally {
+      rmSync(dir, { recursive: true, force: true });
+    }
+  });
+
+  test("standard has full four-applications layout and no business domain", () => {
+    const dir = mkdtempSync(join(tmpdir(), "create-oke-standard-assert-"));
+    try {
+      const result = scaffold({
+        targetDir: join(dir, "standard"),
+        name: "standard-app",
+        source: { kind: "template", id: "standard" },
+      });
+      for (const path of [
+        "src/gates.ts",
+        "src/vault.ts",
+        "src/channels.ts",
+        "src/locales/en.ts",
+        "src/locales/ar.ts",
+        "src/flows/main/shapes.ts",
+        "src/flows/main/signals.ts",
+        "src/core.ts",
+        "src/schema.ts",
+        "src/app.ts",
+      ]) {
+        expect(result.files).toContain(path);
+      }
+      const all = result.files
+        .filter((f) => f.endsWith(".ts") || f.endsWith(".md"))
+        .map((f) => readFileSync(join(result.targetDir, f), "utf8"))
+        .join("\n");
+      expect(all).not.toMatch(/\bbookings\b|\borders\b|\blinks\b|\bstripe\b/i);
+    } finally {
+      rmSync(dir, { recursive: true, force: true });
+    }
+  });
+
+  test("--from-example notes matches today's example tree", () => {
+    const dir = mkdtempSync(join(tmpdir(), "create-oke-from-example-"));
+    try {
+      const exampleDir = resolveExampleDir("notes");
+      const expected = listTemplateFiles(exampleDir);
+      const result = scaffold({
+        targetDir: join(dir, "notes"),
+        name: "from-notes",
+        source: { kind: "example", id: "notes" },
+      });
+      expect([...result.files].sort()).toEqual(expected);
+      expect(result.files).toContain("src/flows/notes/index.ts");
+    } finally {
+      rmSync(dir, { recursive: true, force: true });
+    }
+  });
+});
+
+describe("non-TTY CLI", () => {
+  test("no args + non-TTY → zero prompts, exit 1", async () => {
+    const code = await run([], { stdinIsTTY: false });
+    expect(code).toBe(1);
+  });
+
+  test("spawned with piped stdin shows no clack prompts", async () => {
+    const proc = Bun.spawn(["bun", "run", join(packageRoot(), "src/index.ts")], {
+      cwd: packageRoot(),
+      stdin: "pipe",
+      stdout: "pipe",
+      stderr: "pipe",
+    });
+    proc.stdin.end();
+    const code = await proc.exited;
+    const out = await new Response(proc.stdout).text();
+    const err = await new Response(proc.stderr).text();
+    expect(code).toBe(1);
+    expect(err).toMatch(/missing <name>/);
+    // Clack intro / select chrome must not appear.
+    expect(out + err).not.toMatch(/Project name|Start from a worked example/);
+    expect(out + err).not.toMatch(/│/);
+  });
+
+  test("flag-driven scaffold prints the shared next-steps text", async () => {
+    const root = mkdtempSync(join(tmpdir(), "create-oke-flag-"));
+    const target = join(root, "flag-app");
+    try {
+      const code = await run(
+        [target, "--template", "hello"],
+        { stdinIsTTY: false },
+      );
+      expect(code).toBe(0);
+      expect(readdirSync(target).length).toBeGreaterThan(0);
+      const result = {
+        targetDir: target,
+        name: "flag-app",
+        source: { kind: "template" as const, id: "hello" as const },
+        label: "hello",
+        okengineDependency: "x",
+        files: [],
+      };
+      expect(nextStepsText(result)).toContain("oke dev");
+      expect(nextStepsText(result)).toContain("bun install");
+    } finally {
+      rmSync(root, { recursive: true, force: true });
+    }
+  });
+});
+
+describe("examples catalogue", () => {
+  test("all four teaching examples resolve", () => {
+    for (const id of EXAMPLES) {
+      expect(resolveExampleDir(id)).toBeTruthy();
+    }
+  });
 });