@shadowob/sdk 0.3.4 → 0.4.1

package/package.json

@@ -1,18 +1,38 @@
 {
   "name": "@shadowob/sdk",
-  "version": "0.3.4",
+  "version": "0.4.1",
   "description": "Shadow SDK — typed REST client and real-time Socket.IO event listener for Shadow servers",
   "type": "module",
-  "main": "./src/index.ts",
-  "types": "./src/index.ts",
+  "main": "./dist/index.js",
+  "module": "./dist/index.js",
+  "require": "./dist/index.cjs",
+  "types": "./dist/index.d.ts",
   "exports": {
-    ".": "./src/index.ts"
+    ".": {
+      "types": "./dist/index.d.ts",
+      "development": "./src/index.ts",
+      "import": "./dist/index.js",
+      "require": "./dist/index.cjs",
+      "default": "./dist/index.js"
+    }
   },
+  "files": [
+    "dist"
+  ],
   "dependencies": {
     "socket.io-client": "^4.8.1",
-    "@shadowob/shared": "0.3.4"
+    "@shadowob/shared": "0.4.1"
+  },
+  "devDependencies": {
+    "tsup": "^8.5.0",
+    "typescript": "^5.9.3"
+  },
+  "publishConfig": {
+    "access": "public"
   },
   "scripts": {
+    "build": "tsup",
+    "dev": "tsup --watch",
     "test": "vitest run",
     "test:watch": "vitest"
   }

package/__tests__/client.test.ts

@@ -1,256 +0,0 @@
-import { afterEach, beforeEach, describe, expect, it, vi } from 'vitest'
-import { ShadowClient } from '../src/client'
-
-describe('ShadowClient', () => {
-  let client: ShadowClient
-
-  beforeEach(() => {
-    client = new ShadowClient('https://api.example.com', 'test-token-123')
-  })
-
-  describe('constructor', () => {
-    it('should strip trailing /api from baseUrl', () => {
-      const c = new ShadowClient('https://api.example.com/api', 'token')
-      // Access internal state via a request that would expose the URL
-      expect(c).toBeDefined()
-    })
-
-    it('should strip trailing /api/ from baseUrl', () => {
-      const c = new ShadowClient('https://api.example.com/api/', 'token')
-      expect(c).toBeDefined()
-    })
-
-    it('should leave baseUrl without /api suffix unchanged', () => {
-      const c = new ShadowClient('https://api.example.com', 'token')
-      expect(c).toBeDefined()
-    })
-  })
-
-  describe('request error handling', () => {
-    beforeEach(() => {
-      vi.stubGlobal('fetch', vi.fn())
-    })
-
-    afterEach(() => {
-      vi.unstubAllGlobals()
-    })
-
-    it('should throw on non-ok response with status and body', async () => {
-      const mockFetch = vi.fn().mockResolvedValue({
-        ok: false,
-        status: 401,
-        text: () => Promise.resolve('Unauthorized'),
-      })
-      vi.stubGlobal('fetch', mockFetch)
-
-      await expect(client.getMe()).rejects.toThrow(
-        'Shadow API GET /api/auth/me failed (401): Unauthorized',
-      )
-    })
-
-    it('should include authorization header in requests', async () => {
-      const mockFetch = vi.fn().mockResolvedValue({
-        ok: true,
-        json: () => Promise.resolve({ id: '1', username: 'test' }),
-      })
-      vi.stubGlobal('fetch', mockFetch)
-
-      await client.getMe()
-
-      expect(mockFetch).toHaveBeenCalledWith(
-        expect.any(String),
-        expect.objectContaining({
-          headers: expect.objectContaining({
-            Authorization: 'Bearer test-token-123',
-          }),
-        }),
-      )
-    })
-
-    it('should set Content-Type to application/json', async () => {
-      const mockFetch = vi.fn().mockResolvedValue({
-        ok: true,
-        json: () => Promise.resolve({ token: 'new', user: {} }),
-      })
-      vi.stubGlobal('fetch', mockFetch)
-
-      await client.login({ email: 'a@b.com', password: '12345678' })
-
-      expect(mockFetch).toHaveBeenCalledWith(
-        expect.any(String),
-        expect.objectContaining({
-          headers: expect.objectContaining({
-            'Content-Type': 'application/json',
-          }),
-        }),
-      )
-    })
-
-    it('should handle fetch text() error gracefully', async () => {
-      const mockFetch = vi.fn().mockResolvedValue({
-        ok: false,
-        status: 500,
-        text: () => Promise.reject(new Error('parse error')),
-      })
-      vi.stubGlobal('fetch', mockFetch)
-
-      await expect(client.getMe()).rejects.toThrow('failed (500)')
-    })
-  })
-
-  describe('auth methods', () => {
-    beforeEach(() => {
-      vi.stubGlobal('fetch', vi.fn())
-    })
-
-    afterEach(() => {
-      vi.unstubAllGlobals()
-    })
-
-    it('should call register with correct path and body', async () => {
-      const mockFetch = vi.fn().mockResolvedValue({
-        ok: true,
-        json: () => Promise.resolve({ token: 'jwt', user: { id: '1' } }),
-      })
-      vi.stubGlobal('fetch', mockFetch)
-
-      const data = {
-        email: 'test@example.com',
-        password: 'password123',
-        username: 'testuser',
-        inviteCode: 'ABCD1234',
-      }
-      await client.register(data)
-
-      expect(mockFetch).toHaveBeenCalledWith(
-        'https://api.example.com/api/auth/register',
-        expect.objectContaining({
-          method: 'POST',
-          body: JSON.stringify(data),
-        }),
-      )
-    })
-
-    it('should call login with correct path and body', async () => {
-      const mockFetch = vi.fn().mockResolvedValue({
-        ok: true,
-        json: () => Promise.resolve({ token: 'jwt', user: { id: '1' } }),
-      })
-      vi.stubGlobal('fetch', mockFetch)
-
-      await client.login({ email: 'test@example.com', password: 'password123' })
-
-      expect(mockFetch).toHaveBeenCalledWith(
-        'https://api.example.com/api/auth/login',
-        expect.objectContaining({
-          method: 'POST',
-        }),
-      )
-    })
-
-    it('should call getMe with correct path', async () => {
-      const mockFetch = vi.fn().mockResolvedValue({
-        ok: true,
-        json: () => Promise.resolve({ id: '1', username: 'test' }),
-      })
-      vi.stubGlobal('fetch', mockFetch)
-
-      await client.getMe()
-
-      expect(mockFetch).toHaveBeenCalledWith(
-        'https://api.example.com/api/auth/me',
-        expect.objectContaining({
-          headers: expect.objectContaining({
-            Authorization: 'Bearer test-token-123',
-          }),
-        }),
-      )
-    })
-  })
-
-  describe('server methods', () => {
-    beforeEach(() => {
-      vi.stubGlobal('fetch', vi.fn())
-    })
-
-    afterEach(() => {
-      vi.unstubAllGlobals()
-    })
-
-    it('should call listServers with GET', async () => {
-      const mockFetch = vi.fn().mockResolvedValue({
-        ok: true,
-        json: () => Promise.resolve([]),
-      })
-      vi.stubGlobal('fetch', mockFetch)
-
-      await client.listServers()
-
-      expect(mockFetch).toHaveBeenCalledWith(
-        'https://api.example.com/api/servers',
-        expect.any(Object),
-      )
-    })
-
-    it('should call createServer with POST and body', async () => {
-      const mockFetch = vi.fn().mockResolvedValue({
-        ok: true,
-        json: () => Promise.resolve({ id: 's1', name: 'Test' }),
-      })
-      vi.stubGlobal('fetch', mockFetch)
-
-      await client.createServer({ name: 'Test' })
-
-      expect(mockFetch).toHaveBeenCalledWith(
-        'https://api.example.com/api/servers',
-        expect.objectContaining({
-          method: 'POST',
-          body: JSON.stringify({ name: 'Test' }),
-        }),
-      )
-    })
-  })
-
-  describe('message methods', () => {
-    beforeEach(() => {
-      vi.stubGlobal('fetch', vi.fn())
-    })
-
-    afterEach(() => {
-      vi.unstubAllGlobals()
-    })
-
-    it('should call sendMessage with correct path', async () => {
-      const mockFetch = vi.fn().mockResolvedValue({
-        ok: true,
-        json: () => Promise.resolve({ id: 'm1' }),
-      })
-      vi.stubGlobal('fetch', mockFetch)
-
-      await client.sendMessage('ch1', 'Hello')
-
-      expect(mockFetch).toHaveBeenCalledWith(
-        'https://api.example.com/api/channels/ch1/messages',
-        expect.objectContaining({
-          method: 'POST',
-          body: JSON.stringify({ content: 'Hello' }),
-        }),
-      )
-    })
-
-    it('should call getMessages with channel ID', async () => {
-      const mockFetch = vi.fn().mockResolvedValue({
-        ok: true,
-        json: () => Promise.resolve([]),
-      })
-      vi.stubGlobal('fetch', mockFetch)
-
-      await client.getMessages('ch1')
-
-      expect(mockFetch).toHaveBeenCalledWith(
-        expect.stringContaining('/api/channels/ch1/messages'),
-        expect.any(Object),
-      )
-    })
-  })
-})

package/__tests__/constants.test.ts

@@ -1,38 +0,0 @@
-import { describe, expect, it } from 'vitest'
-import { CLIENT_EVENTS, channelRoom, SERVER_EVENTS, threadRoom, userRoom } from '../src/constants'
-
-describe('room helpers', () => {
-  describe('channelRoom', () => {
-    it('should build channel room name', () => {
-      expect(channelRoom('abc123')).toBe('channel:abc123')
-    })
-
-    it('should handle empty string', () => {
-      expect(channelRoom('')).toBe('channel:')
-    })
-  })
-
-  describe('threadRoom', () => {
-    it('should build thread room name', () => {
-      expect(threadRoom('t456')).toBe('thread:t456')
-    })
-  })
-
-  describe('userRoom', () => {
-    it('should build user room name', () => {
-      expect(userRoom('u789')).toBe('user:u789')
-    })
-  })
-})
-
-describe('re-exported constants', () => {
-  it('should re-export CLIENT_EVENTS from shared', () => {
-    expect(CLIENT_EVENTS).toBeDefined()
-    expect(CLIENT_EVENTS.CHANNEL_JOIN).toBe('channel:join')
-  })
-
-  it('should re-export SERVER_EVENTS from shared', () => {
-    expect(SERVER_EVENTS).toBeDefined()
-    expect(SERVER_EVENTS.MESSAGE_NEW).toBe('message:new')
-  })
-})