fix: match counter names by suffix for preset filtering

PerfMon returns full paths (\\host\Object\Counter) but presets
filter by bare name (Counter). Now uses endsWith matching.

Author: calltelemetry-jason

src/tools/counter-tools.ts

@@ -125,7 +125,9 @@ export async function handleCounterTool(name: string, args: Record<string, unkno
     if (name === "counter_snapshot") {
       const { object, counters: filterCounters } = resolvePreset(args);
       const values = await perfmonCollectCounterData(creds, perfmonHost, object, args.timeoutMs as number | undefined);
-      const filtered = filterCounters ? values.filter(v => filterCounters.includes(v.name)) : values;
+      const filtered = filterCounters
+        ? values.filter(v => filterCounters.some(fc => v.name === fc || v.name.endsWith(`\\${fc}`)))
+        : values;
       return jsonResponse({ object, host: perfmonHost, counters: filtered });
     }
 

src/types/perfmon-types.ts

@@ -72,9 +72,16 @@ export const COUNTER_PRESETS: Record<string, { object: string; counters?: string
   system: { object: "Processor" },
 };
 
+/** Extract bare counter name from full PerfMon path (e.g., \\\\host\\Object\\Counter → Counter) */
+export function bareCounterName(name: string): string {
+  const lastSlash = name.lastIndexOf("\\");
+  return lastSlash >= 0 ? name.slice(lastSlash + 1) : name;
+}
+
 export function classifyCounter(name: string): CounterType {
-  if (GAUGE_COUNTERS.has(name)) return "gauge";
-  if (MONOTONIC_COUNTERS.has(name)) return "counter";
+  const bare = bareCounterName(name);
+  if (GAUGE_COUNTERS.has(bare)) return "gauge";
+  if (MONOTONIC_COUNTERS.has(bare)) return "counter";
   return "unknown";
 }