@reclaimprotocol/agent 0.1.6 → 0.1.7

package/lib/old/client.d.ts

@@ -44,6 +44,10 @@ export interface RegisterProviderBody {
     };
     /** Accepted at register too. */
     pageTitle?: string;
+    /** Accepted at register too — verified straight from the register
+     *  controller's destructure + its `providerConfig` write, both of which
+     *  include it alongside `pageTitle`/`userAgent`. */
+    stepsToFollow?: string;
 }
 export interface UpdateProviderConfigBody {
     providerConfig: Record<string, unknown>;

package/lib/old/to-register.d.ts

@@ -41,8 +41,8 @@ export interface OldProviderOpts {
      *  fresh entries with {@link injectedRequestDataFrom}; a carried-forward
      *  value (read back from an existing version) passes through unchanged. */
     allowedInjectedRequestData?: OldInjectedRequestData[];
-    /** Config (add-version) ONLY — the register endpoint never reads this
-     *  field from the request body. */
+    /** Accepted at register too (verified from the register controller's
+     *  destructure + providerConfig write — same as `userAgent`/`pageTitle`). */
     stepsToFollow?: string;
     /** Config (add-version) ONLY — the register endpoint never reads this
      *  field from the request body. */

package/lib/old/to-register.js

@@ -114,15 +114,18 @@ export function providerToRegisterBody(providers, opts) {
     if (opts.customInjection !== undefined) {
         body.customInjection = opts.customInjection;
     }
-    // Register also accepts these two (unlike allowedInjectedRequestData /
-    // stepsToFollow / useIncognitoWebview / extensionConfig, which the
-    // register controller never reads from the body at all).
+    // Register also accepts these three (unlike allowedInjectedRequestData /
+    // useIncognitoWebview / extensionConfig, which the register controller
+    // never reads from the body at all).
     if (opts.userAgent !== undefined) {
         body.userAgent = opts.userAgent;
     }
     if (opts.pageTitle !== undefined) {
         body.pageTitle = opts.pageTitle;
     }
+    if (opts.stepsToFollow !== undefined) {
+        body.stepsToFollow = opts.stepsToFollow;
+    }
     return body;
 }
 /** Body for `POST /api/providers/:providerId/config` — adds a new version to

package/lib/old/tools/publish.js

@@ -7,9 +7,9 @@ const DEFAULT_VERSION_INFO = 'Updated from MCP browser capture';
 /** Fields with no partial-patch equivalent on this backend: a version always
  *  replaces the WHOLE `providerConfig`, so omitting one of these on
  *  add-version must carry forward the current value rather than publish it
- *  blank. `allowedInjectedRequestData` is handled alongside these (see
- *  `resolveInjectedRequests`) since it needs translation when freshly
- *  authored, unlike these — which pass through unchanged either way. */
+ *  blank. `allowedInjectedRequestData` is handled alongside these (further
+ *  down, inline) since it needs translation when freshly authored, unlike
+ *  these — which pass through unchanged either way. */
 const PRESERVABLE_KEYS = [
     'customInjection',
     'userAgent',
@@ -30,12 +30,22 @@ function extractProviderConfig(resp) {
     return undefined;
 }
 /** Prefer the caller's explicit value; otherwise fall back to `current`'s
- *  value (if any) and record `fieldName` as carried forward. */
+ *  value (if any) and record `fieldName` as carried forward.
+ *
+ *  `current === null` is treated the SAME as `undefined` (nothing to carry),
+ *  not as a real value to preserve. These are all optional fields with no
+ *  Prisma default — a version where one was never set reads back as
+ *  `null`, not a missing key, because Prisma normalizes an absent optional
+ *  field to `null` on read. Carrying that `null` "forward" would falsely
+ *  claim the field was preserved when really nothing was ever there — it
+ *  most likely means an EARLIER version already lost it (e.g. published
+ *  before this field had carry-forward support at all), not that this call
+ *  is doing anything wrong. */
 function resolveField(explicit, current, fieldName, carried) {
     if (explicit !== undefined) {
         return explicit;
     }
-    if (current !== undefined) {
+    if (current !== undefined && current !== null) {
         carried.push(fieldName);
         return current;
     }
@@ -108,10 +118,16 @@ export function createVersionFromCaptureTool(client) {
             '`useIncognitoWebview`, `extensionConfig` that you OMIT is carried ' +
             'forward unchanged from the currently-published version (so ' +
             're-publishing requestData never wipes a working login script or ' +
-            'other config someone set via the dashboard). Pass an explicit ' +
-            'value (`""`, `[]`, `false`, ...) to intentionally clear one of ' +
-            'them instead, or call get_provider_version_config first to see ' +
-            'exactly what is live today before deciding what to change. ' +
+            'other config someone set via the dashboard). This can only carry ' +
+            'forward what the CURRENT version actually has — if an earlier ' +
+            'version already lost a field (e.g. published before this tool ' +
+            'carried it forward at all), there is nothing left to preserve and ' +
+            'the result legitimately comes back empty/null for that field; ' +
+            'that is not this call failing to carry it forward. Pass an ' +
+            'explicit value (`""`, `[]`, `false`, ...) to intentionally clear ' +
+            'one of them instead, or call get_provider_version_config FIRST ' +
+            'to see exactly what is live today, rather than assuming this ' +
+            'call recovered something that may already be gone. ' +
             '`allowedInjectedRequestData`/`useIncognitoWebview`/' +
             '`extensionConfig` only take effect on add-version — the register ' +
             'endpoint (no `providerId`) ignores them. \n\n' +
@@ -241,7 +257,7 @@ export function createVersionFromCaptureTool(client) {
                 stepsToFollow: {
                     type: 'string',
                     description: 'Free-text setup/manual-verification steps shown to the ' +
-                        'dev/verifier. Add-version ONLY.',
+                        'dev/verifier. Accepted on register too.',
                 },
                 useIncognitoWebview: {
                     type: 'boolean',
@@ -320,6 +336,9 @@ export function createVersionFromCaptureTool(client) {
             if (args.pageTitle !== undefined) {
                 opts.pageTitle = args.pageTitle;
             }
+            if (args.stepsToFollow !== undefined) {
+                opts.stepsToFollow = args.stepsToFollow;
+            }
             const body = providerToRegisterBody(providers, opts);
             return withNotes(await client.registerProvider(body));
         }

package/package.json

@@ -1,7 +1,7 @@
 {
 	"name": "@reclaimprotocol/agent",
 	"title": "Reclaim Agent",
-	"version": "0.1.6",
+	"version": "0.1.7",
 	"description": "Reclaim Protocol agent — SDK, CLI, and MCP server in one package. SDK + CLI talk to the builder backend; MCP server adds Chrome CDP capture, provider synthesis, replay, and zkTLS proof.",
 	"type": "module",
 	"main": "./lib/index.js",

package/src/old/client.ts

@@ -43,6 +43,10 @@ export interface RegisterProviderBody {
 	userAgent?: { ios?: string, android?: string }
 	/** Accepted at register too. */
 	pageTitle?: string
+	/** Accepted at register too — verified straight from the register
+	 *  controller's destructure + its `providerConfig` write, both of which
+	 *  include it alongside `pageTitle`/`userAgent`. */
+	stepsToFollow?: string
 }
 
 export interface UpdateProviderConfigBody {

package/src/old/to-register.ts

@@ -50,8 +50,8 @@ export interface OldProviderOpts {
 	 *  fresh entries with {@link injectedRequestDataFrom}; a carried-forward
 	 *  value (read back from an existing version) passes through unchanged. */
 	allowedInjectedRequestData?: OldInjectedRequestData[]
-	/** Config (add-version) ONLY — the register endpoint never reads this
-	 *  field from the request body. */
+	/** Accepted at register too (verified from the register controller's
+	 *  destructure + providerConfig write — same as `userAgent`/`pageTitle`). */
 	stepsToFollow?: string
 	/** Config (add-version) ONLY — the register endpoint never reads this
 	 *  field from the request body. */
@@ -231,9 +231,9 @@ export function providerToRegisterBody(
 		body.customInjection = opts.customInjection
 	}
 
-	// Register also accepts these two (unlike allowedInjectedRequestData /
-	// stepsToFollow / useIncognitoWebview / extensionConfig, which the
-	// register controller never reads from the body at all).
+	// Register also accepts these three (unlike allowedInjectedRequestData /
+	// useIncognitoWebview / extensionConfig, which the register controller
+	// never reads from the body at all).
 	if(opts.userAgent !== undefined) {
 		body.userAgent = opts.userAgent
 	}
@@ -242,6 +242,10 @@ export function providerToRegisterBody(
 		body.pageTitle = opts.pageTitle
 	}
 
+	if(opts.stepsToFollow !== undefined) {
+		body.stepsToFollow = opts.stepsToFollow
+	}
+
 	return body
 }
 

package/src/old/tools/publish.ts

@@ -23,9 +23,9 @@ const DEFAULT_VERSION_INFO = 'Updated from MCP browser capture'
 /** Fields with no partial-patch equivalent on this backend: a version always
  *  replaces the WHOLE `providerConfig`, so omitting one of these on
  *  add-version must carry forward the current value rather than publish it
- *  blank. `allowedInjectedRequestData` is handled alongside these (see
- *  `resolveInjectedRequests`) since it needs translation when freshly
- *  authored, unlike these — which pass through unchanged either way. */
+ *  blank. `allowedInjectedRequestData` is handled alongside these (further
+ *  down, inline) since it needs translation when freshly authored, unlike
+ *  these — which pass through unchanged either way. */
 const PRESERVABLE_KEYS = [
 	'customInjection',
 	'userAgent',
@@ -51,7 +51,17 @@ function extractProviderConfig(
 }
 
 /** Prefer the caller's explicit value; otherwise fall back to `current`'s
- *  value (if any) and record `fieldName` as carried forward. */
+ *  value (if any) and record `fieldName` as carried forward.
+ *
+ *  `current === null` is treated the SAME as `undefined` (nothing to carry),
+ *  not as a real value to preserve. These are all optional fields with no
+ *  Prisma default — a version where one was never set reads back as
+ *  `null`, not a missing key, because Prisma normalizes an absent optional
+ *  field to `null` on read. Carrying that `null` "forward" would falsely
+ *  claim the field was preserved when really nothing was ever there — it
+ *  most likely means an EARLIER version already lost it (e.g. published
+ *  before this field had carry-forward support at all), not that this call
+ *  is doing anything wrong. */
 function resolveField<T>(
 	explicit: T | undefined,
 	current: unknown,
@@ -62,7 +72,7 @@ function resolveField<T>(
 		return explicit
 	}
 
-	if(current !== undefined) {
+	if(current !== undefined && current !== null) {
 		carried.push(fieldName)
 		return current as T
 	}
@@ -163,10 +173,16 @@ export function createVersionFromCaptureTool(
 				'`useIncognitoWebview`, `extensionConfig` that you OMIT is carried ' +
 				'forward unchanged from the currently-published version (so ' +
 				're-publishing requestData never wipes a working login script or ' +
-				'other config someone set via the dashboard). Pass an explicit ' +
-				'value (`""`, `[]`, `false`, ...) to intentionally clear one of ' +
-				'them instead, or call get_provider_version_config first to see ' +
-				'exactly what is live today before deciding what to change. ' +
+				'other config someone set via the dashboard). This can only carry ' +
+				'forward what the CURRENT version actually has — if an earlier ' +
+				'version already lost a field (e.g. published before this tool ' +
+				'carried it forward at all), there is nothing left to preserve and ' +
+				'the result legitimately comes back empty/null for that field; ' +
+				'that is not this call failing to carry it forward. Pass an ' +
+				'explicit value (`""`, `[]`, `false`, ...) to intentionally clear ' +
+				'one of them instead, or call get_provider_version_config FIRST ' +
+				'to see exactly what is live today, rather than assuming this ' +
+				'call recovered something that may already be gone. ' +
 				'`allowedInjectedRequestData`/`useIncognitoWebview`/' +
 				'`extensionConfig` only take effect on add-version — the register ' +
 				'endpoint (no `providerId`) ignores them. \n\n' +
@@ -308,7 +324,7 @@ export function createVersionFromCaptureTool(
 						type: 'string',
 						description:
 							'Free-text setup/manual-verification steps shown to the ' +
-							'dev/verifier. Add-version ONLY.',
+							'dev/verifier. Accepted on register too.',
 					},
 					useIncognitoWebview: {
 						type: 'boolean',
@@ -415,6 +431,10 @@ export function createVersionFromCaptureTool(
 					opts.pageTitle = args.pageTitle
 				}
 
+				if(args.stepsToFollow !== undefined) {
+					opts.stepsToFollow = args.stepsToFollow
+				}
+
 				const body = providerToRegisterBody(providers, opts)
 				return withNotes(await client.registerProvider(body))
 			}