@openchambery/web 1.19.22-beta.1 → 2.0.0-beta.3

package/server/lib/opencode/proxy.test.js

@@ -8,8 +8,21 @@ import {
   normalizeForwardedDirectoryHeaders,
   registerOpenCodeProxy,
   resolveSessionTurnAdmissionRequest,
+  toUpstreamOpenCodeApiPath,
 } from './proxy.js';
 
+describe('toUpstreamOpenCodeApiPath', () => {
+  it('keeps v2 /api paths and restores the prefix Express strips at the /api mount', () => {
+    expect(toUpstreamOpenCodeApiPath('/api/agent')).toBe('/api/agent');
+    expect(toUpstreamOpenCodeApiPath('/api/session?directory=/repo')).toBe('/api/session?directory=/repo');
+    expect(toUpstreamOpenCodeApiPath('/api')).toBe('/api');
+    expect(toUpstreamOpenCodeApiPath('/agent')).toBe('/api/agent');
+    expect(toUpstreamOpenCodeApiPath('/session/ses_1/message')).toBe('/api/session/ses_1/message');
+    expect(toUpstreamOpenCodeApiPath('/')).toBe('/api');
+    expect(toUpstreamOpenCodeApiPath('')).toBe('/api');
+  });
+});
+
 const originalFetch = globalThis.fetch;
 
 afterEach(() => {
@@ -182,6 +195,175 @@ describe('resolveSessionTurnAdmissionRequest', () => {
   });
 });
 
