@abide/abide 0.110.2 → 0.110.4

package/CHANGELOG.md

@@ -1,5 +1,17 @@
 # @abide/abide
 
+## 0.110.4
+
+### Patch Changes
+
+- 2838836: an MCP tool call under a mount is asked under the mount, so a route whose first segment is the mount's name no longer 404s ([`0ceed49`](https://github.com/briancray/abide/commit/0ceed4966d55cd1b56902dc88c7961725319418f))
+
+## 0.110.3
+
+### Patch Changes
+
+- 804b901: the root below a mount is the bare mount, url('/') writing /docs and /docs/ redirected to it ([`9c5d3c3`](https://github.com/briancray/abide/commit/9c5d3c360c27f2d9918fb663d7aff83b22078707))
+
 ## 0.110.2
 
 ### Patch Changes

package/docs/RULEBOOK.md

@@ -1236,8 +1236,9 @@ be refused at construction, naming its `Args` member, and the build MUST refuse
 19.16 `route.url` MUST hold the address a page is served at, the path `APP_URL` names included, on
 the server as in the browser.
 
-19.17 A request for the path `APP_URL` names, without its trailing `/`, MUST be redirected 308 to
-that path with one, by path alone.
+19.17 The root of an app served below the path `APP_URL` names MUST be that path without a trailing
+`/`, as `url('/')` writes it, and a request for the path with one MUST be redirected 308 to it, by
+path alone.
 
 19.18 A `Schema` derived per 19.9 MUST refuse a member its type does not declare, wherever the type
 carries no index signature.

package/package.json

@@ -1,6 +1,6 @@
 {
     "name": "@abide/abide",
-    "version": "0.110.2",
+    "version": "0.110.4",
     "type": "module",
     "description": "Isomorphic type-safe framework for reactive async interfaces for humans and machines",
     "license": "MIT",

package/src/server/createApp.ts

@@ -507,14 +507,14 @@ function stamp(answered: Response): Response {
 
 /**
  * The request with the mount taken off its address, where it is under the mount — or the redirect
- * to the mount's root, where it asked for the mount with no `/` after it.
+ * to the mount's root, where it asked for the mount with a `/` after it.
  *
  * 16.10 — A REQUEST UNDER THE MOUNT IS ANSWERED AS THE ADDRESS BELOW IT; one that is not — a proxy
  * that already took the prefix off — is answered as it came.
  *
- * THE MOUNT'S ROOT IS `/docs/`, as `url('/')` writes it. A document answered at `/docs` resolves a
- * relative address against `/`, which is outside the app, so the bare mount is sent to the root a
- * relative address resolves under.
+ * 19.17 — THE MOUNT'S ROOT IS `/docs`, as `url('/')` writes it, and `/docs/` is sent there so the
+ * root has one address. Every address abide writes is absolute under the mount, so none resolves
+ * against the page's.
  */
 function belowMount(request: Request): Request | Response {
     const mount = MOUNT.current()
@@ -522,10 +522,10 @@ function belowMount(request: Request): Request | Response {
     const url = new URL(request.url)
     // A PATH RATHER THAN A URL: the origin this request names is the one it reached, which behind
     // a proxy is not the one the browser asked.
-    if (url.pathname === mount)
+    if (url.pathname === `${mount}/`)
         return new Response(null, {
             status: 308,
-            headers: { location: `${mount}/${url.search}` },
+            headers: { location: `${mount}${url.search}` },
         })
     const below = unmounted(url.pathname)
     if (below === undefined || below === url.pathname) return request

package/src/server/generatedSurfaces.ts

@@ -15,6 +15,7 @@
 
 import { isFailed } from '#shared/Failed.ts'
 import { namedLogger } from '#shared/log.ts'
+import { mounted } from '#shared/mount.ts'
 import { type Reactive, reactive } from '#shared/reactive/index.ts'
 import { parseRoutePattern } from '#shared/routePattern.ts'
 import { argsInURL, rpcRequest } from '#shared/rpcRequest.ts'
@@ -510,7 +511,9 @@ function callOf(
     // 38.68 — the answer is read here rather than sent on, and nothing in process decodes it.
     headers.delete('accept-encoding')
     if (call.body !== undefined) headers.set('content-type', 'application/json')
-    return new Request(new URL(call.url, asked.url), {
+    // 16.10 — under the mount, as `rpc.url` is: the app takes the mount off what it is asked, and a
+    // route whose first segment is the mount's name would lose that segment instead.
+    return new Request(new URL(mounted(call.url), asked.url), {
         method,
         headers,
         body: call.body ?? null,

package/src/shared/mount.ts

@@ -25,10 +25,14 @@ function documentMount(): string {
     return documentMountRead
 }
 
-/** An address below the app's root, as the address it is served at. */
+/**
+ * An address below the app's root, as the address it is served at. 19.17 — the root below a mount
+ * is the bare mount, `/docs` rather than `/docs/`.
+ */
 export function mounted(address: string): string {
     const mount = MOUNT.current()
-    return mount === '' ? address : `${mount}${address}`
+    if (mount === '') return address
+    return address === '/' ? mount : `${mount}${address}`
 }
 
 /**