fix(page-script): intercept only conversation endpoints (#16)

Author: 11me

extension/src/page/page-script.ts

@@ -186,12 +186,19 @@ function isConversationRequest(method: string, url: URL): boolean {
     return false;
   }
 
-  // Only /backend-api/ endpoints
-  if (!url.pathname.startsWith('/backend-api/')) {
-    return false;
-  }
-
-  return true;
+  // Only endpoints that return the conversation tree we can trim.
+  // ChatGPT performs many GET /backend-api/* requests on load (/me, /models, /settings, etc.).
+  // Intercepting those adds unnecessary overhead (clone/json) and config gating delay.
+  //
+  // Allowed:
+  // - /backend-api/conversation/<id>
+  // - /backend-api/shared_conversation/<id> (share links)
+  //
+  // Explicitly excluded by pattern (extra path segments):
+  // - /backend-api/conversation/<id>/stream_status
+  // - /backend-api/conversation/<id>/textdocs
+  const path = url.pathname;
+  return /^\/backend-api\/(conversation|shared_conversation)\/[^/]+\/?$/.test(path);
 }
 
 /**

tests/unit/page-script.test.ts

@@ -82,14 +82,13 @@ describe('isConversationRequest logic', () => {
   // Testing the logic that would be in isConversationRequest
   const isConversationRequest = (method: string, pathname: string): boolean => {
     if (method !== 'GET') return false;
-    if (!pathname.startsWith('/backend-api/')) return false;
-    return true;
+    return /^\/backend-api\/(conversation|shared_conversation)\/[^/]+\/?$/.test(pathname);
   };
 
-  it('returns true for GET /backend-api/ requests', () => {
+  it('returns true only for conversation endpoints', () => {
     expect(isConversationRequest('GET', '/backend-api/conversation/123')).toBe(true);
-    expect(isConversationRequest('GET', '/backend-api/conversation')).toBe(true);
-    expect(isConversationRequest('GET', '/backend-api/')).toBe(true);
+    expect(isConversationRequest('GET', '/backend-api/conversation/123/')).toBe(true);
+    expect(isConversationRequest('GET', '/backend-api/shared_conversation/abc-xyz')).toBe(true);
   });
 
   it('returns false for non-GET methods', () => {
@@ -98,6 +97,16 @@ describe('isConversationRequest logic', () => {
     expect(isConversationRequest('DELETE', '/backend-api/conversation')).toBe(false);
   });
 
+  it('returns false for non-conversation backend-api paths', () => {
+    expect(isConversationRequest('GET', '/backend-api/')).toBe(false);
+    expect(isConversationRequest('GET', '/backend-api/conversation')).toBe(false);
+    expect(isConversationRequest('GET', '/backend-api/me')).toBe(false);
+    expect(isConversationRequest('GET', '/backend-api/settings/user')).toBe(false);
+    expect(isConversationRequest('GET', '/backend-api/models')).toBe(false);
+    expect(isConversationRequest('GET', '/backend-api/conversation/123/stream_status')).toBe(false);
+    expect(isConversationRequest('GET', '/backend-api/conversation/123/textdocs')).toBe(false);
+  });
+
   it('returns false for non-backend-api paths', () => {
     expect(isConversationRequest('GET', '/api/conversation')).toBe(false);
     expect(isConversationRequest('GET', '/conversation')).toBe(false);