@doikayt/typescript-build-config 0.1.19 → 0.1.20

package/README.md

@@ -123,7 +123,9 @@ small single-purpose scripts, and the check side mirrors the write side:
 publish config accordingly (non-destructively):
 
 - **Library** — `main` / `types` / `exports` → `dist/index.js`, `files: ["dist"]`,
-  and `prepack`, so it publishes a compiled, typed ESM package on push.
+  and `prepack`, so it publishes a compiled, typed ESM package on push. `init`
+  also prompts for the package name (default `@doikayt/<name>`; a bare name is
+  scoped to `@doikayt`, an already-scoped name is kept).
 - **App / CLI** (the default) — marked `private: true`, so a release versions and
   tags it but never publishes to npm.
 
@@ -337,8 +339,8 @@ rely on it.
 ## Release Pipeline
 
 On install, the postinstall script copies the release pipeline files into your
-project automatically. The package name in `auto-changeset.sh` is substituted
-from your project's `package.json` at install time.
+project automatically. `auto-changeset.sh` reads the package name from your
+`package.json` at runtime, so renaming the package needs no pipeline edits.
 
 On `npm update`, the behavior per file is:
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@doikayt/typescript-build-config",
-  "version": "0.1.19",
+  "version": "0.1.20",
   "description": "Shared build configuration for TypeScript projects — ESLint, Prettier, tsconfig, and Playwright presets",
   "_comments": {
     "exports": "This block lists the config files that will be extended by files in in src/toplevel - which are copied to consuming project root folder. An example: eslint.config.js imports from '@doikayt/typescript-build-config/eslint'.",

package/src/init.js

@@ -62,21 +62,29 @@ function createAskYesNo() {
     closed = true;
     while (waiters.length) waiters.shift()("");
   });
-  const ask = (question) =>
+  const nextLine = (question) =>
     new Promise((res) => {
       process.stdout.write(question);
-      const answer = (line) => res(/^y(es)?$/i.test(line.trim()));
-      if (lines.length) answer(lines.shift());
-      else if (closed) answer("");
-      else waiters.push(answer);
+      if (lines.length) res(lines.shift());
+      else if (closed) res("");
+      else waiters.push(res);
     });
+  const ask = async (question) =>
+    /^y(es)?$/i.test((await nextLine(question)).trim());
+  ask.text = async (question) => (await nextLine(question)).trim();
   ask.close = () => rl.close();
   return ask;
 }
 
