project-auto-wizard 0.16.1 → 0.17.0

package/src/index.js

@@ -8,13 +8,13 @@ import { readFileSync, existsSync } from "node:fs";
 import { createInterface } from "node:readline/promises";
 import { parseArgs, parsePathsCsv, CliError, TYPE_CLI_FLAGS } from "./cli/args.js";
 import { helpText } from "./cli/help.js";
-import { resolveLanguage, setLanguage, t, DEFAULT_LANGUAGE, LANG_ENV_VAR, normalizeLanguage } from "./i18n/index.js";
+import { resolveLanguage, setLanguage, t, DEFAULT_LANGUAGE, LANG_ENV_VAR, SUPPORTED_LANGUAGES, normalizeLanguage } from "./i18n/index.js";
 import { fallbackStyleTypes } from "./core/deploy-style.js";
 import { hooksFor, mergeHookResults } from "./core/types.js";
 import { PATHS } from "./core/paths.js";
 import { resolvePayloadRoot, assertPayload, readTemplateVersion } from "./core/assets.js";
 import { detectTypes, detectDefaultBranch, detectRepoName, makeResolvers, detectMarkers } from "./core/detect-fs.js";
-import { parseExisting, droppedPathLines } from "./core/version-yml.js";
+import { parseExisting, droppedPathLines, invalidOptionLines } from "./core/version-yml.js";
 import { resolveReleaseOptions } from "./core/release-options.js";
 import { explicitFromContext, optionContextFields } from "./core/options.js";
 import { runBreakingCheck } from "./core/breaking-check.js";
