@rei-standard/amsg-server 2.1.0 → 2.1.1

package/README.md

@@ -50,6 +50,10 @@ const rei = await createReiServer({
 // GET  /api/v1/messages             -> rei.handlers.messages.GET
 ```
 
+## 关于 `messageType: 'instant'`
+
+> **Note**：新代码的 instant 消息请用 [@rei-standard/amsg-instant](../instant/README.md)，跳过本端点的"建任务 → 处理 → 删任务" DB 来回。本端点的 `instant` 分支为兼容保留，行为不变、不会有运行时警告。
+
 ## AI 接口 `apiUrl` 约束
 
 当 `messageType` 为 `prompted` / `auto`，或 `instant` 使用 AI 配置时：

package/dist/index.cjs

@@ -554,7 +554,7 @@ function buildAiRequestBody(payload) {
 function normalizeAiApiUrl(apiUrl) {
   if (typeof apiUrl !== "string" || !apiUrl.trim()) {
     throw new Error(
-      "Invalid apiUrl: apiUrl is required. Please provide a full chat endpoint URL (for example: https://api.openai.com/v1/chat/completions)."
+      "Invalid apiUrl: apiUrl is required. Please provide a chat endpoint URL (for example: https://api.openai.com or https://api.openai.com/v1/chat/completions)."
     );
   }
   const trimmedApiUrl = apiUrl.trim();
@@ -563,10 +563,17 @@ function normalizeAiApiUrl(apiUrl) {
     parsedUrl = new URL(trimmedApiUrl);
   } catch (e3) {
     throw new Error(
-      `Invalid apiUrl: "${apiUrl}". Please provide a valid absolute URL that points to a full chat endpoint.`
+      `Invalid apiUrl: "${apiUrl}". Please provide a valid absolute URL.`
     );
   }
-  parsedUrl.pathname = parsedUrl.pathname.replace(/\/+$/, "") || "/";
+  let path = parsedUrl.pathname.replace(/\/+$/, "") || "/";
+  if (/\/chat\/completions$/.test(path)) {
+  } else if (path === "/") {
+    path = "/v1/chat/completions";
+  } else if (/\/v\d+$/.test(path)) {
+    path = `${path}/chat/completions`;
+  }
+  parsedUrl.pathname = path;
   return parsedUrl.toString();
 }
 

package/dist/index.d.cts

@@ -906,12 +906,27 @@ function buildAiRequestBody(payload) {
 }
 
 /**
- * Normalize AI API URL with minimal, non-breaking rules:
- * - trim whitespace
- * - remove redundant trailing slashes from pathname
- * - keep query string untouched
+ * Normalize AI API URL for OpenAI-compatible chat endpoints.
  *
- * Note: This function does NOT auto-append /v1 or /chat/completions.
+ * **Keep in sync** with `@rei-standard/amsg-instant`'s
+ * `src/message-processor.js` `normalizeAiApiUrl` — same rules, same
+ * tests. The two packages share this logic but each carry their own copy
+ * to avoid an architectural dependency (server should not depend on the
+ * stateless worker package).
+ *
+ * Rules (idempotent — running it twice equals running it once):
+ *   - Already ends with `/chat/completions`             → leave as-is.
+ *   - Bare host (no path or just `/`)                   → append `/v1/chat/completions`.
+ *   - Path ends with a version segment like `/v1`,
+ *     `/v2`, … (with or without trailing slash)         → append only
+ *     `/chat/completions`; never doubles `/v1` for
+ *     callers who already include it.
+ *   - Anything else (custom path that doesn't match the
+ *     OpenAI shape, e.g. `/v1/messages` for
+ *     Anthropic-style proxies)                          → leave as-is. The caller
+ *     knows their own routing.
+ *
+ * Query string is preserved verbatim.
  *
  * @param {string} apiUrl
  * @returns {string}
@@ -920,7 +935,8 @@ function normalizeAiApiUrl(apiUrl) {
   if (typeof apiUrl !== 'string' || !apiUrl.trim()) {
     throw new Error(
       'Invalid apiUrl: apiUrl is required. ' +
-      'Please provide a full chat endpoint URL (for example: https://api.openai.com/v1/chat/completions).'
+      'Please provide a chat endpoint URL ' +
+      '(for example: https://api.openai.com or https://api.openai.com/v1/chat/completions).'
     );
   }
 
@@ -931,12 +947,23 @@ function normalizeAiApiUrl(apiUrl) {
     parsedUrl = new URL(trimmedApiUrl);
   } catch {
     throw new Error(
-      `Invalid apiUrl: "${apiUrl}". ` +
-      'Please provide a valid absolute URL that points to a full chat endpoint.'
+      `Invalid apiUrl: "${apiUrl}". Please provide a valid absolute URL.`
     );
   }
 
-  parsedUrl.pathname = parsedUrl.pathname.replace(/\/+$/, '') || '/';
+  let path = parsedUrl.pathname.replace(/\/+$/, '') || '/';
+
+  if (/\/chat\/completions$/.test(path)) ; else if (path === '/') {
+    // Bare host → assume OpenAI shape.
+    path = '/v1/chat/completions';
+  } else if (/\/v\d+$/.test(path)) {
+    // Path ends in `/v1`, `/v2`, … — caller already versioned the URL.
+    // Append only `/chat/completions`; never re-add `/v1`.
+    path = `${path}/chat/completions`;
+  }
+  // Any other custom path is left untouched on purpose.
+
+  parsedUrl.pathname = path;
   return parsedUrl.toString();
 }
 
@@ -1033,6 +1060,14 @@ function createScheduleMessageHandler(ctx) {
 
     const encryptedPayload = encryptForStorage(JSON.stringify(fullTaskData), userKey);
 
+    /**
+     * @deprecated Soft-deprecated. For new code, use @rei-standard/amsg-instant.
+     *   This branch is kept for backward compatibility and the existing behavior
+     *   (create task → process → delete) is unchanged. The dedicated amsg-instant
+     *   package is stateless (no DB roundtrip), deployable to Cloudflare Workers,
+     *   and locks the encryption + push-payload contract behind a single version.
+     *   See packages/rei-standard-amsg/instant/README.md.
+     */
     // Instant type: check VAPID before creating the task to avoid orphaned rows
     if (payload.messageType === 'instant') {
       if (!ctx.vapid.email || !ctx.vapid.publicKey || !ctx.vapid.privateKey) {
@@ -1086,6 +1121,13 @@ function createScheduleMessageHandler(ctx) {
       return { status: 500, body: { success: false, error: { code: 'TASK_CREATE_FAILED', message: '创建任务失败' } } };
     }
 
+    /**
+     * @deprecated Soft-deprecated. For new code, use @rei-standard/amsg-instant.
+     *   The "create-task → process-by-uuid → delete-task" sequence below is
+     *   preserved verbatim so existing clients keep working. New integrations
+     *   should call the dedicated amsg-instant endpoint instead — it skips this
+     *   DB round-trip entirely. See packages/rei-standard-amsg/instant/README.md.
+     */
     // Instant type: send immediately
     if (payload.messageType === 'instant') {
       try {

package/dist/index.d.ts

@@ -906,12 +906,27 @@ function buildAiRequestBody(payload) {
 }
 
 /**
- * Normalize AI API URL with minimal, non-breaking rules:
- * - trim whitespace
- * - remove redundant trailing slashes from pathname
- * - keep query string untouched
+ * Normalize AI API URL for OpenAI-compatible chat endpoints.
  *
- * Note: This function does NOT auto-append /v1 or /chat/completions.
+ * **Keep in sync** with `@rei-standard/amsg-instant`'s
+ * `src/message-processor.js` `normalizeAiApiUrl` — same rules, same
+ * tests. The two packages share this logic but each carry their own copy
+ * to avoid an architectural dependency (server should not depend on the
+ * stateless worker package).
+ *
+ * Rules (idempotent — running it twice equals running it once):
+ *   - Already ends with `/chat/completions`             → leave as-is.
+ *   - Bare host (no path or just `/`)                   → append `/v1/chat/completions`.
+ *   - Path ends with a version segment like `/v1`,
+ *     `/v2`, … (with or without trailing slash)         → append only
+ *     `/chat/completions`; never doubles `/v1` for
+ *     callers who already include it.
+ *   - Anything else (custom path that doesn't match the
+ *     OpenAI shape, e.g. `/v1/messages` for
+ *     Anthropic-style proxies)                          → leave as-is. The caller
+ *     knows their own routing.
+ *
+ * Query string is preserved verbatim.
  *
  * @param {string} apiUrl
  * @returns {string}
@@ -920,7 +935,8 @@ function normalizeAiApiUrl(apiUrl) {
   if (typeof apiUrl !== 'string' || !apiUrl.trim()) {
     throw new Error(
       'Invalid apiUrl: apiUrl is required. ' +
-      'Please provide a full chat endpoint URL (for example: https://api.openai.com/v1/chat/completions).'
+      'Please provide a chat endpoint URL ' +
+      '(for example: https://api.openai.com or https://api.openai.com/v1/chat/completions).'
     );
   }
 
@@ -931,12 +947,23 @@ function normalizeAiApiUrl(apiUrl) {
     parsedUrl = new URL(trimmedApiUrl);
   } catch {
     throw new Error(
-      `Invalid apiUrl: "${apiUrl}". ` +
-      'Please provide a valid absolute URL that points to a full chat endpoint.'
+      `Invalid apiUrl: "${apiUrl}". Please provide a valid absolute URL.`
     );
   }
 
-  parsedUrl.pathname = parsedUrl.pathname.replace(/\/+$/, '') || '/';
+  let path = parsedUrl.pathname.replace(/\/+$/, '') || '/';
+
+  if (/\/chat\/completions$/.test(path)) ; else if (path === '/') {
+    // Bare host → assume OpenAI shape.
+    path = '/v1/chat/completions';
+  } else if (/\/v\d+$/.test(path)) {
+    // Path ends in `/v1`, `/v2`, … — caller already versioned the URL.
+    // Append only `/chat/completions`; never re-add `/v1`.
+    path = `${path}/chat/completions`;
+  }
+  // Any other custom path is left untouched on purpose.
+
+  parsedUrl.pathname = path;
   return parsedUrl.toString();
 }
 
@@ -1033,6 +1060,14 @@ function createScheduleMessageHandler(ctx) {
 
     const encryptedPayload = encryptForStorage(JSON.stringify(fullTaskData), userKey);
 
+    /**
+     * @deprecated Soft-deprecated. For new code, use @rei-standard/amsg-instant.
+     *   This branch is kept for backward compatibility and the existing behavior
+     *   (create task → process → delete) is unchanged. The dedicated amsg-instant
+     *   package is stateless (no DB roundtrip), deployable to Cloudflare Workers,
+     *   and locks the encryption + push-payload contract behind a single version.
+     *   See packages/rei-standard-amsg/instant/README.md.
+     */
     // Instant type: check VAPID before creating the task to avoid orphaned rows
     if (payload.messageType === 'instant') {
       if (!ctx.vapid.email || !ctx.vapid.publicKey || !ctx.vapid.privateKey) {
@@ -1086,6 +1121,13 @@ function createScheduleMessageHandler(ctx) {
       return { status: 500, body: { success: false, error: { code: 'TASK_CREATE_FAILED', message: '创建任务失败' } } };
     }
 
+    /**
+     * @deprecated Soft-deprecated. For new code, use @rei-standard/amsg-instant.
+     *   The "create-task → process-by-uuid → delete-task" sequence below is
+     *   preserved verbatim so existing clients keep working. New integrations
+     *   should call the dedicated amsg-instant endpoint instead — it skips this
+     *   DB round-trip entirely. See packages/rei-standard-amsg/instant/README.md.
+     */
     // Instant type: send immediately
     if (payload.messageType === 'instant') {
       try {

package/dist/index.mjs

@@ -554,7 +554,7 @@ function buildAiRequestBody(payload) {
 function normalizeAiApiUrl(apiUrl) {
   if (typeof apiUrl !== "string" || !apiUrl.trim()) {
     throw new Error(
-      "Invalid apiUrl: apiUrl is required. Please provide a full chat endpoint URL (for example: https://api.openai.com/v1/chat/completions)."
+      "Invalid apiUrl: apiUrl is required. Please provide a chat endpoint URL (for example: https://api.openai.com or https://api.openai.com/v1/chat/completions)."
     );
   }
   const trimmedApiUrl = apiUrl.trim();
@@ -563,10 +563,17 @@ function normalizeAiApiUrl(apiUrl) {
     parsedUrl = new URL(trimmedApiUrl);
   } catch {
     throw new Error(
-      `Invalid apiUrl: "${apiUrl}". Please provide a valid absolute URL that points to a full chat endpoint.`
+      `Invalid apiUrl: "${apiUrl}". Please provide a valid absolute URL.`
     );
   }
-  parsedUrl.pathname = parsedUrl.pathname.replace(/\/+$/, "") || "/";
+  let path = parsedUrl.pathname.replace(/\/+$/, "") || "/";
+  if (/\/chat\/completions$/.test(path)) {
+  } else if (path === "/") {
+    path = "/v1/chat/completions";
+  } else if (/\/v\d+$/.test(path)) {
+    path = `${path}/chat/completions`;
+  }
+  parsedUrl.pathname = path;
   return parsedUrl.toString();
 }
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@rei-standard/amsg-server",
-  "version": "2.1.0",
+  "version": "2.1.1",
   "description": "ReiStandard Active Messaging server SDK with pluggable database adapters",
   "repository": {
     "type": "git",