@mnci/cli 4.10.5 → 4.10.7

package/README.md

@@ -1371,8 +1371,21 @@ pipeline installs `golangci-lint` itself (see below).
   generators, in that order — `convert-to-one-mod` refuses once `go.work`
   lists any module, so it has to happen before the first Go project exists.
   Every Go project then shares that module and imports its siblings as
-  `<module>/libs/<name>`, with no per-project manifests and no `replace`
-  directives.
+  `<module>/libs/<name>/<slice>`, with no per-project manifests and no
+  `replace` directives.
+- **A Go library is a capability of slice packages.** The plugin's library
+  generator writes `<name>.go` at the project root, which makes the root
+  package the whole library. mnci replaces it with a root `doc.go` and moves
+  the sample into one starter slice package (`libs/markdown-workspace/
+  markdownworkspace/markdown_workspace_use_case.go`), the Go spelling of the
+  `<kebab>.<role>.ts` files the TypeScript scaffolds get. Rename the starter
+  after the outcome it delivers and add one package per further outcome.
+- **`test` and `lint` reach every slice package.** The plugin's executors run
+  `go test ./...` and `golangci-lint run ./...` from the project root
+  (measured on `@nx-go/nx-go` 4.1.1), so mnci passes no package list. The e2e
+  plants a failing test and a lint finding in a nested package and asserts
+  both fail the project target, so a plugin change that stopped recursing
+  would surface there rather than as a silently green lib.
 - **The `go.work` multi-module layout was rejected deliberately.** Besides
   splitting dependencies across per-project manifests, it is brittle: a
   single stale `use` entry — a project directory removed by hand — makes

package/dist/cli.js

