tkeron 4.0.0-beta.17 → 4.0.0-beta.18

package/changelog.md

@@ -1,3 +1,28 @@
+# v4.0.0-beta.18
+
+## Code Quality & Architecture
+
+### Bug Fixes
+
+- **Fixed**: Removed `process.exit()` from library code (`build.ts`, `develop.ts`, `init.ts`) — errors now propagate via `throw`, letting wrappers handle exit
+- **Fixed**: Moved `reloadClients` Set inside `develop()` to prevent shared state across multiple server instances
+- **Fixed**: Removed `await` on synchronous `path.resolve()` calls in `build.ts` and `develop.ts`
+- **Fixed**: `log.error()` called without arguments in `processPre.ts` — now passes empty string
+
+### Code Style
+
+- **Changed**: Converted all `function` declarations to arrow functions (`initWrapper`, `processComponents`, `processComponentsTs`)
+- **Removed**: Comments in source code (`processCom.ts`, `processComTs.ts`)
+- **Removed**: Dead re-export of `setupSigintHandler` from `develop.ts`
+
+### Tests
+
+- **Fixed**: TS2532 errors in `getBuildResult.test.ts` (proper handling of possibly undefined values)
+- **Updated**: Tests adapted to new error propagation (no more `process.exit` mocking in library tests)
+- **Removed**: Redundant/dead test cases for removed behavior
+
+---
+
 # v4.0.0-beta.17
 
 ## MCP Server Updates

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "tkeron",
-  "version": "4.0.0-beta.17",
+  "version": "4.0.0-beta.18",
   "description": "CLI tool for backend-driven frontend development with TypeScript.",
   "type": "module",
   "main": "./src/index.ts",

package/src/build.ts

@@ -24,10 +24,10 @@ export const build = async (options: BuildOptions) => {
     return;
   }
 
-  const source = await resolve(options.sourceDir || "websrc");
+  const source = resolve(options.sourceDir || "websrc");
   const target = options.targetDir
-    ? await resolve(options.targetDir)
-    : await resolve(dirname(source), "web");
+    ? resolve(options.targetDir)
+    : resolve(dirname(source), "web");
 
   const sourceParent = dirname(source);
   const tempDir = join(
@@ -67,14 +67,6 @@ export const build = async (options: BuildOptions) => {
         `\n💡 Tip: Create the directory first, check the path, or run 'tk init' to create a new project.`,
       );
       log.error(`   Expected: ${source}\n`);
-      process.exit(1);
-    }
-
-    if (
-      error.message &&
-      error.message.includes("must contain at least one hyphen")
-    ) {
-      process.exit(1);
     }
 
     throw error;

package/src/develop.ts

@@ -13,8 +13,6 @@ export interface TkeronDevOptions {
   logger?: Logger;
 }
 
