@omnidev-ai/core 0.1.0 → 0.1.1

package/package.json

@@ -1,6 +1,6 @@
 {
 	"name": "@omnidev-ai/core",
-	"version": "0.1.0",
+	"version": "0.1.1",
 	"type": "module",
 	"license": "MIT",
 	"repository": {
@@ -9,7 +9,8 @@
 		"directory": "packages/core"
 	},
 	"exports": {
-		".": "./src/index.ts"
+		".": "./src/index.ts",
+		"./test-utils": "./src/test-utils/index.ts"
 	},
 	"files": [
 		"src",

package/src/capability/commands.ts

@@ -24,7 +24,9 @@ export async function loadCommands(
 	}
 
 	const commands: Command[] = [];
-	const entries = readdirSync(commandsDir, { withFileTypes: true });
+	const entries = readdirSync(commandsDir, { withFileTypes: true }).sort((a, b) =>
+		a.name.localeCompare(b.name),
+	);
 
 	for (const entry of entries) {
 		if (entry.isDirectory()) {

package/src/capability/skills.ts

@@ -16,7 +16,9 @@ export async function loadSkills(capabilityPath: string, capabilityId: string):
 	}
 
 	const skills: Skill[] = [];
-	const entries = readdirSync(skillsDir, { withFileTypes: true });
+	const entries = readdirSync(skillsDir, { withFileTypes: true }).sort((a, b) =>
+		a.name.localeCompare(b.name),
+	);
 
 	for (const entry of entries) {
 		if (entry.isDirectory()) {

package/src/config/env.test.ts

@@ -1,18 +1,16 @@
 import { afterEach, beforeEach, describe, expect, test } from "bun:test";
 import { existsSync, mkdirSync, rmSync, writeFileSync } from "node:fs";
+import { tmpdir } from "../test-utils/index.js";
 import type { EnvDeclaration } from "../types";
 import { isSecretEnvVar, loadEnvironment, validateEnv } from "./env";
 
 describe("loadEnvironment", () => {
 	const originalCwd = process.cwd();
-	const testDir = "/tmp/omnidev-test-env";
+	let testDir: string;
 
 	beforeEach(() => {
-		// Clean up and create test directory
-		if (existsSync(testDir)) {
-			rmSync(testDir, { recursive: true });
-		}
-		mkdirSync(testDir, { recursive: true });
+		// Create test directory in /tmp
+		testDir = tmpdir("env-test-");
 		process.chdir(testDir);
 	});
 

package/src/config/loader.test.ts

@@ -1,23 +1,21 @@
 import { afterEach, beforeEach, describe, expect, test } from "bun:test";
 import { existsSync, mkdirSync, rmSync, writeFileSync } from "node:fs";
+import { tmpdir } from "../test-utils/index.js";
 import { loadConfig } from "./loader";
 
-const TEST_DIR = "/tmp/omnidev-test-loader";
 const CONFIG_PATH = "omni.toml";
 const LOCAL_CONFIG = "omni.local.toml";
 
 // Save and restore the current working directory
 let originalCwd: string;
+let TEST_DIR: string;
 
 beforeEach(() => {
 	// Save original cwd
 	originalCwd = process.cwd();
 
-	// Clean up test directory
-	if (existsSync(TEST_DIR)) {
-		rmSync(TEST_DIR, { recursive: true });
-	}
-	mkdirSync(TEST_DIR, { recursive: true });
+	// Create test directory in /tmp
+	TEST_DIR = tmpdir("loader-test-");
 	process.chdir(TEST_DIR);
 });
 

package/src/config/profiles.test.ts

@@ -1,22 +1,19 @@
 import { afterEach, beforeEach, describe, expect, test } from "bun:test";
 import { existsSync, mkdirSync, rmSync, writeFileSync } from "node:fs";
+import { tmpdir } from "../test-utils/index.js";
 import { readActiveProfileState } from "../state/active-profile.js";
 import type { OmniConfig } from "../types/index.js";
 import { getActiveProfile, resolveEnabledCapabilities, setActiveProfile } from "./profiles.js";
 
 describe("getActiveProfile", () => {
-	const TEST_DIR = ".omni-test-profiles";
+	let TEST_DIR: string;
 	let originalCwd: string;
 
 	beforeEach(() => {
-		if (!existsSync(TEST_DIR)) {
-			mkdirSync(TEST_DIR, { recursive: true });
-		}
 		originalCwd = process.cwd();
+		TEST_DIR = tmpdir("profiles-test-");
 		process.chdir(TEST_DIR);
-		if (!existsSync(".omni")) {
-			mkdirSync(".omni", { recursive: true });
-		}
+		mkdirSync(".omni", { recursive: true });
 	});
 
 	afterEach(() => {

package/src/config/provider.test.ts

@@ -1,17 +1,18 @@
 import { afterEach, beforeEach, describe, expect, test } from "bun:test";
-import { existsSync, mkdirSync, rmSync } from "node:fs";
+import { existsSync, rmSync } from "node:fs";
+import { tmpdir } from "../test-utils/index.js";
 import { parseProviderFlag } from "./provider.js";
 
-const TEST_DIR = ".test-omni";
+let TEST_DIR: string;
+const originalCwd = process.cwd();
 
 beforeEach(() => {
-	if (existsSync(TEST_DIR)) {
-		rmSync(TEST_DIR, { recursive: true });
-	}
-	mkdirSync(TEST_DIR, { recursive: true });
+	TEST_DIR = tmpdir("provider-test-");
+	process.chdir(TEST_DIR);
 });
 
 afterEach(() => {
+	process.chdir(originalCwd);
 	if (existsSync(TEST_DIR)) {
 		rmSync(TEST_DIR, { recursive: true });
 	}

package/src/gitignore/manager.test.ts

@@ -1,5 +1,6 @@
 import { afterEach, beforeEach, describe, expect, test } from "bun:test";
-import { existsSync, mkdirSync, rmSync } from "node:fs";
+import { existsSync, rmSync } from "node:fs";
+import { tmpdir } from "../test-utils/index.js";
 import {
 	addCapabilityPatterns,
 	buildGitignoreContent,
@@ -10,23 +11,19 @@ import {
 	writeGitignore,
 } from "./manager.js";
 
-const TEST_DIR = ".omni-test-gitignore";
+let TEST_DIR: string;
+const originalCwd = process.cwd();
 
 describe("Gitignore Manager", () => {
 	beforeEach(() => {
-		// Create test directory
-		if (existsSync(TEST_DIR)) {
-			rmSync(TEST_DIR, { recursive: true });
-		}
-		mkdirSync(TEST_DIR, { recursive: true });
-
-		// Change working directory
+		// Create test directory in /tmp
+		TEST_DIR = tmpdir("gitignore-test-");
 		process.chdir(TEST_DIR);
 	});
 
 	afterEach(() => {
 		// Restore working directory
-		process.chdir("..");
+		process.chdir(originalCwd);
 
 		// Clean up test directory
 		if (existsSync(TEST_DIR)) {

package/src/state/active-profile.test.ts

@@ -1,7 +1,7 @@
 import { afterEach, beforeEach, describe, expect, test } from "bun:test";
-import { existsSync, mkdirSync, mkdtempSync, rmSync } from "node:fs";
-import { tmpdir } from "node:os";
+import { existsSync, mkdirSync, rmSync } from "node:fs";
 import { join } from "node:path";
+import { tmpdir } from "../test-utils/index.js";
 import {
 	clearActiveProfileState,
 	readActiveProfileState,
@@ -14,7 +14,7 @@ describe("active-profile state", () => {
 
 	beforeEach(() => {
 		originalCwd = process.cwd();
-		tempDir = mkdtempSync(join(tmpdir(), "active-profile-test-"));
+		tempDir = tmpdir("active-profile-test-");
 		mkdirSync(join(tempDir, ".omni"), { recursive: true });
 		process.chdir(tempDir);
 	});

package/src/test-utils/helpers.ts

@@ -3,6 +3,9 @@
  */
 
 import { expect } from "bun:test";
+import { mkdtempSync } from "node:fs";
+import { tmpdir as osTmpdir } from "node:os";
+import { join } from "node:path";
 
 /**
  * Expects an async function to throw an error
@@ -185,3 +188,12 @@ export async function captureConsole<T>(
 		console.warn = originalWarn;
 	}
 }
+
+/**
+ * Creates a unique temporary directory for tests in /tmp
+ * @param prefix - Optional prefix for the directory name (default: "omnidev-test-")
+ * @returns Path to the created temporary directory
+ */
+export function tmpdir(prefix = "omnidev-test-"): string {
+	return mkdtempSync(join(osTmpdir(), prefix));
+}

package/src/test-utils/index.ts

@@ -15,6 +15,7 @@ export {
 	createSpy,
 	delay,
 	expectToThrowAsync,
+	tmpdir,
 	waitForCondition,
 } from "./helpers";
 // Re-export all mock factories