@oh-my-pi/pi-coding-agent 8.1.0 → 8.2.0

package/src/discovery/builtin.ts

@@ -4,29 +4,24 @@
  * Primary provider for OMP native configs. Supports all capabilities.
  * .pi is an alias for backwards compatibility.
  */
-
-import { registerProvider } from "@oh-my-pi/pi-coding-agent/capability";
-import { type ContextFile, contextFileCapability } from "@oh-my-pi/pi-coding-agent/capability/context-file";
-import {
-	type Extension,
-	type ExtensionManifest,
-	extensionCapability,
-} from "@oh-my-pi/pi-coding-agent/capability/extension";
-import { type ExtensionModule, extensionModuleCapability } from "@oh-my-pi/pi-coding-agent/capability/extension-module";
-import { readDirEntries, readFile } from "@oh-my-pi/pi-coding-agent/capability/fs";
-import { type Hook, hookCapability } from "@oh-my-pi/pi-coding-agent/capability/hook";
-import { type Instruction, instructionCapability } from "@oh-my-pi/pi-coding-agent/capability/instruction";
-import { type MCPServer, mcpCapability } from "@oh-my-pi/pi-coding-agent/capability/mcp";
-import { type Prompt, promptCapability } from "@oh-my-pi/pi-coding-agent/capability/prompt";
-import { type Rule, ruleCapability } from "@oh-my-pi/pi-coding-agent/capability/rule";
-import { type Settings, settingsCapability } from "@oh-my-pi/pi-coding-agent/capability/settings";
-import { type Skill, skillCapability } from "@oh-my-pi/pi-coding-agent/capability/skill";
-import { type SlashCommand, slashCommandCapability } from "@oh-my-pi/pi-coding-agent/capability/slash-command";
-import { type SystemPrompt, systemPromptCapability } from "@oh-my-pi/pi-coding-agent/capability/system-prompt";
-import { type CustomTool, toolCapability } from "@oh-my-pi/pi-coding-agent/capability/tool";
-import type { LoadContext, LoadResult } from "@oh-my-pi/pi-coding-agent/capability/types";
-import { parseFrontmatter } from "@oh-my-pi/pi-coding-agent/utils/frontmatter";
-import { dirname, isAbsolute, join, resolve } from "path";
+import * as path from "node:path";
+import { registerProvider } from "../capability";
+import { type ContextFile, contextFileCapability } from "../capability/context-file";
+import { type Extension, type ExtensionManifest, extensionCapability } from "../capability/extension";
+import { type ExtensionModule, extensionModuleCapability } from "../capability/extension-module";
+import { readDirEntries, readFile } from "../capability/fs";
+import { type Hook, hookCapability } from "../capability/hook";
+import { type Instruction, instructionCapability } from "../capability/instruction";
+import { type MCPServer, mcpCapability } from "../capability/mcp";
+import { type Prompt, promptCapability } from "../capability/prompt";
+import { type Rule, ruleCapability } from "../capability/rule";
+import { type Settings, settingsCapability } from "../capability/settings";
+import { type Skill, skillCapability } from "../capability/skill";
+import { type SlashCommand, slashCommandCapability } from "../capability/slash-command";
+import { type SystemPrompt, systemPromptCapability } from "../capability/system-prompt";
+import { type CustomTool, toolCapability } from "../capability/tool";
+import type { LoadContext, LoadResult } from "../capability/types";
+import { parseFrontmatter } from "../utils/frontmatter";
 import {
 	createSourceMeta,
 	discoverExtensionModulePaths,
@@ -51,7 +46,7 @@ async function getConfigDirs(ctx: LoadContext): Promise<Array<{ dir: string; lev
 	const result: Array<{ dir: string; level: "user" | "project" }> = [];
 
 	for (const name of PROJECT_DIRS) {
-		const projectDir = join(ctx.cwd, name);
+		const projectDir = path.join(ctx.cwd, name);
 		const entries = await readDirEntries(projectDir);
 		if (entries.length > 0) {
 			result.push({ dir: projectDir, level: "project" });
@@ -60,7 +55,7 @@ async function getConfigDirs(ctx: LoadContext): Promise<Array<{ dir: string; lev
 	}
 
 	for (const name of USER_DIRS) {
-		const userDir = join(ctx.home, name, PATHS.userAgent.replace(`${PATHS.userBase}/`, ""));
+		const userDir = path.join(ctx.home, name, PATHS.userAgent.replace(`${PATHS.userBase}/`, ""));
 		const entries = await readDirEntries(userDir);
 		if (entries.length > 0) {
 			result.push({ dir: userDir, level: "user" });
@@ -99,18 +94,18 @@ async function loadMCPServers(ctx: LoadContext): Promise<LoadResult<MCPServer>>
 	};
 
 	const projectDirs = await Promise.all(
-		PROJECT_DIRS.map(async (name) => {
-			const dir = join(ctx.cwd, name);
+		PROJECT_DIRS.map(async name => {
+			const dir = path.join(ctx.cwd, name);
 			const entries = await readDirEntries(dir);
 			return entries.length > 0 ? dir : null;
 		}),
 	);
-	const userPaths = USER_DIRS.map((name) => join(ctx.home, name, "mcp.json"));
+	const userPaths = USER_DIRS.map(name => path.join(ctx.home, name, "mcp.json"));
 
-	const projectDir = projectDirs.find((dir) => dir !== null);
+	const projectDir = projectDirs.find(dir => dir !== null);
 	if (projectDir) {
-		const projectCandidates = ["mcp.json", ".mcp.json"].map((filename) => join(projectDir, filename));
-		const projectContents = await Promise.all(projectCandidates.map((path) => readFile(path)));
+		const projectCandidates = ["mcp.json", ".mcp.json"].map(filename => path.join(projectDir, filename));
+		const projectContents = await Promise.all(projectCandidates.map(path => readFile(path)));
 		for (let i = 0; i < projectCandidates.length; i++) {
 			const content = projectContents[i];
 			if (content) {
@@ -120,7 +115,7 @@ async function loadMCPServers(ctx: LoadContext): Promise<LoadResult<MCPServer>>
 		}
 	}
 
-	const userContents = await Promise.all(userPaths.map((path) => readFile(path)));
+	const userContents = await Promise.all(userPaths.map(path => readFile(path)));
 	for (let i = 0; i < userPaths.length; i++) {
 		const content = userContents[i];
 		if (content) {
@@ -144,10 +139,10 @@ registerProvider<MCPServer>(mcpCapability.id, {
 async function loadSystemPrompt(ctx: LoadContext): Promise<LoadResult<SystemPrompt>> {
 	const items: SystemPrompt[] = [];
 
-	const userPaths = USER_DIRS.map((name) =>
-		join(ctx.home, name, PATHS.userAgent.replace(`${PATHS.userBase}/`, ""), "SYSTEM.md"),
+	const userPaths = USER_DIRS.map(name =>
+		path.join(ctx.home, name, PATHS.userAgent.replace(`${PATHS.userBase}/`, ""), "SYSTEM.md"),
 	);
-	const userContents = await Promise.all(userPaths.map((p) => readFile(p)));
+	const userContents = await Promise.all(userPaths.map(p => readFile(p)));
 	for (let i = 0; i < userPaths.length; i++) {
 		const content = userContents[i];
 		if (content) {
@@ -165,18 +160,18 @@ async function loadSystemPrompt(ctx: LoadContext): Promise<LoadResult<SystemProm
 	let current = ctx.cwd;
 	while (true) {
 		ancestors.push(current);
-		const parent = dirname(current);
+		const parent = path.dirname(current);
 		if (parent === current) break;
 		current = parent;
 	}
 
 	for (const dir of ancestors) {
-		const configDirs = PROJECT_DIRS.map((name) => join(dir, name));
-		const entriesResults = await Promise.all(configDirs.map((d) => readDirEntries(d)));
+		const configDirs = PROJECT_DIRS.map(name => path.join(dir, name));
+		const entriesResults = await Promise.all(configDirs.map(d => readDirEntries(d)));
 		const validConfigDir = configDirs.find((_, i) => entriesResults[i].length > 0);
 		if (!validConfigDir) continue;
 
-		const projectPath = join(validConfigDir, "SYSTEM.md");
+		const projectPath = path.join(validConfigDir, "SYSTEM.md");
 		const content = await readFile(projectPath);
 		if (content) {
 			items.push({
@@ -206,7 +201,7 @@ async function loadSkills(ctx: LoadContext): Promise<LoadResult<Skill>> {
 	const results = await Promise.all(
 		configDirs.map(({ dir, level }) =>
 			loadSkillsFromDir(ctx, {
-				dir: join(dir, "skills"),
+				dir: path.join(dir, "skills"),
 				providerId: PROVIDER_ID,
 				level,
 				requireDescription: true,
@@ -215,8 +210,8 @@ async function loadSkills(ctx: LoadContext): Promise<LoadResult<Skill>> {
 	);
 
 	return {
-		items: results.flatMap((r) => r.items),
-		warnings: results.flatMap((r) => r.warnings ?? []),
+		items: results.flatMap(r => r.items),
+		warnings: results.flatMap(r => r.warnings ?? []),
 	};
 }
 
@@ -234,7 +229,7 @@ async function loadSlashCommands(ctx: LoadContext): Promise<LoadResult<SlashComm
 	const warnings: string[] = [];
 
 	for (const { dir, level } of await getConfigDirs(ctx)) {
-		const commandsDir = join(dir, "commands");
+		const commandsDir = path.join(dir, "commands");
 		const result = await loadFilesFromDir<SlashCommand>(ctx, commandsDir, PROVIDER_ID, level, {
 			extensions: ["md"],
 			transform: (name, content, path, source) => ({
@@ -266,7 +261,7 @@ async function loadRules(ctx: LoadContext): Promise<LoadResult<Rule>> {
 	const warnings: string[] = [];
 
 	for (const { dir, level } of await getConfigDirs(ctx)) {
-		const rulesDir = join(dir, "rules");
+		const rulesDir = path.join(dir, "rules");
 		const result = await loadFilesFromDir<Rule>(ctx, rulesDir, PROVIDER_ID, level, {
 			extensions: ["md", "mdc"],
 			transform: (name, content, path, source) => {
@@ -304,7 +299,7 @@ async function loadPrompts(ctx: LoadContext): Promise<LoadResult<Prompt>> {
 	const warnings: string[] = [];
 
 	for (const { dir, level } of await getConfigDirs(ctx)) {
-		const promptsDir = join(dir, "prompts");
+		const promptsDir = path.join(dir, "prompts");
 		const result = await loadFilesFromDir<Prompt>(ctx, promptsDir, PROVIDER_ID, level, {
 			extensions: ["md"],
 			transform: (name, content, path, source) => ({
@@ -336,15 +331,15 @@ async function loadExtensionModules(ctx: LoadContext): Promise<LoadResult<Extens
 
 	const resolveExtensionPath = (rawPath: string): string => {
 		if (rawPath.startsWith("~/")) {
-			return join(ctx.home, rawPath.slice(2));
+			return path.join(ctx.home, rawPath.slice(2));
 		}
 		if (rawPath.startsWith("~")) {
-			return join(ctx.home, rawPath.slice(1));
+			return path.join(ctx.home, rawPath.slice(1));
 		}
-		if (isAbsolute(rawPath)) {
+		if (path.isAbsolute(rawPath)) {
 			return rawPath;
 		}
-		return resolve(ctx.cwd, rawPath);
+		return path.resolve(ctx.cwd, rawPath);
 	};
 
 	const createExtensionModule = (extPath: string, level: "user" | "project"): ExtensionModule => ({
@@ -357,8 +352,8 @@ async function loadExtensionModules(ctx: LoadContext): Promise<LoadResult<Extens
 	const configDirs = await getConfigDirs(ctx);
 
 	const [discoveredResults, settingsResults] = await Promise.all([
-		Promise.all(configDirs.map(({ dir }) => discoverExtensionModulePaths(ctx, join(dir, "extensions")))),
-		Promise.all(configDirs.map(({ dir }) => readFile(join(dir, "settings.json")))),
+		Promise.all(configDirs.map(({ dir }) => discoverExtensionModulePaths(ctx, path.join(dir, "extensions")))),
+		Promise.all(configDirs.map(({ dir }) => readFile(path.join(dir, "settings.json")))),
 	]);
 
 	for (let i = 0; i < configDirs.length; i++) {
@@ -379,7 +374,7 @@ async function loadExtensionModules(ctx: LoadContext): Promise<LoadResult<Extens
 		const settingsContent = settingsResults[i];
 		if (!settingsContent) continue;
 
-		const settingsPath = join(dir, "settings.json");
+		const settingsPath = path.join(dir, "settings.json");
 		const settingsData = parseJSON<{ extensions?: unknown }>(settingsContent);
 		const extensions = settingsData?.extensions;
 		if (!Array.isArray(extensions)) continue;
@@ -424,7 +419,7 @@ async function loadExtensionModules(ctx: LoadContext): Promise<LoadResult<Extens
 		}
 	}
 
-	const dirDiscoveryResults = await Promise.all(dirDiscoveryPromises.map((d) => d.promise));
+	const dirDiscoveryResults = await Promise.all(dirDiscoveryPromises.map(d => d.promise));
 	for (let i = 0; i < dirDiscoveryPromises.length; i++) {
 		const { level } = dirDiscoveryPromises[i];
 		for (const extPath of dirDiscoveryResults[i]) {
@@ -449,7 +444,7 @@ async function loadExtensions(ctx: LoadContext): Promise<LoadResult<Extension>>
 	const warnings: string[] = [];
 
 	const configDirs = await getConfigDirs(ctx);
-	const entriesResults = await Promise.all(configDirs.map(({ dir }) => readDirEntries(join(dir, "extensions"))));
+	const entriesResults = await Promise.all(configDirs.map(({ dir }) => readDirEntries(path.join(dir, "extensions"))));
 
 	const manifestCandidates: Array<{
 		extDir: string;
@@ -461,16 +456,16 @@ async function loadExtensions(ctx: LoadContext): Promise<LoadResult<Extension>>
 	for (let i = 0; i < configDirs.length; i++) {
 		const { dir, level } = configDirs[i];
 		const entries = entriesResults[i];
-		const extensionsDir = join(dir, "extensions");
+		const extensionsDir = path.join(dir, "extensions");
 
 		for (const entry of entries) {
 			if (entry.name.startsWith(".")) continue;
 			if (!entry.isDirectory()) continue;
 
-			const extDir = join(extensionsDir, entry.name);
+			const extDir = path.join(extensionsDir, entry.name);
 			manifestCandidates.push({
 				extDir,
-				manifestPath: join(extDir, "gemini-extension.json"),
+				manifestPath: path.join(extDir, "gemini-extension.json"),
 				entryName: entry.name,
 				level,
 			});
@@ -516,7 +511,7 @@ async function loadInstructions(ctx: LoadContext): Promise<LoadResult<Instructio
 	const warnings: string[] = [];
 
 	for (const { dir, level } of await getConfigDirs(ctx)) {
-		const instructionsDir = join(dir, "instructions");
+		const instructionsDir = path.join(dir, "instructions");
 		const result = await loadFilesFromDir<Instruction>(ctx, instructionsDir, PROVIDER_ID, level, {
 			extensions: ["md"],
 			transform: (name, content, path, source) => {
@@ -561,7 +556,7 @@ async function loadHooks(ctx: LoadContext): Promise<LoadResult<Hook>> {
 	for (const { dir, level } of configDirs) {
 		for (const hookType of hookTypes) {
 			typeDirRequests.push({
-				typeDir: join(dir, "hooks", hookType),
+				typeDir: path.join(dir, "hooks", hookType),
 				hookType,
 				level,
 			});
@@ -578,17 +573,17 @@ async function loadHooks(ctx: LoadContext): Promise<LoadResult<Hook>> {
 			if (entry.name.startsWith(".")) continue;
 			if (!entry.isFile()) continue;
 
-			const path = join(typeDir, entry.name);
+			const hookPath = path.join(typeDir, entry.name);
 			const baseName = entry.name.includes(".") ? entry.name.slice(0, entry.name.lastIndexOf(".")) : entry.name;
 			const tool = baseName === "*" ? "*" : baseName;
 
 			items.push({
 				name: entry.name,
-				path,
+				path: hookPath,
 				type: hookType,
 				tool,
 				level,
-				_source: createSourceMeta(PROVIDER_ID, path, level),
+				_source: createSourceMeta(PROVIDER_ID, hookPath, level),
 			});
 		}
 	}
@@ -610,7 +605,7 @@ async function loadTools(ctx: LoadContext): Promise<LoadResult<CustomTool>> {
 	const warnings: string[] = [];
 
 	const configDirs = await getConfigDirs(ctx);
-	const entriesResults = await Promise.all(configDirs.map(({ dir }) => readDirEntries(join(dir, "tools"))));
+	const entriesResults = await Promise.all(configDirs.map(({ dir }) => readDirEntries(path.join(dir, "tools"))));
 
 	const fileLoadPromises: Array<Promise<{ items: CustomTool[]; warnings?: string[] }>> = [];
 	const subDirCandidates: Array<{
@@ -624,7 +619,7 @@ async function loadTools(ctx: LoadContext): Promise<LoadResult<CustomTool>> {
 		const toolEntries = entriesResults[i];
 		if (toolEntries.length === 0) continue;
 
-		const toolsDir = join(dir, "tools");
+		const toolsDir = path.join(dir, "tools");
 
 		fileLoadPromises.push(
 			loadFilesFromDir<CustomTool>(ctx, toolsDir, PROVIDER_ID, level, {
@@ -657,7 +652,7 @@ async function loadTools(ctx: LoadContext): Promise<LoadResult<CustomTool>> {
 			if (!entry.isDirectory()) continue;
 
 			subDirCandidates.push({
-				indexPath: join(toolsDir, entry.name, "index.ts"),
+				indexPath: path.join(toolsDir, entry.name, "index.ts"),
 				entryName: entry.name,
 				level,
 			});
@@ -705,7 +700,7 @@ async function loadSettings(ctx: LoadContext): Promise<LoadResult<Settings>> {
 	const warnings: string[] = [];
 
 	for (const { dir, level } of await getConfigDirs(ctx)) {
-		const settingsPath = join(dir, "settings.json");
+		const settingsPath = path.join(dir, "settings.json");
 		const content = await readFile(settingsPath);
 		if (!content) continue;
 
@@ -739,10 +734,10 @@ async function loadContextFiles(ctx: LoadContext): Promise<LoadResult<ContextFil
 	const items: ContextFile[] = [];
 	const warnings: string[] = [];
 
-	const userPaths = USER_DIRS.map((name) =>
-		join(ctx.home, name, PATHS.userAgent.replace(`${PATHS.userBase}/`, ""), "AGENTS.md"),
+	const userPaths = USER_DIRS.map(name =>
+		path.join(ctx.home, name, PATHS.userAgent.replace(`${PATHS.userBase}/`, ""), "AGENTS.md"),
 	);
-	const userContents = await Promise.all(userPaths.map((p) => readFile(p)));
+	const userContents = await Promise.all(userPaths.map(p => readFile(p)));
 	for (let i = 0; i < userPaths.length; i++) {
 		const content = userContents[i];
 		if (content) {
@@ -761,19 +756,19 @@ async function loadContextFiles(ctx: LoadContext): Promise<LoadResult<ContextFil
 	let depth = 0;
 	while (true) {
 		ancestors.push({ dir: current, depth });
-		const parent = dirname(current);
+		const parent = path.dirname(current);
 		if (parent === current) break;
 		current = parent;
 		depth++;
 	}
 
 	for (const { dir, depth: ancestorDepth } of ancestors) {
-		const configDirs = PROJECT_DIRS.map((name) => join(dir, name));
-		const entriesResults = await Promise.all(configDirs.map((d) => readDirEntries(d)));
+		const configDirs = PROJECT_DIRS.map(name => path.join(dir, name));
+		const entriesResults = await Promise.all(configDirs.map(d => readDirEntries(d)));
 		const validConfigDir = configDirs.find((_, i) => entriesResults[i].length > 0);
 		if (!validConfigDir) continue;
 
-		const projectPath = join(validConfigDir, "AGENTS.md");
+		const projectPath = path.join(validConfigDir, "AGENTS.md");
 		const content = await readFile(projectPath);
 		if (content) {
 			items.push({

package/src/discovery/claude.ts

@@ -4,20 +4,19 @@
  * Loads configuration from .claude directories.
  * Priority: 80 (tool-specific, below builtin but above shared standards)
  */
-
-import { join, sep } from "node:path";
-import { registerProvider } from "@oh-my-pi/pi-coding-agent/capability";
-import { type ContextFile, contextFileCapability } from "@oh-my-pi/pi-coding-agent/capability/context-file";
-import { type ExtensionModule, extensionModuleCapability } from "@oh-my-pi/pi-coding-agent/capability/extension-module";
-import { readFile } from "@oh-my-pi/pi-coding-agent/capability/fs";
-import { type Hook, hookCapability } from "@oh-my-pi/pi-coding-agent/capability/hook";
-import { type MCPServer, mcpCapability } from "@oh-my-pi/pi-coding-agent/capability/mcp";
-import { type Settings, settingsCapability } from "@oh-my-pi/pi-coding-agent/capability/settings";
-import { type Skill, skillCapability } from "@oh-my-pi/pi-coding-agent/capability/skill";
-import { type SlashCommand, slashCommandCapability } from "@oh-my-pi/pi-coding-agent/capability/slash-command";
-import { type SystemPrompt, systemPromptCapability } from "@oh-my-pi/pi-coding-agent/capability/system-prompt";
-import { type CustomTool, toolCapability } from "@oh-my-pi/pi-coding-agent/capability/tool";
-import type { LoadContext, LoadResult } from "@oh-my-pi/pi-coding-agent/capability/types";
+import * as path from "node:path";
+import { registerProvider } from "../capability";
+import { type ContextFile, contextFileCapability } from "../capability/context-file";
+import { type ExtensionModule, extensionModuleCapability } from "../capability/extension-module";
+import { readFile } from "../capability/fs";
+import { type Hook, hookCapability } from "../capability/hook";
+import { type MCPServer, mcpCapability } from "../capability/mcp";
+import { type Settings, settingsCapability } from "../capability/settings";
+import { type Skill, skillCapability } from "../capability/skill";
+import { type SlashCommand, slashCommandCapability } from "../capability/slash-command";
+import { type SystemPrompt, systemPromptCapability } from "../capability/system-prompt";
+import { type CustomTool, toolCapability } from "../capability/tool";
+import type { LoadContext, LoadResult } from "../capability/types";
 import {
 	calculateDepth,
 	createSourceMeta,
@@ -38,14 +37,14 @@ const CONFIG_DIR = ".claude";
  * Get user-level .claude path.
  */
 function getUserClaude(ctx: LoadContext): string {
-	return join(ctx.home, CONFIG_DIR);
+	return path.join(ctx.home, CONFIG_DIR);
 }
 
 /**
  * Get project-level .claude path (cwd only).
  */
 function getProjectClaude(ctx: LoadContext): string {
-	return join(ctx.cwd, CONFIG_DIR);
+	return path.join(ctx.cwd, CONFIG_DIR);
 }
 
 // =============================================================================
@@ -57,12 +56,12 @@ async function loadMCPServers(ctx: LoadContext): Promise<LoadResult<MCPServer>>
 	const warnings: string[] = [];
 
 	const userBase = getUserClaude(ctx);
-	const userClaudeJson = join(ctx.home, ".claude.json");
-	const userMcpJson = join(userBase, "mcp.json");
+	const userClaudeJson = path.join(ctx.home, ".claude.json");
+	const userMcpJson = path.join(userBase, "mcp.json");
 
-	const projectBase = join(ctx.cwd, CONFIG_DIR);
-	const projectMcpJson = join(projectBase, ".mcp.json");
-	const projectMcpJsonAlt = join(projectBase, "mcp.json");
+	const projectBase = path.join(ctx.cwd, CONFIG_DIR);
+	const projectMcpJson = path.join(projectBase, ".mcp.json");
+	const projectMcpJsonAlt = path.join(projectBase, "mcp.json");
 
 	const userPaths = [
 		{ path: userClaudeJson, level: "user" as const },
@@ -126,7 +125,7 @@ async function loadContextFiles(ctx: LoadContext): Promise<LoadResult<ContextFil
 	const warnings: string[] = [];
 
 	const userBase = getUserClaude(ctx);
-	const userClaudeMd = join(userBase, "CLAUDE.md");
+	const userClaudeMd = path.join(userBase, "CLAUDE.md");
 
 	const userContent = await readFile(userClaudeMd);
 	if (userContent !== null) {
@@ -139,10 +138,10 @@ async function loadContextFiles(ctx: LoadContext): Promise<LoadResult<ContextFil
 	}
 
 	const projectBase = getProjectClaude(ctx);
-	const projectClaudeMd = join(projectBase, "CLAUDE.md");
+	const projectClaudeMd = path.join(projectBase, "CLAUDE.md");
 	const projectContent = await readFile(projectClaudeMd);
 	if (projectContent !== null) {
-		const depth = calculateDepth(ctx.cwd, projectBase, sep);
+		const depth = calculateDepth(ctx.cwd, projectBase, path.sep);
 		items.push({
 			path: projectClaudeMd,
 			content: projectContent,
@@ -160,8 +159,8 @@ async function loadContextFiles(ctx: LoadContext): Promise<LoadResult<ContextFil
 // =============================================================================
 
 async function loadSkills(ctx: LoadContext): Promise<LoadResult<Skill>> {
-	const userSkillsDir = join(getUserClaude(ctx), "skills");
-	const projectSkillsDir = join(getProjectClaude(ctx), "skills");
+	const userSkillsDir = path.join(getUserClaude(ctx), "skills");
+	const projectSkillsDir = path.join(getProjectClaude(ctx), "skills");
 
 	const results = await Promise.all([
 		loadSkillsFromDir(ctx, { dir: userSkillsDir, providerId: PROVIDER_ID, level: "user" }),
@@ -169,8 +168,8 @@ async function loadSkills(ctx: LoadContext): Promise<LoadResult<Skill>> {
 	]);
 
 	return {
-		items: results.flatMap((r) => r.items),
-		warnings: results.flatMap((r) => r.warnings ?? []),
+		items: results.flatMap(r => r.items),
+		warnings: results.flatMap(r => r.warnings ?? []),
 	};
 }
 
@@ -183,8 +182,8 @@ async function loadExtensionModules(ctx: LoadContext): Promise<LoadResult<Extens
 	const warnings: string[] = [];
 
 	const userBase = getUserClaude(ctx);
-	const userExtensionsDir = join(userBase, "extensions");
-	const projectExtensionsDir = join(ctx.cwd, CONFIG_DIR, "extensions");
+	const userExtensionsDir = path.join(userBase, "extensions");
+	const projectExtensionsDir = path.join(ctx.cwd, CONFIG_DIR, "extensions");
 
 	const dirsToDiscover: { dir: string; level: "user" | "project" }[] = [
 		{ dir: userExtensionsDir, level: "user" },
@@ -194,7 +193,7 @@ async function loadExtensionModules(ctx: LoadContext): Promise<LoadResult<Extens
 	const pathsByLevel = await Promise.all(
 		dirsToDiscover.map(async ({ dir, level }) => {
 			const paths = await discoverExtensionModulePaths(ctx, dir);
-			return paths.map((extPath) => ({ extPath, level }));
+			return paths.map(extPath => ({ extPath, level }));
 		}),
 	);
 
@@ -221,7 +220,7 @@ async function loadSlashCommands(ctx: LoadContext): Promise<LoadResult<SlashComm
 	const warnings: string[] = [];
 
 	const userBase = getUserClaude(ctx);
-	const userCommandsDir = join(userBase, "commands");
+	const userCommandsDir = path.join(userBase, "commands");
 
 	const userResult = await loadFilesFromDir<SlashCommand>(ctx, userCommandsDir, PROVIDER_ID, "user", {
 		extensions: ["md"],
@@ -240,7 +239,7 @@ async function loadSlashCommands(ctx: LoadContext): Promise<LoadResult<SlashComm
 	items.push(...userResult.items);
 	if (userResult.warnings) warnings.push(...userResult.warnings);
 
-	const projectCommandsDir = join(ctx.cwd, CONFIG_DIR, "commands");
+	const projectCommandsDir = path.join(ctx.cwd, CONFIG_DIR, "commands");
 
 	const projectResult = await loadFilesFromDir<SlashCommand>(ctx, projectCommandsDir, PROVIDER_ID, "project", {
 		extensions: ["md"],
@@ -271,18 +270,18 @@ async function loadHooks(ctx: LoadContext): Promise<LoadResult<Hook>> {
 	const warnings: string[] = [];
 
 	const userBase = getUserClaude(ctx);
-	const userHooksDir = join(userBase, "hooks");
+	const userHooksDir = path.join(userBase, "hooks");
 	const projectBase = getProjectClaude(ctx);
-	const projectHooksDir = join(projectBase, "hooks");
+	const projectHooksDir = path.join(projectBase, "hooks");
 
 	const hookTypes = ["pre", "post"] as const;
 
 	const loadTasks: { dir: string; hookType: "pre" | "post"; level: "user" | "project" }[] = [];
 	for (const hookType of hookTypes) {
-		loadTasks.push({ dir: join(userHooksDir, hookType), hookType, level: "user" });
+		loadTasks.push({ dir: path.join(userHooksDir, hookType), hookType, level: "user" });
 	}
 	for (const hookType of hookTypes) {
-		loadTasks.push({ dir: join(projectHooksDir, hookType), hookType, level: "project" });
+		loadTasks.push({ dir: path.join(projectHooksDir, hookType), hookType, level: "project" });
 	}
 
 	const results = await Promise.all(
@@ -320,7 +319,7 @@ async function loadTools(ctx: LoadContext): Promise<LoadResult<CustomTool>> {
 	const warnings: string[] = [];
 
 	const userBase = getUserClaude(ctx);
-	const userToolsDir = join(userBase, "tools");
+	const userToolsDir = path.join(userBase, "tools");
 
 	const userResult = await loadFilesFromDir<CustomTool>(ctx, userToolsDir, PROVIDER_ID, "user", {
 		transform: (name, _content, path, source) => {
@@ -339,7 +338,7 @@ async function loadTools(ctx: LoadContext): Promise<LoadResult<CustomTool>> {
 	if (userResult.warnings) warnings.push(...userResult.warnings);
 
 	const projectBase = getProjectClaude(ctx);
-	const projectToolsDir = join(projectBase, "tools");
+	const projectToolsDir = path.join(projectBase, "tools");
 
 	const projectResult = await loadFilesFromDir<CustomTool>(ctx, projectToolsDir, PROVIDER_ID, "project", {
 		transform: (name, _content, path, source) => {
@@ -369,7 +368,7 @@ async function loadSystemPrompts(ctx: LoadContext): Promise<LoadResult<SystemPro
 	const warnings: string[] = [];
 
 	const userBase = getUserClaude(ctx);
-	const userSystemMd = join(userBase, "SYSTEM.md");
+	const userSystemMd = path.join(userBase, "SYSTEM.md");
 
 	const content = await readFile(userSystemMd);
 	if (content !== null) {
@@ -393,7 +392,7 @@ async function loadSettings(ctx: LoadContext): Promise<LoadResult<Settings>> {
 	const warnings: string[] = [];
 
 	const userBase = getUserClaude(ctx);
-	const userSettingsJson = join(userBase, "settings.json");
+	const userSettingsJson = path.join(userBase, "settings.json");
 
 	const userContent = await readFile(userSettingsJson);
 	if (userContent) {
@@ -411,7 +410,7 @@ async function loadSettings(ctx: LoadContext): Promise<LoadResult<Settings>> {
 	}
 
 	const projectBase = getProjectClaude(ctx);
-	const projectSettingsJson = join(projectBase, "settings.json");
+	const projectSettingsJson = path.join(projectBase, "settings.json");
 	const projectContent = await readFile(projectSettingsJson);
 	if (projectContent) {
 		const data = parseJSON<Record<string, unknown>>(projectContent);

package/src/discovery/cline.ts

@@ -4,14 +4,13 @@
  * Loads rules from .clinerules (can be single file or directory with *.md files).
  * Project-only (no user-level config).
  */
-
-import { dirname, resolve } from "node:path";
-import { registerProvider } from "@oh-my-pi/pi-coding-agent/capability";
-import { readDirEntries, readFile } from "@oh-my-pi/pi-coding-agent/capability/fs";
-import type { Rule } from "@oh-my-pi/pi-coding-agent/capability/rule";
-import { ruleCapability } from "@oh-my-pi/pi-coding-agent/capability/rule";
-import type { LoadContext, LoadResult } from "@oh-my-pi/pi-coding-agent/capability/types";
-import { parseFrontmatter } from "@oh-my-pi/pi-coding-agent/utils/frontmatter";
+import * as path from "node:path";
+import { registerProvider } from "../capability";
+import { readDirEntries, readFile } from "../capability/fs";
+import type { Rule } from "../capability/rule";
+import { ruleCapability } from "../capability/rule";
+import type { LoadContext, LoadResult } from "../capability/types";
+import { parseFrontmatter } from "../utils/frontmatter";
 import { createSourceMeta, loadFilesFromDir } from "./helpers";
 
 const PROVIDER_ID = "cline";
@@ -19,18 +18,18 @@ const DISPLAY_NAME = "Cline";
 const PRIORITY = 40;
 
 async function findClinerules(startDir: string): Promise<{ path: string; isDir: boolean } | null> {
-	let current = resolve(startDir);
+	let current = path.resolve(startDir);
 
 	while (true) {
 		const entries = await readDirEntries(current);
-		const entry = entries.find((e) => e.name === ".clinerules");
+		const entry = entries.find(e => e.name === ".clinerules");
 		if (entry) {
 			return {
-				path: resolve(current, ".clinerules"),
+				path: path.resolve(current, ".clinerules"),
 				isDir: entry.isDirectory(),
 			};
 		}
-		const parent = dirname(current);
+		const parent = path.dirname(current);
 		if (parent === current) return null;
 		current = parent;
 	}