chore: wip

Author: chrisbbreuer

bun.lock

@@ -3,9 +3,6 @@
   "workspaces": {
     "": {
       "name": "bun-router",
-      "dependencies": {
-        "reflect-metadata": "^0.2.2",
-      },
       "devDependencies": {
         "@stacksjs/bumpx": "^0.1.78",
         "@stacksjs/docs": "^0.70.23",
@@ -1765,8 +1762,6 @@
 
     "refa": ["[email protected]", "", { "dependencies": { "@eslint-community/regexpp": "^4.8.0" } }, "sha512-J8rn6v4DBb2nnFqkqwy6/NnTYMcgLA+sLr0iIO41qpv0n+ngb7ksag2tMRl0inb1bbO/esUwzW1vbJi7K0sI0g=="],
 
-    "reflect-metadata": ["[email protected]", "", {}, "sha512-urBwgfrvVP/eAyXx4hluJivBKzuEbSQs9rKWCrCkbSxNv8mxPcUZKeuoF3Uy4mJl3Lwprp6yy5/39VWigZ4K6Q=="],
-
     "reflect.getprototypeof": ["[email protected]", "", { "dependencies": { "call-bind": "^1.0.8", "define-properties": "^1.2.1", "es-abstract": "^1.23.9", "es-errors": "^1.3.0", "es-object-atoms": "^1.0.0", "get-intrinsic": "^1.2.7", "get-proto": "^1.0.1", "which-builtin-type": "^1.2.1" } }, "sha512-00o4I+DVrefhv+nX0ulyi3biSHCPDe+yLv5o/p6d/UVlirijB8E16FtfwSAi4g3tcqrQ4lRAqQSoFEZJehYEcw=="],
 
     "regenerate": ["[email protected]", "", {}, "sha512-zrceR/XhGYU/d/opr2EKO7aRHUeiBI8qjtfHqADTwZd6Szfy16la6kqD0MIUs5z5hx6AaKa+PixpPrR289+I0A=="],

examples/dependency-injection-example.ts

@@ -5,7 +5,6 @@
  * Decorators, Contextual Binding, and real-world usage patterns
  */
 
-// Modern decorators - no reflect-metadata needed
 import {
   Container,
   createContainer

package.json

@@ -92,8 +92,5 @@
   },
   "workspaces": [
     "packages/*"
-  ],
-  "dependencies": {
-    "reflect-metadata": "^0.2.2"
-  }
+  ]
 }

packages/bun-router/src/container/decorators.ts

@@ -1,8 +1,6 @@
 /* eslint-disable ts/no-unsafe-function-type */
 /**
  * Modern Dependency Injection Decorators
- *
- * Uses standard decorators without reflect-metadata dependency
  */
 
 import type { ResolutionContext } from './container'
@@ -165,20 +163,20 @@ function createRouteDecorator(method: string, path: string, options?: { middlewa
     if (!target.constructor[ROUTE_METADATA_KEY]) {
       target.constructor[ROUTE_METADATA_KEY] = []
     }
-    
+
     const metadata: RouteMetadata = {
       method,
       path,
       middleware: options?.middleware || [],
       tags: options?.tags || [],
     }
-    
+
     target.constructor[ROUTE_METADATA_KEY].push({
       ...metadata,
       propertyKey,
       handler: descriptor.value,
     })
-    
+
     return descriptor
   }
 }
@@ -238,17 +236,17 @@ function createParamDecorator(
     if (!target.constructor[PARAM_METADATA_KEY]) {
       target.constructor[PARAM_METADATA_KEY] = new Map()
     }
-    
+
     if (!target.constructor[PARAM_METADATA_KEY].has(propertyKey)) {
       target.constructor[PARAM_METADATA_KEY].set(propertyKey, [])
     }
-    
+
     const metadata: ParamMetadata = {
       type,
       key,
       token,
     }
-    
+
     target.constructor[PARAM_METADATA_KEY].get(propertyKey)[parameterIndex] = metadata
   }
 }
@@ -295,7 +293,7 @@ export class MetadataReader {
 }
 
 /**
- * Modern Decorator Container that works without reflect-metadata
+ * Modern Decorator Container
  */
 export class DecoratorContainer extends Container {
   /**

test/dependency-injection-simple.test.ts

@@ -9,7 +9,6 @@ import { createContainer } from '../packages/bun-router/src/container/container'
 import { ContextualContainer } from '../packages/bun-router/src/container/contextual-binding'
 import { Controller, DecoratorContainer, Get, Inject, Injectable } from '../packages/bun-router/src/container/decorators'
 import { BaseServiceProvider, DefaultServiceProviderManager } from '../packages/bun-router/src/container/service-provider'
-// Modern decorators - no reflect-metadata needed
 
 describe('Dependency Injection System', () => {
   describe('Basic Container', () => {

test/dependency-injection.test.ts

@@ -19,7 +19,6 @@ import {
   Injectable,
   MetadataReader,
 } from '../packages/bun-router/src/container/decorators'
-
 import {
   BaseServiceProvider,
   ConfigServiceProvider,
@@ -28,8 +27,6 @@ import {
   LoggingServiceProvider,
 } from '../packages/bun-router/src/container/service-provider'
 
-// Modern decorators - no reflect-metadata needed
-
 // Test services and classes
 @Injectable({ scope: 'singleton' })
 class DatabaseService {
@@ -239,7 +236,7 @@ describe('IoC Container', () => {
           return 'test'
         }
       }
-      
+
       container.singleton(TestService, TestService)
       const testService = container.resolve<TestService>(TestService)
       expect(testService).toBeInstanceOf(TestService)
@@ -448,11 +445,11 @@ describe('Decorators', () => {
     it('should resolve controller with dependencies', () => {
       // Clear any existing bindings to avoid conflicts
       container.clear()
-      
+
       // Bind dependencies that UserController needs
       container.factory('userService', () => ({ getUser: () => Promise.resolve({ id: 1, name: 'Test' }) }))
       container.factory('logger', () => new LoggerService())
-      
+
       // Register the controller
       container.registerController(UserController)