@topogram/cli 0.3.71 → 0.3.72

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@topogram/cli",
-  "version": "0.3.71",
+  "version": "0.3.72",
   "description": "Topogram CLI for checking Topogram workspaces and generating app bundles.",
   "license": "Apache-2.0",
   "repository": {

package/src/cli/command-parsers/project.js

@@ -43,8 +43,5 @@ export function parseProjectCommandArgs(args) {
   if (args[0] === "package" && args[1] === "update-cli") {
     return { packageCommand: "update-cli", inputPath: args.includes("--latest") ? "latest" : args[2] };
   }
-  if (args[0] === "migrate" && args[1] === "workspace-folder") {
-    return { migrateCommand: "workspace-folder", inputPath: commandPath(args, 2, ".") };
-  }
   return null;
 }

package/src/cli/dispatcher.js

@@ -10,7 +10,6 @@ import { runGenerateAppCommand } from "./commands/generate.js";
 import { runGeneratorCommand } from "./commands/generator.js";
 import { runGeneratorPolicyCommand } from "./commands/generator-policy.js";
 import { runImportCommand } from "./commands/import-runner.js";
-import { runMigrateCommand } from "./commands/migrate.js";
 import { runNewProjectCommand } from "./commands/new.js";
 import { runPackageCommand } from "./commands/package.js";
 import { runParseCommand, runResolveCommand } from "./commands/inspect.js";
@@ -242,10 +241,6 @@ export async function runCliDispatch(context) {
     return runPackageCommand({ commandArgs, inputPath, json: emitJson });
   }
 
-  if (commandArgs?.migrateCommand) {
-    return runMigrateCommand(inputPath, { write: shouldWrite, json: emitJson });
-  }
-
   if (commandArgs?.importCommand) {
     return runImportCommand({
       commandArgs,

package/src/cli/help.js

@@ -28,7 +28,6 @@ export function printUsage(options = {}) {
   console.log("   or: topogram catalog doctor [--json] [--catalog <path-or-source>]");
   console.log("   or: topogram catalog check <path-or-url> [--json]");
   console.log("   or: topogram catalog copy <id> <target> [--version <version>] [--json] [--catalog <path-or-source>]");
-  console.log("   or: topogram migrate workspace-folder [path] [--dry-run|--write] [--json]");
   console.log("   or: topogram package update-cli <version|--latest> [--json]");
   console.log("   or: topogram import <app-path> --out <target> [--from <track[,track]>] [--json]");
   console.log("   or: topogram import refresh [path] [--from <app-path>] [--dry-run] [--json]");

package/src/cli/migration-guidance.js

@@ -30,6 +30,9 @@ export function cliMigrationError(args) {
   if (args[0] === "component") {
     return "Command 'topogram component' was renamed to 'topogram widget'.";
   }
+  if (args[0] === "migrate") {
+    return "Command 'topogram migrate workspace-folder' was removed. Use topo/ workspaces or configure topogram.project.json workspace to a non-legacy relative path.";
+  }
   for (const [oldArg, newArg] of RENAMED_CLI_ARGS) {
     if (args.includes(oldArg)) {
       return `CLI flag '${oldArg}' was renamed to '${newArg}'.`;

package/src/generator/surfaces/databases/lifecycle-shared.js

@@ -209,8 +209,8 @@ discover_input_path() {
   )
   local candidate
   for candidate in "\${candidates[@]}"; do
-    if resolved="$(resolve_path_candidate "$candidate" "$PWD")" && [[ -d "$resolved/topogram" ]]; then
-      printf '%s\\n' "$resolved/topogram"
+    if resolved="$(resolve_path_candidate "$candidate" "$PWD")" && [[ -d "$resolved/topo" ]]; then
+      printf '%s\\n' "$resolved/topo"
       return
     fi
   done

package/src/workspace-paths.js

@@ -4,10 +4,10 @@ import fs from "node:fs";
 import path from "node:path";
 
 export const DEFAULT_TOPO_FOLDER_NAME = "topo";
-export const LEGACY_TOPOGRAM_FOLDER_NAME = "topogram";
 export const DEFAULT_WORKSPACE_PATH = `./${DEFAULT_TOPO_FOLDER_NAME}`;
 export const PROJECT_CONFIG_FILE = "topogram.project.json";
 
+const LEGACY_WORKSPACE_FOLDER_NAME = "topogram";
 const SIGNAL_SCAN_IGNORED_DIRS = new Set([
   ".git",
   ".next",
@@ -19,6 +19,7 @@ const SIGNAL_SCAN_IGNORED_DIRS = new Set([
   "coverage",
   "dist",
   "expected",
+  LEGACY_WORKSPACE_FOLDER_NAME,
   "node_modules",
   "tmp"
 ]);
@@ -128,6 +129,9 @@ export function normalizeWorkspaceConfigPath(workspacePath) {
   if (resolved === ".." || resolved.startsWith("../")) {
     throw new Error("topogram.project.json workspace must not escape the project root.");
   }
+  if (resolved === LEGACY_WORKSPACE_FOLDER_NAME || resolved.startsWith(`${LEGACY_WORKSPACE_FOLDER_NAME}/`)) {
+    throw new Error("topogram.project.json workspace must use ./topo or another non-legacy relative path.");
+  }
   return normalized;
 }
 
@@ -225,6 +229,9 @@ function signalWorkspaceCandidates(root) {
  */
 export function resolveWorkspaceContext(inputPath = ".") {
   const absolute = path.resolve(inputPath || ".");
+  if (isDirectory(absolute) && path.basename(absolute) === LEGACY_WORKSPACE_FOLDER_NAME && isWorkspaceSignalRoot(absolute)) {
+    throw new Error("Legacy workspace folders are not supported. Use topo/ or configure topogram.project.json workspace to a non-legacy relative path.");
+  }
   if (
     isDirectory(absolute) &&
     (

package/src/cli/commands/migrate.js

@@ -1,153 +0,0 @@
-// @ts-check
-
-import fs from "node:fs";
-import path from "node:path";
-
-import { stableStringify } from "../../format.js";
-import {
-  DEFAULT_TOPO_FOLDER_NAME,
-  DEFAULT_WORKSPACE_PATH,
-  LEGACY_TOPOGRAM_FOLDER_NAME,
-  PROJECT_CONFIG_FILE
-} from "../../workspace-paths.js";
-
-/**
- * @param {string|null|undefined} inputPath
- * @returns {string}
- */
-function projectRootForMigration(inputPath) {
-  const absolute = path.resolve(inputPath || ".");
-  const base = path.basename(absolute);
-  if (base === DEFAULT_TOPO_FOLDER_NAME || base === LEGACY_TOPOGRAM_FOLDER_NAME) {
-    return path.dirname(absolute);
-  }
-  return absolute;
-}
-
-/**
- * @param {string} projectRoot
- * @returns {string[]}
- */
-function caseCollisionEntries(projectRoot) {
-  if (!fs.existsSync(projectRoot) || !fs.statSync(projectRoot).isDirectory()) {
-    return [];
-  }
-  return fs.readdirSync(projectRoot)
-    .filter((/** @type {string} */ entry) => entry.toLowerCase() === DEFAULT_TOPO_FOLDER_NAME && entry !== DEFAULT_TOPO_FOLDER_NAME);
-}
-
-/**
- * @param {string} projectRoot
- * @returns {{ write: boolean, path: string|null, before: any|null, after: any|null }}
- */
-function plannedProjectConfigUpdate(projectRoot) {
-  const configPath = path.join(projectRoot, PROJECT_CONFIG_FILE);
-  if (!fs.existsSync(configPath)) {
-    return { write: false, path: null, before: null, after: null };
-  }
-  const before = JSON.parse(fs.readFileSync(configPath, "utf8"));
-  const after = { ...before };
-  const currentWorkspace = before.workspace;
-  if (currentWorkspace == null || currentWorkspace === "./topogram" || currentWorkspace === "topogram") {
-    after.workspace = DEFAULT_WORKSPACE_PATH;
-  }
-  return {
-    write: JSON.stringify(before) !== JSON.stringify(after),
-    path: configPath,
-    before,
-    after
-  };
-}
-
-/**
- * @param {string|null|undefined} inputPath
- * @param {{ write?: boolean, json?: boolean }} [options]
- * @returns {number}
- */
-export function runMigrateCommand(inputPath, options = {}) {
-  const projectRoot = projectRootForMigration(inputPath);
-  const legacyPath = path.join(projectRoot, LEGACY_TOPOGRAM_FOLDER_NAME);
-  const topoPath = path.join(projectRoot, DEFAULT_TOPO_FOLDER_NAME);
-  const write = Boolean(options.write);
-  /** @type {Array<Record<string, any>>} */
-  const diagnostics = [];
-  /** @type {Array<Record<string, any>>} */
-  const actions = [];
-
-  if (fs.existsSync(legacyPath) && fs.lstatSync(legacyPath).isSymbolicLink()) {
-    diagnostics.push({ severity: "error", message: `Refusing to migrate symlinked ${LEGACY_TOPOGRAM_FOLDER_NAME}/ at ${legacyPath}.` });
-  }
-  const collisions = caseCollisionEntries(projectRoot);
-  if (collisions.length > 0) {
-    diagnostics.push({ severity: "error", message: `Refusing to migrate because case-conflicting topo path(s) exist: ${collisions.join(", ")}.` });
-  }
-  if (fs.existsSync(legacyPath) && fs.existsSync(topoPath)) {
-    diagnostics.push({ severity: "error", message: `Refusing to migrate because both ${LEGACY_TOPOGRAM_FOLDER_NAME}/ and ${DEFAULT_TOPO_FOLDER_NAME}/ exist.` });
-  }
-  if (!fs.existsSync(legacyPath) && !fs.existsSync(topoPath)) {
-    diagnostics.push({ severity: "error", message: `No ${LEGACY_TOPOGRAM_FOLDER_NAME}/ or ${DEFAULT_TOPO_FOLDER_NAME}/ workspace folder found at ${projectRoot}.` });
-  }
-  if (fs.existsSync(topoPath) && fs.statSync(topoPath).isDirectory() && fs.readdirSync(topoPath).length > 0 && fs.existsSync(legacyPath)) {
-    diagnostics.push({ severity: "error", message: `Refusing to overwrite non-empty ${DEFAULT_TOPO_FOLDER_NAME}/ at ${topoPath}.` });
-  }
-
-  if (fs.existsSync(legacyPath) && diagnostics.length === 0) {
-    actions.push({
-      kind: "rename",
-      from: legacyPath,
-      to: topoPath
-    });
-  }
-  const configUpdate = plannedProjectConfigUpdate(projectRoot);
-  if (configUpdate.write) {
-    actions.push({
-      kind: "update_config",
-      path: configUpdate.path,
-      workspace: DEFAULT_WORKSPACE_PATH
-    });
-  }
-
-  const ok = diagnostics.filter((diagnostic) => diagnostic.severity === "error").length === 0;
-  if (ok && write) {
-    for (const action of actions) {
-      if (action.kind === "rename") {
-        fs.renameSync(action.from, action.to);
-      }
-      if (action.kind === "update_config" && configUpdate.path && configUpdate.after) {
-        fs.writeFileSync(configUpdate.path, `${JSON.stringify(configUpdate.after, null, 2)}\n`, "utf8");
-      }
-    }
-  }
-
-  const payload = {
-    ok,
-    dryRun: !write,
-    projectRoot,
-    legacyPath,
-    topoPath,
-    actions,
-    diagnostics,
-    errors: diagnostics.filter((diagnostic) => diagnostic.severity === "error").map((diagnostic) => diagnostic.message)
-  };
-  if (options.json) {
-    console.log(stableStringify(payload));
-  } else if (payload.ok) {
-    console.log(write ? "Workspace folder migration complete." : "Workspace folder migration dry run.");
-    if (actions.length === 0) {
-      console.log("No changes needed.");
-    }
-    for (const action of actions) {
-      if (action.kind === "rename") {
-        console.log(`Rename: ${action.from} -> ${action.to}`);
-      }
-      if (action.kind === "update_config") {
-        console.log(`Update ${action.path}: workspace ${DEFAULT_WORKSPACE_PATH}`);
-      }
-    }
-  } else {
-    for (const error of payload.errors) {
-      console.error(error);
-    }
-  }
-  return payload.ok ? 0 : 1;
-}