@crossdelta/platform-sdk 0.3.41 → 0.3.42

package/bin/templates/hono-microservice/tsconfig.json.hbs

@@ -4,6 +4,7 @@
     "module": "ES2022",
     "moduleResolution": "Bundler",
     "strict": true,
+    "lib": ["ES2022"],
     "jsx": "react-jsx",
     "jsxImportSource": "hono/jsx",
     "esModuleInterop": true,

package/bin/templates/workspace/.github/copilot-instructions.md.hbs

@@ -339,38 +339,39 @@ describe('Service Health Check', () => {
 
 #### `src/use-cases/business-logic.use-case.test.ts`
 ```typescript
-import { describe, expect, it, beforeEach, vi } from 'bun:test'
+import { describe, expect, it } from 'bun:test'
 import { myUseCase } from './business-logic.use-case'
 
 describe('BusinessLogic Use Case', () => {
-  beforeEach(() => {
-    vi.clearAllMocks()
-  })
-  
   it('should handle valid input correctly', async () => {
     const result = await myUseCase({ id: '123' })
     expect(result).toBeDefined()
   })
   
   it('should throw error for invalid input', async () => {
-    await expect(myUseCase({ id: '' })).rejects.toThrow()
+    await expect(myUseCase({ id: '' })).rejects.toThrow('Invalid input')
   })
 })
 ```
 
 **Test Guidelines:**
+- **Keep tests simple and lightweight** - Bun's test runner is minimal, not a full-featured test framework
 - Write tests ONLY for use cases (NOT event handlers)
 - Event handlers are thin wrappers - the use cases contain the testable logic
-- Use Bun's native test runner: `import { describe, expect, it, beforeEach, afterEach, vi } from 'bun:test'`
-- Use `vi` from `bun:test` for mocking (NOT jest or vitest)
+- Use Bun's native test runner: `import { describe, expect, it, beforeEach, afterEach } from 'bun:test'`
+- **NO mocking available** - Bun's test runner does NOT have `vi`, `mock`, or similar utilities
+  - ❌ NO `vi` object (that's Vitest, not Bun)
+  - ❌ NO `jest.fn()` or `jest.mock()`
+  - ❌ NO module mocking
+  - ✅ Use simple, direct testing without mocks
 - Use descriptive test names in English
 - Test both happy paths and error cases
-- Focus on validation and error handling
-- DO NOT mock external libraries (Pusher, AWS SDK, etc.) - they are hard to mock correctly
+- Focus on validation and error handling (input params, env vars)
+- **DO NOT mock external libraries** - Bun doesn't support mocking, keep tests simple
 - Test environment variable validation and input validation
-- Use `beforeEach`/`afterEach` to clean up mocks: `vi.clearAllMocks()` or `vi.restoreAllMocks()`
+- Keep cleanup simple in `afterEach`: restore `process.env` only
 
-**Example: Testing validation and error handling**
+**Example: Simple validation and error handling tests**
 ```typescript
 import { describe, expect, it, beforeEach, afterEach } from 'bun:test'
 import { sendNotification } from './send-notification.use-case'
@@ -379,10 +380,12 @@ describe('Send Notification Use Case', () => {
   const originalEnv = process.env
 
   beforeEach(() => {
+    // Save original environment
     process.env = { ...originalEnv }
   })
 
   afterEach(() => {
+    // Restore original environment (no vi.resetModules needed)
     process.env = originalEnv
   })
 
@@ -402,6 +405,71 @@ describe('Send Notification Use Case', () => {
 })
 ```
 
+**What to test:**
+- ✅ Input validation (missing/empty parameters)
+- ✅ Environment variable validation
+- ✅ Error messages and error types
+- ✅ Basic logic flows
+- ❌ Complex mocking scenarios
+- ❌ External API integrations (Pusher, AWS, etc.)
+- ❌ Module reloading/resetting
+
+**Test Strategy:**
+- Focus on **validation** (input parameters, environment variables)
+- Test **error handling** and edge cases
+- Keep tests **simple** - avoid complex mocking of external libraries
+- Test the **business logic**, not the external API calls
+- If a function calls an external API, test the validation BEFORE the call, not the call itself
+
+**Example of what NOT to do:**
+```typescript
+// ❌ BAD - Trying to mock Pusher (not possible in Bun)
+import { describe, expect, it, beforeEach, afterEach, vi } from 'bun:test'
+
+describe('Bad Test', () => {
+  afterEach(() => {
+    vi.resetModules()  // ❌ This doesn't exist in Bun!
+  })
+  
+  it('should send notification', async () => {
+    // ❌ Trying to mock external library
+    const mockPusher = vi.fn()  // ❌ vi doesn't exist!
+  })
+})
+```
+
+**Example of what TO do:**
+```typescript
+// ✅ GOOD - Simple validation tests
+import { describe, expect, it, beforeEach, afterEach } from 'bun:test'
+
+describe('Good Test', () => {
+  const originalEnv = process.env
+
+  beforeEach(() => {
+    process.env = { ...originalEnv }
+  })
+
+  afterEach(() => {
+    process.env = originalEnv
+  })
+
+  it('should throw error if orderId is missing', async () => {
+    await expect(
+      sendNotification({ orderId: '', customerId: 'cust-1' })
+    ).rejects.toThrow('Missing orderId')
+  })
+
+  it('should throw error if credentials not set', async () => {
+    delete process.env.PUSHER_BEAMS_INSTANCE_ID
+    
+    await expect(
+      sendNotification({ orderId: '123', customerId: 'cust-1' })
+    ).rejects.toThrow('Missing Pusher Beams credentials')
+  })
+})
+```
+
 ### README
 
 Generate a service-specific README documenting:

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@crossdelta/platform-sdk",
-  "version": "0.3.41",
+  "version": "0.3.42",
   "description": "CLI toolkit for scaffolding Turborepo workspaces with Pulumi infrastructure and Hono/NestJS microservices",
   "keywords": [
     "cli",