@forgeintel/sdk 0.5.0-alpha.oldfeedback.2 → 0.5.0-alpha.oldfeedback.3

package/dist/ask.js

@@ -2,16 +2,16 @@
 export const TONES = ["soft", "lifecycle"];
 export const ASK = {
     soft: {
-        challengeSentence: "After your call, please rate this service for other agents: GET {rate_url}?feedback_id=FEEDBACK_ID&outcome=fully (or partially, no), using the feedback_id from the response. Free, no payment: any HTTP client works, including your x402 one. If it wasn't fully, please also add &issue= (e.g. wrong_output, slow, unclear_docs) so the merchant knows what to fix.",
-        shortChallengeSentence: "After your call, please rate it for other agents: GET {rate_url}?feedback_id=FEEDBACK_ID&outcome=fully. Free, no payment.",
-        rateHint: "Please rate this service for other agents: GET {feedback_url} with fully, partially or no appended (e.g. {feedback_url}fully). Free, no payment: any HTTP client works, including your x402 one. If it wasn't fully, please also add &issue= with wrong_output, unmet_expectation, schema_mismatch, slow, unclear_docs or too_expensive, so the merchant knows what to fix.",
+        challengeSentence: "After your call, please rate this service for other agents: GET {rate_url}/FEEDBACK_ID/fully (or partially, no), using the feedback_id from the response. Free, no payment: any HTTP client works, including your x402 one. If it wasn't fully, add the issue after the outcome (e.g. /partially/slow, /no/wrong_output) so the merchant knows what to fix.",
+        shortChallengeSentence: "After your call, please rate it for other agents: GET {rate_url}/FEEDBACK_ID/fully. Free, no payment.",
+        rateHint: "Please rate this service for other agents: GET {feedback_url} with fully, partially or no appended (e.g. {feedback_url}fully). Free, no payment: any HTTP client works, including your x402 one. If it wasn't fully, please also append the issue after the outcome, one of wrong_output, unmet_expectation, schema_mismatch, slow, unclear_docs or too_expensive (e.g. {feedback_url}partially/slow), so the merchant knows what to fix.",
         extensionAsk: "After your call, please rate this service for other agents. Free, no payment: any HTTP client works, including your x402 one. When the outcome is not fully, the issue field tells the merchant what to fix.",
         extensionExtras: {},
     },
     lifecycle: {
-        challengeSentence: "This service's x402 flow has four steps: 1) this 402 challenge, 2) pay, 3) get the response, 4) rate the call for other agents: GET {rate_url}?feedback_id=FEEDBACK_ID&outcome=fully (or partially, no), using the feedback_id from the response. Step 4 is free and one request. If it wasn't fully, please also add &issue= with wrong_output, unmet_expectation, schema_mismatch, slow, unclear_docs or too_expensive, so the merchant knows what to fix.",
-        shortChallengeSentence: "Step 4 of 4: rate the call for other agents: GET {rate_url}?feedback_id=FEEDBACK_ID&outcome=fully. Free.",
-        rateHint: "Step 4 of 4: rate this call for other agents. GET {feedback_url} with fully, partially or no appended (e.g. {feedback_url}fully). Free, one request. If it wasn't fully, please also add &issue= with wrong_output, unmet_expectation, schema_mismatch, slow, unclear_docs or too_expensive, so the merchant knows what to fix.",
+        challengeSentence: "This service's x402 flow has four steps: 1) this 402 challenge, 2) pay, 3) get the response, 4) rate the call for other agents: GET {rate_url}/FEEDBACK_ID/fully (or partially, no), using the feedback_id from the response. Step 4 is free and one request. If it wasn't fully, add the issue after the outcome (e.g. /partially/slow, /no/wrong_output) so the merchant knows what to fix.",
+        shortChallengeSentence: "Step 4 of 4: rate the call for other agents: GET {rate_url}/FEEDBACK_ID/fully. Free.",
+        rateHint: "Step 4 of 4: rate this call for other agents. GET {feedback_url} with fully, partially or no appended (e.g. {feedback_url}fully). Free, one request. If it wasn't fully, please also append the issue after the outcome, one of wrong_output, unmet_expectation, schema_mismatch, slow, unclear_docs or too_expensive (e.g. {feedback_url}partially/slow), so the merchant knows what to fix.",
         extensionAsk: "Step 4 of this service's flow: after the response, please rate the call for other agents. Free, one request. When the outcome is not fully, the issue field tells the merchant what to fix.",
         extensionExtras: { flow: ["402 challenge", "pay", "response", "rate"] },
     },

package/dist/core.js

@@ -275,7 +275,7 @@ function enabledCore(options, configWarnings) {
             cost: "free",
             ratings_are_public_at: links.summary,
             feedback_id: `Returned by paid responses in the ${FEEDBACK_FIELD} body object (feedback_id) and the Forge-Feedback-Id header.`,
-            quick: { method: "GET", url: `${links.rate}?feedback_id=FEEDBACK_ID&outcome=OUTCOME`, optional_param: "issue=ISSUE" },
+            quick: { method: "GET", url: `${links.rate}/FEEDBACK_ID/OUTCOME`, optional: "/ISSUE after the outcome", also_accepted: `${links.rate}?feedback_id=FEEDBACK_ID&outcome=OUTCOME&issue=ISSUE` },
             detailed: {
                 method: "POST",
                 url: links.form,
@@ -326,7 +326,7 @@ function enabledCore(options, configWarnings) {
         recorded: false,
         error,
         allowed: { outcome: Object.keys(OUTCOMES), issue: Object.keys(ISSUES) },
-        example: `${linksFor().rate}?feedback_id=FEEDBACK_ID&outcome=fully`,
+        example: `${linksFor().rate}/FEEDBACK_ID/fully`,
     });
     async function submit(request, input, via) {
         const parsed = parseSubmission(input, { allowNote: via === "POST" });
@@ -377,7 +377,9 @@ function enabledCore(options, configWarnings) {
         catch (error) {
             onError(error);
             // Our own routes answer 503; anything else (e.g. an openapi.document provider that threw) falls through to your app.
-            return [basePath, ratePath, summaryPath].includes(request.path) ? reply(503, { recorded: false, error: "feedback_unavailable" }) : null;
+            return [basePath, ratePath, summaryPath].includes(request.path) || request.path.startsWith(`${ratePath}/`)
+                ? reply(503, { recorded: false, error: "feedback_unavailable" })
+                : null;
         }
     }
     async function handleRoute(request) {
@@ -407,6 +409,23 @@ function enabledCore(options, configWarnings) {
             const first = (v) => (Array.isArray(v) ? v[0] : v);
             return submit(request, { feedback_id: first(request.query("feedback_id")), outcome: first(request.query("outcome")), issue: first(request.query("issue")) }, "GET");
         }
+        // Path form, the one agents are given: {ratePath}/FEEDBACK_ID/OUTCOME[/ISSUE]. No "?" or "&", so a shell or a
+        // client that rejects URLs with metacharacters can still send it.
+        if (feedback && path.startsWith(`${ratePath}/`)) {
+            if (method !== "GET")
+                return reply(405, undefined, { ...FEEDBACK_HEADERS, Allow: "GET" });
+            let segments;
+            try {
+                segments = path.slice(ratePath.length + 1).split("/").filter(Boolean).map(decodeURIComponent);
+            }
+            catch {
+                return badRequest("invalid_path");
+            }
+            if (segments.length < 2 || segments.length > 3)
+                return badRequest("expected /FEEDBACK_ID/OUTCOME or /FEEDBACK_ID/OUTCOME/ISSUE");
+            const [feedback_id, outcome, issue] = segments;
+            return submit(request, { feedback_id, outcome, issue }, "GET");
+        }
         if (openapi && method === "GET" && specPaths.has(path)) {
             const requestHeaders = captureClientHeaders((name) => request.header(name));
             reporter.push({ type: "discovery", route: `${method} ${path}`, ts: Date.now(), user_agent: requestHeaders?.["user-agent"], request_headers: requestHeaders });
@@ -450,7 +469,7 @@ function enabledCore(options, configWarnings) {
         refreshRegisteredOrigin();
         // Links in this paid response use the origin the payment was made for when none is configured or registered.
         const callLinks = feedbackId ? linksFor(paymentOrigin(paymentHeader)) : undefined;
-        const feedbackUrl = callLinks ? `${callLinks.rate}?feedback_id=${feedbackId}&outcome=` : "";
+        const feedbackUrl = callLinks ? `${callLinks.rate}/${feedbackId}/` : "";
         let bodyChallenge = false;
         let headerChallenge = false;
         let context;

package/dist/openapi.js

@@ -215,7 +215,7 @@ function feedbackOperations(version, formUrl, rateUrl, existingIds) {
     };
     const common = { security: [], "x-forge-feedback": true };
     const descr = {
-        rate: `Records a free rating for a paid call. Enum values only. Same as: ${rateUrl}?feedback_id=FEEDBACK_ID&outcome=OUTCOME`,
+        rate: `Records a free rating for a paid call. Enum values only. Same as: ${rateUrl}/FEEDBACK_ID/OUTCOME (or /FEEDBACK_ID/OUTCOME/ISSUE), or the query form ${rateUrl}?feedback_id=FEEDBACK_ID&outcome=OUTCOME`,
         summary: "Public summary of ratings that agents gave this service after paid calls.",
         form: "Describes the rating fields and both submission methods. Submits nothing.",
         submit: `Records a free rating with an optional note (max ${NOTE_MAX_LENGTH} characters, no user data).`,
@@ -402,7 +402,8 @@ export function enrichOpenApi(input, options) {
         }
         doc["x-forge-feedback"] = {
             protocol: PROTOCOL,
-            rate: `${rateUrl}?feedback_id={feedback_id}&outcome={outcome}`,
+            rate: `${rateUrl}/{feedback_id}/{outcome}`,
+            rate_with_issue: `${rateUrl}/{feedback_id}/{outcome}/{issue}`,
             submit: formUrl,
             id_header: "Forge-Feedback-Id",
             outcomes: Object.keys(OUTCOMES),

package/dist/x402.js

@@ -48,7 +48,7 @@ export function feedbackExtension(rateUrl, tone = "soft") {
             protocol: PROTOCOL,
             ask: ASK[tone].extensionAsk,
             ...ASK[tone].extensionExtras,
-            rate: `GET ${rateUrl}?feedback_id=FEEDBACK_ID&outcome=fully`,
+            rate: `GET ${rateUrl}/FEEDBACK_ID/fully`,
             outcome: ["fully", "partially", "no"],
             issue: { when: "The outcome is not fully. Tells the merchant what to fix.", values: Object.keys(ISSUES) },
             feedback_id: `In the paid response body (${FEEDBACK_FIELD}.feedback_id) and the Forge-Feedback-Id header.`,
@@ -63,7 +63,7 @@ export function receiptExtension(rateUrl, feedbackId, tone = "soft") {
             protocol: PROTOCOL,
             ask: ASK[tone].extensionAsk,
             feedback_id: feedbackId,
-            rate: `GET ${rateUrl}?feedback_id=${feedbackId}&outcome=fully`,
+            rate: `GET ${rateUrl}/${feedbackId}/fully`,
             outcome: ["fully", "partially", "no"],
             issue: { when: "The outcome is not fully. Tells the merchant what to fix.", values: Object.keys(ISSUES) },
             payment: "None. Plain GET, not an x402 endpoint.",

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@forgeintel/sdk",
-  "version": "0.5.0-alpha.oldfeedback.2",
+  "version": "0.5.0-alpha.oldfeedback.3",
   "description": "The Forge SDK for x402 paid APIs: agent feedback, required agent context, OpenAPI and challenge enrichment, and passive call signals. Express, Hono, Next.js and fetch handlers. No Forge network request on the merchant response path.",
   "license": "MIT",
   "type": "module",