@lovable.dev/mcp-js 0.21.0-rc.3 → 0.21.0

package/dist/stacks/tanstack/index.cjs

@@ -129,7 +129,7 @@ function describeError(err) {
 }
 
 // package.json
-var version = "0.21.0-rc.3";
+var version = "0.21.0";
 
 // src/metrics/otlp.ts
 var SCOPE_NAME = "@lovable.dev/mcp-js";
@@ -236,7 +236,8 @@ var BaseMetricRecorder = class {
       method: ev.method,
       outcome: ev.outcome,
       durationMs: ev.durationMs,
-      stack: this.stack
+      stack: this.stack,
+      ...ev.errorText !== void 0 && { errorText: ev.errorText }
     });
     if (!this.config.enabled)
       return;
@@ -416,23 +417,9 @@ function parseSafeUrl(subject, raw, ErrorClass = Error) {
 
 // src/auth/discovery.ts
 var OAuthConfigurationError = class extends Error {
-  constructor(message, stage) {
-    super(message);
-    this.stage = stage;
-    this.name = "OAuthConfigurationError";
-  }
-};
-var OAuthUpstreamUnreachableError = class extends Error {
-  constructor(message, stage) {
-    super(message);
-    this.stage = stage;
-    this.name = "OAuthUpstreamUnreachableError";
-  }
-};
-var MetadataUnreachableError = class extends Error {
   constructor(message) {
     super(message);
-    this.name = "MetadataUnreachableError";
+    this.name = "OAuthConfigurationError";
   }
 };
 var METADATA_FETCH_TIMEOUT_MS = 5e3;
@@ -464,28 +451,16 @@ async function fetchFirstValidOAuthServerMetadata(metadataUrls, expectedIssuer)
     try {
       return await fetchOAuthServerMetadata(url, expectedIssuer);
     } catch (err) {
-      const error = err instanceof Error ? err : new Error(String(err));
-      log.debug("oauth.discovery.attempt_failed", { url, ...describeError(error) });
-      errors.push(error);
+      log.debug("oauth.discovery.attempt_failed", { url, ...describeError(err) });
+      errors.push(`${url}: ${err instanceof Error ? err.message : String(err)}`);
     }
   }