@@ -614,7 +614,7 @@ var ESLINT_PEER_OVERRIDES = {
   // both — and forcing those to v5 breaks them. So the blast radius is one
   // dependency edge per named parent, and a test asserts there is no top-level
   // entry.
-  "nx": { "brace-expansion": "^5.0.9", "smol-toml": "^1.7.1" },
+  "nx": { "brace-expansion": "^5.0.9", "smol-toml": "^1.7.1", "axios": "^1.20.0" },
   "@nx/js": { "brace-expansion": "^5.0.9" },
   "@nx/eslint": { "brace-expansion": "^5.0.9" },
   "@nx/eslint-plugin": { "brace-expansion": "^5.0.9" },
@@ -857,7 +857,7 @@ function vscodeWorkspace(workspaceName, existingTasks, existingLaunch, existingS
     (configuration) => !String(configuration.name ?? "").startsWith(LAUNCH_CONFIG_PREFIX)
   );
   const settings = { ...existingSettings, ...vscodeSettings() };
-  return JSON.stringify(
+  return toJson(
     {
       folders: [{ path: ".", name: workspaceName }],
       settings,
@@ -870,9 +870,7 @@ function vscodeWorkspace(workspaceName, existingTasks, existingLaunch, existingS
         version: existingLaunch?.version ?? "0.2.0",
         configurations: [...launchConfigurations(workspaceName), ...userConfigurations]
       }
-    },
-    null,
-    2
+    }
   );
 }
 var ROOT_SCRIPTS = {
@@ -1760,6 +1758,10 @@ function removeNxScaffolding(workspaceRoot) {
 }
 function removeNxAuthoredAgentFiles(workspaceRoot) {
   (0, import_node_fs2.rmSync)((0, import_node_path2.join)(workspaceRoot, NX_CLAUDE_SETTINGS), { force: true });
+  const claudeDirectory = (0, import_node_path2.join)(workspaceRoot, (0, import_node_path2.dirname)(NX_CLAUDE_SETTINGS));
+  if ((0, import_node_fs2.existsSync)(claudeDirectory) && (0, import_node_fs2.readdirSync)(claudeDirectory).length === 0) {
+    (0, import_node_fs2.rmSync)(claudeDirectory, { recursive: true, force: true });
+  }
   for (const name of NX_AUTHORED_AGENT_FILES) {
     const path = (0, import_node_path2.join)(workspaceRoot, name);
     if (!(0, import_node_fs2.existsSync)(path)) continue;
@@ -2638,6 +2640,65 @@ function goStartTarget(name) {
     options: { command: "go run .", cwd: `apps/${name}` }
   };
 }
+function goLibraryIdentifiers(projectName) {
+  const words = projectName.split(/[^a-z0-9]+/i).filter((word) => word.length > 0);
+  return {
+    packageName: words.join("").toLowerCase(),
+    functionName: words.map((word) => word.charAt(0).toUpperCase() + word.slice(1)).join(""),
+    fileStem: words.join("_").toLowerCase()
+  };
+}
+function reshapeGoLibraryScaffold(projectRoot, projectName) {
+  const { packageName, functionName, fileStem } = goLibraryIdentifiers(projectName);
+  (0, import_node_fs5.rmSync)((0, import_node_path6.join)(projectRoot, `${projectName}.go`), { force: true });
+  (0, import_node_fs5.rmSync)((0, import_node_path6.join)(projectRoot, `${projectName}_test.go`), { force: true });
+  const files = [
+    [
+      "doc.go",
+      `// Package ${packageName} is the ${projectName} capability. Its code lives in the
+// slice packages below this directory, one package per outcome.
+package ${packageName}
+`
+    ],
+    [
+      (0, import_node_path6.join)(packageName, "doc.go"),
+      `// Package ${packageName} is the starter slice of ${projectName}: rename it after
+// the outcome it delivers, and add one package per further outcome.
+package ${packageName}
+`
+    ],
+    [
+      (0, import_node_path6.join)(packageName, `${fileStem}_use_case.go`),
+      `package ${packageName}
+
+// ${functionName} is the generator's sample behaviour, kept so the slice builds and tests.
+func ${functionName}(name string) string {
+	return "${functionName} " + name
+}
+`
+    ],
+    [
+      (0, import_node_path6.join)(packageName, `${fileStem}_use_case_test.go`),
+      `package ${packageName}
+
+import "testing"
+
+func Test${functionName}(t *testing.T) {
+	if got := ${functionName}("works"); got != "${functionName} works" {
+		t.Fatalf("got %q", got)
+	}
+}
+`
+    ]
+  ];
+  for (const [relativePath, content] of files) {
+    const path = (0, import_node_path6.join)(projectRoot, relativePath);
+    if (!fileExists(path)) {
+      writeFileEnsured(path, content);
+    }
+  }
+  return packageName;
+}
 function prepareGo(workspaceRoot) {
   ensureGo(workspaceRoot);
   warnIfNoGolangciLint(workspaceRoot);
@@ -2706,10 +2767,11 @@ function addGoLib(workspaceRoot, name) {
     test: goTestTarget(),
     lint: goLintTarget()
   });
+  const slice = reshapeGoLibraryScaffold((0, import_node_path6.join)(workspaceRoot, "packages", name), name);
   registerProjectCommands(workspaceRoot, name, { build: false });
   const module2 = goModulePath(workspaceRoot);
   if (module2) {
-    logger.step(`Import it as ${module2}/packages/${name}`);
+    logger.step(`Import its starter slice as ${module2}/packages/${name}/${slice}`);
   }
 }
 function addGoInternalLib(workspaceRoot, name) {
@@ -2729,10 +2791,11 @@ function addGoInternalLib(workspaceRoot, name) {
     test: goTestTarget(),
     lint: goLintTarget()
   });
+  const slice = reshapeGoLibraryScaffold((0, import_node_path6.join)(workspaceRoot, "libs", name), name);
   registerProjectCommands(workspaceRoot, name, { build: false });
   const module2 = goModulePath(workspaceRoot);
   if (module2) {
-    logger.step(`Import it as ${module2}/libs/${name}`);
+    logger.step(`Import its starter slice as ${module2}/libs/${name}/${slice}`);
   }
 }
 
