@oh-my-pi/pi-coding-agent 13.7.0 → 13.7.1

package/package.json

@@ -1,7 +1,7 @@
 {
 	"type": "module",
 	"name": "@oh-my-pi/pi-coding-agent",
-	"version": "13.7.0",
+	"version": "13.7.1",
 	"description": "Coding agent CLI with read, bash, edit, write tools and session management",
 	"homepage": "https://github.com/can1357/oh-my-pi",
 	"author": "Can Boluk",
@@ -41,12 +41,12 @@
 	},
 	"dependencies": {
 		"@mozilla/readability": "^0.6",
-		"@oh-my-pi/omp-stats": "13.7.0",
-		"@oh-my-pi/pi-agent-core": "13.7.0",
-		"@oh-my-pi/pi-ai": "13.7.0",
-		"@oh-my-pi/pi-natives": "13.7.0",
-		"@oh-my-pi/pi-tui": "13.7.0",
-		"@oh-my-pi/pi-utils": "13.7.0",
+		"@oh-my-pi/omp-stats": "13.7.1",
+		"@oh-my-pi/pi-agent-core": "13.7.1",
+		"@oh-my-pi/pi-ai": "13.7.1",
+		"@oh-my-pi/pi-natives": "13.7.1",
+		"@oh-my-pi/pi-tui": "13.7.1",
+		"@oh-my-pi/pi-utils": "13.7.1",
 		"@sinclair/typebox": "^0.34",
 		"@xterm/headless": "^6.0",
 		"ajv": "^8.18",

package/src/discovery/helpers.ts

@@ -275,6 +275,16 @@ export interface ScanSkillsFromDirOptions {
 	requireDescription?: boolean;
 }
 
+// Stable ordering used for skill lists in prompts: name (case-insensitive), then name, then path.
+export function compareSkillOrder(aName: string, aPath: string, bName: string, bPath: string): number {
+	const cmp = (a: string, b: string): number => (a < b ? -1 : a > b ? 1 : 0);
+	const lowerCompare = cmp(aName.toLowerCase(), bName.toLowerCase());
+	if (lowerCompare !== 0) return lowerCompare;
+	const nameCompare = cmp(aName, bName);
+	if (nameCompare !== 0) return nameCompare;
+	return cmp(aPath, bPath);
+}
+
 export async function scanSkillsFromDir(
 	_ctx: LoadContext,
 	options: ScanSkillsFromDirOptions,
@@ -301,8 +311,10 @@ export async function scanSkillsFromDir(
 				return;
 			}
 			const skillDirName = path.basename(path.dirname(skillPath));
+			const rawName = frontmatter.name;
+			const name = typeof rawName === "string" ? rawName.trim() || skillDirName : skillDirName;
 			items.push({
-				name: (frontmatter.name as string) || skillDirName,
+				name,
 				path: skillPath,
 				content: body,
 				frontmatter: frontmatter as SkillFrontmatter,
@@ -325,6 +337,9 @@ export async function scanSkillsFromDir(
 	}
 	await Promise.all(work);
 
+	// Deterministic ordering: async file reads complete nondeterministically, so sort after loading.
+	items.sort((a, b) => compareSkillOrder(a.name, a.path, b.name, b.path));
+
 	return { items, warnings };
 }
 

package/src/extensibility/skills.ts

@@ -5,7 +5,7 @@ import { skillCapability } from "../capability/skill";
 import type { SourceMeta } from "../capability/types";
 import type { SkillsSettings } from "../config/settings";
 import { type Skill as CapabilitySkill, loadCapability } from "../discovery";
-import { scanSkillsFromDir } from "../discovery/helpers";
+import { compareSkillOrder, scanSkillsFromDir } from "../discovery/helpers";
 import { expandTilde } from "../tools/path-utils";
 
 export interface Skill {
@@ -235,8 +235,12 @@ export async function loadSkills(options: LoadSkillsOptions = {}): Promise<LoadS
 		}
 	}
 
+	const skills = Array.from(skillMap.values());
+	// Deterministic ordering for prompt stability (case-insensitive, then exact name, then path).
+	skills.sort((a, b) => compareSkillOrder(a.name, a.filePath, b.name, b.filePath));
+
 	return {
-		skills: Array.from(skillMap.values()),
+		skills,
 		warnings: [...(result.warnings ?? []).map(w => ({ skillPath: "", message: w })), ...collisionWarnings],
 	};
 }