@modelprofile.com/browser-runtime 4.3.0 → 5.0.0

package/ts/classes.runtimeownership.ts

@@ -40,6 +40,7 @@ export interface IBrowserRuntimeOwnershipTestingOptions {
   now?(): number;
   beforeMetadataPublication?(): Promise<void>;
   afterLegacyLockProbe?(): Promise<void>;
+  afterEmptyProcessCmdline?(pid: number): Promise<void>;
   beforeGenerationRemoval?(reason: TOwnershipGenerationRemovalReason): Promise<void>;
 }
 
@@ -118,6 +119,7 @@ export class BrowserRuntimeOwnership {
   private readonly now: () => number;
   private readonly beforeMetadataPublication?: () => Promise<void>;
   private readonly afterLegacyLockProbe?: () => Promise<void>;
+  private readonly afterEmptyProcessCmdline?: (pid: number) => Promise<void>;
   private readonly beforeGenerationRemoval?: (
     reason: TOwnershipGenerationRemovalReason,
   ) => Promise<void>;
@@ -150,6 +152,7 @@ export class BrowserRuntimeOwnership {
     this.now = optionsArg.testing?.now ?? Date.now;
     this.beforeMetadataPublication = optionsArg.testing?.beforeMetadataPublication;
     this.afterLegacyLockProbe = optionsArg.testing?.afterLegacyLockProbe;
+    this.afterEmptyProcessCmdline = optionsArg.testing?.afterEmptyProcessCmdline;
     this.beforeGenerationRemoval = optionsArg.testing?.beforeGenerationRemoval;
     this.onOwnershipLost = optionsArg.onOwnershipLost;
   }
@@ -1098,7 +1101,8 @@ export class BrowserRuntimeOwnership {
         continue;
       }
       if (cmdline.byteLength === 0) {
-        if (!await this.processIsZombie(pid)) indeterminate = true;
+        await this.afterEmptyProcessCmdline?.(pid);
+        if (!await this.emptyCommandProcessIsClear(pid)) indeterminate = true;
         continue;
       }
       let cmdlineText: string;
@@ -1204,15 +1208,28 @@ export class BrowserRuntimeOwnership {
     return 'indeterminate';
   }
 
-  private async processIsZombie(pidArg: number): Promise<boolean> {
+  private async emptyCommandProcessIsClear(pidArg: number): Promise<boolean> {
+    const processPath = plugins.path.join(this.procRoot, String(pidArg));
     try {
       const status = await this.readBoundedFile(
-        plugins.path.join(this.procRoot, String(pidArg), 'status'),
+        plugins.path.join(processPath, 'status'),
         64 * 1024,
       );
-      return /^State:\s+Z(?:\s|$)/mu.test(status);
-    } catch {
-      return false;
+      const uidState = this.parseProcessUidState(status);
+      if (uidState === 'other') return true;
+      if (uidState !== 'same') return false;
+      const stateLines = status.split('\n').filter((line) => line.startsWith('State:'));
+      return stateLines.length === 1 && /^State:[\t ]+Z(?:[\t ]|$)/u.test(stateLines[0]!);
+    } catch (error) {
+      if (!isMissingError(error)) return false;
+      // An empty command line can precede process exit. A missing status file
+      // alone is insufficient: the PID directory must also have disappeared.
+      try {
+        await plugins.fsPromises.lstat(processPath, { bigint: true });
+        return false;
+      } catch (statError) {
+        return isMissingError(statError);
+      }
     }
   }
 

package/ts/index.ts

@@ -10,7 +10,10 @@ export { BrowserRuntime, BrowserRuntimeLease } from './classes.runtime.js';
 export { BrowserRuntimeError } from './errors.js';
 export { createBrowserRuntimeMcpHttpHandler } from './mcp.js';
 
-export type { TAcquireBrowserRuntimeLeaseOptions } from './classes.runtime.js';
+export type {
+  TAcquireBrowserRuntimeLeaseOptions,
+  IBrowserRuntimeLeaseRenewalOptions,
+} from './classes.runtime.js';
 export type { TBrowserRuntimeErrorCode } from './errors.js';
 export type {
   IAttachTrustedFramedPeerOptions,
@@ -48,6 +51,7 @@ export type {
   IBrowserResourceKey,
   IBrowserResourceRegistration,
   IBrowserRuntimeState,
+  IBrowserRuntimeViewportResult,
   IBrowserRuntimeTabState,
   IBrowserScreenshotAction,
   IBrowserScreenshotResult,

package/ts/interfaces.ts

@@ -83,7 +83,7 @@ export interface IBrowserFlexCapabilityBinding extends IBrowserCapabilityBinding
 export interface IBrowserMcpCapabilityBinding extends IBrowserCapabilityBindingBase {
   role: 'agent';
   source: 'mcp';
-  sessionId: IBrowserOpenCodeSessionId;
+  sessionId: TQualifiedBrowserSessionId;
   scopeId?: never;
   channelId?: never;
   runId?: never;
@@ -118,7 +118,7 @@ type TReadonlyBrowserCapabilityAuthorizationRequest =
       readonly sessionId: Readonly<IBrowserFlexSessionId>;
     })
   | (Omit<Readonly<IBrowserMcpCapabilityBinding>, 'sessionId'> & {
-      readonly sessionId: Readonly<IBrowserOpenCodeSessionId>;
+      readonly sessionId: Readonly<TQualifiedBrowserSessionId>;
     });
 
 export type TBrowserRuntimeOperationIdentity = TReadonlyBrowserCapabilityAuthorizationRequest & {
@@ -138,6 +138,12 @@ export type TBrowserRuntimeOperationClassification =
   | 'tab'
   | 'agent-action';
 
+/** The viewport accepted by the shared resource, which may be smaller than one viewer's request. */
+export interface IBrowserRuntimeViewportResult {
+  viewport: plugins.smartpuppeteer.ILiveBrowserViewport;
+  viewportRevision: number;
+}
+
 export type TBrowserRuntimeLeaseAuthority = TReadonlyBrowserCapabilityAuthorizationRequest & {
   readonly runtimeAuthorityId: string;
   readonly authorityGeneration: number;
@@ -503,7 +509,7 @@ extends Omit<
 export interface IBrowserMcpAuthenticatedBinding extends IBrowserCapabilityBindingBase {
   role: 'agent';
   source: 'mcp';
-  sessionId: IBrowserOpenCodeSessionId;
+  sessionId: TQualifiedBrowserSessionId;
   scopeId?: never;
   channelId?: never;
 }

package/ts/mcp.ts

@@ -231,7 +231,10 @@ const validateMcpBinding = (value: unknown): IBrowserMcpAuthenticatedBinding =>
     throw new BrowserRuntimeError('CAPABILITY_INVALID');
   }
   const session = binding.sessionId as Record<string, unknown> | undefined;
-  if (session?.harnessId !== 'opencode') throw new BrowserRuntimeError('CAPABILITY_INVALID');
+  if (session?.harnessId !== 'opencode' && session?.harnessId !== 'codex'
+    && session?.harnessId !== 'flex') {
+    throw new BrowserRuntimeError('CAPABILITY_INVALID');
+  }
   return {
     projectId: validateBoundedString(binding.projectId, 'projectId', 1, 256),
     browserResourceId: validateBoundedString(
@@ -257,7 +260,7 @@ const validateMcpBinding = (value: unknown): IBrowserMcpAuthenticatedBinding =>
     peerId: validateBoundedString(binding.peerId, 'peerId', 1, 256),
     source: 'mcp',
     sessionId: {
-      harnessId: 'opencode',
+      harnessId: session.harnessId,
       nativeId: validateBoundedString(session.nativeId, 'sessionId.nativeId', 1, 256),
     },
   };