+describe('registerOpenCodeProxy runtime contract gate (ticket 11)', () => {
+  it('blocks execution paths when runtime contract denies execution', async () => {
+    const app = express();
+    registerOpenCodeProxy(app, {
+      fs: { promises: { realpath: async (v) => v } },
+      os: {},
+      path: {},
+      OPEN_CODE_READY_GRACE_MS: 12_000,
+      LONG_REQUEST_TIMEOUT_MS: 60_000,
+      getRuntime: () => ({
+        openCodePort: 4096,
+        isOpenCodeReady: true,
+        openCodeNotReadySince: 0,
+        isRestartingOpenCode: false,
+        runtimeContract: {
+          executionAllowed: false,
+          phase: 'incompatible',
+          reasons: ['below-min-verified'],
+          serveVersion: '2.0.5',
+          minVerifiedVersion: '2.0.12',
+        },
+      }),
+      getOpenCodeAuthHeaders: () => ({}),
+      buildOpenCodeUrl: (pathname) => `http://opencode.test${pathname}`,
+      ensureOpenCodeApiPrefix: () => {},
+      getUiNotificationClients: () => [],
+    });
+
+    const response = await request(app)
+      .post('/api/session/ses_1/prompt')
+      .send({ parts: [] })
+      .expect(409);
+
+    expect(response.body.errorCode).toBe('RUNTIME_CONTRACT_EXECUTION_BLOCKED');
+    expect(response.body.phase).toBe('incompatible');
+  });
+
+  it('keeps interrupt available when execution is limited', async () => {
+    const upstreamUrls = [];
+    globalThis.fetch = vi.fn(async (url) => {
+      upstreamUrls.push(String(url));
+      return {
+        ok: true,
+        status: 200,
+        headers: new Headers({ 'content-type': 'application/json' }),
+        json: async () => ({}),
+        text: async () => '{}',
+        body: null,
+      };
+    });
+
+    const app = express();
+    registerOpenCodeProxy(app, {
+      fs: { promises: { realpath: async (v) => v } },
+      os: {},
+      path: {},
+      OPEN_CODE_READY_GRACE_MS: 12_000,
+      LONG_REQUEST_TIMEOUT_MS: 60_000,
+      getRuntime: () => ({
+        openCodePort: 4096,
+        isOpenCodeReady: true,
+        openCodeNotReadySince: 0,
+        isRestartingOpenCode: false,
+        runtimeContract: {
+          executionAllowed: false,
+          phase: 'ready-unverified',
+          reasons: ['unverified-newer'],
+          serveVersion: '2.1.0',
+          minVerifiedVersion: '2.0.12',
+        },
+      }),
+      getOpenCodeAuthHeaders: () => ({}),
+      buildOpenCodeUrl: (pathname) => `http://opencode.test${pathname}`,
+      ensureOpenCodeApiPrefix: () => {},
+      getUiNotificationClients: () => [],
+    });
+
+    // Gate must not 409 interrupt; proxy may still forward or fail transport — not contract block.
+    const response = await request(app)
+      .post('/api/session/ses_1/interrupt')
+      .send({});
+    expect(response.status).not.toBe(409);
+    expect(response.body?.errorCode).not.toBe('RUNTIME_CONTRACT_EXECUTION_BLOCKED');
+  });
+
+  it('blocks writes when runtimeContract is null (production default, no bypass)', async () => {
+    const app = express();
+    registerOpenCodeProxy(app, {
+      fs: { promises: { realpath: async (v) => v } },
+      os: {},
+      path: {},
+      OPEN_CODE_READY_GRACE_MS: 12_000,
+      LONG_REQUEST_TIMEOUT_MS: 60_000,
+      getRuntime: () => ({
+        openCodePort: 4096,
+        isOpenCodeReady: true,
+        openCodeNotReadySince: 0,
+        isRestartingOpenCode: false,
+        runtimeContract: null,
+      }),
+      getOpenCodeAuthHeaders: () => ({}),
+      buildOpenCodeUrl: (pathname) => `http://opencode.test${pathname}`,
+      ensureOpenCodeApiPrefix: () => {},
+      getUiNotificationClients: () => [],
+    });
+
+    const response = await request(app)
+      .post('/api/session/ses_1/prompt')
+      .send({ parts: [] })
+      .expect(409);
+    expect(response.body.errorCode).toBe('RUNTIME_CONTRACT_EXECUTION_BLOCKED');
+  });
+
+  it('re-gates after readiness hold when stale permit becomes incompatible', async () => {
+    // Race: early gate sees executionAllowed true (old instance), hold waits for
+    // restart, then new instance publishes incompatible — must 409, not forward.
+    const notReadySince = Date.now();
+    let tick = 0;
+    const app = express();
+    registerOpenCodeProxy(app, {
+      fs: { promises: { realpath: async (v) => v } },
+      os: {},
+      path: {},
+      OPEN_CODE_READY_GRACE_MS: 12_000,
+      LONG_REQUEST_TIMEOUT_MS: 60_000,
+      getRuntime: () => {
+        tick += 1;
+        if (tick <= 2) {
+          return {
+            openCodePort: 4096,
+            isOpenCodeReady: false,
+            openCodeNotReadySince: notReadySince,
+            isRestartingOpenCode: true,
+            runtimeContract: {
+              executionAllowed: true,
+              phase: 'ready',
+              serveVersion: '2.0.12',
+            },
+          };
+        }
+        return {
+          openCodePort: 4096,
+          isOpenCodeReady: true,
+          openCodeNotReadySince: 0,
+          isRestartingOpenCode: false,
+          runtimeContract: {
+            executionAllowed: false,
+            phase: 'incompatible',
+            reasons: ['below-min-verified'],
+            serveVersion: '2.0.5',
+            minVerifiedVersion: '2.0.12',
+          },
+        };
+      },
+      getOpenCodeAuthHeaders: () => ({}),
+      buildOpenCodeUrl: (pathname) => `http://opencode.test${pathname}`,
+      ensureOpenCodeApiPrefix: () => {},
+      getUiNotificationClients: () => [],
+    });
+
+    const response = await request(app)
+      .post('/api/session/ses_1/prompt')
+      .send({ parts: [] })
+      .expect(409);
+    expect(response.body.errorCode).toBe('RUNTIME_CONTRACT_EXECUTION_BLOCKED');
+    expect(response.body.phase).toBe('incompatible');
+  });
+});
+
 describe('registerOpenCodeProxy reasoning projection routes', () => {
   it('strips includeReasoning from SSE upstream and drops reasoning events when false', async () => {
     const upstreamUrls = [];
@@ -300,3 +482,159 @@ describe('registerOpenCodeProxy reasoning projection routes', () => {
     expect(res.body[0].parts[0].text).toBe('keep-me');
   });
 });
+
+describe('registerOpenCodeProxy session metadata overlay', () => {
+  const mountProxyWithMetadata = (getStoredSessionMetadata) => {
+    const app = express();
+    registerOpenCodeProxy(app, {
+      fs: { promises: { realpath: async (v) => v } },
+      os: {},
+      path: {},
+      OPEN_CODE_READY_GRACE_MS: 1000,
+      LONG_REQUEST_TIMEOUT_MS: 60_000,
+      SSE_UPSTREAM_CONNECT_TIMEOUT_MS: 5_000,
+      SSE_UPSTREAM_STALL_TIMEOUT_MS: 5_000,
+      getRuntime: readyRuntime,
+      getOpenCodeAuthHeaders: () => ({}),
+      buildOpenCodeUrl: (path) => `http://127.0.0.1:4096${path}`,
+      ensureOpenCodeApiPrefix: () => {},
+      getStoredSessionMetadata,
+    });
+    return app;
+  };
+
+  it('merges Host store metadata onto session list and detail (store keys win)', async () => {
+    globalThis.fetch = vi.fn(async (url) => {
+      const path = String(url).split('?')[0];
+      if (path.endsWith('/api/session') || path.endsWith('/session')) {
+        return {
+          ok: true,
+          status: 200,
+          headers: new Headers({ 'content-type': 'application/json' }),
+          text: async () => JSON.stringify([
+            {
+              id: 'ses_1',
+              title: 'Alpha',
+              metadata: { fromOpenCode: true, shared: 'theirs' },
+            },
+            { id: 'ses_2', title: 'Beta' },
+          ]),
+        };
+      }
+      if (path.includes('/session/ses_1')) {
+        return {
+          ok: true,
+          status: 200,
+          headers: new Headers({ 'content-type': 'application/json' }),
+          text: async () => JSON.stringify({
+            id: 'ses_1',
+            title: 'Alpha',
+            metadata: { fromOpenCode: true, shared: 'theirs' },
+          }),
+        };
+      }
+      return {
+        ok: false,
+        status: 404,
+        headers: new Headers({ 'content-type': 'application/json' }),
+        text: async () => JSON.stringify({ error: 'not found' }),
+      };
+    });
+
+    const app = mountProxyWithMetadata(async () => ({
+      ses_1: { openchamber: { goal: { status: 'active' } }, shared: 'ours' },
+    }));
+
+    const list = await request(app).get('/api/session');
+    expect(list.status).toBe(200);
+    expect(list.body[0]).toMatchObject({
+      id: 'ses_1',
+      metadata: {
+        fromOpenCode: true,
+        shared: 'ours',
+        openchamber: { goal: { status: 'active' } },
+      },
+    });
+    expect(list.body[1].metadata).toBeUndefined();
+
+    const detail = await request(app).get('/api/session/ses_1');
+    expect(detail.status).toBe(200);
+    expect(detail.body).toMatchObject({
+      id: 'ses_1',
+      metadata: {
+        fromOpenCode: true,
+        shared: 'ours',
+        openchamber: { goal: { status: 'active' } },
+      },
+    });
+  });
+
+  it('returns retryable 503 when the configured store throws (never silent upstream)', async () => {
+    vi.spyOn(console, 'warn').mockImplementation(() => {});
+    globalThis.fetch = vi.fn(async () => ({
+      ok: true,
+      status: 200,
+      headers: new Headers({ 'content-type': 'application/json' }),
+      text: async () => JSON.stringify([
+        { id: 'ses_1', title: 'Alpha', metadata: { fromOpenCode: true } },
+      ]),
+    }));
+
+    const app = mountProxyWithMetadata(async () => {
+      throw new Error('store down');
+    });
+
+    const list = await request(app).get('/api/session');
+    expect(list.status).toBe(503);
+    expect(list.body).toMatchObject({
+      retryable: true,
+      code: 'session_metadata_unavailable',
+    });
+  });
+
+  it('projects Host archive onto time.archived and preserves list cursor envelopes', async () => {
+    globalThis.fetch = vi.fn(async () => ({
+      ok: true,
+      status: 200,
+      headers: new Headers({ 'content-type': 'application/json' }),
+      text: async () => JSON.stringify({
+        data: [
+          {
+            id: 'ses_1',
+            title: 'Alpha',
+            time: { created: 1, updated: 2, archived: 50 },
+          },
+        ],
+        cursor: 'next-page',
+      }),
+    }));
+
+    const app = mountProxyWithMetadata(async () => ({
+      ses_1: { openchamber: { archive: { archivedAt: 99 } } },
+    }));
+
+    const list = await request(app).get('/api/session');
+    expect(list.status).toBe(200);
+    expect(list.body.cursor).toBe('next-page');
+    expect(list.body.data[0].time.archived).toBe(99);
+    expect(list.body.data[0].metadata.openchamber.archive.archivedAt).toBe(99);
+  });
+
+  it('explicit Host archivedAt 0 clears upstream historical time.archived', async () => {
+    globalThis.fetch = vi.fn(async () => ({
+      ok: true,
+      status: 200,
+      headers: new Headers({ 'content-type': 'application/json' }),
+      text: async () => JSON.stringify([
+        { id: 'ses_1', time: { created: 1, updated: 2, archived: 50 } },
+      ]),
+    }));
+
+    const app = mountProxyWithMetadata(async () => ({
+      ses_1: { openchamber: { archive: { archivedAt: 0 } } },
+    }));
+
+    const list = await request(app).get('/api/session');
+    expect(list.body[0].time.archived).toBeUndefined();
+  });
+});