-  const unreachable = errors.some((e) => e instanceof MetadataUnreachableError);
-  const detail = errors.map((e) => e.message).join("; ");
-  log.error("oauth.discovery.exhausted", { expectedIssuer, urlsTried: metadataUrls, errors: detail, unreachable });
-  const message = `failed to discover OAuth server metadata (${detail})`;
-  throw unreachable ? new MetadataUnreachableError(message) : new Error(message);
+  log.error("oauth.discovery.exhausted", { expectedIssuer, urlsTried: metadataUrls, errors });
+  throw new Error(`failed to discover OAuth server metadata (${errors.join("; ")})`);
 }
 async function fetchOAuthServerMetadata(url, expectedIssuer) {
   log.debug("oauth.discovery.fetch", { url });
-  let response;
-  try {
-    response = await fetch(url, {
-      signal: AbortSignal.timeout(METADATA_FETCH_TIMEOUT_MS),
-      redirect: "manual"
-    });
-  } catch (err) {
-    throw new MetadataUnreachableError(err instanceof Error ? err.message : String(err));
-  }
+  const response = await fetch(url, { signal: AbortSignal.timeout(METADATA_FETCH_TIMEOUT_MS), redirect: "manual" });
   if (!response.ok) {
     throw new Error(String(response.status));
   }
@@ -508,21 +483,14 @@ async function fetchIssuerOAuthServerMetadata(issuer) {
   try {
     return await fetchFirstValidOAuthServerMetadata(oauthMetadataUrlsForIssuer(issuer), issuer);
   } catch (err) {
-    const message = `OAuth issuer discovery failed: ${err instanceof Error ? err.message : String(err)}`;
-    if (err instanceof MetadataUnreachableError) {
-      log.error("oauth.discovery.unreachable", {
-        issuer,
-        ...describeError(err),
-        outcome: "503 authorization server unreachable"
-      });
-      throw new OAuthUpstreamUnreachableError(message, "discovery");
-    }
     log.error("oauth.discovery.config_error", {
       issuer,
       ...describeError(err),
       outcome: "500 oauth configuration error"
     });
-    throw new OAuthConfigurationError(message, "discovery");
+    throw new OAuthConfigurationError(
+      `OAuth issuer discovery failed: ${err instanceof Error ? err.message : String(err)}`
+    );
   }
 }
 function createOAuthDiscoveryResolver(auth) {
@@ -582,13 +550,18 @@ function readString(value) {
 function splitScopes(value) {
   if (typeof value === "string")
     return value.split(/\s+/).filter(Boolean);
-  if (Array.isArray(value))
-    return value.filter((entry) => typeof entry === "string" && entry.length > 0);
-  return [];
+  return stringClaimList(value);
 }
 function stringClaim(claims, name) {
   return readString(claims[name]);
 }
+function stringClaimList(value) {
+  if (typeof value === "string")
+    return value === "" ? [] : [value];
+  if (Array.isArray(value))
+    return value.filter((entry) => typeof entry === "string" && entry !== "");
+  return [];
+}
 
 // src/auth/verifier.ts
 var DEFAULT_JWT_ALGORITHMS = ["RS256", "RS384", "RS512", "ES256", "ES384", "ES512", "EdDSA"];
@@ -615,24 +588,15 @@ function tokenHeaderFields(token) {
   }
 }
 async function fetchVerificationKeySet(jwksUri) {
-  let response;
-  try {
-    response = await fetch(jwksUri, { signal: AbortSignal.timeout(JWKS_FETCH_TIMEOUT_MS), redirect: "manual" });
-  } catch (err) {
-    log.error("oauth.jwks.unreachable", { ...describeError(err), outcome: "503 authorization server unreachable" });
-    throw new OAuthUpstreamUnreachableError(
-      `JWKS fetch failed: ${err instanceof Error ? err.message : String(err)}`,
-      "jwks"
-    );
-  }
   try {
+    const response = await fetch(jwksUri, { signal: AbortSignal.timeout(JWKS_FETCH_TIMEOUT_MS), redirect: "manual" });
     if (!response.ok)
       throw new Error(`JWKS endpoint returned ${response.status}`);
     const json = await response.json();
     return (0, import_jose.createLocalJWKSet)(json);
   } catch (err) {
     log.error("oauth.jwks.fetch_failed", { ...describeError(err), outcome: "500 oauth configuration error" });
-    throw new OAuthConfigurationError(`JWKS fetch failed: ${err instanceof Error ? err.message : String(err)}`, "jwks");
+    throw new OAuthConfigurationError(`JWKS fetch failed: ${err instanceof Error ? err.message : String(err)}`);
   }
 }
 function assertAccessTokenTyp(token, allowed) {
@@ -649,13 +613,12 @@ function assertAccessTokenTyp(token, allowed) {
     throw new OAuthTokenError(401, "invalid_token", "Access token typ header is not accepted");
   }
 }
-async function verifyJwtClaims(token, keySet, issuer, audience, auth) {
+async function verifyJwtClaims(token, keySet, issuer, auth) {
   try {
     const { payload } = await (0, import_jose.jwtVerify)(token, keySet, {
       // `issuer` is trimmed of any trailing slash; accept both forms so a token whose
       // `iss` carries the slash the AS publishes still verifies.
       issuer: [issuer, `${issuer}/`],
-      audience: [...audience],
       algorithms: auth.algorithms ? [...auth.algorithms] : DEFAULT_JWT_ALGORITHMS,
       requiredClaims: ["sub", "exp"],
       clockTolerance: auth.clockToleranceSeconds ?? DEFAULT_CLOCK_TOLERANCE_SECONDS
@@ -666,6 +629,22 @@ async function verifyJwtClaims(token, keySet, issuer, audience, auth) {
     throw err;
   }
 }
+function checkAcceptedAudience(claims, accepted, auth) {
+  const audPopulated = claims.aud !== void 0 && !(Array.isArray(claims.aud) && claims.aud.length === 0);
+  if (audPopulated) {
+    if (stringClaimList(claims.aud).some((audience) => accepted.includes(audience)))
+      return "aud";
+    log.debug("oauth.verify.bad_audience", { tokenAud: claims.aud, accepted, outcome: "401 invalid_token" });
+  } else if (auth.acceptResourceClaim !== false) {
+    const acceptedTrimmed = accepted.map(trimTrailingSlash);
+    if (stringClaimList(claims.resource).some((resource) => acceptedTrimmed.includes(trimTrailingSlash(resource))))
+      return "resource";
+    log.debug("oauth.verify.bad_resource", { tokenResource: claims.resource, accepted, outcome: "401 invalid_token" });
+  } else {
+    log.debug("oauth.verify.empty_audience", { accepted, outcome: "401 invalid_token" });
+  }
+  throw new OAuthTokenError(401, "invalid_token", "token audience is not accepted");
+}
 function assertNonEmptySubject(claims) {
   const sub = claims["sub"];
   if (typeof sub !== "string" || sub.trim() === "") {
@@ -710,14 +689,16 @@ function createOAuthTokenVerifier(auth, discovery) {
     log.debug("oauth.jwks.fetch", { jwksUri });
     const keySet = await fetchVerificationKeySet(jwksUri);
     assertAccessTokenTyp(token, allowedTyps);
-    const claims = await verifyJwtClaims(token, keySet, issuer, acceptedAudiences, auth);
+    const claims = await verifyJwtClaims(token, keySet, issuer, auth);
+    const audienceVia = checkAcceptedAudience(claims, acceptedAudiences, auth);
     assertNonEmptySubject(claims);
     const context = buildMcpAuthContext({ token, claims, issuer, resource, acceptedAudiences });
     assertOAuthClientClaim(auth, context.principal.clientId);
     log.info("oauth.verify.ok", {
       sub: context.principal.sub,
       clientId: context.principal.clientId,
-      scopes: context.principal.scopes
+      scopes: context.principal.scopes,
+      audienceVia
     });
     return context;
   };
@@ -766,12 +747,6 @@ function oauthConfigurationErrorResponse() {
     headers: { ...JSON_HEADERS, "Cache-Control": "no-store" }
   });
 }
-function oauthUpstreamUnreachableResponse() {
-  return new Response(JSON.stringify({ error: "authorization server unreachable" }), {
-    status: 503,
-    headers: { ...JSON_HEADERS, "Cache-Control": "no-store", "Retry-After": "30" }
-  });
-}
 function parseBearerToken(request) {
   const header = request.headers.get("Authorization");
   if (!header)
@@ -844,22 +819,12 @@ function createRequestAuthorizer(mcp, options = {}) {
         assertRequiredScopes(runtime.auth, auth);
         return { ok: true, auth };
       } catch (err) {
-        if (err instanceof OAuthUpstreamUnreachableError) {
-          log.error("auth.upstream_unreachable", { ...describeError(err), stage: err.stage, outcome: "503" });
-          await recorder?.emit({
-            tool: null,
-            method: "authorize",
-            outcome: err.stage === "jwks" ? "auth_jwks_unreachable" : "auth_discovery_unreachable",
-            durationMs: nowMs() - startedAt
-          });
-          return { ok: false, response: oauthUpstreamUnreachableResponse() };
-        }
         if (err instanceof OAuthConfigurationError) {
-          log.error("auth.config_error", { ...describeError(err), stage: err.stage, outcome: "500" });
+          log.error("auth.config_error", { ...describeError(err), outcome: "500" });
           await recorder?.emit({
             tool: null,
             method: "authorize",
-            outcome: err.stage === "jwks" ? "auth_jwks_config_error" : "auth_discovery_config_error",
+            outcome: "auth_config_error",
             durationMs: nowMs() - startedAt
           });
           return { ok: false, response: oauthConfigurationErrorResponse() };
@@ -938,6 +903,14 @@ var ToolContext = class {
   }
 };
 
+// src/core/content.ts
+function extractTextContent(content) {
+  if (!content)
+    return void 0;
+  const text = content.filter((block) => block.type === "text").map((block) => block.text).join("\n").trim();
+  return text.length > 0 ? text : void 0;
+}
+
 // src/core/cors.ts
 var EXPOSE_HEADERS = "WWW-Authenticate, Mcp-Session-Id, Mcp-Protocol-Version";
 var ALLOW_HEADERS = "Authorization, Content-Type, Mcp-Session-Id, Mcp-Protocol-Version, Last-Event-ID";
@@ -988,7 +961,8 @@ function adaptToolToSdkCallback(tool, auth, recorder) {
       tool: tool.name,
       method: "tools/call",
       outcome: result.isError ? "tool_error" : "ok",
-      durationMs: nowMs() - start
+      durationMs: nowMs() - start,
+      errorText: result.isError ? extractTextContent(result.content) : void 0
     });
     return { content: result.content ?? [], structuredContent: result.structuredContent, isError: result.isError };
   };
@@ -1244,7 +1218,8 @@ function createInvokeToolHandler(mcp, options = {}) {
       tool: tool.name,
       method: "tools/call",
       outcome: result.isError ? "tool_error" : "ok",
-      durationMs: nowMs() - start
+      durationMs: nowMs() - start,
+      errorText: result.isError ? extractTextContent(result.content) : void 0
     });
     return Response.json({
       content: result.content ?? [],

package/dist/stacks/tanstack/index.d.cts

@@ -1,5 +1,5 @@
 import { M as McpRuntimeOptions } from '../../authorize-BMeXd_Sh.js';
-import { d as McpDefinition } from '../../types-CPkhCRxc.js';
+import { d as McpDefinition } from '../../types-qqrmJ736.js';
 import 'zod';
 
 interface TanStackRouteCtx {

package/dist/stacks/tanstack/index.d.ts

@@ -1,5 +1,5 @@
 import { M as McpRuntimeOptions } from '../../authorize-BMeXd_Sh.js';
-import { d as McpDefinition } from '../../types-CPkhCRxc.js';
+import { d as McpDefinition } from '../../types-qqrmJ736.js';
 import 'zod';
 
 interface TanStackRouteCtx {

package/dist/stacks/tanstack/index.js

@@ -3,23 +3,24 @@ import {
 } from "../../chunk-UQK5UO6C.js";
 import {
   createMcpProtocolHandler
-} from "../../chunk-5W6JK6OS.js";
+} from "../../chunk-CI7JYLV3.js";
 import {
   createOAuthProtectedResourceMetadataHandler
-} from "../../chunk-NENAFXZJ.js";
+} from "../../chunk-ENJDNFEB.js";
 import {
   createInvokeToolHandler
-} from "../../chunk-JIOOUIDX.js";
+} from "../../chunk-QPKPYDPB.js";
 import {
   createListToolsHandler
-} from "../../chunk-TVU5FL2T.js";
+} from "../../chunk-LFXK3FNF.js";
+import "../../chunk-6QFZYYUV.js";
 import "../../chunk-MA5H6PSF.js";
 import {
   createRecorderForRuntime
-} from "../../chunk-CIAVINZ5.js";
+} from "../../chunk-YLU2JTHL.js";
 import "../../chunk-H37EB22A.js";
 import "../../chunk-6DXGZZA4.js";
-import "../../chunk-SIDWSSWV.js";
+import "../../chunk-E3Q3FRKQ.js";
 
 // src/stacks/tanstack/handlers.ts
 var STACK = "tanstack";

package/dist/{types-CPkhCRxc.d.ts → types-qqrmJ736.d.ts}

@@ -10,6 +10,8 @@ type JwtClaims = Record<string, unknown> & {
     scope?: string;
     client_id?: string;
     email?: string;
+    /** RFC 8707 resource indicator(s) the token was granted for (Lovable OAuth convention). */
+    resource?: string | string[];
 };
 /**
  * The safe-to-surface half of the auth context, reachable by tools only through
@@ -60,6 +62,14 @@ interface McpAuthConfig {
     readonly protectedResourceMetadataUrl?: string;
     readonly requireOAuthClientClaim?: boolean;
     readonly acceptedAudiences?: readonly string[];
+    /**
+     * Accept a token whose RFC 8707 `resource` claim matches an accepted
+     * audience (trailing slashes ignored) when its `aud` is an empty array or
+     * absent — the shape Lovable's OAuth server mints; applies to any configured
+     * issuer. A populated `aud` is always authoritative and is never overridden.
+     * Defaults to true. Set false to require a matching `aud` (strict RFC 9068).
+     */
+    readonly acceptResourceClaim?: boolean;
     readonly requiredScopes?: readonly string[];
     readonly jwksUri?: string;
     readonly algorithms?: readonly string[];

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@lovable.dev/mcp-js",
-  "version": "0.21.0-rc.3",
+  "version": "0.21.0",
   "description": "Author MCP servers for Lovable apps. Declare tools with defineTool, register them in defineMcp, and a framework adapter (TanStack or Supabase Edge Functions) emits the route(s) at build time.",
   "type": "module",
   "repository": {