openuispec 0.1.29 → 0.1.30

package/README.md

@@ -188,13 +188,15 @@ Use the commands like this:
 - `openuispec validate` checks schema correctness
 - `openuispec validate semantic` checks cross-references such as locale keys, formatters, mappers, contracts, icons, navigation targets, and API endpoints
 - `openuispec init --no-configure-targets` scaffolds the spec project without running the target-stack wizard
-- `openuispec configure-target <t>` records target stack choices in `platform/<target>.yaml` using preset defaults, while still allowing custom framework/library values when the project uses something outside the catalog
+- `openuispec configure-target <t>` records and confirms target stack choices in `platform/<target>.yaml`, while still allowing custom framework/library values when the project uses something outside the catalog
 - `openuispec drift --target <t> --explain` explains semantic spec changes since that target's accepted baseline
 - `openuispec prepare --target <t>` builds the target work bundle for either first-time generation or drift-based updates
 - `openuispec status` shows every target's snapshot state, baseline commit, and whether that target is behind the current spec, still needs a baseline, or has not been generated yet
 
 In first-time generation mode, `prepare` also carries target-specific generation constraints such as native localization requirements, multi-file output rules, target folder layout expectations, and a requirement to refresh current platform/framework setup knowledge before code generation.
 
+If stack choices were auto-applied via `configure-target --defaults` or `init --defaults`, they remain unconfirmed. `prepare` will block implementation readiness until the user explicitly confirms the target stack, and AI agents should ask the user to confirm or change those choices instead of silently proceeding to code generation.
+
 When target stack choices come from the preset catalog, `prepare --json` also exposes install-oriented refs for the selected options:
 - Android: Gradle plugin ids and library coordinates
 - Web: npm package specs

package/cli/configure-target.ts

@@ -4,11 +4,9 @@ import { existsSync, mkdirSync, readFileSync, writeFileSync } from "node:fs";
 import { dirname, join, relative, resolve } from "node:path";
 import { fileURLToPath } from "node:url";
 import YAML from "yaml";
-import { findProjectDir, readManifest } from "../drift/index.js";
+import { findProjectDir, isSupportedTarget, readManifest, type SupportedTarget } from "../drift/index.js";
 import { ask, askChoice } from "./init.js";
 
