Clean up unused exports (#1101)

Author: JasonYeMSFT

tests/.eslintrc

@@ -0,0 +1,5 @@
+{
+    "ignorePatterns": [
+        "jest.config.ts"
+    ]
+}

tests/azure-hosted-copilot-sdk/util.ts

@@ -39,7 +39,7 @@ export async function setupCopilotSdkApp(workspace: string): Promise<void> {
 
 // --- Invocation rate helpers ---
 
-export function logRate(skillName: string, label: string, successCount: number, runsPerPrompt: number): number {
+function logRate(skillName: string, label: string, successCount: number, runsPerPrompt: number): number {
   const rate = successCount / runsPerPrompt;
   const msg = `${skillName} invocation rate for ${label}: ${(rate * 100).toFixed(1)}% (${successCount}/${runsPerPrompt})`;
   console.log(msg);
@@ -92,7 +92,7 @@ const TOKEN_PATTERNS = [
  * Redact token-like values from a string.
  * Returns the string with tokens replaced by `[REDACTED]`.
  */
-export function redactTokens(text: string): string {
+function redactTokens(text: string): string {
   let result = text;
   for (const pattern of TOKEN_PATTERNS) {
     // Reset lastIndex for global regexes

tests/azure-messaging/integration.test.ts

@@ -14,8 +14,7 @@ import {
   shouldSkipIntegrationTests,
   getIntegrationSkipReason
 } from "../utils/agent-runner";
-import { softCheckSkill } from "../utils/evaluate";
-import { isSkillInvoked } from "../utils/evaluate";
+import { softCheckSkill, isSkillInvoked } from "../utils/evaluate";
 
 const SKILL_NAME = "azure-messaging";
 const RUNS_PER_PROMPT = 3;

tests/azure-prepare/integration.test.ts

@@ -17,8 +17,7 @@ import {
 import { hasValidationCommand } from "../azure-validate/utils";
 import { hasPlanReadyForValidation, getDockerContext, hasServicesSection, getServiceProject } from "./utils";
 import { cloneRepo } from "../utils/git-clone";
-import { expectFiles, getToolCalls, softCheckSkill } from "../utils/evaluate";
-import { isSkillInvoked } from "../utils/evaluate";
+import { expectFiles, getToolCalls, softCheckSkill, isSkillInvoked } from "../utils/evaluate";
 
 const SKILL_NAME = "azure-prepare";
 const RUNS_PER_PROMPT = 1;

tests/azure-rbac/integration.test.ts

@@ -16,8 +16,7 @@ import {
   shouldSkipIntegrationTests,
   getIntegrationSkipReason
 } from "../utils/agent-runner";
-import { softCheckSkill } from "../utils/evaluate";
-import { isSkillInvoked } from "../utils/evaluate";
+import { softCheckSkill, isSkillInvoked } from "../utils/evaluate";
 
 const SKILL_NAME = "azure-rbac";
 const RUNS_PER_PROMPT = 5;

tests/azure-resource-visualizer/integration.test.ts

@@ -17,9 +17,7 @@ import {
   getIntegrationSkipReason
 } from "../utils/agent-runner";
 import type { AgentMetadata } from "../utils/agent-runner";
-import { softCheckSkill } from "../utils/evaluate";
-import { isSkillInvoked } from "../utils/evaluate";
-import { getToolCalls } from "../utils/evaluate";
+import { softCheckSkill, isSkillInvoked, getToolCalls } from "../utils/evaluate";
 
 const SKILL_NAME = "azure-resource-visualizer";
 const RUNS_PER_PROMPT = 5;

tests/azure-validate/integration.test.ts

@@ -19,8 +19,7 @@ import {
   matchesFileEdit,
 } from "./utils";
 import { cloneRepo } from "../utils/git-clone";
-import { matchesCommand, softCheckSkill } from "../utils/evaluate";
-import { isSkillInvoked } from "../utils/evaluate";
+import { matchesCommand, softCheckSkill, isSkillInvoked } from "../utils/evaluate";
 
 const SKILL_NAME = "azure-validate";
 const RUNS_PER_PROMPT = 5;

tests/azure-validate/utils.ts

@@ -29,30 +29,10 @@ export function hasValidationCommand(metadata: AgentMetadata): boolean {
   });
 }
 
-/**
- * Check whether any tool call's serialized arguments match the given
- * pattern. Searches across all tool types (powershell, create, edit, etc.)
- * unless a specific toolName is provided.
- */
-export function matchesToolCallArgs(
-  metadata: AgentMetadata,
-  pattern: RegExp,
-  toolName?: string,
-): boolean {
-  return getToolCalls(metadata, toolName).some(event => {
-    const argsStr = JSON.stringify(event.data);
-    return pattern.test(argsStr);
-  });
-}
-
 /**
  * Check whether any file-mutating tool call (create or edit) targets a file
  * whose path matches {@link pathPattern} AND whose serialized arguments
  * match {@link contentPattern}.
- *
- * This is stricter than {@link matchesToolCallArgs} because it ensures the
- * content appeared in a write to a specific file, not just in any tool call
- * (e.g. a plan document).
  */
 export function matchesFileEdit(
   metadata: AgentMetadata,

tests/eslint.config.mjs

@@ -3,6 +3,7 @@ import { defineConfig } from "eslint/config";
 import tseslint from "typescript-eslint";
 import jest from "eslint-plugin-jest";
 import integrationTestNameRule from "./eslint-rules/integration-test-name.mjs";
+import importPlugin from "eslint-plugin-import";
 
 const tsFiles = ["**/*.ts"];
 const jsFiles = ["**/*.js", "**/*.mjs"];
@@ -59,6 +60,8 @@ export default defineConfig(
     extends: [
       eslint.configs.recommended,
       ...tseslint.configs.recommended,
+      importPlugin.flatConfigs.recommended,
+      importPlugin.flatConfigs.typescript
     ],
     plugins: {
       jest,
@@ -81,7 +84,8 @@ export default defineConfig(
       "@typescript-eslint/no-require-imports": "error",
       // A significant number of integration tests generate reports for human review.
       // We don't have deterministic metrics for them yet.
-      "jest/expect-expect": "off"
+      "jest/expect-expect": "off",
+      "import/no-unused-modules": [1, { "unusedExports": true }]
     },
   },
   // JavaScript files - no TypeScript project needed

tests/microsoft-foundry/foundry-agent/create/integration.test.ts

@@ -15,8 +15,7 @@ import {
   shouldSkipIntegrationTests,
   getIntegrationSkipReason,
 } from "../../../utils/agent-runner";
-import { softCheckSkill } from "../../../utils/evaluate";
-import { getToolCalls } from "../../../utils/evaluate";
+import { softCheckSkill, getToolCalls } from "../../../utils/evaluate";
 
 const SKILL_NAME = "microsoft-foundry";
 const RUNS_PER_PROMPT = 5;