From cf9b0f72f3c68153dc263fbc0f7c37f3ca3dc53a Mon Sep 17 00:00:00 2001
From: Leo310 <leodev310@gmail.com>
Date: Mon, 28 Sep 2026 11:39:01 +0200
Subject: [PATCH] chore(skills): drop the skill usage counters; keep the author
 stamp

The load and revision counters were built for a curator that is on hold, and
for a person in the Agent editor they rarely change anything: every revision
already shows up as a tool card in its chat and as a change to the note. They
cost a plugin-data write, and so a sync, on every skill load. Removed: the
recording in load_skill and manage_skills, the stored list, the delete-path
bookkeeping, and the usage line in the editor. The `metadata.author: agent`
stamp on created skills stays as plain provenance, like `author: S2B` on the
bundled ones. The 2.3.0 changelog entry for the usage line goes with it.

Co-Authored-By: Claude <noreply@anthropic.com>
---
 CHANGELOG.md                                 |  1 -
 src/agent/AgentManager.ts                    |  1 -
 src/agent/tools/loadSkill.ts                 | 14 +---
 src/agent/tools/manageSkills.ts              | 12 ++-
 src/components/modal/AgentEditorModal.svelte |  4 -
 src/skills/SkillsService.ts                  | 10 ---
 src/skills/usageSummary.ts                   | 55 -------------
 src/stores/dataStore.svelte.ts               | 58 --------------
 src/types/plugin.ts                          | 29 -------
 test/agent/loadSkill.test.ts                 | 24 ------
 test/agent/manageSkills.test.ts              | 55 +------------
 test/skills/deleteSkill.test.ts              | 20 +----
 test/skills/usageSummary.test.ts             | 49 ------------
 test/stores/dataStore.test.ts                | 84 --------------------
 14 files changed, 11 insertions(+), 405 deletions(-)
 delete mode 100644 src/skills/usageSummary.ts
 delete mode 100644 test/skills/usageSummary.test.ts