-const reloadClients = new Set<ReadableStreamDefaultController>();
-
 export interface DevelopServer {
   stop: () => Promise<void>;
   port: number;
@@ -28,6 +26,8 @@ export const develop = async (
     throw new Error("Invalid options provided for develop");
   }
 
+  const reloadClients = new Set<ReadableStreamDefaultController>();
+
   const {
     port = 3000,
     host = "localhost",
@@ -36,28 +36,13 @@ export const develop = async (
     logger: log = defaultLogger,
   } = options;
 
-  const source = await resolve(sourceDir || "websrc");
+  const source = resolve(sourceDir || "websrc");
   const target = outputDir
-    ? await resolve(outputDir)
-    : await resolve(dirname(source), "web");
+    ? resolve(outputDir)
+    : resolve(dirname(source), "web");
 
   log.log("🔨 Building project...");
-  try {
-    await build({ sourceDir: source, targetDir: target, logger: log });
-  } catch (error: any) {
-    if (error.code === "ENOENT") {
-      const actualSourceDir = sourceDir || "websrc";
-      log.error(
-        `\n❌ Error: Source directory "${actualSourceDir}" does not exist.`,
-      );
-      log.error(
-        `\n💡 Tip: Create the directory first, check the path, or run 'tk init' to create a new project.`,
-      );
-      log.error(`   Expected: ${source}\n`);
-      process.exit(1);
-    }
-    throw error;
-  }
+  await build({ sourceDir: source, targetDir: target, logger: log });
   log.log("✅ Build complete!");
 
   const server = Bun.serve({
@@ -182,5 +167,3 @@ export const develop = async (
     host: server.hostname ?? host,
   };
 };
-
-export { setupSigintHandler } from "./setupSigintHandler";

package/src/init.ts

@@ -57,7 +57,7 @@ export const init = async (options: InitOptions) => {
 
           if (!shouldContinue) {
             log.log("\n❌ Operation cancelled.");
-            process.exit(0);
+            return;
           }
         }
 

package/src/initWrapper.ts

@@ -9,7 +9,7 @@ export interface InitWrapperOptions {
   [key: string]: any;
 }
 
-export async function initWrapper(options: InitWrapperOptions) {
+export const initWrapper = async (options: InitWrapperOptions) => {
   const log = options?.logger || defaultLogger;
 
   if (!options || typeof options !== "object") {
@@ -60,4 +60,4 @@ export async function initWrapper(options: InitWrapperOptions) {
     }
     process.exit(1);
   }
-}
+};

package/src/processCom.ts

@@ -60,14 +60,14 @@ export const processCom = async (
   return hasChanges;
 };
 
-async function processComponents(
+const processComponents = async (
   element: any,
   currentDir: string,
   rootDir: string,
   componentStack: string[],
   depth: number = 0,
   log: Logger = silentLogger,
-): Promise<boolean> {
+): Promise<boolean> => {
   let hasChanges = false;
   const MAX_DEPTH = 50;
   if (depth > MAX_DEPTH) {
@@ -183,7 +183,6 @@ async function processComponents(
       }
     }
 
-    // Recursively process child elements
     const childChanged = await processComponents(
       child,
       currentDir,
@@ -196,4 +195,4 @@ async function processComponents(
   }
 
   return hasChanges;
-}
+};

package/src/processComTs.ts

@@ -61,7 +61,7 @@ export const processComTs = async (
   return hasChanges;
 };
 
-async function processComponentsTs(
+const processComponentsTs = async (
   element: any,
   currentDir: string,
   rootDir: string,
@@ -69,7 +69,7 @@ async function processComponentsTs(
   depth: number = 0,
   log: Logger = silentLogger,
   options: ProcessComTsOptions = {},
-): Promise<boolean> {
+): Promise<boolean> => {
   let hasChanges = false;
   const MAX_DEPTH = 50;
   if (depth > MAX_DEPTH) {
@@ -251,7 +251,6 @@ ${codeWithoutImports}
       }
     }
 
-    // Recursively process child elements
     const childChanged = await processComponentsTs(
       child,
       currentDir,
@@ -265,4 +264,4 @@ ${codeWithoutImports}
   }
 
   return hasChanges;
-}
+};

package/src/processPre.ts

@@ -89,7 +89,7 @@ export const processPre = async (
       if (stdout) {
         log.error(`\nOutput:\n${stdout}`);
       }
-      log.error();
+      log.error("");
       throw new Error(`Pre-rendering failed for ${preFile}`);
     }
   }

package/tests/test-helpers.ts

@@ -11,7 +11,7 @@ const hashString = (str: string): number => {
   for (let i = 0; i < str.length; i++) {
     const char = str.charCodeAt(i);
     hash = (hash << 5) - hash + char;
-    hash = hash & hash; // Convert to 32bit integer
+    hash = hash & hash;
   }
   return Math.abs(hash);
 };
@@ -22,7 +22,7 @@ export const getTestResources = (
   testName: string,
 ): { port: number; dir: string } => {
   const hash = hashString(testName);
-  const port = 9000 + (hash % 10000); // Port range: 9000-18999
+  const port = 9000 + (hash % 10000);
   const uniqueId = `${hash}-${resourceCounter++}`;
   const dir = join(tmpdir(), `tkeron-test-${uniqueId}`);