@@ -5271,7 +5334,25 @@ function runDoctor(workspaceRoot) {
 var import_node_fs13 = require("fs");
 var import_node_path20 = require("path");
 var NEVER_COPIED = /* @__PURE__ */ new Set([".git", "node_modules"]);
-var EXISTING_WINS = /* @__PURE__ */ new Set(["README.md", "LICENSE", "LICENSE.md", "LICENSE.txt"]);
+var AGENT_AND_CI_ENTRIES = [
+  "AGENTS.md",
+  "CLAUDE.md",
+  ".claude",
+  ".agents",
+  ".codex",
+  ".cursor",
+  ".gemini",
+  ".opencode",
+  "opencode.json",
+  ".github"
+];
+var EXISTING_WINS = /* @__PURE__ */ new Set([
+  "README.md",
+  "LICENSE",
+  "LICENSE.md",
+  "LICENSE.txt",
+  ...AGENT_AND_CI_ENTRIES
+]);
 var PREFLIGHT_BLOCKERS = [
   "package.json",
   "nx.json",
@@ -5398,27 +5479,30 @@ async function runNew(name, options) {
   const stack = await resolveStack(options);
   const nxCloud = options.nxCloud ?? (options.yes ? false : await promptNxCloud());
   const stagingParent = adoptTarget === void 0 ? process.cwd() : (0, import_node_fs14.mkdtempSync)((0, import_node_path21.join)((0, import_node_os.tmpdir)(), "mnci-new-"));
-  logger.step(`Creating Nx workspace '${workspaceName}' (preset: ts)`);
-  runNpx(
-    [
-      "--yes",
-      "create-nx-workspace@latest",
-      workspaceName,
-      "--preset=ts",
-      "--pm=npm",
-      nxCloud ? `--nxCloud=${nxCloudProviderValue(ci)}` : "--nxCloud=skip",
-      "--no-interactive"
-    ],
-    stagingParent
-  );
   let workspaceRoot = (0, import_node_path21.join)(stagingParent, workspaceName);
-  if (adoptTarget !== void 0) {
-    logger.step(`Adopting ${adoptTarget} (keeping its .git)`);
-    const adoption = adoptGeneratedWorkspace(workspaceRoot, adoptTarget);
-    for (const kept of adoption.kept) logger.detail(`kept the existing ${kept}`);
-    if (adoption.mergedGitignore) logger.detail("merged the generated .gitignore into the existing one");
-    (0, import_node_fs14.rmSync)(stagingParent, { recursive: true, force: true });
-    workspaceRoot = adoptTarget;
+  try {
+    logger.step(`Creating Nx workspace '${workspaceName}' (preset: ts)`);
+    runNpx(
+      [
+        "--yes",
+        "create-nx-workspace@latest",
+        workspaceName,
+        "--preset=ts",
+        "--pm=npm",
+        nxCloud ? `--nxCloud=${nxCloudProviderValue(ci)}` : "--nxCloud=skip",
+        "--no-interactive"
+      ],
+      stagingParent
+    );
+    if (adoptTarget !== void 0) {
+      logger.step(`Adopting ${adoptTarget} (keeping its .git)`);
+      const adoption = adoptGeneratedWorkspace(workspaceRoot, adoptTarget);
+      for (const kept of adoption.kept) logger.detail(`kept the existing ${kept}`);
+      if (adoption.mergedGitignore) logger.detail("merged the generated .gitignore into the existing one");
+      workspaceRoot = adoptTarget;
+    }
+  } finally {
+    if (adoptTarget !== void 0) (0, import_node_fs14.rmSync)(stagingParent, { recursive: true, force: true });
   }
   logger.step(
     "Applying MoNecromanCI overlay (VS Code workspace, release config, .npmrc, commitlint, pipeline, stack)"