@ryuenn3123/agentic-senior-core 6.5.0 → 6.5.2

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

@@ -1,6 +1,6 @@
 {
   "name": "agentic-senior-core",
-  "version": "6.5.0",
+  "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

@@ -20,6 +20,63 @@ const JSCPD_TIMEOUT_MS = 10000;
 // Recognized source root directories — scan scope walks up to the first match
 const SOURCE_ROOTS = ['src', 'lib', 'app'];
 
+// Framework-conventional filenames that MUST be identical across directories by design.
+// Matches where both files share one of these basenames in different dirs are not duplication.
+const FRAMEWORK_CONVENTIONAL_BASENAMES = new Set([
+  // Next.js App Router
+  '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',
+  // Remix
+  'root.tsx', 'root.jsx', 'root.ts', 'root.js',
+  'entry.server.tsx', 'entry.server.ts', 'entry.client.tsx', 'entry.client.ts',
+  // Expo Router
+  '_layout.tsx', '_layout.jsx', '_layout.ts', '_layout.js',
+  // Nuxt
+  'index.vue', 'app.vue',
+  // SvelteKit
+  '+page.svelte', '+layout.svelte', '+page.server.ts', '+page.server.js',
+  '+error.svelte', '+layout.server.ts', '+layout.server.js',
+  // Common convention / barrel files
+  'index.ts', 'index.js', 'index.tsx', 'index.jsx',
+  'types.ts', 'types.d.ts',
+  // Config files (declarative, structurally similar across projects)
+  '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',
+]);
+
+// Suffix patterns for frameworks that mandate a naming convention (e.g. Angular).
+// Files matching the same suffix in different dirs are structural, not copy-paste.
+const FRAMEWORK_CONVENTIONAL_SUFFIXES = [
+  // Angular (*.component.ts, *.module.ts, *.service.ts, *.pipe.ts, *.directive.ts)
+  '.component.ts', '.component.js', '.module.ts', '.service.ts',
+  '.pipe.ts', '.directive.ts', '.guard.ts', '.resolver.ts',
+  // Storybook
+  '.stories.tsx', '.stories.jsx', '.stories.ts', '.stories.js',
+  // Test / spec (structurally similar boilerplate across suites)
+  '.spec.ts', '.spec.tsx', '.spec.js', '.spec.jsx',
+  '.test.ts', '.test.tsx', '.test.js', '.test.jsx',
+];
+
+function hasConventionalSuffix(basename) {
+  for (var i = 0; i < FRAMEWORK_CONVENTIONAL_SUFFIXES.length; i++) {
+    if (basename.endsWith(FRAMEWORK_CONVENTIONAL_SUFFIXES[i])) return true;
+  }
+  return false;
+}
+
+function isFrameworkConventional(basename) {
+  return FRAMEWORK_CONVENTIONAL_BASENAMES.has(basename) || hasConventionalSuffix(basename);
+}
+
 let inputBuffer = '';
 process.stdin.setEncoding('utf8');
 process.stdin.on('data', chunk => {
@@ -201,7 +258,15 @@ function loadDedupConfig() {
       }
     } catch (_) {}
   }
-  return { mode: 'advisory', minTokens: 30, ignoreDirs: ['tests', 'migrations', 'generated', 'node_modules'] };
+  return {
+    mode: 'advisory',
+    minTokens: 30,
+    ignoreDirs: [
+      'tests', 'test', '__tests__', 'migrations', 'generated', 'node_modules',
+      'dist', 'build', '.next', '.nuxt', '.expo', 'coverage', '.storybook',
+      'prisma/migrations', 'android', 'ios',
+    ],
+  };
 }
 
 // minimal: attempt-then-fallback — try scan directly, fall back on failure.
@@ -238,6 +303,7 @@ function checkForDuplicates(report, filePath) {
   if (duplicates.length === 0) return null;
 
   var normalizedTarget = path.resolve(filePath).replace(/\\/g, '/').toLowerCase();
+  var targetBasename = path.basename(filePath);
 
   for (var i = 0; i < duplicates.length; i++) {
     var dup = duplicates[i];
@@ -245,13 +311,35 @@ function checkForDuplicates(report, filePath) {
     var secondName = path.resolve(dup.secondFile.name).replace(/\\/g, '/').toLowerCase();
 
     if (firstName === normalizedTarget || secondName === normalizedTarget) {
-      var matchedFile = firstName === normalizedTarget
-        ? path.basename(dup.secondFile.name)
-        : path.basename(dup.firstFile.name);
+      var otherRaw = firstName === normalizedTarget ? dup.secondFile.name : dup.firstFile.name;
+      var otherBasename = path.basename(otherRaw);
+
+      // Skip framework-conventional filenames in different directories — identical names
+      // are mandated by the framework (e.g. Next.js page.tsx, Angular *.component.ts),
+      // not copy-paste duplication.
+      if (isFrameworkConventional(targetBasename)
+          && isFrameworkConventional(otherBasename)
+          && path.dirname(path.resolve(filePath)) !== path.dirname(path.resolve(otherRaw))) {
+        continue;
+      }
+
+      // Show relative path instead of bare basename so the developer knows WHICH file.
+      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.0",
+  "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

@@ -2,5 +2,9 @@
   "mode": "advisory",
   "minTokens": 30,
   "scanRoot": null,
-  "ignoreDirs": ["tests", "migrations", "generated", "node_modules"]
+  "ignoreDirs": [
+    "tests", "test", "__tests__", "migrations", "generated", "node_modules",
+    "dist", "build", ".next", ".nuxt", ".expo", "coverage", ".storybook",
+    "prisma/migrations", "android", "ios"
+  ]
 }

package/gemini-extension.json

@@ -1,6 +1,6 @@
 {
   "name": "agentic-senior-core",
-  "version": "6.5.0",
+  "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.0",
+  "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.0
+version: 6.5.2
 description: Universal AI coding rules. Write code like a staff engineer.
 author: fatidaprilian
 provides_hooks: