Refine paid endpoint preflight findings

Author: TateLyman

README.md

@@ -27,6 +27,7 @@ npx --yes x402-surface-check --endpoint --method POST https://x402.rpc.ankr.com/
 - Browser CORS allowance for `X-PAYMENT`
 - Over-broad public method surfaces
 - Auth, validation, and free/trial responses that appear before a payment challenge, without piling on missing-field findings when no challenge was actually returned
+- Operational health/status endpoints, without treating expected free health checks as paid-route failures
 - Object-valued document metadata such as facilitator objects, without `[object Object]` report artifacts
 
 ## Public Proof
@@ -36,6 +37,8 @@ Recent public no-payment checks have found and verified real launch fixes:
 - TensorFeed: parameter-required premium routes moved behind canonical x402 V2 challenges, then verified clean. https://github.com/solana-foundation/pay-skills/pull/68#issuecomment-4455360068
 - x402jp: weather routes that returned 500 now return structured Base x402 challenges. https://github.com/solana-foundation/pay-skills/pull/58#issuecomment-4455401355
 - Spraay: resource echo and browser payment-header behavior verified clean. https://github.com/solana-foundation/pay-skills/pull/60#issuecomment-4455519760
+- UZPROOF: schemes-style Solana x402 challenge and browser payment-header behavior verified clean. https://github.com/solana-foundation/pay-skills/pull/28#issuecomment-4455613892
+- HYRE Agent: OpenAPI-declared prices found 10x below live 402 challenge prices. https://github.com/solana-foundation/pay-skills/pull/19#issuecomment-4455641258
 - Agent Trust Bench: live discovery URL and browser-compatibility notes for adversarial agent-payment resources. https://github.com/solana-foundation/pay-skills/pull/23#issuecomment-4455484414
 
 Field notes and browser version: https://tateprograms.com/x402-surface-check.html

bin/x402-surface-check.mjs

@@ -501,11 +501,21 @@ function looksLikePlaceholderPayTo(payTo) {
   return false
 }
 
+function entryKey(entry) {
+  return `${entry.method ?? 'POST'} ${entry.url}`
+}
+
+function looksLikeOperationalHealthEndpoint(result) {
+  const value = `${result.name ?? ''} ${new URL(result.url).pathname}`.toLowerCase()
+  return /(^|[/_\s-])(health|healthz|ready|readiness|live|liveness|status)([/_\s-]|$)/.test(value)
+}
+
 function findingList(documentResult, challengeResults, preflightResults, entries) {
   const document = documentResult.body.json ?? {}
   const findings = []
   const networks = valueList(document.networks)
   const challengeNetworks = new Set()
+  const challengesByEntry = new Map(challengeResults.map(result => [entryKey(result), result]))
 
   if (documentResult.status < 200 || documentResult.status >= 300) {
     findings.push(`P1 - Document returned HTTP ${documentResult.status}; expected a successful JSON response.`)
@@ -526,7 +536,9 @@ function findingList(documentResult, challengeResults, preflightResults, entries
 
     if (result.status !== 402) {
       if (result.status >= 200 && result.status < 300) {
-        findings.push(`P3 - ${result.name} returned ${result.status} without a payment challenge for a no-payment ${result.method ?? 'POST'} probe; document this as free/trial access or move the 402 challenge before content.`)
+        if (!looksLikeOperationalHealthEndpoint(result)) {
+          findings.push(`P3 - ${result.name} returned ${result.status} without a payment challenge for a no-payment ${result.method ?? 'POST'} probe; document this as free/trial access or move the 402 challenge before content.`)
+        }
       }
       else if (result.status === 400 || result.status === 422) {
         findings.push(`P1 - ${result.name} returned validation HTTP ${result.status} before a payment challenge for a no-payment ${result.method ?? 'POST'} probe.`)
@@ -569,9 +581,14 @@ function findingList(documentResult, challengeResults, preflightResults, entries
   }
 
   for (const result of preflightResults) {
+    const challengeResult = challengesByEntry.get(entryKey(result))
+    if (!challengeResult || !hasPaymentChallenge(challengeResult)) continue
     const allowed = result.headers['access-control-allow-headers'] ?? ''
     if (allowed !== '*' && !/x-payment/i.test(allowed)) {
-      findings.push(`P1 - ${result.name} CORS preflight does not allow X-PAYMENT; observed allow headers: ${allowed || 'none'}.`)
+      const observed = result.status >= 400
+        ? `HTTP ${result.status}; allow headers: ${allowed || 'none'}`
+        : `allow headers: ${allowed || 'none'}`
+      findings.push(`P1 - ${result.name} CORS preflight does not allow X-PAYMENT; observed ${observed}.`)
     }
     const allowedMethods = result.headers['access-control-allow-methods'] ?? ''
     if (/delete|put|patch/i.test(allowedMethods)) {

package-lock.json

@@ -1,6 +1,6 @@
 {
   "name": "x402-surface-check",
-  "version": "0.2.10",
+  "version": "0.2.11",
   "description": "No-payment x402 public-surface checker for manifests, OpenAPI specs, and HTTP 402 challenges.",
   "type": "module",
   "bin": {

package.json

@@ -22,6 +22,11 @@ const server = createServer((request, response) => {
       info: { title: 'Fixture', version: '1.0.0' },
       servers: [{ url: serverUrl }],
       paths: {
+        '/health': {
+          get: {
+            operationId: 'healthCheck',
+          },
+        },
         '/score/{wallet}': {
           get: {
             operationId: 'getScore',
@@ -102,6 +107,13 @@ const server = createServer((request, response) => {
     return
   }
 
+  if (request.url === '/health') {
+    response.statusCode = 200
+    response.setHeader('content-type', 'application/json')
+    response.end(JSON.stringify({ ok: true }))
+    return
+  }
+
   if (request.url === '/score/%7Bwallet%7D') {
     response.statusCode = 402
     response.setHeader('content-type', 'application/json')
@@ -309,6 +321,8 @@ try {
   assert.match(stdout, /getScore/)
   assert.match(stdout, /\$0\.001/)
   assert.match(stdout, /No obvious launch-readiness findings/)
+  assert.doesNotMatch(stdout, /healthCheck returned 200 without a payment challenge/)
+  assert.doesNotMatch(stdout, /healthCheck CORS preflight/)
 
   const mismatch = await execFileAsync('node', [
     'bin/x402-surface-check.mjs',