diff --git a/CHANGELOG.md b/CHANGELOG.md
index 2fce26d0..c2cdb3fd 100644
--- a/CHANGELOG.md
+++ b/CHANGELOG.md
@@ -28,7 +28,6 @@ A reworked chat composer, especially on mobile, agents that keep their memory an
 - **Chat: attachments as preview tiles.** Images show a thumbnail and PDFs their first page; other files show a card. Click a tile to open the file, or use its corner × to remove it (#526).
 - **Chat: Take photo on mobile.** The attach menu has a Take photo row that goes straight to the camera (#520).
 - **Chat: one jump-to-bottom button** replaces the message-navigation arrows. It appears once you scroll away from the end, including while a reply streams below the fold (#522, #523, #527, #528).
-- **Agents: where a skill came from and how it's used.** Skills the agent writes are marked as its own, and the Agent editor shows how often each custom skill has been used and revised (#516).
 - **What's new, in the plugin.** After an update, a notice opens a What's new tab with the notes for every release since the one you last saw. The Show release notes command and a row in Settings → Troubleshooting open it any time (#533).
 - **Update notices.** Once a day the plugin asks GitHub whether a newer version exists and says so, with a link to Community plugins. It sends nothing about you or your vault, stays quiet while Obsidian's own automatic update check is on, and can be turned off under Settings → General → Privacy (#534).
 
diff --git a/src/agent/AgentManager.ts b/src/agent/AgentManager.ts
index c11177e7..2f586ce5 100644
--- a/src/agent/AgentManager.ts
+++ b/src/agent/AgentManager.ts
@@ -982,7 +982,6 @@ export class AgentManager {
 				tools.push(
 					createLoadSkillTool(this.plugin.skillsService, {
 						skillNames: loadableSkills,
-						recordUsage: (name) => getData().recordSkillLoad(name),
 						isToolAvailable: (toolId) => {
 							if (BUILT_IN_TOOL_IDS.includes(toolId as BuiltInToolId)) {
 								return boundBuiltInTools.has(toolId);
diff --git a/src/agent/tools/loadSkill.ts b/src/agent/tools/loadSkill.ts
index c2a589d6..03e6016b 100644
--- a/src/agent/tools/loadSkill.ts
+++ b/src/agent/tools/loadSkill.ts
@@ -3,7 +3,6 @@ import { tool } from "@langchain/core/tools";
 import { z } from "zod";
 import type { SkillsService } from "../../skills/SkillsService";
 import type { CommunityPluginStatus } from "../integrations/pluginIntegrations";
-import { Logger as Log } from "../../utils/logging";
 import { recordSkillLoaded } from "./skillLoadRegistry";
 
 /** Frontmatter key (under `metadata:`) listing community plugin ids a skill can make use of. */
@@ -47,12 +46,6 @@ export interface LoadSkillToolOptions {
 	 * that path exists, instead of letting it find out from a failed result.
 	 */
 	pluginStatus?: (pluginId: string) => { status: CommunityPluginStatus; displayName: string };
-	/**
-	 * Called after a skill's body was returned to the model. The plugin data store counts
-	 * loads per skill (see `SkillUsageEntry`); injected rather than imported so the tool
-	 * stays testable without a live store, and a missed record never fails a load.
-	 */
-	recordUsage?: (skillName: string) => void;
 }
 
 /**
@@ -64,7 +57,7 @@ export interface LoadSkillToolOptions {
  * @returns A LangChain tool for loading skill content
  */
 export function createLoadSkillTool(skillsService: SkillsService, options: LoadSkillToolOptions) {
-	const { skillNames, isToolAvailable, pluginStatus, recordUsage } = options;
+	const { skillNames, isToolAvailable, pluginStatus } = options;
 
 	// If no skills available, return a tool that explains this
 	if (skillNames.length === 0) {
@@ -95,11 +88,6 @@ export function createLoadSkillTool(skillsService: SkillsService, options: LoadS
 			// The model has now seen this skill's current text in this conversation, which is
 			// what `manage_skills` requires before it will patch or rewrite it.
 			recordSkillLoaded(config?.configurable?.thread_id, skillName, skill.content);
-			try {
-				recordUsage?.(skillName);
-			} catch (error) {
-				Log.warn(`load_skill: could not record usage of ${skillName}`, error);
-			}
 
 			// Return the skill content with metadata
 			const lines: string[] = [];
diff --git a/src/agent/tools/manageSkills.ts b/src/agent/tools/manageSkills.ts
index 34515c88..8a8f4bd9 100644
--- a/src/agent/tools/manageSkills.ts
+++ b/src/agent/tools/manageSkills.ts
@@ -130,11 +130,11 @@ function splitSkillMd(raw: string): { head: string; body: string } | null {
 }
 
 /**
- * `metadata.author` value stamped on every skill the agent creates. Bundled skills carry
- * `S2B` in the same key and hand-written ones carry whatever the user put there (or nothing),
- * so one existing key tells the three apart — which is what lets anything autonomous later
- * (a curator) confine itself to the agent's own skills. A fixed word rather than the agent's
- * name: consumers need no list of agent names, and a renamed or deleted agent changes nothing.
+ * `metadata.author` value stamped on every skill the agent creates: plain provenance in the
+ * same key the bundled skills use for `S2B`, written once and never read by the plugin. It is
+ * the one fact about a skill that cannot be reconstructed later, so it is recorded even though
+ * nothing acts on it today. A fixed word rather than the agent's name, so a renamed or deleted
+ * agent changes nothing.
  */
 export const AGENT_SKILL_AUTHOR = "agent";
 
@@ -331,7 +331,6 @@ export function createManageSkillsTool(skillsService: SkillsService | undefined,
 				// counters so a later skill of the same name starts from zero.
 				const agent = getData().getAgent(agentId) ?? getData().getSelectedAgent();
 				if (agent?.skills[input.name]) delete agent.skills[input.name];
-				getData().forgetSkillUsage(input.name);
 
 				return `Deleted the "${input.name}" skill.`;
 			}
@@ -408,7 +407,6 @@ export function createManageSkillsTool(skillsService: SkillsService | undefined,
 			// history, so a revised core skill is flagged as customized rather than overwritten
 			// by the next upgrade.
 			await skillsService.writeSkillFile(skillName, newContent);
-			getData().recordSkillRevision(skillName);
 			// The model wrote this text too, so a follow-up revision in the same turn needs no reload.
 			recordSkillLoaded(threadId, skillName, parseFrontmatter(newContent).body);
 
diff --git a/src/components/modal/AgentEditorModal.svelte b/src/components/modal/AgentEditorModal.svelte
index 790fd64d..6878a681 100644
--- a/src/components/modal/AgentEditorModal.svelte
+++ b/src/components/modal/AgentEditorModal.svelte
@@ -38,7 +38,6 @@ import { agentDefinitionPath, agentDir } from "../../utils/agentPaths";
 import { Logger } from "../../utils/logging";
 import { extractErrorMessage } from "../../utils/errorMessage";
 import { humanizeSkillName } from "../../skills";
-import { skillUsageSummary } from "../../skills/usageSummary";
 import {
 	DEFAULT_AGENT_ICON,
 	type AgentConfig,
@@ -299,8 +298,6 @@ const skills = $derived.by(() => {
 			category: metadata.category ?? "custom",
 			corePluginId: metadata.corePluginId,
 			linkedPluginId: metadata.linkedPluginId,
-			author: metadata.frontmatter.metadata?.author,
-			usage: pluginData.getSkillUsage(skillName),
 		});
 	}
 	return result;
@@ -1052,7 +1049,6 @@ function getServerToolsState(serverId: string): MCPServerToolsState | undefined
             class="skill-entity"
             name={ext.displayName}
             desc={ext.description}
-            meta={skillUsageSummary(ext.author, ext.usage)}
           >
             {#snippet actions()}
               <Button
diff --git a/src/skills/SkillsService.ts b/src/skills/SkillsService.ts
index ef111da4..ca7b275e 100644
--- a/src/skills/SkillsService.ts
+++ b/src/skills/SkillsService.ts
@@ -1018,16 +1018,6 @@ export class SkillsService {
 			}
 
 			this.skillsCache.delete(skillName);
-			// Drop the usage counters with the skill, whichever surface deleted it (the Agent
-			// editor's trash button lands here; manage_skills removes the folder itself and
-			// forgets on its own). A later skill of the same name must start from zero.
-			// Advisory data: a store that is not up yet must not turn a completed delete
-			// into a reported failure.
-			try {
-				getData().forgetSkillUsage(skillName);
-			} catch {
-				Log.debug(`Could not clear usage counters for ${skillName}`);
-			}
 			Log.info(`Deleted skill: ${skillName}`);
 			return true;
 		} catch (error) {
diff --git a/src/skills/usageSummary.ts b/src/skills/usageSummary.ts
deleted file mode 100644
index 047d9825..00000000
--- a/src/skills/usageSummary.ts
+++ /dev/null
@@ -1,55 +0,0 @@
-/**
- * The one-line origin-and-usage summary shown under a skill in the Agent editor, e.g.
- * "Created by the agent · used 3 times, last 2 days ago · revised once". Pure, so the
- * wording is unit-tested rather than eyeballed.
- */
-
-import type { SkillUsageEntry } from "../types/plugin";
-
-const AGENT_AUTHOR = "agent";
-
-/** Coarse relative age: the editor row is a glance, not a log. */
-export function describeAge(timestamp: number, now: number): string {
-	const seconds = Math.max(0, Math.round((now - timestamp) / 1000));
-	if (seconds < 60) return "just now";
-	const minutes = Math.round(seconds / 60);
-	if (minutes < 60) return `${minutes} min ago`;
-	const hours = Math.round(minutes / 60);
-	if (hours < 24) return hours === 1 ? "1 hour ago" : `${hours} hours ago`;
-	const days = Math.round(hours / 24);
-	if (days < 30) return days === 1 ? "yesterday" : `${days} days ago`;
-	const months = Math.round(days / 30);
-	if (months < 12) return months === 1 ? "1 month ago" : `${months} months ago`;
-	const years = Math.round(days / 365);
-	return years === 1 ? "1 year ago" : `${years} years ago`;
-}
-
-function times(count: number): string {
-	return count === 1 ? "once" : count === 2 ? "twice" : `${count} times`;
-}
-
-/**
- * Summarize where a custom skill came from and how it has been used. `author` is the
- * frontmatter `metadata.author`; `usage` the plugin-data counters, absent when the skill was
- * never loaded or revised.
- */
-export function skillUsageSummary(
-	author: string | undefined,
-	usage: SkillUsageEntry | undefined,
-	now = Date.now(),
-): string {
-	const origin = author === AGENT_AUTHOR ? "Created by the agent" : author ? `Author: ${author}` : "Your skill";
-
-	const parts = [origin];
-	if (!usage || usage.loadCount === 0) {
-		parts.push("never used");
-	} else {
-		const last = usage.lastLoadedAt === null ? "" : `, last ${describeAge(usage.lastLoadedAt, now)}`;
-		parts.push(`used ${times(usage.loadCount)}${last}`);
-	}
-	if (usage && usage.revisionCount > 0) {
-		const last = usage.lastRevisedAt === null ? "" : ` (last ${describeAge(usage.lastRevisedAt, now)})`;
-		parts.push(`revised ${times(usage.revisionCount)}${last}`);
-	}
-	return parts.join(" · ");
-}
diff --git a/src/stores/dataStore.svelte.ts b/src/stores/dataStore.svelte.ts
index f152a226..a8a19737 100644
--- a/src/stores/dataStore.svelte.ts
+++ b/src/stores/dataStore.svelte.ts
@@ -29,7 +29,6 @@ import type {
 	PrivacyMode,
 	PromptFileReader,
 	RecentNoteEntry,
-	SkillUsageEntry,
 	StaleGuidance,
 	ToolConfig,
 } from "../types/plugin";
@@ -259,7 +258,6 @@ export const DEFAULT_SETTINGS: PluginData = {
 	searchShowMatchContext: true,
 	searchShowKeyboardHints: true,
 	recentNotes: [],
-	skillUsage: [],
 	embeddingIndexes: [],
 	searchEmbedIndex: null,
 	graphEmbedIndex: null,
@@ -1352,62 +1350,6 @@ export class PluginDataStore {
 		void this.saveSettings();
 	}
 
-	// --- Skill usage ---
-
-	/**
-	 * The usage list, healed to an empty list when the field is absent or not a list (a data
-	 * file from before the field existed, or one written while the shape was still changing).
-	 */
-	private usageList(): SkillUsageEntry[] {
-		if (!Array.isArray(this.#data.skillUsage)) this.#data.skillUsage = [];
-		return this.#data.skillUsage;
-	}
-
-	/** Usage counters for a skill, or undefined when it has never been loaded or revised. */
-	getSkillUsage(skillName: string): SkillUsageEntry | undefined {
-		return this.usageList().find((entry) => entry.name === skillName);
-	}
-
-	/** All usage counters. */
-	get skillUsage(): readonly SkillUsageEntry[] {
-		return this.usageList();
-	}
-
-	private bumpSkillUsage(skillName: string, mutate: (entry: SkillUsageEntry) => void): void {
-		const usage = this.usageList();
-		let entry = usage.find((candidate) => candidate.name === skillName);
-		if (!entry) {
-			entry = { name: skillName, loadCount: 0, lastLoadedAt: null, revisionCount: 0, lastRevisedAt: null };
-			usage.push(entry);
-		}
-		mutate(entry);
-		void this.saveSettings();
-	}
-
-	/** `load_skill` returned this skill's body. */
-	recordSkillLoad(skillName: string): void {
-		this.bumpSkillUsage(skillName, (entry) => {
-			entry.loadCount += 1;
-			entry.lastLoadedAt = Date.now();
-		});
-	}
-
-	/** `manage_skills` patched or rewrote this skill. */
-	recordSkillRevision(skillName: string): void {
-		this.bumpSkillUsage(skillName, (entry) => {
-			entry.revisionCount += 1;
-			entry.lastRevisedAt = Date.now();
-		});
-	}
-
-	/** The skill is gone; a later skill of the same name starts from zero. */
-	forgetSkillUsage(skillName: string): void {
-		const usage = this.usageList();
-		if (!usage.some((entry) => entry.name === skillName)) return;
-		this.#data.skillUsage = usage.filter((entry) => entry.name !== skillName);
-		void this.saveSettings();
-	}
-
 	// --- Embedding Indexes (Multi-Index) ---
 
 	get embeddingIndexes(): EmbeddingIndexConfig[] {
diff --git a/src/types/plugin.ts b/src/types/plugin.ts
index 62e96568..107b5ad2 100644
--- a/src/types/plugin.ts
+++ b/src/types/plugin.ts
@@ -30,29 +30,6 @@ export interface RecentNoteEntry {
 	lastOpenedAt: number;
 }
 
-/**
- * How a skill has been used, stored in plugin data (never in the SKILL.md: a counter in
- * frontmatter would turn every load into a vault write, break the bundled skills'
- * shipped-history fingerprint, and diverge across synced devices). Advisory only — a missed
- * bump costs nothing — and per device, since plugin data is.
- *
- * A list rather than a record keyed by name, like `recentNotes`: skill names are free-form
- * ("constructor" is a valid one), and Svelte's `$state` proxy silently drops a write to a key
- * that exists on the prototype chain, so a name-keyed object could never store such a skill.
- */
-export interface SkillUsageEntry {
-	/** The skill's frontmatter name. */
-	name: string;
-	/** Times `load_skill` returned this skill's body. */
-	loadCount: number;
-	/** Epoch ms of the most recent load, or null when never loaded. */
-	lastLoadedAt: number | null;
-	/** Times `manage_skills` patched or rewrote this skill. */
-	revisionCount: number;
-	/** Epoch ms of the most recent revision, or null when never revised. */
-	lastRevisedAt: number | null;
-}
-
 /**
  * How long after being opened a note still counts as "recent".
  *
@@ -323,10 +300,6 @@ export interface SkillDisplayInfo {
 	linkedPluginId?: string;
 	/** Linked core plugin ID (if any) */
 	corePluginId?: string;
-	/** `metadata.author` from the frontmatter: `S2B` for bundled skills, `agent` for agent-created ones. */
-	author?: string;
-	/** Usage counters from plugin data, when the skill has ever been loaded or revised. */
-	usage?: SkillUsageEntry;
 }
 
 // ============================================================================
@@ -646,8 +619,6 @@ export interface PluginData {
 	searchShowMatchContext: boolean;
 	searchShowKeyboardHints: boolean;
 	recentNotes: RecentNoteEntry[];
-	/** Per-skill load and revision counters (see {@link SkillUsageEntry}); one entry per skill name. */
-	skillUsage: SkillUsageEntry[];
 
 	/**
 	 * Registry of all known embedding indexes.
diff --git a/test/agent/loadSkill.test.ts b/test/agent/loadSkill.test.ts
index 8f58b5f3..b68666d8 100644
--- a/test/agent/loadSkill.test.ts
+++ b/test/agent/loadSkill.test.ts
@@ -115,28 +115,4 @@ describe("load_skill tool", () => {
 		expect(skillLoadState("t-load", "views", "views instructions")).toBe("not-loaded");
 		expect(skillLoadState("t-other", "web", "web instructions")).toBe("not-loaded");
 	});
-
-	it("reports a successful load to the usage recorder, and only then", async () => {
-		const recordUsage = vi.fn();
-		const service = mockSkillsService({ web: {} });
-		const tool = createLoadSkillTool(service, { skillNames: ["web", "gone"], recordUsage });
-
-		await tool.invoke({ skillName: "web" });
-		await tool.invoke({ skillName: "gone" }); // known to the tool, missing from the service
-
-		expect(recordUsage).toHaveBeenCalledTimes(1);
-		expect(recordUsage).toHaveBeenCalledWith("web");
-	});
-
-	// A recorder that throws must never cost the model the skill it asked for.
-	it("still returns the skill when the usage recorder throws", async () => {
-		const service = mockSkillsService({ web: { body: "Use fetch_url." } });
-		const tool = createLoadSkillTool(service, {
-			skillNames: ["web"],
-			recordUsage: () => {
-				throw new Error("store gone");
-			},
-		});
-		expect(String(await tool.invoke({ skillName: "web" }))).toContain("Use fetch_url.");
-	});
 });
diff --git a/test/agent/manageSkills.test.ts b/test/agent/manageSkills.test.ts
index 31fe6418..ba8eaccb 100644
--- a/test/agent/manageSkills.test.ts
+++ b/test/agent/manageSkills.test.ts
@@ -15,8 +15,6 @@ import { recordSkillLoaded, resetSkillLoadRegistry } from "../../src/agent/tools
 import { parseFrontmatter } from "../../src/skills/SkillsService";
 
 const RUN_CONFIG = { configurable: { thread_id: "t1" }, runId: "run-1" };
-const recordSkillRevision = vi.fn();
-const forgetSkillUsage = vi.fn();
 
 const DATAVIEW_MD = `---
 name: dataview
@@ -116,11 +114,7 @@ describe("manage_skills tool", () => {
 		mockGetData.mockReturnValue({
 			getAgent: () => agentWithSkills({ other: { enabled: false } }),
 			getSelectedAgent: () => agentWithSkills({ other: { enabled: false } }),
-			recordSkillRevision,
-			forgetSkillUsage,
 		});
-		recordSkillRevision.mockClear();
-		forgetSkillUsage.mockClear();
 		// Revisions require the skill to have been loaded in this thread; most tests below
 		// exercise what happens after that, so satisfy the gate up front.
 		resetSkillLoadRegistry();
@@ -368,7 +362,7 @@ describe("manage_skills tool", () => {
 		});
 	});
 
-	describe("usage and provenance", () => {
+	describe("provenance", () => {
 		it("stamps metadata.author: agent on a created skill", async () => {
 			const cache = dataviewCache();
 			const svc = makeSkillsService(cache);
@@ -384,51 +378,6 @@ describe("manage_skills tool", () => {
 			expect(written).toContain("metadata:\n  author: agent\n---");
 			expect(parseFrontmatter(written).frontmatter.metadata?.author).toBe("agent");
 		});
-
-		it("counts a patch and an update as revisions", async () => {
-			const svc = makeSkillsService(dataviewCache());
-			const { app } = makeApp({ "Agents/Skills/dataview/SKILL.md": DATAVIEW_MD });
-			const t = createManageSkillsTool(svc, app, "agent-1");
-
-			await t.invoke(
-				{ operation: { type: "patch", skillName: "dataview", oldText: "Old body.", newText: "A." } },
-				RUN_CONFIG,
-			);
-			await t.invoke(
-				{ operation: { type: "update", skillName: "dataview", newBody: "# Dataview\n\nB." } },
-				RUN_CONFIG,
-			);
-
-			expect(recordSkillRevision).toHaveBeenCalledTimes(2);
-			expect(recordSkillRevision).toHaveBeenCalledWith("dataview");
-		});
-
-		it("does not count a refused or no-op revision", async () => {
-			const svc = makeSkillsService(dataviewCache());
-			const { app } = makeApp({ "Agents/Skills/dataview/SKILL.md": DATAVIEW_MD });
-			const t = createManageSkillsTool(svc, app, "agent-1");
-
-			await t.invoke(
-				{ operation: { type: "patch", skillName: "dataview", oldText: "missing", newText: "x" } },
-				RUN_CONFIG,
-			);
-			await t.invoke(
-				{ operation: { type: "update", skillName: "dataview", newBody: "# Dataview\n\nOld body." } },
-				RUN_CONFIG,
-			);
-
-			expect(recordSkillRevision).not.toHaveBeenCalled();
-		});
-
-		it("forgets a deleted skill's counters", async () => {
-			const svc = makeSkillsService(dataviewCache());
-			const { app } = makeApp({ "Agents/Skills/dataview/SKILL.md": DATAVIEW_MD });
-			const t = createManageSkillsTool(svc, app, "agent-1");
-
-			await t.invoke({ operation: { type: "delete", name: "dataview" } }, RUN_CONFIG);
-
-			expect(forgetSkillUsage).toHaveBeenCalledWith("dataview");
-		});
 	});
 
 	describe("update operation", () => {
@@ -635,8 +584,6 @@ describe("manage_skills tool", () => {
 			const agent = agentWithSkills({ other: { enabled: false }, dataview: { enabled: true } });
 			mockGetData.mockReturnValue({
 				getAgent: () => agent,
-				recordSkillRevision,
-				forgetSkillUsage,
 				getSelectedAgent: () => agent,
 			});
 			const { app } = makeApp({ "Agents/Skills/dataview/SKILL.md": DATAVIEW_MD });
diff --git a/test/skills/deleteSkill.test.ts b/test/skills/deleteSkill.test.ts
index a82b50bf..5c538f1a 100644
--- a/test/skills/deleteSkill.test.ts
+++ b/test/skills/deleteSkill.test.ts
@@ -1,15 +1,9 @@
-import { beforeEach, describe, expect, it, vi } from "vitest";
+import { describe, expect, it, vi } from "vitest";
 
 vi.mock("obsidian", () => import("../__mocks__/obsidian"));
 
-// deleteSkill clears the skill's usage counters through the data store; the Agent editor's
-// trash button is the path that reaches it (manage_skills removes the folder itself).
-const forgetSkillUsage = vi.fn();
 vi.mock("../../src/stores/dataStore.svelte", () => ({
-	getData: () => ({
-		agentFolder: "Agents",
-		forgetSkillUsage,
-	}),
+	getData: () => ({ agentFolder: "Agents" }),
 }));
 
 import { SkillsService } from "../../src/skills/SkillsService";
@@ -55,11 +49,7 @@ function makeAdapter(initial: Record<string, string>) {
 }
 
 describe("SkillsService.deleteSkill", () => {
-	beforeEach(() => {
-		forgetSkillUsage.mockClear();
-	});
-
-	it("removes the file and forgets the skill's usage counters", async () => {
+	it("removes the skill's folder and its cache entry", async () => {
 		const adapter = makeAdapter({ "Agents/Skills/weekly-review/SKILL.md": SKILL_MD });
 		const plugin = { app: { vault: { adapter, configDir: ".obsidian" } } } as never;
 		const svc = new SkillsService(plugin);
@@ -70,14 +60,12 @@ describe("SkillsService.deleteSkill", () => {
 
 		expect(adapter.files.has("Agents/Skills/weekly-review/SKILL.md")).toBe(false);
 		expect(svc.getCachedSkills().has("weekly-review")).toBe(false);
-		expect(forgetSkillUsage).toHaveBeenCalledWith("weekly-review");
 	});
 
-	it("does not touch counters for a skill it does not know", async () => {
+	it("reports false for a skill it does not know", async () => {
 		const adapter = makeAdapter({});
 		const plugin = { app: { vault: { adapter, configDir: ".obsidian" } } } as never;
 		const svc = new SkillsService(plugin);
 		expect(await svc.deleteSkill("ghost")).toBe(false);
-		expect(forgetSkillUsage).not.toHaveBeenCalled();
 	});
 });
diff --git a/test/skills/usageSummary.test.ts b/test/skills/usageSummary.test.ts
deleted file mode 100644
index ca715672..00000000
--- a/test/skills/usageSummary.test.ts
+++ /dev/null
@@ -1,49 +0,0 @@
-/**
- * The origin-and-usage line under a custom skill in the Agent editor. Its wording is the
- * only place a user sees what the counters mean, so it is pinned here.
- */
-
-import { describe, expect, it } from "vitest";
-
-import { describeAge, skillUsageSummary } from "../../src/skills/usageSummary";
-
-const NOW = 1_800_000_000_000;
-const DAY = 24 * 60 * 60 * 1000;
-
-describe("describeAge", () => {
-	it("rounds to the coarsest unit that reads naturally", () => {
-		expect(describeAge(NOW - 5_000, NOW)).toBe("just now");
-		expect(describeAge(NOW - 3 * 60_000, NOW)).toBe("3 min ago");
-		expect(describeAge(NOW - 60 * 60_000, NOW)).toBe("1 hour ago");
-		expect(describeAge(NOW - DAY, NOW)).toBe("yesterday");
-		expect(describeAge(NOW - 12 * DAY, NOW)).toBe("12 days ago");
-		expect(describeAge(NOW - 45 * DAY, NOW)).toBe("2 months ago");
-		expect(describeAge(NOW - 400 * DAY, NOW)).toBe("1 year ago");
-	});
-});
-
-describe("skillUsageSummary", () => {
-	it("names the origin from the author key", () => {
-		expect(skillUsageSummary("agent", undefined, NOW)).toBe("Created by the agent · never used");
-		expect(skillUsageSummary(undefined, undefined, NOW)).toBe("Your skill · never used");
-		expect(skillUsageSummary("Leo", undefined, NOW)).toBe("Author: Leo · never used");
-	});
-
-	it("reads counts and ages in words", () => {
-		const usage = {
-			name: "x",
-			loadCount: 3,
-			lastLoadedAt: NOW - 2 * DAY,
-			revisionCount: 1,
-			lastRevisedAt: NOW - 60_000,
-		};
-		expect(skillUsageSummary("agent", usage, NOW)).toBe(
-			"Created by the agent · used 3 times, last 2 days ago · revised once (last 1 min ago)",
-		);
-	});
-
-	it("omits the revision part when nothing was revised", () => {
-		const usage = { name: "x", loadCount: 1, lastLoadedAt: NOW, revisionCount: 0, lastRevisedAt: null };
-		expect(skillUsageSummary(undefined, usage, NOW)).toBe("Your skill · used once, last just now");
-	});
-});
diff --git a/test/stores/dataStore.test.ts b/test/stores/dataStore.test.ts
index d139b1bc..49942210 100644
--- a/test/stores/dataStore.test.ts
+++ b/test/stores/dataStore.test.ts
@@ -1112,87 +1112,3 @@ describe("PluginDataStore graph index availability", () => {
 		expect(store.getGraphEmbedModel()).toBeNull();
 	});
 });
-
-describe("PluginDataStore – skill usage", () => {
-	beforeEach(() => {
-		__resetPluginDataStoreForTests();
-	});
-
-	async function freshStore() {
-		const plugin = {
-			...createMockPlugin(),
-			loadData: vi.fn().mockResolvedValue(structuredClone(DEFAULT_SETTINGS)),
-		};
-		return { store: await createData(plugin as never), plugin };
-	}
-
-	it("starts with no usage for an unknown skill", async () => {
-		const { store } = await freshStore();
-		expect(store.getSkillUsage("weekly-review")).toBeUndefined();
-	});
-
-	it("counts loads and revisions separately, with timestamps", async () => {
-		const { store, plugin } = await freshStore();
-		const before = Date.now();
-
-		store.recordSkillLoad("weekly-review");
-		store.recordSkillLoad("weekly-review");
-		store.recordSkillRevision("weekly-review");
-
-		const usage = store.getSkillUsage("weekly-review")!;
-		expect(usage.loadCount).toBe(2);
-		expect(usage.revisionCount).toBe(1);
-		expect(usage.lastLoadedAt).toBeGreaterThanOrEqual(before);
-		expect(usage.lastRevisedAt).toBeGreaterThanOrEqual(before);
-		// Persisted, not just held in memory.
-		expect(plugin.saveData).toHaveBeenCalled();
-		const saved = plugin.saveData.mock.calls.at(-1)![0];
-		expect(saved.skillUsage.find((entry: { name: string }) => entry.name === "weekly-review").loadCount).toBe(2);
-	});
-
-	it("a revision on a never-loaded skill leaves the load side empty", async () => {
-		const { store } = await freshStore();
-		store.recordSkillRevision("weekly-review");
-		expect(store.getSkillUsage("weekly-review")).toMatchObject({
-			loadCount: 0,
-			lastLoadedAt: null,
-			revisionCount: 1,
-		});
-	});
-
-	it("forgets a skill so a later one of the same name starts from zero", async () => {
-		const { store } = await freshStore();
-		store.recordSkillLoad("weekly-review");
-		store.forgetSkillUsage("weekly-review");
-		expect(store.getSkillUsage("weekly-review")).toBeUndefined();
-		expect(store.skillUsage).toEqual([]);
-	});
-
-	// Data written before this field existed has no `skillUsage` key at all.
-	it("tolerates saved data without a skillUsage field, or with a non-list one", async () => {
-		__resetPluginDataStoreForTests();
-		const data = structuredClone(DEFAULT_SETTINGS) as unknown as Record<string, unknown>;
-		data.skillUsage = { "weekly-review": { loadCount: 9 } }; // a shape that never shipped
-		const plugin = { ...createMockPlugin(), loadData: vi.fn().mockResolvedValue(data) };
-		const store = await createData(plugin as never);
-		expect(store.getSkillUsage("x")).toBeUndefined();
-		store.recordSkillLoad("x");
-		expect(store.getSkillUsage("x")?.loadCount).toBe(1);
-	});
-
-	// "constructor" and "toString" are valid skill names. With a name-keyed object Svelte's
-	// $state proxy silently dropped the write (the key exists on the prototype chain), which
-	// is why the counters are a list; this pins that such names round-trip.
-	it("stores a skill named like an Object.prototype member", async () => {
-		const { store } = await freshStore();
-		expect(store.getSkillUsage("constructor")).toBeUndefined();
-		expect(store.getSkillUsage("toString")).toBeUndefined();
-
-		store.recordSkillLoad("constructor");
-		expect(store.getSkillUsage("constructor")).toMatchObject({ loadCount: 1, revisionCount: 0 });
-
-		store.forgetSkillUsage("toString"); // must be a no-op, not a delete of the prototype
-		store.forgetSkillUsage("constructor");
-		expect(store.getSkillUsage("constructor")).toBeUndefined();
-	});
-});