@titan-design/code-graph 0.9.0 → 0.9.1

package/README.md

@@ -234,7 +234,9 @@ fail a check.
 flags every symbol strictly above the 95th percentile of `symbol_body_lines` over all symbols
 that carry it, interpolated linearly between ranks. `percentile` runs from 50 to 100. The rule
 stays silent until `minSample` nodes (default 20) carry the metric. Each violation's
-`threshold` is the computed percentile value.
+`threshold` is the computed percentile value. For sparse metrics whose percentile sits at or
+near zero, `floor` flags a node only if its value also exceeds that absolute number, and
+`rankNonZero: true` ranks and gates on non-zero carriers only.
 
 ```ts
 import { checkSnapshot, loadCheckRules, openCodeGraph } from "@titan-design/code-graph";

package/dist/index.js

@@ -1634,6 +1634,9 @@ function isInsideGitRepo(repoRoot) {
 function resolveGitRef(repoRoot, ref) {
   return runGit(repoRoot, ["rev-parse", ref]);
 }
+function isGitAncestor(repoRoot, ancestor, descendant) {
+  return runGit(repoRoot, ["merge-base", "--is-ancestor", ancestor, descendant]) !== null;
+}
 function detectRenames(options) {
   const threshold = options.similarityThreshold ?? 80;
   const args = [
@@ -1805,9 +1808,15 @@ function priorSnapshotForRef(store, ref, options = {}) {
   const byCommit = commit ? candidates2.find((s) => s.commitHash === commit) : void 0;
   return byCommit ?? candidates2.find((s) => s.ref === ref) ?? null;
 }
-function aliasBaseFor(store, ref) {
+function aliasTargetCommit(target) {
+  return target.commit ?? detectGitHead(target.repoRoot);
+}
+function aliasBaseFor(store, ref, target) {
   const committed = allSnapshots(store).filter((s) => s.commitHash).reverse();
-  return committed.find((s) => s.ref === ref) ?? committed[0] ?? null;
+  const candidate = committed.find((s) => s.ref === ref) ?? committed[0];
+  if (!candidate?.commitHash) return null;
+  const commit = aliasTargetCommit(target);
+  return commit && isGitAncestor(target.repoRoot, candidate.commitHash, commit) ? candidate : null;
 }
 
 // src/identity/index-aliases.ts
@@ -1828,17 +1837,17 @@ function movedSymbolAliases(store, baseSnapshotId, fileAliases, nodes) {
   }
   return out;
 }
-function renameAliases(input, baseCommit) {
-  if (input.detectRenames === false || !isInsideGitRepo(input.rootDir)) return [];
-  const target = input.commitHash ?? detectGitHead(input.rootDir) ?? void 0;
-  if (target === baseCommit) return [];
+function renameAliases(input, baseCommit, target) {
+  if (input.detectRenames === false || target === baseCommit) return [];
   const pairs = detectRenames({ repoRoot: input.rootDir, fromCommit: baseCommit, toCommit: target });
   return buildAliases(input.idRoot, pairs);
 }
 function computeAliasBridge(store, input) {
-  const base = aliasBaseFor(store, input.ref);
-  if (!base?.commitHash) return { baseSnapshotId: null, aliases: [] };
-  const fileAliases = renameAliases(input, base.commitHash);
+  const target = { repoRoot: input.rootDir, commit: input.commitHash };
+  const base = aliasBaseFor(store, input.ref, target);
+  const commit = base ? aliasTargetCommit(target) : null;
+  if (!base?.commitHash || !commit) return { baseSnapshotId: null, aliases: [] };
+  const fileAliases = renameAliases(input, base.commitHash, commit);
   const symbols = movedSymbolAliases(store, base.id, fileAliases, input.nodes);
   return { baseSnapshotId: base.id, aliases: [...fileAliases, ...symbols] };
 }