bro-framework 2.2.0 → 2.2.1

package/README.md

@@ -109,7 +109,7 @@ Stop importing singleton database connections and socket instances into every fi
 If you've ever hand-written OpenAPI YAML, you know the pain. `bro.js` parses your Zod schemas and automatically serves a stunning, interactive [Scalar](https://scalar.com/) API playground at `/docs`. It's highly secure: by default, these internal docs are disabled in production mode.
 
 ### The Frontend SDK Generator
-Tired of writing frontend `fetch` wrappers? Run `bro sdk`. The CLI will parse your backend routes and compile a `bro-client.js` file for your frontend. It features built-in token management, request stringification, and type-safe deep tree traversal (e.g., `api.users.id("123").post(data)`).
+Tired of writing frontend `fetch` wrappers? Run `bro sdk`. The CLI will parse your backend routes and compile a `bro-sdk.js` file for your frontend. It features built-in token management, request stringification, and type-safe deep tree traversal (e.g., `api.users.id("123").post(data)`).
 
 ### Background Task Scheduler
 Don't spin up a separate worker server. Drop a JavaScript file anywhere in the `tasks/` folder, export a cron string (e.g., `"0 0 * * *"`), and an async handler. `bro.js` natively schedules it as a background worker with full access to your injected database and WebSocket contexts.

package/bin/bro.js

@@ -9,7 +9,7 @@ register();
 
 import { createServer } from '../src/server.js';
 import { colors, printBanner, printRoute, printHotReload } from '../src/logger.js';
-import { scanTasks, stopTasks } from '../src/tasks.js';
+import { scanTasks } from '../src/tasks.js';
 import { generateSDK } from '../src/sdk.js';
 import dotenv from 'dotenv';
 import chokidar from 'chokidar';
@@ -204,7 +204,7 @@ async function bootstrap() {
       console.log(`[bro.js] Server running in production on port ${port}`);
     }
     
-    await scanTasks({ db, io });
+    let taskManager = await scanTasks({ db, io });
     
     if (command === 'dev') {
       const printCurrentRoutes = (routesToPrint) => {
@@ -240,7 +240,7 @@ async function bootstrap() {
 
     const handleShutdown = async (signal) => {
       console.log(`\n[bro.js] Received ${signal}. Shutting down gracefully...`);
-      stopTasks();
+      if (taskManager) taskManager.stopAll();
       if (io) io.close();
       server.close(() => {
         console.log('[bro.js] HTTP server closed.');

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "bro-framework",
-  "version": "2.2.0",
+  "version": "2.2.1",
   "description": "The No-BS Backend Framework for Node.js",
   "repository": {
     "type": "git",

package/src/logger.js

@@ -1,3 +1,7 @@
+import fs from 'fs';
+import path from 'path';
+import { fileURLToPath } from 'url';
+
 export const colors = {
   reset: "\x1b[0m",
   bold: "\x1b[1m",
@@ -25,7 +29,16 @@ export function formatMethod(method) {
 
 export function printBanner(port, durationMs) {
   const time = durationMs.toFixed(0);
-  const version = "2.0.1";
+  
+  let version = "2.2.0";
+  try {
+    const __filename = fileURLToPath(import.meta.url);
+    const __dirname = path.dirname(__filename);
+    const pkgPath = path.join(__dirname, '..', 'package.json');
+    const pkg = JSON.parse(fs.readFileSync(pkgPath, 'utf-8'));
+    version = pkg.version;
+  } catch(e) {}
+  
   const innerWidth = 47;
 
   const stripAnsi = (str) => str.replace(/\x1B\[[0-9;]*[a-zA-Z]/g, "");

package/src/router.js

@@ -43,6 +43,12 @@ export function parseRouteFile(filePath, routesDir) {
   if (parts.length < 2) return null; 
   
   const method = parts.pop().toLowerCase();
+  
+  const allowedMethods = new Set(['get', 'post', 'put', 'delete', 'patch', 'options', 'head']);
+  if (!allowedMethods.has(method)) {
+    throw new Error(`Invalid HTTP method "${method}" in file: ${filePath}`);
+  }
+  
   const namePart = parts.join('.');
   
   let routePath = '/' + path.dirname(relativePath).replace(/\\/g, '/');
@@ -52,6 +58,15 @@ export function parseRouteFile(filePath, routesDir) {
     routePath += `/${namePart}`;
   }
   
+  const bracketRegex = /\[(.*?)\]/g;
+  let match;
+  while ((match = bracketRegex.exec(routePath)) !== null) {
+    const paramName = match[1];
+    if (!/^[a-zA-Z_$][a-zA-Z0-9_$]*$/.test(paramName)) {
+      throw new Error(`Invalid dynamic parameter "[${paramName}]" in file: ${filePath}. Must be a valid JavaScript identifier.`);
+    }
+  }
+  
   routePath = routePath.replace(/\[(.*?)\]/g, ':$1');
   
   if (routePath === '') routePath = '/';

package/src/sdk.js

@@ -98,6 +98,10 @@ function generateApiObject(endpoints) {
       
       if (!current[name]) {
         current[name] = { _isParam: isParam, _methods: {}, _children: {}, _path: pathAcc };
+      } else {
+        if (current[name]._isParam !== isParam) {
+          throw new Error(`SDK Collision: Route segment "${name}" conflicts between static and dynamic parameters at path "${pathAcc}"`);
+        }
       }
       
       if (i === parts.length - 1) {

package/src/tasks.js

@@ -5,21 +5,24 @@ import cron from 'node-cron';
 import { colors } from './logger.js';
 import { scanDir } from './router.js';
 
-let taskHandles = [];
-
-export function stopTasks() {
-  taskHandles.forEach(t => t.stop());
-  taskHandles = [];
+export class TaskManager {
+  constructor() {
+    this.taskHandles = [];
+  }
+  
+  stopAll() {
+    this.taskHandles.forEach(t => t.stop());
+    this.taskHandles = [];
+  }
 }
 
 export async function scanTasks(ctx) {
+  const manager = new TaskManager();
   const tasksDir = path.join(process.cwd(), 'tasks');
-  if (!fs.existsSync(tasksDir)) return;
+  if (!fs.existsSync(tasksDir)) return manager;
   
-  stopTasks();
-
   const files = scanDir(tasksDir);
-  if (files.length === 0) return;
+  if (files.length === 0) return manager;
   
   let count = 0;
   for (const file of files) {
@@ -35,7 +38,7 @@ export async function scanTasks(ctx) {
             console.error(`\n  ${colors.red}❌ Task Error (${file}):${colors.reset}`, err);
           }
         });
-        taskHandles.push(task);
+        manager.taskHandles.push(task);
         count++;
       }
     } catch (err) {
@@ -46,4 +49,6 @@ export async function scanTasks(ctx) {
   if (count > 0) {
     console.log(`  ${colors.dim}├──${colors.reset} ${colors.cyan}Scheduled ${count} background task(s)${colors.reset}`);
   }
+  
+  return manager;
 }