@biffo/cli 0.278.4 → 0.279.1

package/dist/index.js

@@ -3012,6 +3012,35 @@ function standardArguments(pluginName, handler) {
     // `terraform plan` for a whole environment. Filtered by declaration below,
     // so a Lambda-backed legacy module that does not declare it is unaffected.
     ["cdn_distribution_arn", "module.cdn.distribution_arn"],
+    // biffo-template#1456: an installed plugin has no channel for
+    // instance-specific configuration at all, and the first thing that needs
+    // one is the deployment's own public origin — a plugin minting a
+    // user-visible URL (e.g. a tracked link) cannot derive that from
+    // BIFFO_CORE_API_URL, which is the execute-api gateway host, not the
+    // branded domain.
+    //
+    // Routed through the module's existing `environment_variables` map
+    // (declared in modules/plugins/_template/variables.tf, merged over
+    // BIFFO_CORE_API_URL/BIFFO_PLUGIN_NAME in that module's main.tf) rather
+    // than a new bespoke module variable — that hook already exists, already
+    // reaches the Lambda, and plugins.core.tf already uses the identical
+    // shape for orchestrator's email_branding_env. Adding a dedicated
+    // `public_base_url` variable would be a second channel doing the same
+    // job.
+    //
+    // `local.portal_url` is defined once in the *user-owned* main.tf
+    // (custom_domain when set, else the CloudFront distribution domain) and
+    // is reachable here because Terraform loads every *.tf file in a
+    // directory as one module — no argument is being added to a module block
+    // declared in another file (the #1538 trap), only a local referenced
+    // across files in the same directory, which Terraform has always allowed.
+    // It is safe to assume present in every environment this generator writes
+    // into: `listEnvironments` already requires `enabled_plugins` to be
+    // declared, and `portal_url` was extracted as a local two days before
+    // `enabled_plugins` existed at all (8dd54065, predating 64677295) — so
+    // any environment old enough to lack it predates the plugin system
+    // entirely and is already excluded by that check.
+    ["environment_variables", "{ BIFFO_PUBLIC_BASE_URL = local.portal_url }"],
     ["tags", "local.tags"]
   ];
 }
@@ -9248,7 +9277,7 @@ async function runPluginUpgrade(target, options, deps) {
     }
     if (manifest.tables.length > 0) {
       log.info(
-        `Generating migration for services/${entry.name}/'s ${manifest.tables.length} table(s)...`
+        `Checking for a migration for services/${entry.name}/'s ${manifest.tables.length} table(s)...`
       );
       const generatedPaths = await deps.migrations.generate(options.cwd, [entry.name]);
       for (const absPath of generatedPaths) {
@@ -9256,6 +9285,10 @@ async function runPluginUpgrade(target, options, deps) {
       }
       if (generatedPaths.length > 0) {
         log.success(`Generated migration: ${relative5(options.cwd, generatedPaths[0])}`);
+      } else {
+        log.info(
+          `${entry.name}'s tables and columns already match the manifest \u2014 no migration needed.`
+        );
       }
     }
     const label = currentVersion ? `${entry.name} ${currentVersion} -> ${entry.version}` : `${entry.name} to ${entry.version}`;
@@ -9341,7 +9374,9 @@ async function runLocalPluginRefresh(localPath, options, deps) {
       if (generatedPaths.length > 0) {
         log.success(`Generated migration: ${relative5(options.cwd, generatedPaths[0])}`);
       } else {
-        log.info(`${source.name}'s table set is unchanged \u2014 no migration needed.`);
+        log.info(
+          `${source.name}'s tables and columns already match the manifest \u2014 no migration needed.`
+        );
       }
     } else {
       log.info(`${source.name} declares no tables \u2014 nothing to migrate.`);
@@ -9419,7 +9454,7 @@ function printLocalDryRun(source, currentVersion, inTreeSource) {
   );
   if (source.manifest.tables.length > 0) {
     console.log(
-      `  Would check for a migration for ${source.manifest.tables.length} table(s) (only generated if the table set changed)`
+      `  Would check for a migration for ${source.manifest.tables.length} table(s) (generated for a new table or an added column on an already-migrated table; a removed/retyped/nullability-changed column stops the refresh instead \u2014 #1539)`
     );
   }
   console.log(`  Would commit:  chore(plugins): refresh ${source.name} from local checkout

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@biffo/cli",
-  "version": "0.278.4",
+  "version": "0.279.1",
   "description": "Biffo project scaffolding CLI",
   "license": "MIT",
   "type": "module",