@flighthq/shell 0.5.1-next.820.97b3830 → 0.5.1-next.903.f434bc2

package/README.md

@@ -13,5 +13,5 @@ Import the supported application-facing API from `@flighthq/shell`. The `@flight
 This package is part of the locked-version Flight SDK graph. Applications may instead install and import `@flighthq/sdk` when package-level tree shaking is sufficient.
 
 - [Flight project](https://github.com/flighthq/flight)
-- [Source for @flighthq/shell@0.5.1-next.820.97b3830](https://github.com/flighthq/flight/tree/97b3830e843209d92da169e02a52c0574efe5c09/packages/shell)
-- [License](https://github.com/flighthq/flight/blob/97b3830e843209d92da169e02a52c0574efe5c09/LICENSE.md)
+- [Source for @flighthq/shell@0.5.1-next.903.f434bc2](https://github.com/flighthq/flight/tree/f434bc233778d75aeea61995df3a8cfaec459a85/packages/shell)
+- [License](https://github.com/flighthq/flight/blob/f434bc233778d75aeea61995df3a8cfaec459a85/LICENSE.md)

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@flighthq/shell",
-  "version": "0.5.1-next.820.97b3830",
+  "version": "0.5.1-next.903.f434bc2",
   "author": "Joshua Granick and other contributors",
   "license": "MIT",
   "repository": {
@@ -38,7 +38,7 @@
     "clean:dist": "tsx ../../scripts/clean-package-dist.ts"
   },
   "dependencies": {
-    "@flighthq/types": "0.5.1-next.820.97b3830"
+    "@flighthq/types": "0.5.1-next.903.f434bc2"
   },
   "devDependencies": {
     "@flighthq/entity": "*",

package/src/shell.test.ts

@@ -1,4 +1,4 @@
-import { createEntity } from '@flighthq/entity/contract';
+import { allocateEntity, finishEntity } from '@flighthq/entity/contract';
 import type {
   EntityRuntimeKey,
   HasShellBeep,
@@ -50,12 +50,14 @@ describe('moveShellItemsToTrash', () => {
     const paths: string[] = [];
     const resolvers: Array<(outcome: { reason: 'ok' | 'operation-failed' }) => void> = [];
     const host = trashHost(
-      createEntity({
-        moveToTrash(path) {
+      (() => {
+        const out = allocateEntity<any>();
+        out.moveToTrash = (path: string) => {
           paths.push(path);
           return new Promise((resolve) => resolvers.push(resolve));
-        },
-      } satisfies Omit<ShellTrashBackend, typeof EntityRuntimeKey>),
+        };
+        return finishEntity(out);
+      })(),
     );
 
     const outcomes = moveShellItemsToTrash(host, ['/first', '/second']);
@@ -75,7 +77,18 @@ describe('moveShellItemsToTrash', () => {
 describe('moveShellItemToTrash', () => {
   it('routes through the supplied trash provider', async () => {
     const moveToTrash = vi.fn(async () => ({ reason: 'ok' as const }));
-    await expect(moveShellItemToTrash(trashHost(createEntity({ moveToTrash })), '/item')).resolves.toEqual({
+    await expect(
+      moveShellItemToTrash(
+        trashHost(
+          (() => {
+            const out = allocateEntity<any>();
+            out.moveToTrash = moveToTrash;
+            return finishEntity(out);
+          })(),
+        ),
+        '/item',
+      ),
+    ).resolves.toEqual({
       reason: 'ok',
     });
     expect(moveToTrash).toHaveBeenCalledWith('/item');
@@ -91,7 +104,13 @@ describe('openShellExternalUrl', () => {
 
   it('blocks before dispatch when the scheme is not allowed', async () => {
     const open = vi.fn(async () => ({ reason: 'ok' as const }));
-    const host = externalHost(createEntity({ open }));
+    const host = externalHost(
+      (() => {
+        const out = allocateEntity<any>();
+        out.open = open;
+        return finishEntity(out);
+      })(),
+    );
     await expect(openShellExternalUrl(host, 'file:///etc/passwd', { allowedSchemes: ['https'] })).resolves.toEqual({
       reason: 'blocked-scheme',
     });
@@ -101,20 +120,24 @@ describe('openShellExternalUrl', () => {
   it('keeps two live hosts isolated', async () => {
     const calls: string[] = [];
     const first = externalHost(
-      createEntity({
-        async open(url) {
+      (() => {
+        const out = allocateEntity<any>();
+        out.open = async (url: string) => {
           calls.push(`first:${url}`);
           return { reason: 'ok' };
-        },
-      } satisfies Omit<ShellExternalBackend, typeof EntityRuntimeKey>),
+        };
+        return finishEntity(out);
+      })(),
     );
     const second = externalHost(
-      createEntity({
-        async open(url) {
+      (() => {
+        const out = allocateEntity<any>();
+        out.open = async (url: string) => {
           calls.push(`second:${url}`);
           return { reason: 'operation-failed' };
-        },
-      } satisfies Omit<ShellExternalBackend, typeof EntityRuntimeKey>),
+        };
+        return finishEntity(out);
+      })(),
     );
     await expect(openShellExternalUrl(first, 'https://one.test', HTTPS_ONLY)).resolves.toEqual({ reason: 'ok' });
     await expect(openShellExternalUrl(second, 'https://two.test', HTTPS_ONLY)).resolves.toEqual({
@@ -126,12 +149,11 @@ describe('openShellExternalUrl', () => {
 
 describe('openShellPath', () => {
   it('preserves a provider failure and its message', async () => {
-    const provider = createEntity({
-      async open() {
-        return { message: '', reason: 'operation-failed' as const };
-      },
-    });
-    await expect(openShellPath(pathOpenHost(provider), '/missing')).resolves.toEqual({
+    const provider = allocateEntity<ShellPathOpenBackend>();
+    provider.open = async () => {
+      return { message: '', reason: 'operation-failed' as const };
+    };
+    await expect(openShellPath(pathOpenHost(finishEntity(provider)), '/missing')).resolves.toEqual({
       message: '',
       reason: 'operation-failed',
     });
@@ -151,7 +173,18 @@ describe('readShellShortcutLink', () => {
 describe('revealShellPath', () => {
   it('returns the reveal-specific outcome', async () => {
     const reveal = vi.fn(async () => ({ reason: 'operation-failed' as const }));
-    await expect(revealShellPath(pathRevealHost(createEntity({ reveal })), '/item')).resolves.toEqual({
+    await expect(
+      revealShellPath(
+        pathRevealHost(
+          (() => {
+            const out = allocateEntity<any>();
+            out.reveal = reveal;
+            return finishEntity(out);
+          })(),
+        ),
+        '/item',
+      ),
+    ).resolves.toEqual({
       reason: 'operation-failed',
     });
   });
@@ -160,7 +193,15 @@ describe('revealShellPath', () => {
 describe('shellBeep', () => {
   it('dispatches synchronously', () => {
     const beep = vi.fn();
-    shellBeep(beepHost(createEntity({ beep })));
+    shellBeep(
+      beepHost(
+        (() => {
+          const out = allocateEntity<any>();
+          out.beep = beep;
+          return finishEntity(out);
+        })(),
+      ),
+    );
     expect(beep).toHaveBeenCalledOnce();
   });
 });
@@ -176,7 +217,13 @@ describe('spawnShellProcess', () => {
   it('forwards the command, argument vector, and options to the host backend unchanged', () => {
     const process = shellProcess();
     const spawn = vi.fn(() => process);
-    const host = processHost(createEntity({ spawn }));
+    const host = processHost(
+      (() => {
+        const out = allocateEntity<any>();
+        out.spawn = spawn;
+        return finishEntity(out);
+      })(),
+    );
     const args = ['--flag', 'value'] as const;
     const options = { cwd: '/work', environment: { MODE: 'test' } } as const;
 
@@ -222,13 +269,13 @@ function processHost(process: ShellProcessBackend): ShellProcessHost {
 }
 
 function shellProcess(): ShellProcess {
-  return createEntity({
-    exit: Promise.resolve({ code: 0, signal: null }),
-    stderr: new ReadableStream<Uint8Array>(),
-    stdin: new WritableStream<Uint8Array>(),
-    stdout: new ReadableStream<Uint8Array>(),
-    terminate: vi.fn(),
-  });
+  const out = allocateEntity<any>();
+  out.exit = Promise.resolve({ code: 0, signal: null });
+  out.stderr = new ReadableStream<Uint8Array>();
+  out.stdin = new WritableStream<Uint8Array>();
+  out.stdout = new ReadableStream<Uint8Array>();
+  out.terminate = vi.fn();
+  return finishEntity(out);
 }
 
 function shortcutLinkHost(shortcutLink: ShellShortcutLinkBackend): HasShellShortcutLink {
@@ -236,12 +283,12 @@ function shortcutLinkHost(shortcutLink: ShellShortcutLinkBackend): HasShellShort
 }
 
 function shortcutLinkProvider(): ShellShortcutLinkBackend & { write: ReturnType<typeof vi.fn> } {
-  return createEntity({
-    async read() {
-      return { link: { target: '/app' }, reason: 'ok' };
-    },
-    write: vi.fn(async () => ({ reason: 'ok' as const })),
-  } satisfies Omit<ShellShortcutLinkBackend, typeof EntityRuntimeKey>);
+  const out = allocateEntity<any>();
+  out.read = async () => {
+    return { link: { target: '/app' }, reason: 'ok' };
+  };
+  out.write = vi.fn(async () => ({ reason: 'ok' as const }));
+  return finishEntity(out);
 }
 
 function trashHost(trash: ShellTrashBackend): HasShellTrash {