memorix 1.1.7 → 1.1.8

package/src/git/noise-filter.ts

@@ -1,210 +1,210 @@
-/**
- * Git Commit Noise Filter
- *
- * Filters out low-value commits that would dilute memory quality.
- * Codex identified this as the #1 priority for Git-Memory maturity:
- * "Not every commit is worth becoming a long-term memory."
- *
- * Built-in heuristics (no LLM needed):
- *   - Merge commit detection
- *   - Low-signal commit message patterns (typo, format, bump, lint, etc.)
- *   - File-only commits (only lockfiles, generated files, etc.)
- *
- * All thresholds are configurable via memorix.yml `git` section.
- */
-
-import type { CommitInfo } from './extractor.js';
-
-// ─── Types ───
-
-export interface NoiseFilterConfig {
-  /** Skip merge commits entirely (default: true) */
-  skipMergeCommits?: boolean;
-  /** File glob patterns to exclude (default: lockfiles, generated files) */
-  excludePatterns?: string[];
-  /** Additional commit message patterns to skip (regex strings) */
-  noiseKeywords?: string[];
-}
-
-export interface FilterResult {
-  /** Whether this commit should be skipped */
-  skip: boolean;
-  /** Human-readable reason for skipping */
-  reason?: string;
-}
-
-// ─── Built-in Noise Patterns ───
-
-/**
- * Commit message patterns that indicate low-value commits.
- * These are checked case-insensitively against the full subject line.
- */
-const NOISE_MESSAGE_PATTERNS: { pattern: RegExp; reason: string }[] = [
-  // Typo / formatting
-  { pattern: /^fix(?:ed)?\s*typo/i, reason: 'typo fix' },
-  { pattern: /^typo\b/i, reason: 'typo fix' },
-  { pattern: /^(?:fix|correct)\s*(?:spelling|grammar|whitespace|indent)/i, reason: 'formatting fix' },
-  { pattern: /^(?:format|lint|prettier|eslint|style)\b/i, reason: 'code formatting' },
-  { pattern: /^(?:auto-?format|code\s*style)/i, reason: 'auto-formatting' },
-
-  // Lockfile / dependency bumps
-  { pattern: /^(?:chore|build)?\(?deps?\)?[:/]?\s*(?:bump|update|upgrade)\b/i, reason: 'dependency bump' },
-  { pattern: /^bump\s+\S+\s+(?:from|to)\b/i, reason: 'dependency bump' },
-  { pattern: /^(?:yarn|npm|pnpm)\s+(?:lock|update)\b/i, reason: 'lockfile update' },
-  { pattern: /^update\s+(?:lock|yarn\.lock|package-lock)/i, reason: 'lockfile update' },
-
-  // Merge noise
-  { pattern: /^Merge\s+(?:branch|pull\s+request|remote)/i, reason: 'merge commit' },
-  { pattern: /^Merge\s+\S+\s+into\s+\S+/i, reason: 'merge commit' },
-
-  // WIP / temporary
-  { pattern: /^wip\b/i, reason: 'work in progress' },
-  { pattern: /^(?:temp|tmp|todo|fixme|hack)\b/i, reason: 'temporary commit' },
-  { pattern: /^(?:save|saving|checkpoint|backup)\b/i, reason: 'checkpoint commit' },
-
-  // Empty / trivial
-  { pattern: /^(?:initial\s+commit|first\s+commit|init)\s*$/i, reason: 'initial commit (no content)' },
-  { pattern: /^\.$/i, reason: 'empty message' },
-  { pattern: /^(?:update|change|fix|edit|modify)\s*$/i, reason: 'non-descriptive message' },
-
-  // CI automation
-  { pattern: /^\[(?:ci|cd)\s+skip\]/i, reason: 'CI skip marker' },
-
-  // Generated
-  { pattern: /^(?:auto-?generated|generated\s+by)\b/i, reason: 'auto-generated' },
-];
-
-/**
- * File patterns that indicate a commit is "lockfile-only" or "generated-only".
- * If ALL changed files match these patterns, the commit is considered noise.
- */
-const NOISE_FILE_PATTERNS: RegExp[] = [
-  /^package-lock\.json$/,
-  /^yarn\.lock$/,
-  /^pnpm-lock\.yaml$/,
-  /^Gemfile\.lock$/,
-  /^Cargo\.lock$/,
-  /^poetry\.lock$/,
-  /^composer\.lock$/,
-  /^go\.sum$/,
-  /\.min\.(js|css)$/,
-  /\.map$/,              // source maps
-  /^dist\//,             // build output
-  /^build\//,            // build output
-  /^\.DS_Store$/,
-  /^Thumbs\.db$/,
-];
-
-// ─── Filter Logic ───
-
-/**
- * Check if a commit is a merge commit (has 2+ parents).
- */
-function isMergeCommit(commit: CommitInfo): boolean {
-  // Merge commits typically have a subject starting with "Merge"
-  return /^Merge\s+/i.test(commit.subject);
-}
-
-/**
- * Check if ALL changed files match noise file patterns.
- * (Commit that only touches lockfiles/generated files.)
- */
-function isAllFilesNoise(files: string[], extraPatterns?: string[]): boolean {
-  if (files.length === 0) return false;
-
-  const allPatterns = [...NOISE_FILE_PATTERNS];
-  if (extraPatterns) {
-    for (const p of extraPatterns) {
-      try {
-        // Convert glob-like patterns to regex: *.lock → \.lock$, dist/* → ^dist\/
-        const escaped = p
-          .replace(/[.+^${}()|[\]\\]/g, '\\$&')
-          .replace(/\*/g, '.*')
-          .replace(/\?/g, '.');
-        allPatterns.push(new RegExp(escaped));
-      } catch { /* invalid pattern — skip */ }
-    }
-  }
-
-  return files.every(file =>
-    allPatterns.some(pattern => pattern.test(file)),
-  );
-}
-
-/**
- * Determine if a commit should be filtered out as noise.
- *
- * @param commit - The commit to evaluate
- * @param config - Optional config overrides from memorix.yml git section
- * @returns FilterResult with skip=true if the commit should be skipped
- */
-export function shouldFilterCommit(
-  commit: CommitInfo,
-  config?: NoiseFilterConfig,
-): FilterResult {
-  const skipMerge = config?.skipMergeCommits ?? true;
-
-  // 1. Merge commits
-  if (skipMerge && isMergeCommit(commit)) {
-    return { skip: true, reason: 'merge commit (git.skipMergeCommits: true)' };
-  }
-
-  // 2. Built-in message pattern matching
-  for (const { pattern, reason } of NOISE_MESSAGE_PATTERNS) {
-    if (pattern.test(commit.subject)) {
-      return { skip: true, reason: `noise pattern: ${reason}` };
-    }
-  }
-
-  // 3. User-defined noise keywords from memorix.yml
-  if (config?.noiseKeywords) {
-    for (const keyword of config.noiseKeywords) {
-      try {
-        const re = new RegExp(keyword, 'i');
-        if (re.test(commit.subject) || re.test(commit.body)) {
-          return { skip: true, reason: `user noise pattern: ${keyword}` };
-        }
-      } catch { /* invalid regex — skip */ }
-    }
-  }
-
-  // 4. All-noise-files check (lockfile-only, generated-only commits)
-  if (commit.filesChanged.length > 0 && isAllFilesNoise(commit.filesChanged, config?.excludePatterns)) {
-    return { skip: true, reason: 'all changed files are lockfiles/generated files' };
-  }
-
-  // 5. Extremely small commits with non-descriptive messages
-  // (e.g., single-character change with vague subject)
-  if (
-    commit.insertions + commit.deletions <= 2 &&
-    commit.subject.length < 15 &&
-    !/\b(fix|feat|refactor|security|break)/i.test(commit.subject)
-  ) {
-    return { skip: true, reason: 'trivial change with non-descriptive message' };
-  }
-
-  return { skip: false };
-}
-
-/**
- * Filter an array of commits, returning only signal-worthy ones.
- * Returns both kept and skipped with reasons (for CLI reporting).
- */
-export function filterCommits(
-  commits: CommitInfo[],
-  config?: NoiseFilterConfig,
-): { kept: CommitInfo[]; skipped: { commit: CommitInfo; reason: string }[] } {
-  const kept: CommitInfo[] = [];
-  const skipped: { commit: CommitInfo; reason: string }[] = [];
-
-  for (const commit of commits) {
-    const result = shouldFilterCommit(commit, config);
-    if (result.skip) {
-      skipped.push({ commit, reason: result.reason! });
-    } else {
-      kept.push(commit);
-    }
-  }
-
-  return { kept, skipped };
-}
+/**
+ * Git Commit Noise Filter
+ *
+ * Filters out low-value commits that would dilute memory quality.
+ * Codex identified this as the #1 priority for Git-Memory maturity:
+ * "Not every commit is worth becoming a long-term memory."
+ *
+ * Built-in heuristics (no LLM needed):
+ *   - Merge commit detection
+ *   - Low-signal commit message patterns (typo, format, bump, lint, etc.)
+ *   - File-only commits (only lockfiles, generated files, etc.)
+ *
+ * All thresholds are configurable via memorix.yml `git` section.
+ */
+
+import type { CommitInfo } from './extractor.js';
+
+// ─── Types ───
+
+export interface NoiseFilterConfig {
+  /** Skip merge commits entirely (default: true) */
+  skipMergeCommits?: boolean;
+  /** File glob patterns to exclude (default: lockfiles, generated files) */
+  excludePatterns?: string[];
+  /** Additional commit message patterns to skip (regex strings) */
+  noiseKeywords?: string[];
+}
+
+export interface FilterResult {
+  /** Whether this commit should be skipped */
+  skip: boolean;
+  /** Human-readable reason for skipping */
+  reason?: string;
+}
+
+// ─── Built-in Noise Patterns ───
+
+/**
+ * Commit message patterns that indicate low-value commits.
+ * These are checked case-insensitively against the full subject line.
+ */
+const NOISE_MESSAGE_PATTERNS: { pattern: RegExp; reason: string }[] = [
+  // Typo / formatting
+  { pattern: /^fix(?:ed)?\s*typo/i, reason: 'typo fix' },
+  { pattern: /^typo\b/i, reason: 'typo fix' },
+  { pattern: /^(?:fix|correct)\s*(?:spelling|grammar|whitespace|indent)/i, reason: 'formatting fix' },
+  { pattern: /^(?:format|lint|prettier|eslint|style)\b/i, reason: 'code formatting' },
+  { pattern: /^(?:auto-?format|code\s*style)/i, reason: 'auto-formatting' },
+
+  // Lockfile / dependency bumps
+  { pattern: /^(?:chore|build)?\(?deps?\)?[:/]?\s*(?:bump|update|upgrade)\b/i, reason: 'dependency bump' },
+  { pattern: /^bump\s+\S+\s+(?:from|to)\b/i, reason: 'dependency bump' },
+  { pattern: /^(?:yarn|npm|pnpm)\s+(?:lock|update)\b/i, reason: 'lockfile update' },
+  { pattern: /^update\s+(?:lock|yarn\.lock|package-lock)/i, reason: 'lockfile update' },
+
+  // Merge noise
+  { pattern: /^Merge\s+(?:branch|pull\s+request|remote)/i, reason: 'merge commit' },
+  { pattern: /^Merge\s+\S+\s+into\s+\S+/i, reason: 'merge commit' },
+
+  // WIP / temporary
+  { pattern: /^wip\b/i, reason: 'work in progress' },
+  { pattern: /^(?:temp|tmp|todo|fixme|hack)\b/i, reason: 'temporary commit' },
+  { pattern: /^(?:save|saving|checkpoint|backup)\b/i, reason: 'checkpoint commit' },
+
+  // Empty / trivial
+  { pattern: /^(?:initial\s+commit|first\s+commit|init)\s*$/i, reason: 'initial commit (no content)' },
+  { pattern: /^\.$/i, reason: 'empty message' },
+  { pattern: /^(?:update|change|fix|edit|modify)\s*$/i, reason: 'non-descriptive message' },
+
+  // CI automation
+  { pattern: /^\[(?:ci|cd)\s+skip\]/i, reason: 'CI skip marker' },
+
+  // Generated
+  { pattern: /^(?:auto-?generated|generated\s+by)\b/i, reason: 'auto-generated' },
+];
+
+/**
+ * File patterns that indicate a commit is "lockfile-only" or "generated-only".
+ * If ALL changed files match these patterns, the commit is considered noise.
+ */
+const NOISE_FILE_PATTERNS: RegExp[] = [
+  /^package-lock\.json$/,
+  /^yarn\.lock$/,
+  /^pnpm-lock\.yaml$/,
+  /^Gemfile\.lock$/,
+  /^Cargo\.lock$/,
+  /^poetry\.lock$/,
+  /^composer\.lock$/,
+  /^go\.sum$/,
+  /\.min\.(js|css)$/,
+  /\.map$/,              // source maps
+  /^dist\//,             // build output
+  /^build\//,            // build output
+  /^\.DS_Store$/,
+  /^Thumbs\.db$/,
+];
+
+// ─── Filter Logic ───
+
+/**
+ * Check if a commit is a merge commit (has 2+ parents).
+ */
+function isMergeCommit(commit: CommitInfo): boolean {
+  // Merge commits typically have a subject starting with "Merge"
+  return /^Merge\s+/i.test(commit.subject);
+}
+
+/**
+ * Check if ALL changed files match noise file patterns.
+ * (Commit that only touches lockfiles/generated files.)
+ */
+function isAllFilesNoise(files: string[], extraPatterns?: string[]): boolean {
+  if (files.length === 0) return false;
+
+  const allPatterns = [...NOISE_FILE_PATTERNS];
+  if (extraPatterns) {
+    for (const p of extraPatterns) {
+      try {
+        // Convert glob-like patterns to regex: *.lock → \.lock$, dist/* → ^dist\/
+        const escaped = p
+          .replace(/[.+^${}()|[\]\\]/g, '\\$&')
+          .replace(/\*/g, '.*')
+          .replace(/\?/g, '.');
+        allPatterns.push(new RegExp(escaped));
+      } catch { /* invalid pattern — skip */ }
+    }
+  }
+
+  return files.every(file =>
+    allPatterns.some(pattern => pattern.test(file)),
+  );
+}
+
+/**
+ * Determine if a commit should be filtered out as noise.
+ *
+ * @param commit - The commit to evaluate
+ * @param config - Optional config overrides from memorix.yml git section
+ * @returns FilterResult with skip=true if the commit should be skipped
+ */
+export function shouldFilterCommit(
+  commit: CommitInfo,
+  config?: NoiseFilterConfig,
+): FilterResult {
+  const skipMerge = config?.skipMergeCommits ?? true;
+
+  // 1. Merge commits
+  if (skipMerge && isMergeCommit(commit)) {
+    return { skip: true, reason: 'merge commit (git.skipMergeCommits: true)' };
+  }
+
+  // 2. Built-in message pattern matching
+  for (const { pattern, reason } of NOISE_MESSAGE_PATTERNS) {
+    if (pattern.test(commit.subject)) {
+      return { skip: true, reason: `noise pattern: ${reason}` };
+    }
+  }
+
+  // 3. User-defined noise keywords from memorix.yml
+  if (config?.noiseKeywords) {
+    for (const keyword of config.noiseKeywords) {
+      try {
+        const re = new RegExp(keyword, 'i');
+        if (re.test(commit.subject) || re.test(commit.body)) {
+          return { skip: true, reason: `user noise pattern: ${keyword}` };
+        }
+      } catch { /* invalid regex — skip */ }
+    }
+  }
+
+  // 4. All-noise-files check (lockfile-only, generated-only commits)
+  if (commit.filesChanged.length > 0 && isAllFilesNoise(commit.filesChanged, config?.excludePatterns)) {
+    return { skip: true, reason: 'all changed files are lockfiles/generated files' };
+  }
+
+  // 5. Extremely small commits with non-descriptive messages
+  // (e.g., single-character change with vague subject)
+  if (
+    commit.insertions + commit.deletions <= 2 &&
+    commit.subject.length < 15 &&
+    !/\b(fix|feat|refactor|security|break)/i.test(commit.subject)
+  ) {
+    return { skip: true, reason: 'trivial change with non-descriptive message' };
+  }
+
+  return { skip: false };
+}
+
+/**
+ * Filter an array of commits, returning only signal-worthy ones.
+ * Returns both kept and skipped with reasons (for CLI reporting).
+ */
+export function filterCommits(
+  commits: CommitInfo[],
+  config?: NoiseFilterConfig,
+): { kept: CommitInfo[]; skipped: { commit: CommitInfo; reason: string }[] } {
+  const kept: CommitInfo[] = [];
+  const skipped: { commit: CommitInfo; reason: string }[] = [];
+
+  for (const commit of commits) {
+    const result = shouldFilterCommit(commit, config);
+    if (result.skip) {
+      skipped.push({ commit, reason: result.reason! });
+    } else {
+      kept.push(commit);
+    }
+  }
+
+  return { kept, skipped };
+}