-type SupportedTarget = "ios" | "android" | "web";
-
 type WizardOptionPreset = {
   value: string;
   generation_value?: string;
@@ -44,6 +42,27 @@ type TargetWizardPreset = {
   questions: ChoiceQuestionPreset[];
 };
 
+export type TargetWizardOptionsResponse = {
+  target: SupportedTarget;
+  defaults_are_unconfirmed: true;
+  confirmation_required_before_implementation: true;
+  interactive_command: string;
+  defaults_command: string;
+  framework: {
+    prompt: string;
+    recommended: string;
+    options: string[];
+    custom_allowed: true;
+  };
+  questions: Array<{
+    key: string;
+    prompt: string;
+    recommended: string;
+    custom_allowed: true;
+    options: WizardOptionPreset[];
+  }>;
+};
+
 function readWizardPresets(): Record<SupportedTarget, TargetWizardPreset> {
   const presetsPath = join(dirname(fileURLToPath(import.meta.url)), "target-presets.json");
   return JSON.parse(readFileSync(presetsPath, "utf-8")) as Record<SupportedTarget, TargetWizardPreset>;
@@ -51,6 +70,34 @@ function readWizardPresets(): Record<SupportedTarget, TargetWizardPreset> {
 
 const TARGET_WIZARDS = readWizardPresets();
 
+function listFrameworkOptions(wizard: TargetWizardPreset): string[] {
+  return [...new Set([...(wizard.framework_options ?? [wizard.framework]), "other"])];
+}
+
+export function listTargetWizardOptions(target: SupportedTarget): TargetWizardOptionsResponse {
+  const wizard = TARGET_WIZARDS[target];
+  return {
+    target,
+    defaults_are_unconfirmed: true,
+    confirmation_required_before_implementation: true,
+    interactive_command: `openuispec configure-target ${target}`,
+    defaults_command: `openuispec configure-target ${target} --defaults`,
+    framework: {
+      prompt: wizard.framework_prompt ?? `${target} framework`,
+      recommended: wizard.framework,
+      options: listFrameworkOptions(wizard),
+      custom_allowed: true,
+    },
+    questions: wizard.questions.map((question) => ({
+      key: question.key,
+      prompt: question.prompt,
+      recommended: question.recommended,
+      custom_allowed: true,
+      options: question.options,
+    })),
+  };
+}
+
 function filterOptionsForFramework(
   question: ChoiceQuestionPreset,
   framework: string
@@ -279,32 +326,72 @@ function buildGeneration(
   return generation;
 }
 
+function stackConfirmation(useDefaults: boolean): Record<string, string> {
+  const now = new Date().toISOString();
+  return useDefaults
+    ? {
+        status: "pending_user_confirmation",
+        source: "defaults",
+        updated_at: now,
+      }
+    : {
+        status: "confirmed",
+        source: "user",
+        confirmed_at: now,
+      };
+}
+
 function parseTarget(argv: string[]): SupportedTarget | null {
   const direct = argv[0];
-  if (direct && ["ios", "android", "web"].includes(direct)) {
-    return direct as SupportedTarget;
+  if (direct && isSupportedTarget(direct)) {
+    return direct;
   }
   const targetIdx = argv.indexOf("--target");
-  if (targetIdx !== -1 && argv[targetIdx + 1] && ["ios", "android", "web"].includes(argv[targetIdx + 1])) {
-    return argv[targetIdx + 1] as SupportedTarget;
+  if (targetIdx !== -1 && argv[targetIdx + 1] && isSupportedTarget(argv[targetIdx + 1])) {
+    return argv[targetIdx + 1];
   }
   return null;
 }
 
+function parseSetPairs(argv: string[]): Record<string, string> {
+  const pairs: Record<string, string> = {};
+  for (let i = 0; i < argv.length; i++) {
+    if (argv[i] === "--set" && argv[i + 1]) {
+      const raw = argv[i + 1];
+      const eqIdx = raw.indexOf("=");
+      if (eqIdx > 0) {
+        pairs[raw.slice(0, eqIdx)] = raw.slice(eqIdx + 1);
+      }
+      i++;
+    }
+  }
+  return pairs;
+}
+
 export async function runConfigureTarget(argv: string[]): Promise<void> {
   const target = parseTarget(argv);
+  const listOptions = argv.includes("--list-options");
   const useDefaults = argv.includes("--defaults");
-  const interactive = stdin.isTTY && stdout.isTTY && !useDefaults;
+  const quiet = argv.includes("--quiet");
+  const setPairs = parseSetPairs(argv);
+  const hasSetPairs = Object.keys(setPairs).length > 0;
+  const interactive = stdin.isTTY && stdout.isTTY && !useDefaults && !hasSetPairs;
   if (!target) {
     console.error("Error: target is required for configure-target");
-    console.error("Usage: openuispec configure-target <ios|android|web> [--defaults]");
+    console.error("Usage: openuispec configure-target <ios|android|web> [--defaults] [--list-options] [--set key=value]");
     process.exit(1);
   }
 
-  if (!interactive && !useDefaults) {
+  if (listOptions) {
+    console.log(JSON.stringify(listTargetWizardOptions(target), null, 2));
+    return;
+  }
+
+  if (!interactive && !useDefaults && !hasSetPairs) {
     console.error(
       "Error: `openuispec configure-target` needs a TTY for prompts.\n" +
-        "Run with `--defaults` in non-interactive environments."
+        "Preferred: ask the user to confirm the target stack, then run `openuispec configure-target <target>` in an interactive terminal.\n" +
+        "Fallback: run with `--defaults` only for unattended setup; those values remain unconfirmed and `prepare` will block implementation until the user confirms them."
     );
     process.exit(1);
   }
@@ -351,7 +438,16 @@ export async function runConfigureTarget(argv: string[]): Promise<void> {
 
   let answers = computeDefaultAnswers(framework);
 
-  if (interactive) {
+  if (hasSetPairs) {
+    // Non-interactive --set path: merge provided values with existing/defaults
+    const knownKeys = new Set(wizard.questions.map((q) => q.key));
+    for (const [key, value] of Object.entries(setPairs)) {
+      if (!knownKeys.has(key)) {
+        console.error(`Warning: "${key}" does not match any wizard question; setting as custom value.`);
+      }
+      answers[key] = value;
+    }
+  } else if (interactive) {
     const rl = createInterface({ input: stdin, output: stdout });
 
     try {
@@ -393,10 +489,14 @@ export async function runConfigureTarget(argv: string[]): Promise<void> {
     }
   }
 
+  // --set implies confirmed (user explicitly chose values); --defaults without --set is pending
   const updatedPlatform: Record<string, any> = {
     ...existingPlatform,
     framework,
-    generation: buildGeneration(wizard, answers, existingGeneration, framework),
+    generation: {
+      ...buildGeneration(wizard, answers, existingGeneration, framework),
+      stack_confirmation: stackConfirmation(useDefaults && !hasSetPairs),
+    },
   };
 
   if (wizard.language) updatedPlatform.language = wizard.language;
@@ -408,9 +508,16 @@ export async function runConfigureTarget(argv: string[]): Promise<void> {
 
   writeFileSync(platformPath, YAML.stringify({ [target]: updatedPlatform }));
 
-  console.log(`\nSaved ${relative(process.cwd(), platformPath)}`);
-  console.log("Configured values:");
-  for (const [key, value] of Object.entries(answers)) {
-    console.log(`  - ${key}: ${value}`);
+  const savedPath = relative(process.cwd(), platformPath);
+  if (argv.includes("--silent")) {
+    // Called as subroutine (e.g. from init --quiet) — no output at all
+  } else if (quiet) {
+    console.log(savedPath);
+  } else {
+    console.log(`\nSaved ${savedPath}`);
+    console.log("Configured values:");
+    for (const [key, value] of Object.entries(answers)) {
+      console.log(`  - ${key}: ${value}`);
+    }
   }
 }

package/cli/index.ts

@@ -4,13 +4,15 @@
  *
  * Usage:
  *   openuispec init                           Create a new spec project
- *   openuispec init --defaults                Scaffold non-interactively with defaults
- *   openuispec configure-target <t>           Configure target stack and managed dependencies
+ *   openuispec init --defaults                Scaffold non-interactively with unconfirmed defaults
+ *   openuispec configure-target <t>           Configure and confirm target stack choices
+ *   openuispec configure-target <t> --list-options  Print target stack prompt options as JSON
  *   openuispec drift [--target <t>]           Check for spec drift
  *   openuispec drift --snapshot --target <t>  Snapshot current state + git baseline
  *   openuispec drift --target <t> --explain   Explain semantic changes since baseline
  *   openuispec prepare --target <t>           Build the target work bundle
  *   openuispec status                         Show cross-target baseline/drift status
+ *   openuispec check --target <t> [--json]    Composite validation + prepare readiness
  *   openuispec validate [group...]            Validate spec files against schemas
  */
 
@@ -78,6 +80,12 @@ async function main(): Promise<void> {
       break;
     }
 
+    case "check": {
+      const { runCheck } = await import("../check/index.js");
+      runCheck(rest);
+      break;
+    }
+
     case "validate": {
       const { runValidate } = await import("../schema/validate.js");
       runValidate(rest);
@@ -93,19 +101,25 @@ OpenUISpec CLI v0.1
 
 Usage:
   openuispec init                           Create a new spec project
-  openuispec init --defaults                Scaffold non-interactively with defaults
+  openuispec init --defaults                Scaffold non-interactively with unconfirmed defaults
   openuispec init --no-configure-targets    Skip target stack setup during init
   openuispec update-rules                   Update AI rules to match installed version
-  openuispec configure-target <t> [--defaults] Configure target stack and managed dependencies
+  openuispec configure-target <t> [--defaults] Configure target stack; --defaults stays unconfirmed
+  openuispec configure-target <t> --set k=v    Set specific stack values (confirmed)
+  openuispec configure-target <t> --list-options Print target stack prompt options as JSON
   openuispec drift [--target <t>]           Check for spec drift
   openuispec drift --snapshot --target <t>  Snapshot current state + git baseline
   openuispec drift --target <t> --explain   Explain semantic changes since baseline
   openuispec prepare --target <t>           Build the target work bundle
   openuispec status                         Show cross-target baseline/drift status
-  openuispec validate [group...]            Validate spec files
+  openuispec check --target <t> [--json]    Composite validation + prepare readiness
+  openuispec validate [group...] [--json]   Validate spec files
+  openuispec validate semantic --json       Semantic validation as JSON
 
 Validate groups: manifest, tokens, screens, flows, platform, locales, contracts, semantic
 
+Exit codes: 0 = success, 1 = missing config/usage error, 2 = validation failure
+
 Docs: https://openuispec.rsteam.uz
 `);
       break;
@@ -117,4 +131,7 @@ Docs: https://openuispec.rsteam.uz
   }
 }
 
-main();
+main().catch((err) => {
+  console.error(err instanceof Error ? err.message : String(err));
+  process.exit(1);
+});

package/cli/init.ts

@@ -17,6 +17,7 @@ import {
 } from "node:fs";
 import { join, relative, dirname, resolve } from "node:path";
 import { fileURLToPath } from "node:url";
+import { SUPPORTED_TARGETS, isSupportedTarget, type SupportedTarget } from "../drift/index.js";
 
 // ── prompts ──────────────────────────────────────────────────────────
 
@@ -75,13 +76,13 @@ function ensureDir(path: string): void {
   mkdirSync(path, { recursive: true });
 }
 
-function writeIfMissing(path: string, content: string): boolean {
+function writeIfMissing(path: string, content: string, quiet = false): boolean {
   if (existsSync(path)) {
-    console.log(`  skip ${relative(process.cwd(), path)} (exists)`);
+    if (!quiet) console.log(`  skip ${relative(process.cwd(), path)} (exists)`);
     return false;
   }
   writeFileSync(path, content);
-  console.log(`  create ${relative(process.cwd(), path)}`);
+  if (!quiet) console.log(`  create ${relative(process.cwd(), path)}`);
   return true;
 }
 
@@ -202,7 +203,7 @@ Do NOT guess the file format — skipping this step will produce invalid YAML th
 openuispec validate             # Validate spec files against schemas
 openuispec validate semantic    # Run semantic cross-reference linting
 openuispec validate screens     # Validate only screens
-openuispec configure-target ${targets[0]} [--defaults] # Configure target stack defaults
+openuispec configure-target ${targets[0]} [--defaults] # Configure target stack; --defaults stays unconfirmed
 openuispec status               # Show cross-target baseline/drift status
 openuispec drift --target ${targets[0]} --explain   # Explain semantic spec drift
 openuispec prepare --target ${targets[0]}          # Build the target work bundle
@@ -213,6 +214,8 @@ The target work bundle has two modes:
 - \`bootstrap\` when no snapshot exists yet, for first-time generation
 - \`update\` after a snapshot exists, for drift-based target updates
 
+If target stack values were written with \`--defaults\`, treat them as unconfirmed. Before generating code, ask the user to confirm or change the stack and run \`openuispec configure-target <target>\` without \`--defaults\`.
+
 ## Learn more
 
 Docs: https://openuispec.rsteam.uz
@@ -302,6 +305,7 @@ Spec-first workflow:
 5. Run \`openuispec validate semantic\`.
 6. Run \`openuispec drift --target <target> --explain\` to inspect semantic changes since that target's baseline.
 7. Run \`openuispec prepare --target <target>\` to build the target work bundle for that target. In \`bootstrap\` mode it provides first-generation constraints; in \`update\` mode it provides drift-based update scope.
+   If the target stack was filled from defaults, stop and ask the user to confirm or change it before implementation.
 8. Verify the affected UI targets build/run if possible.
 9. Only then run \`openuispec drift --snapshot --target <target>\` for affected targets, after that target output directory exists.
 10. Run \`openuispec drift --target <target> --explain\` again to confirm no spec changes remain for that target.
@@ -322,6 +326,7 @@ Platform-first workflow:
 - Do not treat \`openuispec drift\` as proof that generated UI matches the spec.
 - Do not skip \`--explain\` / \`prepare\` when another platform needs to catch up with shared spec changes.
 - Do not modify generated UI without checking whether the spec must change first.
+- Do not use \`configure-target --defaults\` as silent approval for implementation. Ask the user to confirm the stack first.
 
 ## CLI commands
 - \`openuispec init\` — scaffold a new spec project
@@ -330,7 +335,7 @@ Platform-first workflow:
 - \`openuispec drift --target <t>\` — check for spec drift
 - \`openuispec drift --target <t> --explain\` — explain semantic spec drift since the target baseline
 - \`openuispec drift --snapshot --target <t>\` — snapshot current state after the target output exists
-- \`openuispec prepare --target <t>\` — build the target work bundle
+- \`openuispec prepare --target <t>\` — build the target work bundle and check whether stack confirmation is still pending
 - \`openuispec status\` — show cross-target baseline/drift status
 - \`openuispec update-rules\` — update AI rules to match installed package version
 - \`openuispec drift --all\` — include stubs in drift check
@@ -354,7 +359,7 @@ export function updateRules(): void {
   }
 
   // Detect targets from manifest
-  let targets = ["ios", "android", "web"];
+  let targets = [...SUPPORTED_TARGETS];
   try {
     const manifest = readFileSync(
       join(cwd, specDir, "openuispec.yaml"),
@@ -429,6 +434,7 @@ export { getPackageVersion };
 
 interface InitOptions {
   defaults: boolean;
+  quiet: boolean;
   name?: string;
   specDir?: string;
   targets?: string[];
@@ -447,11 +453,10 @@ interface InitAnswers {
 }
 
 function parseTargetsValue(raw: string): string[] {
-  const allowed = new Set(["ios", "android", "web"]);
   return raw
     .split(",")
     .map((value) => value.trim().toLowerCase())
-    .filter((value): value is string => allowed.has(value));
+    .filter((value): value is SupportedTarget => isSupportedTarget(value));
 }
 
 function requireFlagValue(argv: string[], index: number, flag: string): string {
@@ -464,12 +469,13 @@ function requireFlagValue(argv: string[], index: number, flag: string): string {
 }
 
 function parseInitArgs(argv: string[]): InitOptions {
-  const options: InitOptions = { defaults: argv.includes("--defaults") };
+  const options: InitOptions = { defaults: argv.includes("--defaults"), quiet: argv.includes("--quiet") };
 
   for (let index = 0; index < argv.length; index++) {
     const arg = argv[index];
     switch (arg) {
       case "--defaults":
+      case "--quiet":
         break;
       case "--name":
         options.name = requireFlagValue(argv, index, arg);
@@ -516,7 +522,7 @@ function collectDefaults(): InitAnswers {
   return {
     name: defaultName,
     specDir: "openuispec",
-    targets: ["ios", "android", "web"],
+    targets: [...SUPPORTED_TARGETS],
     withApi: true,
     backendPath: "../backend/",
     configureTargets: true,
@@ -527,7 +533,7 @@ async function collectInteractiveAnswers(rl: ReturnType<typeof createInterface>)
   const defaults = collectDefaults();
   const name = await ask(rl, "Project name", defaults.name);
   const specDir = await ask(rl, "Spec directory", defaults.specDir);
-  const targets = await askList(rl, "\nWhich platforms?", ["ios", "android", "web"], defaults.targets);
+  const targets = await askList(rl, "\nWhich platforms?", [...SUPPORTED_TARGETS], defaults.targets);
 
   if (targets.length === 0) {
     console.error("At least one target is required.");
@@ -554,7 +560,7 @@ function collectNonInteractiveAnswers(argv: string[]): InitAnswers {
   const parsed = parseInitArgs(argv);
   const defaults = collectDefaults();
 
-  if (!parsed.defaults && argv.length === 0) {
+  if (!parsed.defaults && argv.filter((a) => a !== "--quiet").length === 0) {
     console.error(
       "Error: `openuispec init` needs a TTY for prompts.\n" +
         "Run with `--defaults` or pass flags such as `--name`, `--targets`, `--with-api`, `--backend`, and `--configure-targets`."
@@ -584,10 +590,11 @@ function collectNonInteractiveAnswers(argv: string[]): InitAnswers {
 // ── main ─────────────────────────────────────────────────────────────
 
 export async function init(argv: string[] = []): Promise<void> {
+  const quiet = argv.includes("--quiet");
   const interactive = stdin.isTTY && stdout.isTTY && !argv.includes("--defaults");
   const rl = interactive ? createInterface({ input: stdin, output: stdout }) : null;
 
-  console.log("\nOpenUISpec — Project Setup\n");
+  if (!quiet) console.log("\nOpenUISpec — Project Setup\n");
 
   try {
     const cwd = process.cwd();
@@ -596,7 +603,7 @@ export async function init(argv: string[] = []): Promise<void> {
 
     // ── create folders ─────────────────────────────────────────────
 
-    console.log("\nScaffolding...\n");
+    if (!quiet) console.log("\nScaffolding...\n");
 
     const root = join(cwd, answers.specDir);
     const dirs = [
@@ -620,14 +627,16 @@ export async function init(argv: string[] = []): Promise<void> {
       manifestTemplate(answers.name, answers.targets, {
         withApi: answers.withApi,
         backendPath: answers.backendPath,
-      })
+      }),
+      quiet
     );
 
     // ── spec README ──────────────────────────────────────────────
 
     writeIfMissing(
       join(root, "README.md"),
-      specReadmeTemplate(answers.name, answers.targets)
+      specReadmeTemplate(answers.name, answers.targets),
+      quiet
     );
 
     // ── .gitkeep for empty dirs ────────────────────────────────────
@@ -641,23 +650,11 @@ export async function init(argv: string[] = []): Promise<void> {
         const gk = join(dir, ".gitkeep");
         if (!existsSync(gk)) {
           writeFileSync(gk, "");
-          console.log(`  create ${relative(cwd, gk)}`);
+          if (!quiet) console.log(`  create ${relative(cwd, gk)}`);
         }
       }
     }
 
-    if (answers.withApi && answers.backendPath) {
-      const backendDir = resolve(root, answers.backendPath);
-      const backendExisted = existsSync(backendDir);
-      ensureDir(backendDir);
-      const backendEntries = readdirSync(backendDir).filter((entry) => entry !== ".gitkeep");
-      const backendGitkeep = join(backendDir, ".gitkeep");
-      if ((!backendExisted || backendEntries.length === 0) && !existsSync(backendGitkeep)) {
-        writeFileSync(backendGitkeep, "");
-        console.log(`  create ${relative(cwd, backendGitkeep)}`);
-      }
-    }
-
     // ── AI assistant rules ─────────────────────────────────────────
 
     const rules = aiRulesBlock(answers.specDir, answers.targets);
@@ -667,28 +664,31 @@ export async function init(argv: string[] = []): Promise<void> {
       if (existsSync(filePath)) {
         const existing = readFileSync(filePath, "utf-8");
         if (existing.includes("OpenUISpec")) {
-          console.log(`  skip ${file} (already has OpenUISpec rules)`);
+          if (!quiet) console.log(`  skip ${file} (already has OpenUISpec rules)`);
           continue;
         }
         appendFileSync(filePath, "\n" + rules);
-        console.log(`  update ${file} (appended rules)`);
+        if (!quiet) console.log(`  update ${file} (appended rules)`);
       } else {
         writeFileSync(filePath, rules.trimStart());
-        console.log(`  create ${file}`);
+        if (!quiet) console.log(`  create ${file}`);
       }
     }
 
     if (answers.configureTargets) {
-      console.log("\nConfiguring target stacks...\n");
+      if (!quiet) console.log("\nConfiguring target stacks...\n");
       const { runConfigureTarget } = await import("./configure-target.js");
       for (const target of answers.targets) {
-        await runConfigureTarget([target, ...(interactive ? [] : ["--defaults"])]);
+        await runConfigureTarget([target, ...(interactive ? [] : ["--defaults"]), ...(quiet ? ["--silent"] : [])]);
       }
     }
 
     // ── done ───────────────────────────────────────────────────────
 
-    console.log(`
+    if (quiet) {
+      console.log(`./${answers.specDir}/`);
+    } else {
+      console.log(`
 Done! Your spec project is ready at ./${answers.specDir}/
 
 Getting started (new project):
@@ -709,7 +709,7 @@ Getting started (existing project):
 Commands:
   openuispec validate             Validate spec files
   openuispec validate semantic   Check semantic cross-references
-  openuispec configure-target ios [--defaults]   Configure target stack defaults
+  openuispec configure-target ios [--defaults]   Configure target stack; --defaults stays unconfirmed
   openuispec status   Show cross-target baseline/drift status
   openuispec drift --target ios --explain   Explain semantic spec changes
   openuispec prepare --target ios   Build the target work bundle
@@ -719,6 +719,7 @@ AI rules have been added to CLAUDE.md and AGENTS.md.
 
 Docs: https://openuispec.rsteam.uz
 `);
+    }
   } catch (err) {
     rl?.close();
     throw err;