@ryuenn3123/agentic-senior-core 6.5.1 → 6.5.2

package/.agents/plugins/agentic-senior-core/.codex-plugin/plugin.json

@@ -1,6 +1,6 @@
 {
   "name": "agentic-senior-core",
-  "version": "6.5.1",
+  "version": "6.5.2",
   "description": "Universal AI coding rules. Write code like a staff engineer.",
   "skills": "./skills/",
   "hooks": "./hooks/hooks.json",

package/.agents/plugins/agentic-senior-core/hooks/dedup-gate.js

@@ -120,7 +120,7 @@ process.stdin.on('data', chunk => {
     const tmpDir = fs.mkdtempSync(path.join(os.tmpdir(), 'asc-dedup-'));
 
     const ignoreFlags = (config.ignoreDirs || []).map(function (d) { return '--ignore "' + d + '"'; }).join(' ');
-    const minTokens = config.minTokens || 50;
+    const minTokens = config.minTokens || 30;
     const scanCmd = ' "' + scanDir + '" --min-tokens ' + minTokens
       + ' --reporters json --silent --output "' + tmpDir + '" ' + ignoreFlags;
 
@@ -260,7 +260,7 @@ function loadDedupConfig() {
   }
   return {
     mode: 'advisory',
-    minTokens: 50,
+    minTokens: 30,
     ignoreDirs: [
       'tests', 'test', '__tests__', 'migrations', 'generated', 'node_modules',
       'dist', 'build', '.next', '.nuxt', '.expo', 'coverage', '.storybook',
@@ -327,9 +327,19 @@ function checkForDuplicates(report, filePath) {
       var cwd = process.cwd();
       var matchedFile = path.relative(cwd, path.resolve(otherRaw)).replace(/\\/g, '/');
       var lines = dup.lines || 0;
-      // jscpd v5 reports fragments; estimate overlap percentage from line count
-      var totalLines = (report.statistics && report.statistics.total && report.statistics.total.lines) || 1;
-      var percent = Math.round((lines / totalLines) * 100);
+      var totalFileLines = 1;
+      try {
+        totalFileLines = fs.readFileSync(path.resolve(filePath), 'utf8').split('\n').length || 1;
+      } catch (_) {
+        totalFileLines = dup.firstFile.lines || dup.secondFile.lines || lines || 1;
+      }
+      var percent = Math.round((lines / totalFileLines) * 100);
+
+      // Skip trivial matches (import boilerplate, small overlaps)
+      if (lines < 10 && percent < 10) {
+        continue;
+      }
+
       return { matchedFile: matchedFile, lines: lines, percent: percent };
     }
   }

package/.agents/plugins/agentic-senior-core/plugin.json

@@ -1,6 +1,6 @@
 {
   "name": "agentic-senior-core",
-  "version": "6.5.1",
+  "version": "6.5.2",
   "description": "Universal AI coding rules. Because your AI writes code like it gets paid by the line.",
   "contextFileName": "rules/agentic-senior-core.md",
   "rules": [

package/.asc/dedup-config.json

@@ -1,6 +1,6 @@
 {
   "mode": "advisory",
-  "minTokens": 50,
+  "minTokens": 30,
   "scanRoot": null,
   "ignoreDirs": [
     "tests", "test", "__tests__", "migrations", "generated", "node_modules",

package/gemini-extension.json

@@ -1,6 +1,6 @@
 {
   "name": "agentic-senior-core",
-  "version": "6.5.1",
+  "version": "6.5.2",
   "description": "Universal AI coding rules. Write code like a staff engineer.",
   "author": "fatidaprilian",
   "license": "MIT",

package/lib/cli/commands/git-hook-generator.mjs

@@ -27,6 +27,51 @@ const SOURCE_EXTENSIONS = new Set([${extensionsList}]);
 
 const JSCPD_TIMEOUT_MS = 10000;
 
+// Framework-conventional filenames that MUST be identical across directories by design.
+const FRAMEWORK_CONVENTIONAL_BASENAMES = new Set([
+  'page.tsx', 'page.jsx', 'page.ts', 'page.js',
+  'layout.tsx', 'layout.jsx', 'layout.ts', 'layout.js',
+  'loading.tsx', 'loading.jsx', 'loading.ts', 'loading.js',
+  'error.tsx', 'error.jsx', 'error.ts', 'error.js',
+  'not-found.tsx', 'not-found.jsx', 'not-found.ts', 'not-found.js',
+  'template.tsx', 'template.jsx', 'template.ts', 'template.js',
+  'route.tsx', 'route.ts', 'route.js',
+  'default.tsx', 'default.jsx', 'default.ts', 'default.js',
+  'root.tsx', 'root.jsx', 'root.ts', 'root.js',
+  'entry.server.tsx', 'entry.server.ts', 'entry.client.tsx', 'entry.client.ts',
+  '_layout.tsx', '_layout.jsx', '_layout.ts', '_layout.js',
+  'index.vue', 'app.vue',
+  '+page.svelte', '+layout.svelte', '+page.server.ts', '+page.server.js',
+  '+error.svelte', '+layout.server.ts', '+layout.server.js',
+  'index.ts', 'index.js', 'index.tsx', 'index.jsx',
+  'types.ts', 'types.d.ts',
+  'tailwind.config.ts', 'tailwind.config.js', 'tailwind.config.mjs',
+  'postcss.config.js', 'postcss.config.mjs', 'postcss.config.cjs',
+  'next.config.ts', 'next.config.js', 'next.config.mjs',
+  'vite.config.ts', 'vite.config.js', 'vite.config.mjs',
+  'tsconfig.json', 'jest.config.ts', 'jest.config.js', 'vitest.config.ts',
+]);
+
+const FRAMEWORK_CONVENTIONAL_SUFFIXES = [
+  '.component.ts', '.component.js', '.module.ts', '.service.ts',
+  '.pipe.ts', '.directive.ts', '.guard.ts', '.resolver.ts',
+  '.stories.tsx', '.stories.jsx', '.stories.ts', '.stories.js',
+  '.spec.ts', '.spec.tsx', '.spec.js', '.spec.jsx',
+  '.test.ts', '.test.tsx', '.test.js', '.test.jsx',
+];
+
+function isFrameworkConventional(basename) {
+  if (FRAMEWORK_CONVENTIONAL_BASENAMES.has(basename)) return true;
+  for (var i = 0; i < FRAMEWORK_CONVENTIONAL_SUFFIXES.length; i++) {
+    if (basename.endsWith(FRAMEWORK_CONVENTIONAL_SUFFIXES[i])) return true;
+  }
+  return false;
+}
+
+// Minimum thresholds — skip trivial matches (import boilerplate, etc.)
+const MIN_LINES_THRESHOLD = 10;
+const MIN_PERCENT_THRESHOLD = 10;
+
 function getStagedFiles(cwd) {
   try {
     const output = execSync('git diff --cached --name-only --diff-filter=ACM', { encoding: 'utf8', cwd });
@@ -48,7 +93,14 @@ function loadDedupConfig(cwd) {
       }
     } catch (_) {}
   }
-  return { minTokens: 30, ignoreDirs: ['tests', 'migrations', 'generated', 'node_modules'] };
+  return {
+    minTokens: 30,
+    ignoreDirs: [
+      'tests', 'test', '__tests__', 'migrations', 'generated', 'node_modules',
+      'dist', 'build', '.next', '.nuxt', '.expo', 'coverage', '.storybook',
+      'prisma/migrations', 'android', 'ios',
+    ],
+  };
 }
 
 function runEslintAutoFix(cwd, stagedFiles) {
@@ -126,10 +178,34 @@ function checkForDuplicates(report, stagedSourceFiles, cwd) {
 
     if (isFirstStaged || isSecondStaged) {
       const stagedFile = isFirstStaged ? normalizedStagedMap.get(firstName) : normalizedStagedMap.get(secondName);
-      const matchedFile = isFirstStaged ? path.basename(dup.secondFile.name) : path.basename(dup.firstFile.name);
+      const otherRaw = isFirstStaged ? dup.secondFile.name : dup.firstFile.name;
+      const otherBasename = path.basename(otherRaw);
+      const stagedBasename = path.basename(stagedFile);
+
+      // Skip framework-conventional filenames in different directories
+      if (isFrameworkConventional(stagedBasename)
+          && isFrameworkConventional(otherBasename)
+          && path.dirname(path.resolve(cwd, stagedFile)) !== path.dirname(path.resolve(cwd, otherRaw))) {
+        continue;
+      }
+
       const lines = dup.lines || 0;
-      const totalLines = (report.statistics && report.statistics.total && report.statistics.total.lines) || 1;
-      const percent = Math.round((lines / totalLines) * 100);
+      // Calculate percent relative to the actual staged file size
+      let totalFileLines = 1;
+      try {
+        totalFileLines = fs.readFileSync(path.resolve(cwd, stagedFile), 'utf8').split('\\n').length || 1;
+      } catch (_) {
+        totalFileLines = dup.firstFile.lines || dup.secondFile.lines || lines || 1;
+      }
+      const percent = Math.round((lines / totalFileLines) * 100);
+
+      // Skip trivial matches (import boilerplate, small overlaps)
+      if (lines < MIN_LINES_THRESHOLD && percent < MIN_PERCENT_THRESHOLD) {
+        continue;
+      }
+
+      // Show relative path for actionable messages
+      const matchedFile = path.relative(cwd, path.resolve(cwd, otherRaw)).replace(/\\\\/g, '/');
       return { stagedFile, matchedFile, lines, percent };
     }
   }
@@ -185,8 +261,11 @@ function runPreCommitGate() {
   })));
 
   const tmpDir = fs.mkdtempSync(path.join(os.tmpdir(), 'asc-git-dedup-'));
-  const ignoreFlags = (config.ignoreDirs || ['tests', 'migrations', 'generated', 'node_modules'])
-    .map(d => \`--ignore "\${d}"\`).join(' ');
+  const ignoreFlags = (config.ignoreDirs || [
+    'tests', 'test', '__tests__', 'migrations', 'generated', 'node_modules',
+    'dist', 'build', '.next', '.nuxt', '.expo', 'coverage', '.storybook',
+    'prisma/migrations', 'android', 'ios',
+  ]).map(d => \`--ignore "\${d}"\`).join(' ');
   const minTokens = config.minTokens || 30;
 
   let finding = null;

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@ryuenn3123/agentic-senior-core",
-  "version": "6.5.1",
+  "version": "6.5.2",
   "type": "module",
   "description": "Agentic Senior Core: Universal AI coding rules and workflows. Write code like a staff engineer, not a junior.",
   "bin": {

package/plugin.yaml

@@ -1,5 +1,5 @@
 name: agentic-senior-core
-version: 6.5.1
+version: 6.5.2
 description: Universal AI coding rules. Write code like a staff engineer.
 author: fatidaprilian
 provides_hooks: