@sdt-tools/cli 0.2.0 → 0.2.6

package/dist/index.cjs

@@ -86,7 +86,10 @@ var logger = {
 // src/commands/init.ts
 function initCommand() {
   const cmd = new import_commander.Command("init");
-  cmd.description("Initialize a new SDT project in the current directory.").option("--name <name>", "Project name", "NewSnowflakeProject").option("--scope <scope>", "Project scope: account | database | schema", "database").option("--db <database>", "Database name (required for database/schema scope)").option("--schema <schema>", "Schema name (required for schema scope)").option("--dir <dir>", "Target directory (default: cwd)", process.cwd()).action(async (opts) => {
+  cmd.description("Initialize a new SDT project in the current directory.").option("--name <name>", "Project name", "NewSnowflakeProject").option("--scope <scope>", "Project scope: account | database | schema", "database").option("--db <database>", "Database name (required for database/schema scope)").option("--schema <schema>", "Schema name (required for schema scope)").option("--dir <dir>", "Target directory (default: cwd)", process.cwd()).option(
+    "--force",
+    "Overwrite an existing <name>.sdtproj in the target directory. Without this flag, init refuses to clobber an existing project file so a re-run never silently discards your project configuration."
+  ).action(async (opts) => {
     const scopeType = String(opts.scope);
     if (!["account", "database", "schema"].includes(scopeType)) {
       throw new Error(`Invalid --scope: ${opts.scope}. Use account | database | schema.`);
@@ -104,6 +107,15 @@ function initCommand() {
     await import_node_fs.promises.mkdir(root, { recursive: true });
     const project10 = (0, import_project.newProjectTemplate)(String(opts.name), scope);
     const projectPath = import_node_path.default.join(root, `${project10.name}.sdtproj`);
+    if (!opts.force) {
+      try {
+        await import_node_fs.promises.access(projectPath);
+        logger.error(`${projectPath} already exists; pass --force to overwrite.`);
+        process.exitCode = 1;
+        return;
+      } catch {
+      }
+    }
     await (0, import_project.saveProject)(projectPath, project10);
     const seedFolders = ["databases", "scripts/pre", "scripts/post"];
     for (const f of seedFolders) await import_node_fs.promises.mkdir(import_node_path.default.join(root, f), { recursive: true });
@@ -380,7 +392,13 @@ function compareCommand() {
   const cmd = new import_commander5.Command("compare");
   cmd.description(
     "Compare two schemas. Sources may be .sdtproj, .sdtpac, or snowflake://<profile>[/db[/schema]]."
-  ).argument("<source>", "Left side of the comparison").argument("<target>", "Right side of the comparison").option("-o, --output <path>", "Write JSON result to this path").option("--format <format>", "Output format: json | summary | markdown", "summary").option("--ignore-case", "Treat unquoted identifiers case-insensitively", false).option(
+  ).argument("[source]", "Left side of the comparison (or use --source)").argument("[target]", "Right side of the comparison (or use --target)").option(
+    "--source <path>",
+    "Left side of the comparison (flag form of the positional source arg)"
+  ).option(
+    "--target <path>",
+    "Right side of the comparison (flag form of the positional target arg)"
+  ).option("-o, --output <path>", "Write JSON result to this path").option("--format <format>", "Output format: json | summary | markdown", "summary").option("--ignore-case", "Treat unquoted identifiers case-insensitively", false).option(
     "--no-slice",
     "Disable the source project's Slice (if it has one). Default: a project's slice is applied automatically."
   ).option(
@@ -405,7 +423,14 @@ function compareCommand() {
   ).option(
     "--no-history",
     "Skip writing the compare-history audit record (AUDITCMP.1). Default: every compare run writes a record to `.sdt/history/compare/`, exportable via `sdt audit-log emit`."
-  ).action(async (sourceArg, targetArg, opts) => {
+  ).action(async (sourceArgPos, targetArgPos, opts) => {
+    const sourceArg = opts.source ? String(opts.source) : sourceArgPos ? String(sourceArgPos) : "";
+    const targetArg = opts.target ? String(opts.target) : targetArgPos ? String(targetArgPos) : "";
+    if (!sourceArg || !targetArg) {
+      throw new Error(
+        "compare needs a source and a target \u2014 pass them positionally (`sdt compare <source> <target>`) or via `--source`/`--target`."
+      );
+    }
     const nameMapping = await buildMappingFromOptions(opts);
     const source = await resolveSource(String(sourceArg));
     const target = await resolveSource(String(targetArg));
@@ -691,11 +716,14 @@ function splitStatements(sql) {
 function publishCommand() {
   const cmd = new import_commander6.Command("publish");
   cmd.description(
-    "Compare a .sdtpac to a live Snowflake target and apply (or dry-run) the migration."
+    "Compare a .sdtpac (the desired state) to a live Snowflake target and apply (or dry-run) the migration. Shared form with `ddt publish`: `--source <desired> --connection <live-target>`."
+  ).option(
+    "--source <path>",
+    "Path to the desired-state .sdtpac (canonical name; the same flag `ddt publish` uses). Required for a normal publish; omit only with --restore-from-snapshot."
   ).option(
     "--pac <path>",
-    "Path to a built .sdtpac. Required for a normal publish; omit only with --restore-from-snapshot."
-  ).requiredOption("-c, --connection <profile>", "Connection profile name").option(
+    "Back-compat alias for --source (the desired-state .sdtpac). Prefer --source for cross-platform parity with `ddt publish`."
+  ).requiredOption("-c, --connection <profile>", "Connection profile name (the live target).").option(
     "--restore-from-snapshot <batchId>",
     "Recovery mode: skip the compare step and emit ALTER TABLE \u2026 SWAP WITH against the snapshot batch <batchId> from the registry. Dry-run by default; pass --apply --yes to execute. This is the command printed by TRUST.4's post-deploy-smoke + TRUST.8's restore-hint when a deploy fails \u2014 they tell the operator to run `sdt publish --restore-from-snapshot <id>`."
   ).option(
@@ -782,12 +810,15 @@ function publishCommand() {
       await runRestoreFromSnapshot(opts);
       return;
     }
-    if (!opts.pac) {
-      logger.error("--pac <path> is required (unless --restore-from-snapshot is given).");
+    const pacRef = opts.source ?? opts.pac;
+    if (!pacRef) {
+      logger.error(
+        "--source <path> (or its alias --pac) is required (unless --restore-from-snapshot is given)."
+      );
       process.exitCode = 1;
       return;
     }
-    const pacPath = import_node_path4.default.resolve(String(opts.pac));
+    const pacPath = import_node_path4.default.resolve(String(pacRef));
     const pac10 = await (0, import_pac2.readPac)(pacPath);
     const freshnessMode = opts.freshness ?? "warn";
     if (freshnessMode !== "skip") {
@@ -1575,12 +1606,17 @@ function validateCommand() {
     logger.dim(`  Profiles: ${Object.keys(loaded.project.deploymentProfiles ?? {}).length}`);
     if (opts.checkVariables) {
       const projRoot = import_node_path6.default.dirname(import_node_path6.default.resolve(String(opts.project)));
-      const fileContents = await Promise.all(
-        files.map(async (f) => {
+      const { results } = await import_core6.catalog.mapPool(
+        files,
+        async (f) => {
           const relPath = import_node_path6.default.relative(projRoot, f).split(import_node_path6.default.sep).join("/");
           const content = await import_node_fs6.promises.readFile(f, "utf8");
           return { path: relPath, content };
-        })
+        },
+        { concurrency: 16, stopOnError: true }
+      );
+      const fileContents = results.filter(
+        (r) => r !== void 0
       );
       const profilesBlock = loaded.project.deploymentProfiles ?? {};
       const profileVariableKeys = Object.values(profilesBlock).map(
@@ -1705,6 +1741,11 @@ function connectionCommand() {
     await (0, import_connection5.upsertProfile)(profile);
     logger.success(`Saved profile "${profile.name}".`);
   });
+  cmd.command("get <name>").description("Print a profile (secrets are redacted).").action(async (name) => {
+    const profile = await (0, import_connection5.getProfile)(String(name));
+    const redacted = { ...profile, auth: redactAuth(profile.auth) };
+    logger.info(JSON.stringify(redacted, null, 2));
+  });
   cmd.command("remove").description("Remove a connection profile.").argument("<name>").action(async (name) => {
     const ok = await (0, import_connection5.removeProfile)(String(name));
     if (ok) logger.success(`Removed profile "${name}".`);
@@ -1722,6 +1763,24 @@ function connectionCommand() {
   });
   return cmd;
 }
+function redactAuth(auth) {
+  const isPlaceholder = (v) => v.startsWith("env:") || v.startsWith("keyring:");
+  switch (auth.method) {
+    case "PASSWORD":
+    case "MFA":
+      return auth.password && !isPlaceholder(auth.password) ? { ...auth, password: "<redacted>" } : auth;
+    case "OAUTH":
+      return auth.token && !isPlaceholder(auth.token) ? { ...auth, token: "<redacted>" } : auth;
+    case "KEY_PAIR":
+      return auth.privateKeyPassphrase && !isPlaceholder(auth.privateKeyPassphrase) ? { ...auth, privateKeyPassphrase: "<redacted>" } : auth;
+    case "EXTERNAL_BROWSER":
+      return auth;
+    default: {
+      const _exhaustive = auth;
+      return _exhaustive;
+    }
+  }
+}
 
 // src/commands/scaffold.ts
 var import_commander10 = require("commander");