@pikku/cli 0.9.11 → 0.9.13

package/CHANGELOG.md

@@ -1,5 +1,17 @@
 # @pikku/cli
 
+## 0.9.13
+
+### Patch Changes
+
+- ccd2a45: fix: adding functions should always be using func config and not pure functions
+
+## 0.9.12
+
+### Patch Changes
+
+- eb8ed09: feat: only write files if the content changed / file doesn't exist, this stops triggering restarts for development
+
 ## 0.9.11
 
 ### Patch Changes

package/dist/src/serialize-pikku-types.js

@@ -204,7 +204,7 @@ export const pikkuFunc = <In, Out = unknown>(
     | PikkuFunction<In, Out>
     | CorePikkuFunctionConfig<PikkuFunction<In, Out>, PikkuPermission<In>>
 ) => {
-  return typeof func === 'function' ? func : func.func
+  return typeof func === 'function' ? { func } : func
 }
 
 /**
@@ -232,7 +232,7 @@ export const pikkuSessionlessFunc = <In, Out = unknown>(
     | PikkuFunctionSessionless<In, Out>
     | CorePikkuFunctionConfig<PikkuFunctionSessionless<In, Out>, PikkuPermission<In>, PikkuMiddleware>
 ) => {
-  return typeof func === 'function' ? func : func.func
+  return typeof func === 'function' ? { func } : func
 }
 
 /**
@@ -263,7 +263,7 @@ export const pikkuChannelConnectionFunc = <Out = unknown, ChannelData = unknown>
         name?: string
       }
 ) => {
-  return typeof func === 'function' ? func : func.func
+  return typeof func === 'function' ? { func } : func
 }
 
 /**
@@ -292,7 +292,7 @@ export const pikkuChannelDisconnectionFunc = <ChannelData = unknown>(
         name?: string
       }
 ) => {
-  return typeof func === 'function' ? func : func.func
+  return typeof func === 'function' ? { func } : func
 }
 
 /**
@@ -319,7 +319,7 @@ export const pikkuChannelFunc = <In = unknown, Out = unknown, ChannelData = unkn
     | PikkuFunctionSessionless<In, Out, ChannelData>
     | CorePikkuFunctionConfig<PikkuFunction<In, Out, ChannelData>, PikkuPermission<In>>
 ) => {
-  return typeof func === 'function' ? func : func.func
+  return typeof func === 'function' ? { func } : func
 }
 
 /**
@@ -343,7 +343,7 @@ export const pikkuVoidFunc = (
     | PikkuFunctionSessionless<void, void>
     | CorePikkuFunctionConfig<PikkuFunctionSessionless<void, void>, PikkuPermission<void>>
 ) => {
-  return typeof func === 'function' ? func : func.func
+  return typeof func === 'function' ? { func } : func
 }
    
 /**
@@ -534,7 +534,7 @@ export const pikkuMCPPromptFunc = <In>(
         name?: string
       }
 ) => {
-  return typeof func === 'function' ? func : func.func
+  return typeof func === 'function' ? { func } : func
 }
 
 /**
@@ -566,7 +566,7 @@ export const pikkuMCPToolFunc = <In>(
       name?: string
     }
 ) => {
-  return typeof func === 'function' ? func : func.func
+  return typeof func === 'function' ? { func } : func
 }
 
 /**
@@ -599,7 +599,7 @@ export const pikkuMCPResourceFunc = <In>(
       name?: string
     }
 ) => {
-  return typeof func === 'function' ? func : func.func
+  return typeof func === 'function' ? { func } : func
 }
 `;
 };

package/dist/src/utils.js

@@ -1,5 +1,5 @@
 import { relative, dirname, resolve } from 'path';
-import { mkdir, writeFile } from 'fs/promises';
+import { mkdir, readFile, writeFile } from 'fs/promises';
 import chalk from 'chalk';
 import { fileURLToPath } from 'url';
 import { readFileSync } from 'fs';
@@ -165,10 +165,21 @@ export const writeFileInDir = async (logger, path, content, { ignoreModifyCommen
     else {
         content = `${ignoreModifyComment ? '' : DO_NOT_MODIFY_COMMENT}${content}`;
     }
-    await mkdir(dirname(path), { recursive: true });
-    await writeFile(path, content, 'utf-8');
-    if (logWrite) {
-        logger.success(`✓ File written to ${path}`);
+    // Try to read existing file content
+    let existingContent;
+    try {
+        existingContent = await readFile(path, 'utf-8');
+    }
+    catch (error) {
+        // File doesn't exist, so we need to write it
+    }
+    // Only write if content has changed or file doesn't exist
+    if (existingContent !== content) {
+        await mkdir(dirname(path), { recursive: true });
+        await writeFile(path, content, 'utf-8');
+        if (logWrite) {
+            logger.success(`✓ File written to ${path}`);
+        }
     }
 };
 export const logCommandInfoAndTime = async (logger, commandStart, commandEnd, [skipCondition, skipMessage = 'none found'], callback) => {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@pikku/cli",
-  "version": "0.9.11",
+  "version": "0.9.13",
   "author": "yasser.fadl@gmail.com",
   "license": "MIT",
   "bin": {

package/src/serialize-pikku-types.ts

@@ -212,7 +212,7 @@ export const pikkuFunc = <In, Out = unknown>(
     | PikkuFunction<In, Out>
     | CorePikkuFunctionConfig<PikkuFunction<In, Out>, PikkuPermission<In>>
 ) => {
-  return typeof func === 'function' ? func : func.func
+  return typeof func === 'function' ? { func } : func
 }
 
 /**
@@ -240,7 +240,7 @@ export const pikkuSessionlessFunc = <In, Out = unknown>(
     | PikkuFunctionSessionless<In, Out>
     | CorePikkuFunctionConfig<PikkuFunctionSessionless<In, Out>, PikkuPermission<In>, PikkuMiddleware>
 ) => {
-  return typeof func === 'function' ? func : func.func
+  return typeof func === 'function' ? { func } : func
 }
 
 /**
@@ -271,7 +271,7 @@ export const pikkuChannelConnectionFunc = <Out = unknown, ChannelData = unknown>
         name?: string
       }
 ) => {
-  return typeof func === 'function' ? func : func.func
+  return typeof func === 'function' ? { func } : func
 }
 
 /**
@@ -300,7 +300,7 @@ export const pikkuChannelDisconnectionFunc = <ChannelData = unknown>(
         name?: string
       }
 ) => {
-  return typeof func === 'function' ? func : func.func
+  return typeof func === 'function' ? { func } : func
 }
 
 /**
@@ -327,7 +327,7 @@ export const pikkuChannelFunc = <In = unknown, Out = unknown, ChannelData = unkn
     | PikkuFunctionSessionless<In, Out, ChannelData>
     | CorePikkuFunctionConfig<PikkuFunction<In, Out, ChannelData>, PikkuPermission<In>>
 ) => {
-  return typeof func === 'function' ? func : func.func
+  return typeof func === 'function' ? { func } : func
 }
 
 /**
@@ -351,7 +351,7 @@ export const pikkuVoidFunc = (
     | PikkuFunctionSessionless<void, void>
     | CorePikkuFunctionConfig<PikkuFunctionSessionless<void, void>, PikkuPermission<void>>
 ) => {
-  return typeof func === 'function' ? func : func.func
+  return typeof func === 'function' ? { func } : func
 }
    
 /**
@@ -542,7 +542,7 @@ export const pikkuMCPPromptFunc = <In>(
         name?: string
       }
 ) => {
-  return typeof func === 'function' ? func : func.func
+  return typeof func === 'function' ? { func } : func
 }
 
 /**
@@ -574,7 +574,7 @@ export const pikkuMCPToolFunc = <In>(
       name?: string
     }
 ) => {
-  return typeof func === 'function' ? func : func.func
+  return typeof func === 'function' ? { func } : func
 }
 
 /**
@@ -607,7 +607,7 @@ export const pikkuMCPResourceFunc = <In>(
       name?: string
     }
 ) => {
-  return typeof func === 'function' ? func : func.func
+  return typeof func === 'function' ? { func } : func
 }
 `
 }

package/src/utils.ts

@@ -1,6 +1,6 @@
 import { relative, dirname, resolve } from 'path'
 import { PathToNameAndType, InspectorState, TypesMap } from '@pikku/inspector'
-import { mkdir, writeFile } from 'fs/promises'
+import { mkdir, readFile, writeFile } from 'fs/promises'
 import chalk from 'chalk'
 import { fileURLToPath } from 'url'
 import { readFileSync } from 'fs'
@@ -301,11 +301,22 @@ export const writeFileInDir = async (
     content = `${ignoreModifyComment ? '' : DO_NOT_MODIFY_COMMENT}${content}`
   }
 
-  await mkdir(dirname(path), { recursive: true })
-  await writeFile(path, content, 'utf-8')
+  // Try to read existing file content
+  let existingContent: string | undefined
+  try {
+    existingContent = await readFile(path, 'utf-8')
+  } catch (error) {
+    // File doesn't exist, so we need to write it
+  }
+
+  // Only write if content has changed or file doesn't exist
+  if (existingContent !== content) {
+    await mkdir(dirname(path), { recursive: true })
+    await writeFile(path, content, 'utf-8')
 
-  if (logWrite) {
-    logger.success(`✓ File written to ${path}`)
+    if (logWrite) {
+      logger.success(`✓ File written to ${path}`)
+    }
   }
 }