stackmem 1.0.16 → 1.0.18

package/dist/cli.js

@@ -353,7 +353,7 @@ async function confirmSupersede(oldId, newContent) {
 }
 async function cmdSave(project, type, content, force = false) {
     if (!MEMORY_TYPES.includes(type)) {
-        fail(`Invalid type "${type}". Must be one of: ${MEMORY_TYPES.join(", ")}`);
+        fail(`Invalid memory type '${type}'. Valid types: decision, constraint, discovery, rejection.`);
     }
     const { data: existingRows, error: existingError } = await supabase
         .from("memories")
@@ -930,6 +930,24 @@ function registerMcpServer(distIndexPath) {
     nodeFs.writeFileSync(claudeConfigPath, JSON.stringify(config, null, 2), "utf-8");
     console.log("stackmem MCP server registered with Claude Code.");
 }
+const DEFAULT_GITIGNORE_CONTENT = `node_modules/
+dist/
+.env
+.env.local
+.DS_Store
+*.log
+coverage/
+`;
+// Returns true if the file was created, false if it already existed
+// (and was left untouched).
+function writeGitignore(resolvedPath) {
+    const gitignorePath = nodePath.join(resolvedPath, ".gitignore");
+    if (nodeFs.existsSync(gitignorePath)) {
+        return false;
+    }
+    nodeFs.writeFileSync(gitignorePath, DEFAULT_GITIGNORE_CONTENT, "utf-8");
+    return true;
+}
 function writeClaudeMd(project, resolvedPath) {
     const claudeMdPath = nodePath.join(resolvedPath, "CLAUDE.md");
     if (nodeFs.existsSync(claudeMdPath)) {
@@ -1171,9 +1189,10 @@ async function cmdInit(project, targetPath) {
     const resolvedPath = nodePath.resolve(targetPath);
     const gitDir = nodePath.join(resolvedPath, ".git");
     if (!nodeFs.existsSync(gitDir)) {
-        console.error(`No git repo found at ${targetPath}`);
+        console.error(`No git repo found at ${targetPath}. Run 'git init' first, then run 'cm init' again.`);
         process.exit(1);
     }
+    const gitignoreCreated = writeGitignore(resolvedPath);
     const codingMemoryDir = nodePath.join(resolvedPath, ".coding-memory");
     nodeFs.mkdirSync(codingMemoryDir, { recursive: true });
     const hookLogicPath = nodePath.join(codingMemoryDir, "post-commit-hook.mjs");
@@ -1193,6 +1212,9 @@ async function cmdInit(project, targetPath) {
     const windsurfRulesWritten = writeWindsurfRules(resolvedPath, rulesContent);
     const backendOk = await verifyBackendConnection(project);
     console.log("");
+    if (gitignoreCreated) {
+        console.log("✓ .gitignore created");
+    }
     console.log("✓ Git hook installed");
     console.log("✓ MCP server registered");
     console.log("✓ CLAUDE.md written");
@@ -1627,7 +1649,14 @@ async function main() {
             let explicitProject;
             let type;
             let rest;
-            if (MEMORY_TYPES.includes(positional[0])) {
+            // Exactly two positional args can only mean <type> <content> with the
+            // project omitted — an explicit project would need a third slot for
+            // content, which wouldn't exist. Without this check, a typo'd type
+            // (failing the MEMORY_TYPES.includes test below) used to fall into
+            // the "explicit project" branch instead, shifting content out of
+            // existence and surfacing a generic usage() dump instead of the
+            // targeted invalid-type error.
+            if (positional.length === 2 || MEMORY_TYPES.includes(positional[0])) {
                 explicitProject = null;
                 [type, ...rest] = positional;
             }
@@ -1635,6 +1664,9 @@ async function main() {
                 explicitProject = positional[0] ?? null;
                 [, type, ...rest] = positional;
             }
+            if (type !== undefined && !MEMORY_TYPES.includes(type)) {
+                fail(`Invalid memory type '${type}'. Valid types: decision, constraint, discovery, rejection.`);
+            }
             if (!type || rest.length === 0)
                 usage();
             const project = requireProject(explicitProject);

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "stackmem",
-  "version": "1.0.16",
+  "version": "1.0.18",
   "description": "",
   "main": "index.js",
   "type": "module",

package/src/cli.ts

@@ -411,7 +411,7 @@ async function confirmSupersede(oldId: string, newContent: string): Promise<bool
 
 async function cmdSave(project: string, type: string, content: string, force = false) {
   if (!MEMORY_TYPES.includes(type as MemoryType)) {
-    fail(`Invalid type "${type}". Must be one of: ${MEMORY_TYPES.join(", ")}`);
+    fail(`Invalid memory type '${type}'. Valid types: decision, constraint, discovery, rejection.`);
   }
 
 
@@ -1059,6 +1059,28 @@ function registerMcpServer(distIndexPath: string): void {
   console.log("stackmem MCP server registered with Claude Code.");
 }
 
+const DEFAULT_GITIGNORE_CONTENT = `node_modules/
+dist/
+.env
+.env.local
+.DS_Store
+*.log
+coverage/
+`;
+
+// Returns true if the file was created, false if it already existed
+// (and was left untouched).
+function writeGitignore(resolvedPath: string): boolean {
+  const gitignorePath = nodePath.join(resolvedPath, ".gitignore");
+
+  if (nodeFs.existsSync(gitignorePath)) {
+    return false;
+  }
+
+  nodeFs.writeFileSync(gitignorePath, DEFAULT_GITIGNORE_CONTENT, "utf-8");
+  return true;
+}
+
 function writeClaudeMd(project: string, resolvedPath: string): void {
   const claudeMdPath = nodePath.join(resolvedPath, "CLAUDE.md");
 
@@ -1333,10 +1355,12 @@ async function cmdInit(project: string, targetPath: string) {
   const gitDir = nodePath.join(resolvedPath, ".git");
 
   if (!nodeFs.existsSync(gitDir)) {
-    console.error(`No git repo found at ${targetPath}`);
+    console.error(`No git repo found at ${targetPath}. Run 'git init' first, then run 'cm init' again.`);
     process.exit(1);
   }
 
+  const gitignoreCreated = writeGitignore(resolvedPath);
+
   const codingMemoryDir = nodePath.join(resolvedPath, ".coding-memory");
   nodeFs.mkdirSync(codingMemoryDir, { recursive: true });
 
@@ -1365,6 +1389,9 @@ async function cmdInit(project: string, targetPath: string) {
   const backendOk = await verifyBackendConnection(project);
 
   console.log("");
+  if (gitignoreCreated) {
+    console.log("✓ .gitignore created");
+  }
   console.log("✓ Git hook installed");
   console.log("✓ MCP server registered");
   console.log("✓ CLAUDE.md written");
@@ -1858,7 +1885,14 @@ async function main() {
       let explicitProject: string | null;
       let type: string | undefined;
       let rest: string[];
-      if (MEMORY_TYPES.includes(positional[0] as MemoryType)) {
+      // Exactly two positional args can only mean <type> <content> with the
+      // project omitted — an explicit project would need a third slot for
+      // content, which wouldn't exist. Without this check, a typo'd type
+      // (failing the MEMORY_TYPES.includes test below) used to fall into
+      // the "explicit project" branch instead, shifting content out of
+      // existence and surfacing a generic usage() dump instead of the
+      // targeted invalid-type error.
+      if (positional.length === 2 || MEMORY_TYPES.includes(positional[0] as MemoryType)) {
         explicitProject = null;
         [type, ...rest] = positional;
       } else {
@@ -1866,6 +1900,10 @@ async function main() {
         [, type, ...rest] = positional;
       }
 
+      if (type !== undefined && !MEMORY_TYPES.includes(type as MemoryType)) {
+        fail(`Invalid memory type '${type}'. Valid types: decision, constraint, discovery, rejection.`);
+      }
+
       if (!type || rest.length === 0) usage();
       const project = requireProject(explicitProject);
       await cmdSave(project, type, rest.join(" "), force);