@wraps.dev/cli 2.12.2 → 2.12.4

package/dist/cli.js

@@ -46,10 +46,6 @@ function isCI() {
     // Azure Pipelines
     "CODEBUILD_BUILD_ID",
     // AWS CodeBuild
-    "NETLIFY",
-    // Netlify
-    "VERCEL",
-    // Vercel
     "HEROKU_TEST_RUN_ID",
     // Heroku CI
     "BUDDY",
@@ -399,7 +395,9 @@ async function readAuthConfig() {
 async function saveAuthConfig(config2) {
   await ensureWrapsDir();
   const path3 = getConfigPath();
-  await writeFile2(path3, JSON.stringify(config2, null, 2), "utf-8");
+  const existing = await readAuthConfig();
+  const merged = existing ? { ...existing, ...config2 } : config2;
+  await writeFile2(path3, JSON.stringify(merged, null, 2), "utf-8");
   await chmod(path3, 384);
 }
 async function clearAuthConfig() {
@@ -418,7 +416,13 @@ async function resolveTokenAsync(flags2) {
     return sync;
   }
   const config2 = await readAuthConfig();
-  return config2?.auth?.token || null;
+  if (!config2?.auth?.token) {
+    return null;
+  }
+  if (config2.auth.expiresAt && new Date(config2.auth.expiresAt) <= /* @__PURE__ */ new Date()) {
+    return null;
+  }
+  return config2.auth.token;
 }
 var CONFIG_FILE;
 var init_config = __esm({
@@ -674,24 +678,34 @@ var init_client = __esm({
         await this.flush();
       }
       /**
-       * Enable telemetry
-       * Note: Won't override environment variable opt-outs (DO_NOT_TRACK, CI, etc.)
+       * Enable telemetry.
+       * Returns null if enabled, or a string describing why an env override prevented it.
        */
       enable() {
         this.config.setEnabled(true);
-        if (process.env.DO_NOT_TRACK === "1" || process.env.DO_NOT_TRACK === "true") {
+        const override = this.getEnvOverride();
+        if (override) {
           this.enabled = false;
-          return;
+          return override;
+        }
+        this.enabled = true;
+        return null;
+      }
+      /**
+       * Check if an environment variable is overriding the config.
+       * Returns a human-readable reason, or null if no override.
+       */
+      getEnvOverride() {
+        if (process.env.DO_NOT_TRACK === "1" || process.env.DO_NOT_TRACK === "true") {
+          return "DO_NOT_TRACK environment variable is set";
         }
         if (process.env.WRAPS_TELEMETRY_DISABLED === "1") {
-          this.enabled = false;
-          return;
+          return "WRAPS_TELEMETRY_DISABLED environment variable is set";
         }
         if (isCI()) {
-          this.enabled = false;
-          return;
+          return "CI environment detected";
         }
-        this.enabled = true;
+        return null;
       }
       /**
        * Disable telemetry
@@ -8552,7 +8566,7 @@ async function login(options) {
     });
     trackError("DEVICE_AUTH_FAILED", "auth:login", { step: "request_code" });
     clack.log.error("Failed to start device authorization.");
-    process.exit(1);
+    throw new Error("Failed to start device authorization.");
   }
   const {
     device_code,
@@ -8603,6 +8617,10 @@ async function login(options) {
         clack.log.info(`Organization: ${pc2.cyan(organizations[0].name)}`);
       } else if (organizations.length > 1) {
         clack.log.info(`${organizations.length} organizations available`);
+      } else {
+        clack.log.info(
+          `No organizations found. Create one at ${pc2.underline(`${baseURL}/onboarding`)} and run ${pc2.cyan("wraps auth login")} again.`
+        );
       }
       if (options.json) {
         console.log(
@@ -8616,7 +8634,8 @@ async function login(options) {
       return;
     }
     if (tokenError) {
-      const errorCode = tokenError.error || tokenError.code;
+      const err = tokenError;
+      const errorCode = err.error || err.code;
       if (errorCode === "authorization_pending") {
         continue;
       }
@@ -8633,7 +8652,7 @@ async function login(options) {
         trackError("ACCESS_DENIED", "auth:login", { step: "poll_token" });
         spinner9.stop("Denied.");
         clack.log.error("Authorization was denied.");
-        process.exit(1);
+        throw new Error("Authorization was denied.");
       }
       if (errorCode === "expired_token") {
         break;
@@ -8648,7 +8667,7 @@ async function login(options) {
   trackError("DEVICE_CODE_EXPIRED", "auth:login", { step: "poll_token" });
   spinner9.stop("Expired.");
   clack.log.error("Device code expired. Run `wraps auth login` to try again.");
-  process.exit(1);
+  throw new Error("Device code expired.");
 }
 
 // src/commands/auth/logout.ts
@@ -29497,8 +29516,12 @@ async function findExistingPhoneNumber(phoneNumberType) {
       }
     }
     return null;
-  } catch {
-    return null;
+  } catch (error) {
+    const name = error instanceof Error ? error.name : "";
+    if (name === "ResourceNotFoundException") {
+      return null;
+    }
+    throw error;
   }
 }
 async function createSMSPhoneNumber(phoneNumberType, optOutList) {
@@ -29885,7 +29908,12 @@ async function createSMSPhonePoolWithSDK(phoneNumberArn, region) {
         }
       }
     }
-  } catch {
+  } catch (error) {
+    const name = error instanceof Error ? error.name : "";
+    if (name === "ResourceNotFoundException") {
+    } else {
+      throw error;
+    }
   }
   try {
     const response = await client.send(
@@ -29901,8 +29929,12 @@ async function createSMSPhonePoolWithSDK(phoneNumberArn, region) {
       })
     );
     return response.PoolId;
-  } catch {
-    return;
+  } catch (error) {
+    const name = error instanceof Error ? error.name : "";
+    if (name === "ConflictException") {
+      return;
+    }
+    throw error;
   }
 }
 async function createSMSEventDestinationWithSDK(configurationSetName, snsTopicArn, region) {
@@ -29928,7 +29960,12 @@ async function createSMSEventDestinationWithSDK(configurationSetName, snsTopicAr
         return existingDest.EventDestinationName;
       }
     }
-  } catch {
+  } catch (error) {
+    const name = error instanceof Error ? error.name : "";
+    if (name === "ResourceNotFoundException") {
+    } else {
+      throw error;
+    }
   }
   try {
     const response = await client.send(
@@ -29942,8 +29979,12 @@ async function createSMSEventDestinationWithSDK(configurationSetName, snsTopicAr
       })
     );
     return response.EventDestination?.EventDestinationName || eventDestinationName;
-  } catch {
-    return eventDestinationName;
+  } catch (error) {
+    const name = error instanceof Error ? error.name : "";
+    if (name === "ConflictException") {
+      return eventDestinationName;
+    }
+    throw error;
   }
 }
 async function deleteSMSPhonePoolWithSDK(region) {
@@ -29972,7 +30013,12 @@ async function deleteSMSPhonePoolWithSDK(region) {
         }
       }
     }
-  } catch {
+  } catch (error) {
+    const name = error instanceof Error ? error.name : "";
+    if (name === "ResourceNotFoundException") {
+    } else {
+      throw error;
+    }
   }
 }
 async function deleteSMSEventDestinationWithSDK(configurationSetName, region) {
@@ -29986,7 +30032,12 @@ async function deleteSMSEventDestinationWithSDK(configurationSetName, region) {
         EventDestinationName: eventDestinationName
       })
     );
-  } catch {
+  } catch (error) {
+    const name = error instanceof Error ? error.name : "";
+    if (name === "ResourceNotFoundException") {
+    } else {
+      throw error;
+    }
   }
 }
 async function createSMSProtectConfigurationWithSDK(configurationSetName, region, options) {
@@ -30021,7 +30072,12 @@ async function createSMSProtectConfigurationWithSDK(configurationSetName, region
         break;
       }
     }
-  } catch {
+  } catch (error) {
+    const name = error instanceof Error ? error.name : "";
+    if (name === "ResourceNotFoundException") {
+    } else {
+      throw error;
+    }
   }
   try {
     let protectConfigId = existingProtectConfigId;
@@ -30087,8 +30143,12 @@ async function createSMSProtectConfigurationWithSDK(configurationSetName, region
       );
     }
     return protectConfigId;
-  } catch {
-    return;
+  } catch (error) {
+    const name = error instanceof Error ? error.name : "";
+    if (name === "ConflictException") {
+      return;
+    }
+    throw error;
   }
 }
 async function deleteSMSProtectConfigurationWithSDK(region) {
@@ -30125,7 +30185,12 @@ async function deleteSMSProtectConfigurationWithSDK(region) {
         }
       }
     }
-  } catch {
+  } catch (error) {
+    const name = error instanceof Error ? error.name : "";
+    if (name === "ResourceNotFoundException") {
+    } else {
+      throw error;
+    }
   }
 }
 
@@ -30148,7 +30213,29 @@ async function smsDestroy(options) {
     "Validating AWS credentials",
     async () => validateAWSCredentials()
   );
-  const region = await getAWSRegion();
+  let region = options.region || await getAWSRegion();
+  if (!(options.region || process.env.AWS_REGION || process.env.AWS_DEFAULT_REGION)) {
+    const smsConnections = await findConnectionsWithService(
+      identity.accountId,
+      "sms"
+    );
+    if (smsConnections.length === 1) {
+      region = smsConnections[0].region;
+    } else if (smsConnections.length > 1) {
+      const selectedRegion = await clack38.select({
+        message: "Multiple SMS deployments found. Which region to destroy?",
+        options: smsConnections.map((conn) => ({
+          value: conn.region,
+          label: conn.region
+        }))
+      });
+      if (clack38.isCancel(selectedRegion)) {
+        clack38.cancel("Operation cancelled");
+        process.exit(0);
+      }
+      region = selectedRegion;
+    }
+  }
   const metadata = await loadConnectionMetadata(identity.accountId, region);
   const smsService = metadata?.services?.sms;
   const storedStackName = smsService?.pulumiStackName;
@@ -30227,6 +30314,7 @@ async function smsDestroy(options) {
   await progress.execute("Cleaning up protect configuration", async () => {
     await deleteSMSProtectConfigurationWithSDK(region);
   });
+  let destroyFailed = false;
   try {
     await progress.execute(
       "Destroying SMS infrastructure (this may take 2-3 minutes)",
@@ -30242,8 +30330,14 @@ async function smsDestroy(options) {
         } catch (_error) {
           throw new Error("No SMS infrastructure found to destroy");
         }
-        await stack.destroy({ onOutput: () => {
+        await stack.refresh({ onOutput: () => {
         } });
+        await withTimeout(
+          stack.destroy({ onOutput: () => {
+          }, continueOnError: true }),
+          DEFAULT_PULUMI_TIMEOUT_MS,
+          "Pulumi destroy"
+        );
         await stack.workspace.removeStack(stackName);
       }
     );
@@ -30263,21 +30357,29 @@ async function smsDestroy(options) {
       throw errors.stackLocked();
     }
     trackError("DESTROY_FAILED", "sms destroy", { step: "destroy" });
-    clack38.log.error("SMS infrastructure destruction failed");
-    throw new Error(`SMS destruction failed: ${errorMessage}`);
+    destroyFailed = true;
+    clack38.log.warn(
+      "Some resources may not have been fully removed. You can re-run this command or clean up manually in the AWS console."
+    );
   }
   if (metadata) {
     removeServiceFromConnection(metadata, "sms");
     await saveConnectionMetadata(metadata);
   }
   progress.stop();
-  clack38.outro(pc41.green("SMS infrastructure has been removed"));
-  console.log(`
+  if (destroyFailed) {
+    clack38.outro(
+      pc41.yellow("SMS infrastructure partially removed. Metadata cleaned up.")
+    );
+  } else {
+    clack38.outro(pc41.green("SMS infrastructure has been removed"));
+    console.log(`
 ${pc41.bold("Cleaned up:")}`);
-  console.log(`  ${pc41.green("\u2713")} Phone number released`);
-  console.log(`  ${pc41.green("\u2713")} Configuration set deleted`);
-  console.log(`  ${pc41.green("\u2713")} Event processing infrastructure removed`);
-  console.log(`  ${pc41.green("\u2713")} IAM role deleted`);
+    console.log(`  ${pc41.green("\u2713")} Phone number released`);
+    console.log(`  ${pc41.green("\u2713")} Configuration set deleted`);
+    console.log(`  ${pc41.green("\u2713")} Event processing infrastructure removed`);
+    console.log(`  ${pc41.green("\u2713")} IAM role deleted`);
+  }
   console.log(
     `
 Run ${pc41.cyan("wraps sms init")} to deploy infrastructure again.
@@ -30285,6 +30387,7 @@ Run ${pc41.cyan("wraps sms init")} to deploy infrastructure again.
   );
   trackServiceRemoved("sms", {
     reason: "user_initiated",
+    partial_failure: destroyFailed,
     duration_ms: Date.now() - startTime
   });
 }
@@ -30988,31 +31091,64 @@ ${pc42.yellow(pc42.bold("Important Notes:"))}`);
         };
       }
     );
+    let sdkResourceWarning = false;
     if (outputs.phoneNumberArn) {
-      await progress.execute("Creating phone pool", async () => {
-        await createSMSPhonePoolWithSDK(outputs.phoneNumberArn, region);
-      });
+      try {
+        await progress.execute("Creating phone pool", async () => {
+          await createSMSPhonePoolWithSDK(outputs.phoneNumberArn, region);
+        });
+      } catch (sdkError) {
+        sdkResourceWarning = true;
+        const msg = sdkError instanceof Error ? sdkError.message : String(sdkError);
+        clack39.log.warn(`Phone pool creation failed: ${msg}`);
+        clack39.log.info(
+          `Run ${pc42.cyan("wraps sms sync")} to retry SDK resource creation.`
+        );
+      }
     }
     if (smsConfig.eventTracking?.enabled && outputs.configSetName && outputs.snsTopicArn) {
-      await progress.execute("Configuring event destination", async () => {
-        await createSMSEventDestinationWithSDK(
-          outputs.configSetName,
-          outputs.snsTopicArn,
-          region
+      try {
+        await progress.execute("Configuring event destination", async () => {
+          await createSMSEventDestinationWithSDK(
+            outputs.configSetName,
+            outputs.snsTopicArn,
+            region
+          );
+        });
+      } catch (sdkError) {
+        sdkResourceWarning = true;
+        const msg = sdkError instanceof Error ? sdkError.message : String(sdkError);
+        clack39.log.warn(`Event destination creation failed: ${msg}`);
+        clack39.log.info(
+          `Run ${pc42.cyan("wraps sms sync")} to retry SDK resource creation.`
         );
-      });
+      }
     }
     if (smsConfig.protectConfiguration?.enabled && outputs.configSetName) {
-      await progress.execute("Configuring fraud protection", async () => {
-        await createSMSProtectConfigurationWithSDK(
-          outputs.configSetName,
-          region,
-          {
-            allowedCountries: smsConfig.protectConfiguration?.allowedCountries,
-            aitFiltering: smsConfig.protectConfiguration?.aitFiltering
-          }
+      try {
+        await progress.execute("Configuring fraud protection", async () => {
+          await createSMSProtectConfigurationWithSDK(
+            outputs.configSetName,
+            region,
+            {
+              allowedCountries: smsConfig.protectConfiguration?.allowedCountries,
+              aitFiltering: smsConfig.protectConfiguration?.aitFiltering
+            }
+          );
+        });
+      } catch (sdkError) {
+        sdkResourceWarning = true;
+        const msg = sdkError instanceof Error ? sdkError.message : String(sdkError);
+        clack39.log.warn(`Protect configuration creation failed: ${msg}`);
+        clack39.log.info(
+          `Run ${pc42.cyan("wraps sms sync")} to retry SDK resource creation.`
         );
-      });
+      }
+    }
+    if (sdkResourceWarning) {
+      clack39.log.warn(
+        "Some SDK resources failed to create. Core infrastructure is deployed."
+      );
     }
   } catch (error) {
     trackServiceInit("sms", false, {
@@ -31141,7 +31277,8 @@ async function getPhoneNumberDetails(region) {
     }
     return null;
   } catch (error) {
-    console.error("Error fetching phone number:", error);
+    const errorMessage = error instanceof Error ? error.message : String(error);
+    clack40.log.error(`Error fetching phone number: ${errorMessage}`);
     return null;
   }
 }
@@ -31535,7 +31672,7 @@ Run ${pc45.cyan("wraps sms init")} to deploy SMS infrastructure first.
         smsConfig.protectConfiguration = {
           enabled: true,
           allowedCountries: ["US"],
-          aitFiltering: true
+          aitFiltering: false
         };
       }
     }
@@ -31585,9 +31722,14 @@ import {
 } from "@aws-sdk/client-pinpoint-sms-voice-v2";
 import * as clack43 from "@clack/prompts";
 import pc46 from "picocolors";
+
+// src/utils/sms/validation.ts
+init_esm_shims();
 function isValidPhoneNumber(phone) {
   return /^\+[1-9]\d{1,14}$/.test(phone);
 }
+
+// src/commands/sms/test.ts
 var SIMULATOR_DESTINATIONS = [
   { number: "+14254147755", country: "United States (US)" },
   { number: "+447860019066", country: "United Kingdom (GB)" },
@@ -31757,18 +31899,39 @@ Run ${pc46.cyan("wraps sms init")} to deploy SMS infrastructure.
       error: errorMessage,
       duration_ms: Date.now() - startTime
     });
-    if (errorMessage.includes("not verified")) {
+    const errorName = error instanceof Error ? error.name : "";
+    if (errorName === "ConflictException" || errorMessage.includes("opt-out")) {
+      clack43.log.error("Destination number has opted out of messages");
+      console.log(
+        `
+The recipient has opted out of receiving SMS messages.
+`
+      );
+    } else if (errorName === "ThrottlingException" || errorMessage.includes("spending limit")) {
+      clack43.log.error("SMS rate or spending limit reached");
+      console.log(
+        `
+Check your AWS account SMS spending limits in the console.
+`
+      );
+    } else if (errorName === "ValidationException") {
+      clack43.log.error(`Invalid request: ${errorMessage}`);
+    } else if (errorMessage.includes("not verified") || errorMessage.includes("not registered")) {
       clack43.log.error("Toll-free number registration is not complete");
       console.log(
         `
-Run ${pc46.cyan("wraps sms register")} to complete registration.
+Run ${pc46.cyan("wraps sms register")} to check registration status.
+`
+      );
+    } else if (errorName === "AccessDeniedException") {
+      clack43.log.error(
+        "Permission denied \u2014 IAM role may be missing SMS send permissions"
+      );
+      console.log(
+        `
+Run ${pc46.cyan("wraps sms upgrade")} to update IAM policies.
 `
       );
-    } else if (errorMessage.includes("opt-out")) {
-      clack43.log.error("Destination number has opted out of messages");
-    } else if (errorMessage.includes("spending limit")) {
-      clack43.log.error("AWS SMS spending limit reached");
-      console.log("\nVisit the AWS console to increase your spending limit.\n");
     } else {
       clack43.log.error(`Failed to send SMS: ${errorMessage}`);
     }
@@ -32721,9 +32884,6 @@ import {
 } from "@aws-sdk/client-pinpoint-sms-voice-v2";
 import * as clack45 from "@clack/prompts";
 import pc48 from "picocolors";
-function isValidPhoneNumber2(phone) {
-  return /^\+[1-9]\d{1,14}$/.test(phone);
-}
 async function smsVerifyNumber(options) {
   const startTime = Date.now();
   const progress = new DeploymentProgress();
@@ -32844,7 +33004,7 @@ Usage: ${pc48.cyan("wraps sms verify-number --delete --phone-number +14155551234
         if (!value) {
           return "Phone number is required";
         }
-        if (!isValidPhoneNumber2(value)) {
+        if (!isValidPhoneNumber(value)) {
           return "Please enter a valid phone number in E.164 format (e.g., +14155551234)";
         }
         return;
@@ -32855,7 +33015,7 @@ Usage: ${pc48.cyan("wraps sms verify-number --delete --phone-number +14155551234
       process.exit(0);
     }
     phoneNumber = result;
-  } else if (!isValidPhoneNumber2(phoneNumber)) {
+  } else if (!isValidPhoneNumber(phoneNumber)) {
     progress.stop();
     clack45.log.error(
       `Invalid phone number format: ${phoneNumber}. Use E.164 format (e.g., +14155551234)`
@@ -33088,12 +33248,21 @@ import * as clack47 from "@clack/prompts";
 import pc50 from "picocolors";
 async function telemetryEnable() {
   const client = getTelemetryClient();
-  client.enable();
-  clack47.log.success(pc50.green("Telemetry enabled"));
-  console.log(`   Config: ${pc50.dim(client.getConfigPath())}`);
-  console.log(`
+  const override = client.enable();
+  if (override) {
+    clack47.log.warn(
+      "Telemetry enabled in config, but overridden by environment"
+    );
+    console.log(`   Reason: ${pc50.yellow(override)}`);
+    console.log(`   Config: ${pc50.dim(client.getConfigPath())}`);
+    console.log();
+  } else {
+    clack47.log.success(pc50.green("Telemetry enabled"));
+    console.log(`   Config: ${pc50.dim(client.getConfigPath())}`);
+    console.log(`
    ${pc50.dim("Thank you for helping improve Wraps!")}
 `);
+  }
 }
 async function telemetryDisable() {
   const client = getTelemetryClient();
@@ -33109,9 +33278,13 @@ async function telemetryDisable() {
 async function telemetryStatus() {
   const client = getTelemetryClient();
   clack47.intro(pc50.bold("Telemetry Status"));
+  const override = client.getEnvOverride();
   const status2 = client.isEnabled() ? pc50.green("Enabled") : pc50.red("Disabled");
   console.log();
   console.log(`  ${pc50.bold("Status:")} ${status2}`);
+  if (!client.isEnabled() && override) {
+    console.log(`  ${pc50.bold("Reason:")} ${pc50.yellow(override)}`);
+  }
   console.log(`  ${pc50.bold("Config file:")} ${pc50.dim(client.getConfigPath())}`);
   if (client.isEnabled()) {
     console.log();
@@ -34108,7 +34281,8 @@ Run ${pc51.cyan("wraps --help")} for available commands.
         case "destroy":
           await smsDestroy({
             force: flags.force,
-            preview: flags.preview
+            preview: flags.preview,
+            region: flags.region
           });
           break;
         case "verify-number":
@@ -34268,12 +34442,6 @@ Available commands: ${pc51.cyan("login")}, ${pc51.cyan("status")}, ${pc51.cyan("
           );
           throw new Error(`Unknown auth command: ${subCommand || "(none)"}`);
       }
-      const authDuration = Date.now() - startTime;
-      const authCommandName = `auth:${subCommand}`;
-      trackCommand(authCommandName, {
-        success: true,
-        duration_ms: authDuration
-      });
       return;
     }
     if (primaryCommand === "aws" && subCommand) {