resolve comments

Author: xtbase

packages/create-onchain/rules/onchainkit.cursor.txt

@@ -19,7 +19,6 @@ vi.mock('./utils.js', () => ({
   createClickableLink: vi.fn(),
   isValidPackageName: vi.fn().mockReturnValue(true),
   toValidPackageName: vi.fn().mockImplementation((name) => name),
-  ensureDir: vi.fn().mockResolvedValue(undefined),
   addToGitignore: vi.fn().mockResolvedValue(undefined),
 }));
 vi.mock('ora', () => ({

packages/create-onchain/rules/onchainkit.copilot.txt renamed to packages/create-onchain/rules/onchainkit.txt

@@ -8,7 +8,6 @@ import {
   toValidPackageName,
   createClickableLink,
   copyDir,
-  ensureDir,
   addToGitignore,
 } from './utils.js';
 import { fileURLToPath } from 'url';
@@ -154,60 +153,45 @@ export async function createOnchainKitTemplate() {
 
   if (aiTool !== 'none') {
     console.log(`${pc.greenBright('\u2713')} ${pc.blueBright(`${aiTool}`)}`);
+    const onchainkitRulesTemplate = await fs.promises.readFile(
+      path.resolve(
+        fileURLToPath(import.meta.url),
+        '../../../rules/onchainkit.txt',
+      ),
+      'utf-8',
+    );
 
     if (aiTool === 'cursor') {
-      const cursorRulesTemplate = await fs.promises.readFile(
-        path.resolve(
-          fileURLToPath(import.meta.url),
-          '../../../rules/onchainkit.cursor.txt',
-        ),
-        'utf-8',
-      );
-
       const rulesDir = path.join(root, '.cursor/rules');
 
       const fileContent = `---
 description: OnchainKit Cursor Rules
 globs:
 alwaysApply: false
 ---
-${cursorRulesTemplate.trim()}`;
-      await ensureDir(rulesDir);
+${onchainkitRulesTemplate.trim()}`;
+      await fs.promises.mkdir(rulesDir, { recursive: true });
       // Write the file
       await fs.promises.writeFile(path.join(rulesDir, 'onchainkit.mdc'), fileContent);
       await addToGitignore({
         gitignorePath: path.join(root, '.gitignore'),
         additionalPath: '.cursor/rules/onchainkit.mdc',
       });
     } else if (aiTool === 'windsurf') {
-      const windsurfRulesTemplate = await fs.promises.readFile(
-        path.resolve(
-          fileURLToPath(import.meta.url),
-          '../../../rules/onchainkit.windsurf.txt',
-        ),
-        'utf-8',
-      );
       await fs.promises.writeFile(
         path.join(root, '.windsurfrules'), 
-        windsurfRulesTemplate,
+        onchainkitRulesTemplate,
       );
       await addToGitignore({
         gitignorePath: path.join(root, '.gitignore'),
         additionalPath: '.windsurfrules',
       });
     } else if (aiTool === 'copilot') {
-      await ensureDir(path.join(root, '.github'));
-
-      const copilotRulesTemplate = await fs.promises.readFile(
-        path.resolve(
-          fileURLToPath(import.meta.url),
-          '../../../rules/onchainkit.copilot.txt',
-        ),
-        'utf-8',
-      );
+      await fs.promises.mkdir(path.join(root, '.github'), { recursive: true });
+
       await fs.promises.writeFile(
         path.join(root, '.github/copilot-instructions.md'),
-        copilotRulesTemplate,
+        onchainkitRulesTemplate,
       );
       await addToGitignore({
         gitignorePath: path.join(root, '.gitignore'),

packages/create-onchain/rules/onchainkit.windsurf.txt

@@ -7,7 +7,6 @@ import {
   toValidPackageName,
   detectPackageManager,
   getVersion,
-  ensureDir,
   addToGitignore,
 } from './utils';
 import { Stats } from 'node:fs';
@@ -109,29 +108,6 @@ describe('utils', () => {
     });
   });
 
-  describe('ensureDir', () => {
-    beforeEach(() => {
-      vi.resetAllMocks();
-    });
-
-    it('should create a directory if it does not exist', async () => {
-      const error = new Error('ENOENT');
-      (error as NodeJS.ErrnoException).code = 'ENOENT';
-      vi.mocked(fs.stat).mockRejectedValue(error);
-
-      await ensureDir('test/foobar/directory');
-
-      expect(fs.mkdir).toHaveBeenCalledWith('test/foobar/directory', { recursive: true });
-    });
-
-    it('should not create a directory if it already exists', async () => {
-      vi.mocked(fs.stat).mockResolvedValue({ isDirectory: () => true } as unknown as Stats);
-
-      await ensureDir('test/foobar/directory');
-      expect(fs.mkdir).not.toHaveBeenCalled();
-    });
-  });
-
   describe('addToGitignore', () => {
     it('should add a path to the gitignore file', async () => {
       vi.mocked(fs.readFile).mockResolvedValue('existing content');

packages/create-onchain/src/onchainkit.test.ts

@@ -109,21 +109,6 @@ export function detectPackageManager(): string {
   return 'npm'; // default to npm if unable to detect
 }
 
-export async function ensureDir(dirPath: string) {
-  try {
-    const stats = await fs.stat(dirPath);
-    if (!stats.isDirectory()) {
-      throw new Error(`Path exists but is not a directory: ${dirPath}`);
-    }
-  } catch (error) {
-    if ((error as NodeJS.ErrnoException).code === 'ENOENT') {
-      await fs.mkdir(dirPath, { recursive: true });
-    } else {
-      throw error;
-    }
-  }
-}
-
 export async function addToGitignore({
   gitignorePath,
   additionalPath,