Fix lint

Author: JasonYeMSFT

scripts/src/frontmatter/cli.ts

@@ -20,7 +20,6 @@
 import { dirname, resolve, basename, relative } from "node:path";
 import { fileURLToPath } from "node:url";
 import { existsSync, readdirSync, readFileSync, statSync } from "node:fs";
-import { execFileSync } from "node:child_process";
 import { parseArgs } from "node:util";
 import { parseSkillContent } from "../shared/skill-helper.js";
 
@@ -382,11 +381,9 @@ const TRIGGER_SECTION_RE = new RegExp(
 const DO_NOT_USE_FOR_RE = /\bDO NOT USE FOR:/i;
 const SANITIZED_DO_NOT_USE_FOR_MARKER = "DO_NOT_USE_FOR:";
 const DISAMBIGUATION_CLAUSE_MARKER_RE = new RegExp(`(?:${SANITIZED_DO_NOT_USE_FOR_MARKER}|PREFER OVER\\b)`, "i");
-const PREFER_OVER_RE = /\bPREFER OVER\b/i;
 const BROAD_SKILL_NAMES = new Set(["azure-prepare", "azure-deploy"]);
 const MIN_TRIGGER_PHRASE_LENGTH = 4;
 const OVERLAP_PREVIEW_LIMIT = 3;
-const DEFAULT_REMOTE_BASE_REF = "origin/main";
 
 function normalizeTriggerPhrase(phrase: string): string {
   return phrase
@@ -430,11 +427,6 @@ export function hasPreferOverClause(description: string | null, competingSkillNa
   return new RegExp(`\\bPREFER OVER\\s+${escapedName}\\b`, "i").test(description);
 }
 
-function hasAnyPreferOverClause(description: string | null): boolean {
-  if (!description) return false;
-  return PREFER_OVER_RE.test(description);
-}
-
 function hasAnyDisambiguationClause(description: string | null, competingSkillName: string): boolean {
   return hasDoNotUseForClause(description) || hasPreferOverClause(description, competingSkillName);
 }
@@ -465,51 +457,6 @@ function isBroadRoutingSkill(name: string): boolean {
   return BROAD_SKILL_NAMES.has(name);
 }
 
-function getMergeBaseRef(): string | null {
-  // Prefer merge-base with common default branches first (remote then local), then
-  // fall back to HEAD~1 for environments where base branches are unavailable.
-  const baseRefCandidates = [DEFAULT_REMOTE_BASE_REF, "origin/master", "main", "master"];
-  for (const baseRef of baseRefCandidates) {
-    try {
-      return execFileSync("git", ["merge-base", "HEAD", baseRef], {
-        cwd: REPO_ROOT,
-        encoding: "utf-8",
-        stdio: ["ignore", "pipe", "ignore"],
-      }).trim();
-    } catch {
-      // Try next candidate base ref.
-    }
-  }
-
-  try {
-    return execFileSync("git", ["rev-parse", "HEAD~1"], {
-      cwd: REPO_ROOT,
-      encoding: "utf-8",
-      stdio: ["ignore", "pipe", "ignore"],
-    }).trim();
-  } catch {
-    return null;
-  }
-}
-
-function getDescriptionFromGitRef(filePath: string, ref: string): string | null {
-  const relativeFilePath = relative(REPO_ROOT, filePath).replace(/\\/g, "/");
-  try {
-    const previousContent = execFileSync("git", ["show", `${ref}:${relativeFilePath}`], {
-      cwd: REPO_ROOT,
-      encoding: "utf-8",
-      stdio: ["ignore", "pipe", "ignore"],
-    });
-    const parsed = parseSkillContent(previousContent);
-    if (parsed === null || typeof parsed.data.description !== "string") {
-      return null;
-    }
-    return parsed.data.description;
-  } catch {
-    return null;
-  }
-}
-
 export function validateTriggerOverlapDisambiguation(
   skill: SkillRoutingContext,
   allSkills: SkillRoutingContext[],
@@ -542,17 +489,6 @@ export function validateTriggerOverlapDisambiguation(
   return issues;
 }
 
-function getRemovedDisambiguationClauses(previousDescription: string | null, currentDescription: string | null): string[] {
-  const removed: string[] = [];
-  if (hasDoNotUseForClause(previousDescription) && !hasDoNotUseForClause(currentDescription)) {
-    removed.push("DO NOT USE FOR");
-  }
-  if (hasAnyPreferOverClause(previousDescription) && !hasAnyPreferOverClause(currentDescription)) {
-    removed.push("PREFER OVER");
-  }
-  return removed;
-}
-
 // ── Validate a single SKILL.md ──────────────────────────────────────────────
 
 export function validateSkillFile(filePath: string): ValidationResult {
@@ -761,7 +697,6 @@ function main(): void {
   const results: ValidationResult[] = [];
   const routingContexts = buildSkillRoutingContexts(getAllSkillFiles());
   const routingContextByName = new Map(routingContexts.map((context) => [context.name, context]));
-  const mergeBaseRef = getMergeBaseRef();
 
   for (const file of skillFiles) {
     const result = validateSkillFile(file);