test(cli): remove stray files and require extensionsDir in createExtension

This commit is contained in:
Jerop Kipruto
2026-03-17 13:35:49 -04:00
parent bb1c48a870
commit f20a01b0db
2 changed files with 17 additions and 3 deletions

View File

@@ -1 +0,0 @@
{ "name": "my-extension", "version": "1.0.0", "mcpServers": {} }

View File

@@ -18,7 +18,7 @@ import {
} from '../config/extensions/variables.js';
export function createExtension({
extensionsDir = 'extensions-dir',
extensionsDir,
name = 'my-extension',
version = '1.0.0',
addContextFile = false,
@@ -27,7 +27,22 @@ export function createExtension({
installMetadata = undefined as ExtensionInstallMetadata | undefined,
settings = undefined as ExtensionSetting[] | undefined,
themes = undefined as CustomTheme[] | undefined,
} = {}): string {
}: {
extensionsDir: string;
name?: string;
version?: string;
addContextFile?: boolean;
contextFileName?: string;
mcpServers?: Record<string, MCPServerConfig>;
installMetadata?: ExtensionInstallMetadata;
settings?: ExtensionSetting[];
themes?: CustomTheme[];
}): string {
if (!extensionsDir) {
throw new Error(
'extensionsDir is required to avoid leaking files during tests',
);
}
const extDir = path.join(extensionsDir, name);
fs.mkdirSync(extDir, { recursive: true });
fs.writeFileSync(