package/src/hooks/installers/index.ts

@@ -642,7 +642,7 @@ export function getGlobalConfigPath(agent: AgentName): string {
   }
 }
 
-function getAgentRulesPath(agent: AgentName, root: string, global = false): string {
+export function getAgentRulesPath(agent: AgentName, root: string, global = false): string {
   if (global) {
     switch (agent) {
       case 'windsurf':
@@ -1153,8 +1153,8 @@ function getAgentRulesContent(agent?: AgentName, scope: 'project' | 'global' = '
     ? '# Memorix — Memory Tools for Active Workspaces'
     : '# Memorix — Project Memory Tools';
   const intro = scope === 'global'
-    ? 'Use Memorix when the active workspace has Memorix tools available and prior context would materially help. Do not assume every workspace is configured for Memorix.'
-    : 'This repository is configured to use Memorix for persistent cross-session memory. Memory tools are available when you need context from prior sessions or when you learn something worth saving.';
+    ? 'Use Memorix when the active workspace has Memorix tools available and prior context would materially help. For non-trivial coding work, Memory Autopilot is the default entry point before local progress notes or broad file exploration. Do not assume every workspace is configured for Memorix.'
+    : 'This repository is configured to use Memorix for persistent cross-session memory. For non-trivial coding work, Memory Autopilot is the default entry point before local progress notes or broad file exploration.';
 
   if (agent === 'windsurf') {
     frontmatter = '---\ntrigger: always_on\n---\n\n';
@@ -1169,7 +1169,7 @@ function getAgentRulesContent(agent?: AgentName, scope: 'project' | 'global' = '
     '',
     '## Start with Memory Autopilot',
     '',
-    `For starting or continuing code work, use \x60memorix_project_context\x60 with the user's actual task before ad-hoc file reads, dev-log reads, or git archaeology. Memorix will choose a task-lensed brief (bugfix, feature, release, onboarding, refactor, docs, test, or general). Treat its "Start here" files as the first ${contextNoun} files to inspect.`,
+    `Default first step for non-trivial coding work: call \x60memorix_project_context\x60 with the user's actual task before progress files, dev-log reads, ad-hoc file reads, or git archaeology. Memorix will choose a task-lensed brief (bugfix, feature, release, onboarding, refactor, docs, test, or general). Treat its "Start here" files as the first ${contextNoun} files to inspect.`,
     '',
     'If the MCP tool is not visible yet but the client supports tool discovery or dynamic loading, search/select \x60memorix_project_context\x60 first. Run \x60memorix context --task "<task>"\x60 only after MCP is unavailable, disabled, or not discoverable, and pass the user\'s real task text. Do not skip memory, wait indefinitely on MCP startup, or hand-write tool-call syntax.',
     '',

package/src/hooks/official-skills.ts

@@ -42,7 +42,7 @@ export const OFFICIAL_MEMORIX_SKILLS: OfficialMemorixSkill[] = [
       '',
       '## Search Rules',
       '',
-      '- Use `memorix_project_context` before broad continuation work, unfamiliar code changes, ad-hoc file reads, or dev-log reads. Pass the user\'s actual task so Memorix can choose a bugfix, feature, release, onboarding, refactor, docs, test, or general brief. Start by reading its suggested files.',
+      '- Default first step for non-trivial coding work: call `memorix_project_context` before progress files, dev-log reads, ad-hoc file reads, or git archaeology. Pass the user\'s actual task so Memorix can choose a bugfix, feature, release, onboarding, refactor, docs, test, or general brief. Start by reading its suggested files.',
       '- In Claude Code headless/print-mode, an initial MCP `pending` status is not a failure by itself. Prefer MCP first: use the client\'s tool discovery/dynamic loading to find `memorix_project_context` when it is not in the first visible tool list.',
       '- Run `memorix context --task "<task>"` from the shell only after MCP is unavailable, disabled, or not discoverable, and pass the user\'s real task text. Do not skip memory, wait indefinitely on `pending`, or hand-write tool-call syntax.',
       '- Use `memorix_search` for a specific decision, bug, file, or prior change that the project context did not answer.',