@@ -100,16 +100,31 @@ async function runInner(argv, {
   // Language: --lang -> env var -> saved version.yml value -> en. Resolved after --help/--version
   // so an invalid env var cannot block the help output.
   let language;
+  // Language written to version.yml: the environment variable only decides this run's output, so it must not
+  // overwrite a saved choice. Only --lang (or a first install / an unusable saved value) sets the stored language.
+  let storedLanguage;
+  // Interactive mode asks for the language itself when nothing says which one to use.
+  let askLanguage = false;
   try {
     const savedVy = join(cwd, "version.yml");
-    const saved = existsSync(savedVy) ? parseExisting(readFileSync(savedVy, "utf8")).language : null;
+    const savedParsed = existsSync(savedVy) ? parseExisting(readFileSync(savedVy, "utf8")) : null;
+    const saved = savedParsed?.language ?? null;
     language = resolveLanguage({ flag: opts.lang, env: process.env[LANG_ENV_VAR], saved });
+    storedLanguage = normalizeLanguage(opts.lang) ? language : (saved ?? language);
     // An existing install without a saved language now falls back to English: say so once, since
-    // its workflow messages switch from Korean on the next update.
+    // its workflow messages switch from Korean on the next update. A hand-edited unknown value also
+    // falls back to English, but it was never the old default, so it gets its own wording.
     const existingWithoutLanguage = existsSync(savedVy) && !saved;
     const unspecified = !normalizeLanguage(opts.lang) && !normalizeLanguage(process.env[LANG_ENV_VAR]);
+    askLanguage = opts.mode === "interactive" && unspecified && !saved;
     if (existingWithoutLanguage && unspecified && ["full", "interactive"].includes(opts.mode)) {
-      console.error(t("cli.lang.defaultNotice", {}, DEFAULT_LANGUAGE));
+      const unknown = savedParsed?.languageUnsupported;
+      if (unknown) {
+        console.error(t("cli.lang.unsupportedSavedNotice", { value: unknown, supported: SUPPORTED_LANGUAGES.join(", ") }, DEFAULT_LANGUAGE));
+      } else if (!askLanguage) {
+        // When the user is asked instead, there is no silent fallback left to explain.
+        console.error(t("cli.lang.defaultNotice", {}, DEFAULT_LANGUAGE));
+      }
     }
   } catch (e) {
     if (e instanceof CliError) { console.error(e.message); return 1; }
@@ -150,7 +165,7 @@ async function runInner(argv, {
     // The log file is created on the first write, so viewing only status/doctor from the menu leaves nothing behind.
     startLog("install");
     return await runInteractive(
-      { ...optionContextFields(opts), language },
+      { ...optionContextFields(opts), language: askLanguage ? null : storedLanguage, askLanguage },
       { cwd, payloadRoot: payload, clock },
     );
   }
@@ -319,6 +334,8 @@ async function runInner(argv, {
 
   // Warn against the final folders (an explicit --paths may keep the other one). The dry-run preview prints its own notice.
   if (!opts.dryRun) for (const line of droppedPathLines(existing?.droppedPaths, paths)) console.error(`⚠️  ${line}`);
+  // An option value that could not be read is rewritten as false by this run - say so instead of doing it silently.
+  for (const line of invalidOptionLines(existing?.invalidOptions)) console.error(`⚠️  ${line}`);
 
   // version: existing version.yml first (SSoT; prevents overwriting on re-run) -> CLI value -> file detection.
   // Non-interactive, so the fallback notice uses the CLI wording (--project-version) as is.
@@ -374,7 +391,7 @@ async function runInner(argv, {
     repoName,
     // resolvers that compute @wizard ask/auto token values
     resolvers: makeResolvers(cwd, repoName, paths, typeOptions),
-    now, today, language,
+    now, today, language: storedLanguage,
     // Extra context for the install log; does not change the install itself.
     markers: detectMarkers(cwd, types), detectWarnings,
   });
@@ -406,6 +423,7 @@ async function runInner(argv, {
   // Completion summary (also printed in CLI mode)
   printSummary({
     mode: opts.mode, types, version, versionCode, branches, developMissing,
+    releaseAutomerge: context.includeReleaseAutomerge !== false,
     copiedFiles: result?.workflows?.copiedFiles ?? [],
     autoUpdated: result?.workflows?.autoUpdated ?? [],
     gitignoreUpdated: result?.gitignoreUpdated === true,

package/src/ui/prompts.js

@@ -3,7 +3,7 @@
 // On ESC each function returns the CANCEL symbol -> the caller interprets it as default/stay.
 // Ctrl+C / Ctrl+D make the engine reject with PromptAbortError -> run() catches it and exits with code 130.
 import * as engine from "./readline-engine.js";
-import { t } from "../i18n/index.js";
+import { t, SUPPORTED_LANGUAGES, DEFAULT_LANGUAGE } from "../i18n/index.js";
 import { DEPLOY_STYLES, NO_DEPLOY_STYLE } from "../core/deploy-style.js";
 import { TYPE_IDS, typeInfo } from "../core/types.js";
 import { askableOptions } from "../core/options.js";
@@ -11,6 +11,16 @@ import { ENV_MODES, DEFAULT_ENV_MODE, STORE_PLATFORMS, DEPLOY_MODES, DEFAULT_DEP
 
 export const CANCEL = engine.CANCEL;
 
+// Language selection - asked before any language is known, so the message and every label are rendered
+// once per supported language instead of in the active one. Returns the language code. CANCEL on cancel.
+export async function selectLanguage() {
+  return engine.select({
+    message: SUPPORTED_LANGUAGES.map((lang) => t("ui.prompts.language.message", {}, lang)).join(" / "),
+    options: SUPPORTED_LANGUAGES.map((lang) => ({ value: lang, label: t("ui.prompts.language.label", {}, lang) })),
+    initialIndex: Math.max(0, SUPPORTED_LANGUAGES.indexOf(DEFAULT_LANGUAGE)),
+  });
+}
+
 // Mode selection - localized labels, returns the internal key. CANCEL on cancel.
 // again=true is a re-entry after running a read-only mode (status/doctor) and returning to the menu -
 // asking "what should we install?" again would feel odd, so the wording changes.

package/src/ui/summary.js

@@ -13,6 +13,8 @@ export function printSummary(ctx) {
   const { mode, types = [], version = "", versionCode = null, copiedFiles = [], autoUpdated = [], branches = null, gitignoreUpdated = false,
     // pr-flow but the develop branch could not be created on the remote - the layout line alone looks ready, so warn separately.
     developMissing = false,
+    // false when version.yml turns the release PR automerge off: the summary must not describe automerge or its PAT then.
+    releaseAutomerge = true,
     // Post-install verification and record
     answers = [], unresolved = [], secrets = new Map(), optionalSecrets = new Map(), logPath = "", legacyMdLogs = false, cleanup = null,
     // Flutter store deployment - app files created/kept and cleanup of deselected stores
@@ -49,7 +51,7 @@ export function printSummary(ctx) {
     if (branches.mode === "trunk-based") {
       err(t("ui.summary.branches.trunk", { main: branches.main }));
     } else {
-      err(t("ui.summary.branches.prFlow", { develop: branches.develop, main: branches.main }));
+      err(t(releaseAutomerge ? "ui.summary.branches.prFlow" : "ui.summary.branches.prFlowManual", { develop: branches.develop, main: branches.main }));
       if (developMissing) {
         err(t("ui.summary.branches.developMissing", { develop: branches.develop }));
         err(t("ui.summary.branches.developHint", { main: branches.main, develop: branches.develop }));
@@ -208,11 +210,18 @@ export function printSummary(ctx) {
     err("");
   }
 
-  err(`  ${num()} ${t("ui.summary.todo.pat")}`);
-  err(t("ui.summary.todo.patPath"));
-  err(t("ui.summary.todo.patName"));
-  err(t("ui.summary.todo.patAccount"));
-  err(t("ui.summary.todo.patOptional"));
+  // With automerge off (pr-flow) nothing merges the release PR on the workflow's behalf, so the PAT guidance does not apply.
+  // In trunk-based mode there is no release PR, and the PAT block is kept as before.
+  if (releaseAutomerge || !branches || branches.mode === "trunk-based") {
+    err(`  ${num()} ${t("ui.summary.todo.pat")}`);
+    err(t("ui.summary.todo.patPath"));
+    err(t("ui.summary.todo.patName"));
+    err(t("ui.summary.todo.patAccount"));
+    err(t("ui.summary.todo.patOptional"));
+  } else {
+    err(`  ${num()} ${t("ui.summary.todo.manualMerge")}`);
+    err(t("ui.summary.todo.manualMergeHow"));
+  }
   err("");
   // Installed workflows declare the permissions they need themselves - report it by the same rule as the doctor guidance.
   err(`  ${num()} ${t("ui.summary.todo.permissions")}`);