@flue/sdk 0.3.4 → 0.3.5

package/dist/index.mjs

@@ -399,7 +399,6 @@ var CloudflarePlugin = class {
 		return `
 // Auto-generated by @flue/sdk build (cloudflare)
 import { Agent, routeAgentRequest } from 'agents';
-import { DurableObject } from 'cloudflare:workers';
 import { Bash, InMemoryFs } from 'just-bash';
 import {
   createFlueContext,
@@ -456,24 +455,34 @@ async function createLocalEnv() {
  * RPC stub, null otherwise.
  *
  * NOTE on detection: The value returned by \`getSandbox()\` is a workerd RPC
- * Proxy. \`in\` and \`typeof\` against it return \`true\`/\`'function'\` for any
- * property name, so structural duck-typing is unreliable.
+ * Proxy. None of the obvious detection strategies work:
  *
- * \`instanceof <UserSandboxClass>\` ALSO does not work: the RPC stub's
- * prototype chain is workerd's internal \`DurableObject\` runtime class, not
- * the user-defined \`Sandbox\` class (the user's class only exists on the
- * in-DO side; the caller side gets a generic stub). Empirically:
- *   typeof stub === 'object'
- *   Object.getPrototypeOf(stub).constructor.name === 'DurableObject'
+ *   - Structural duck-typing (\`'X' in stub\`, \`typeof stub.X === 'function'\`):
+ *     the proxy lies positively for any property name, so any check returns
+ *     \`true\` regardless of what's actually on the remote.
+ *   - \`instanceof <UserSandboxClass>\` (e.g. \`Sandbox\` from
+ *     \`@cloudflare/sandbox\`): the user's class only exists on the in-DO
+ *     side; over RPC the caller gets a generic stub.
+ *   - \`instanceof DurableObject\` (imported from \`cloudflare:workers\`): the
+ *     stub's prototype chain has a class *named* \`DurableObject\`, but it's a
+ *     workerd-internal class with a different identity than the importable
+ *     one. \`instanceof\` checks identity, not name, so it returns \`false\`.
  *
- * \`instanceof DurableObject\` (imported from \`cloudflare:workers\`) is the
- * one signal that holds: it walks the prototype chain via the runtime and
- * matches any DO RPC stub. We treat any DO stub passed to \`init({ sandbox })\`
- * as intended for \`@cloudflare/sandbox\`, since that's the only documented
- * use case for that argument shape on the Cloudflare target.
+ * The one signal that does work — verified by runtime probe — is the string
+ * name of the prototype's constructor. Workerd's internal RPC stub class is
+ * named \`DurableObject\`, and \`Object.getPrototypeOf(stub).constructor.name\`
+ * returns that string. This is a heuristic (it relies on a workerd-internal
+ * naming convention, not a contractual API), but it's empirically correct
+ * today and will misroute only if a user passes some other DO stub to
+ * \`init({ sandbox })\` — in which case \`cfSandboxToSessionEnv\` will fail
+ * loudly on first method call.
  */
 function resolveSandbox(sandbox) {
-  if (sandbox instanceof DurableObject) {
+  if (
+    sandbox &&
+    typeof sandbox === 'object' &&
+    Object.getPrototypeOf(sandbox)?.constructor?.name === 'DurableObject'
+  ) {
     return cfSandboxToSessionEnv(sandbox);
   }
   return null;

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@flue/sdk",
-  "version": "0.3.4",
+  "version": "0.3.5",
   "type": "module",
   "license": "Apache-2.0",
   "exports": {