dsh-session-mng 0.1.3 → 0.2.0

package/lib/api.js

@@ -18,6 +18,9 @@ export const API_PREFIX = '/session-mng/api'
 /** Request body ceiling; these requests are tiny id lists or one title. */
 const MAX_BODY_BYTES = 64 * 1024
 
+/** Budget for one read request: a catalog read plus, at worst, one size scan. */
+const GET_TIMEOUT_MS = 20_000
+
 /** Write one JSON response. */
 function sendJson(res, status, body) {
   if (res.writableEnded) return
@@ -59,7 +62,16 @@ async function readJsonBody(req) {
   }
 }
 
-/** One dispatch table entry. `needsBody` is false only for the state read. */
+/** Read one GET query string into a plain parameter object. */
+function queryOf(url) {
+  const params = {}
+  for (const [key, value] of url.searchParams) {
+    if (!Object.hasOwn(params, key)) params[key] = value
+  }
+  return params
+}
+
+/** One dispatch table entry. `needsBody` is false only for the read routes. */
 const ROUTES = {
   preview: { needsBody: true, run: (service, body) => service.preview(body.sessionId, body.count) },
   'title/propose': {
@@ -74,6 +86,24 @@ const ROUTES = {
   move: { needsBody: true, run: (service, body) => service.move(body.sessionIds, body.workspaceId) },
 }
 
+/**
+ * The read surface. `/state` stays byte-compatible for 0.1.x clients; the
+ * 0.2.0 routes serve the same data through a paginated, catalog-backed shape.
+ * Each entry receives the parsed query object and one request signal.
+ */
+const GET_ROUTES = {
+  state: (service, params, signal) =>
+    service.state({ includeSubagents: params.subagents === '1', signal }),
+  overview: (service, params, signal) =>
+    service.overview({ includeSubagents: params.subagents === '1', signal }),
+  sessions: (service, params, signal) => service.sessions({ ...params, signal }),
+  search: (service, params, signal) => service.search({ ...params, signal }),
+  session: (service, params, signal) => service.session({ ...params, signal }),
+  blanks: (service, params, signal) =>
+    service.blanks({ includeSubagents: params.subagents === '1', signal }),
+  sizes: (service, params, signal) => service.sizes({ ...params, signal }),
+}
+
 /**
  * Build the Node request handler for the plugin's prefix route.
  *
@@ -97,25 +127,28 @@ export function createApiHandler({ ctx, config, service, trustedHosts }) {
         return
       }
 
-      if (relative === '' || relative === 'state') {
+      // A prefix route also answers the bare prefix; it means the legacy `/state`.
+      const key = relative === '' ? 'state' : relative
+
+      const readRoute = GET_ROUTES[key]
+      if (readRoute !== undefined) {
         if (req.method !== 'GET') {
-          sendJson(res, 405, { ok: false, error: { code: 'method-error', message: 'state is GET-only' } })
+          sendJson(res, 405, { ok: false, error: { code: 'method-error', message: `${key} is GET-only` } })
           return
         }
-        const includeSubagents = url.searchParams.get('subagents') === '1'
-        const value = await service.state({ includeSubagents, signal: AbortSignal.timeout(20_000) })
+        const value = await readRoute(service, queryOf(url), AbortSignal.timeout(GET_TIMEOUT_MS))
         sendJson(res, 200, { ok: true, value })
         return
       }
 
-      if (req.method !== 'POST') {
-        sendJson(res, 405, { ok: false, error: { code: 'method-error', message: 'POST only' } })
+      const route = ROUTES[key]
+      if (route === undefined) {
+        sendJson(res, 404, { ok: false, error: { code: 'not-found', message: `unknown route "${key}"` } })
         return
       }
 
-      const route = ROUTES[relative]
-      if (route === undefined) {
-        sendJson(res, 404, { ok: false, error: { code: 'not-found', message: `unknown action "${relative}"` } })
+      if (req.method !== 'POST') {
+        sendJson(res, 405, { ok: false, error: { code: 'method-error', message: 'POST only' } })
         return
       }
 
@@ -123,7 +156,7 @@ export function createApiHandler({ ctx, config, service, trustedHosts }) {
       // A closed connection cancels a read-only model call; destructive
       // operations are deliberately allowed to finish once admitted.
       const controller = new AbortController()
-      if (relative === 'title/propose') {
+      if (key === 'title/propose') {
         res.on('close', () => {
           if (!res.writableEnded) controller.abort()
         })