+// Prepend the @doikayt scope unless the name is already scoped.
+function scopeName(name) {
+  return name.startsWith("@") ? name : `@doikayt/${name}`;
+}
+
 export async function runInit({
   cwd = process.cwd(),
   prompt,
+  promptText,
   resolveDevVersions = resolveVersions,
   log = console.log,
 } = {}) {
@@ -84,16 +92,32 @@ export async function runInit({
   // on error so a leftover interface never keeps the process alive.
   const ownAsk = prompt ? null : createAskYesNo();
   const ask = prompt ?? ownAsk;
+  // Free-text prompt (package name). Falls back to "" (accept the default) when a
+  // yes/no prompt is injected without a text one, so tests need not supply it.
+  const askText = promptText ?? ownAsk?.text ?? (async () => "");
   try {
-    return await collectAndScaffold({ cwd, ask, resolveDevVersions, log });
+    return await collectAndScaffold({
+      cwd,
+      ask,
+      askText,
+      resolveDevVersions,
+      log,
+    });
   } finally {
     ownAsk?.close();
   }
 }
 
-async function collectAndScaffold({ cwd, ask, resolveDevVersions, log }) {
+async function collectAndScaffold({
+  cwd,
+  ask,
+  askText,
+  resolveDevVersions,
+  log,
+}) {
   const pkgPath = resolve(cwd, "package.json");
   const raw = readFileSync(pkgPath, "utf8");
+  const currentName = JSON.parse(raw).name;
 
   // Collect every answer up front, before doing any work or printing next
   // steps, so the flow reads: ask everything → act → report.
@@ -102,10 +126,21 @@ async function collectAndScaffold({ cwd, ask, resolveDevVersions, log }) {
   );
 
   const library = await ask(
-    "Is this a publishable library (published to npm)? Default is No\n" +
-      "  — a non-library (an app/CLI) is marked private so it is never published. [y/N] ",
+    "Is this a publishable library (published to npm)? Default is No.\n" +
+      "  No = an app/CLI (marked private, never published). [y/N] ",
   );
 
+  // The package name is what gets published, so ask only for a library. A bare
+  // name gets the @doikayt scope; an already-scoped name is kept.
+  let libraryName;
+  if (library) {
+    const suggested = scopeName(currentName || "package");
+    const entered = await askText(
+      `Package name for publishing? [${suggested}] `,
+    );
+    libraryName = entered ? scopeName(entered) : suggested;
+  }
+
   const wantDemo = await ask(
     "Scaffold a starter demo (minimal src module + README + project.json) so\n" +
       "you can build, test, and release immediately? Default is No; existing\n" +
@@ -115,17 +150,25 @@ async function collectAndScaffold({ cwd, ask, resolveDevVersions, log }) {
   const scripts = canonicalScripts({ ui, library });
   const devDependencies = resolveDevVersions(devDependencyNames({ ui }));
 
+  const fields = packageFields({ library });
+  // `npm init -y` seeds these placeholders; treat them as unset so the canonical
+  // `test` script and (for a library) `main` replace them instead of shadowing.
+  const replaceDefaults = {
+    scripts: { test: 'echo "Error: no test specified" && exit 1' },
+    fields: { main: "index.js" },
+  };
+  if (library) {
+    // The chosen name is a deliberate publish decision, so it replaces whatever
+    // name npm init left in place.
+    fields.name = libraryName;
+    if (currentName != null) replaceDefaults.fields.name = currentName;
+  }
+
   const { text, added, skipped } = applyToPackageJson(raw, {
     scripts,
     devDependencies,
-    fields: packageFields({ library }),
-    // `npm init -y` seeds these placeholders; treat them as unset so the
-    // canonical `test` script and (for a library) `main` replace them instead of
-    // shadowing them.
-    replaceDefaults: {
-      scripts: { test: 'echo "Error: no test specified" && exit 1' },
-      fields: { main: "index.js" },
-    },
+    fields,
+    replaceDefaults,
   });
   writeFileSync(pkgPath, text);
 

package/src/pipeline/auto-changeset.sh

@@ -15,7 +15,9 @@
 set -euo pipefail
 
 CHANGESET_DIR=".changeset"
-PACKAGES=("__PACKAGE_NAME__")
+# Read the package name from package.json at runtime, so it stays correct even if
+# the project is renamed after install — package.json is the single source of truth.
+PACKAGES=("$(node -p "require('./package.json').name")")
 
 # Skip if a manually-authored changeset already exists
 EXISTING=$(find "$CHANGESET_DIR" -name "*.md" ! -name "README.md" | wc -l)

package/src/postinstall.js

@@ -61,8 +61,6 @@ if (alreadyExist.length > 0) {
 
 // --- Pipeline files: per-file copy/warn/diff ---
 
-const projectName = projectPkg.name;
-
 const pipelineFiles = [
   { src: "release.yml", dest: ".github/workflows/release.yml" },
   { src: "changeset-config.json", dest: ".changeset/config.json" },
@@ -77,14 +75,7 @@ for (const { src, dest } of pipelineFiles) {
   const srcPath = resolve(pipelineDir, src);
   const destPath = resolve(projectRoot, dest);
 
-  let canonical = readFileSync(srcPath, "utf8");
-
-  if (src === "auto-changeset.sh") {
-    canonical = canonical.replace(
-      'PACKAGES=("__PACKAGE_NAME__")',
-      `PACKAGES=("${projectName}")`,
-    );
-  }
+  const canonical = readFileSync(srcPath, "utf8");
 
   if (!existsSync(destPath)) {
     mkdirSync(dirname(destPath), { recursive: true });