deepspace 0.5.0 → 0.5.2

package/features/docs/src/editor/useDocEditor.ts

@@ -175,7 +175,11 @@ export function useDocEditor({
       // in dev (the editor view is destroyed and re-created, and any subscriber
       // that runs in between throws on `editor.view`). Defer to a useEffect.
       immediatelyRender: false,
-      // eslint-disable-next-line @typescript-eslint/no-explicit-any
+      // Bare directive on purpose: naming @typescript-eslint/no-explicit-any
+      // here is a hard "unknown rule" error under the scaffolded app's
+      // minimal eslint config (which never registers that plugin), while the
+      // SDK monorepo's config does enable the rule and needs the suppression.
+      // eslint-disable-next-line
       extensions: extensions as any,
       editorProps: {
         transformPastedHTML: stripCollaborationArtifactsFromHTML,

package/features/messaging/src/ChatPage.tsx

@@ -37,8 +37,13 @@ export default function ChatPage({
   }, [channelId, isMember, markAsRead])
 
   if (status !== 'ready' || !channelId) {
+    // data-testid lets tests (and humans reading a trace) distinguish "stuck
+    // waiting for the default channel" from "route never mounted".
     return (
-      <div className={`flex items-center justify-center h-full ${className ?? ''}`}>
+      <div
+        data-testid="chat-page-loading"
+        className={`flex items-center justify-center h-full ${className ?? ''}`}
+      >
         <div className="text-muted-foreground text-sm">Loading chat...</div>
       </div>
     )

package/features/messaging/src/hooks/useChatChannel.ts

@@ -26,12 +26,24 @@ export function useChatChannel(channelName: string = DEFAULT_CHANNEL_NAME) {
   const { channels, status, create } = useChannels()
   const hasInitialized = useRef(false)
 
-  const defaultChannel = channels.find(
-    (c: RecordData<Channel>) =>
-      c.data.name === channelName &&
-      c.data.type === 'public' &&
-      !c.data.archived
-  )
+  // Nothing enforces name uniqueness server-side, so two first visitors can
+  // race the one-shot init below and BOTH create the default channel. With a
+  // bare .find(), each client then resolves whichever duplicate its local
+  // snapshot orders first — split-brain: A and B post into different rooms
+  // and never see each other. Pick deterministically (oldest createdAt,
+  // recordId tie-break) so every client converges on the same channel once
+  // the duplicates sync.
+  const defaultChannel = channels
+    .filter(
+      (c: RecordData<Channel>) =>
+        c.data.name === channelName &&
+        c.data.type === 'public' &&
+        !c.data.archived,
+    )
+    .sort(
+      (a: RecordData<Channel>, b: RecordData<Channel>) =>
+        a.createdAt.localeCompare(b.createdAt) || a.recordId.localeCompare(b.recordId),
+    )[0]
 
   const { isMember, join, status: membersStatus } = useChannelMembers(defaultChannel?.recordId)
 

package/package.json

@@ -1,6 +1,9 @@
 {
   "name": "deepspace",
-  "version": "0.5.0",
+  "version": "0.5.2",
+  "engines": {
+    "node": ">=22.15.0"
+  },
   "description": "DeepSpace SDK — build real-time collaborative apps on Cloudflare Workers",
   "type": "module",
   "sideEffects": false,
@@ -41,14 +44,15 @@
   },
   "dependencies": {
     "@clack/prompts": "^1.2.0",
-    "better-auth": "1.5.6",
+    "better-auth": "1.6.23",
     "citty": "^0.2.0",
     "cross-spawn": "^7.0.6",
     "esbuild": "^0.25.0",
     "hono": "^4.6.0",
     "jose": "^6.1.0",
     "smol-toml": "^1.6.1",
-    "yjs": "^13.6.0"
+    "yjs": "^13.6.0",
+    "zod": "^4.3.6"
   },
   "peerDependencies": {
     "@ai-sdk/anthropic": "^2.0.0",

package/scripts/add-feature.cjs

@@ -97,16 +97,18 @@ function copyFile(src, dest) {
 }
 
 // Copy + rewrite leading `../` imports. Used when a page file is redirected
-// into src/pages/(protected)/, which adds one directory level — every relative
-// import that traverses upward needs one extra `../`. Matches static
-// `from '...'`, side-effect `import '...'`, and dynamic/type `import('...')`
-// (e.g. `import('../hooks/x').T` in a type position, or `await import('../x')`).
+// into a deeper route group (e.g. src/pages/(app)/ adds one directory level,
+// src/pages/(app)/(protected)/ adds two) — every relative import that
+// traverses upward needs `levels` extra `../`. Matches static `from '...'`,
+// side-effect `import '...'`, and dynamic/type `import('...')` (e.g.
+// `import('../hooks/x').T` in a type position, or `await import('../x')`).
 // Same-directory (`./...`) and bare-package imports are untouched.
-function copyFileWithImportShift(src, dest) {
+function copyFileWithImportShift(src, dest, levels) {
   const destDir = path.dirname(dest)
   if (!fs.existsSync(destDir)) fs.mkdirSync(destDir, { recursive: true })
   const content = fs.readFileSync(src, 'utf-8')
-  const shifted = content.replace(/(from\s+|import\s+|import\()(['"])\.\.\//g, '$1$2../../')
+  const prefix = '../'.repeat(levels)
+  const shifted = content.replace(/(from\s+|import\s+|import\()(['"])(\.\.\/)/g, `$1$2${prefix}$3`)
   fs.writeFileSync(dest, shifted)
 }
 
@@ -114,18 +116,21 @@ function copyFileWithImportShift(src, dest) {
 // Install files
 // ---------------------------------------------------------------------------
 
-// When a feature's route is protected, page files must land under
-// src/pages/(protected)/ so generouted's route group wraps them in <AuthGate>.
-// `(protected)` is a folder-name-only group — the URL path is unchanged.
-// Only page files (src/pages/*) get rewritten; schemas, components, etc.
-// keep their declared destinations.
+// Page files must land under src/pages/(app)/ so the (app)/_layout.tsx route
+// group wraps them in the DeepSpace providers (auth + realtime records) — the
+// top level of src/pages/ is reserved for static pages that mount no
+// providers. Protected pages go one level deeper, under (app)/(protected)/,
+// so the nested layout also wraps them in <AuthGate>. Both are
+// folder-name-only route groups — the URL path is unchanged. Only page files
+// (src/pages/*) get redirected; schemas, components, etc. keep their declared
+// destinations. Page files already placed in the (app) group are left as-is.
 function resolveDestPath(fileDest, route) {
   const PAGES = 'src/pages/'
-  const PROTECTED = 'src/pages/(protected)/'
+  const APP = 'src/pages/(app)/'
+  if (!fileDest.startsWith(PAGES) || fileDest.startsWith(APP)) return fileDest
+  const rest = fileDest.slice(PAGES.length)
   const isProtected = route && route.protected !== false
-  if (!isProtected) return fileDest
-  if (!fileDest.startsWith(PAGES) || fileDest.startsWith(PROTECTED)) return fileDest
-  return PROTECTED + fileDest.slice(PAGES.length)
+  return isProtected ? `${APP}(protected)/${rest}` : `${APP}${rest}`
 }
 
 function installFeature(config, targetDir) {
@@ -137,6 +142,10 @@ function installFeature(config, targetDir) {
     const resolvedDest = resolveDestPath(file.dest, config.route)
     const destPath = path.join(targetDir, resolvedDest)
     const wasRedirected = resolvedDest !== file.dest
+    // Each extra path segment in the resolved dest is one directory level the
+    // page dropped into (e.g. (app)/ = 1, (app)/(protected)/ = 2), which is
+    // exactly how many `../` its upward relative imports must gain.
+    const shiftLevels = resolvedDest.split('/').length - file.dest.split('/').length
 
     if (!fs.existsSync(srcPath)) {
       console.error(`   Warning: source not found: ${file.src}`)
@@ -149,7 +158,7 @@ function installFeature(config, targetDir) {
     } else {
       const replacing = fs.existsSync(destPath)
       if (wasRedirected) {
-        copyFileWithImportShift(srcPath, destPath)
+        copyFileWithImportShift(srcPath, destPath, shiftLevels)
       } else {
         copyFile(srcPath, destPath)
       }