refactor: github chat.

Author: knightedcodemonkey

docs/ai-chat-context-and-payload-strategy.md

@@ -16,7 +16,7 @@ Each request includes a system prompt with policy guidance, then augments that p
 
 Primary implementation:
 
-- src/modules/github/chat-drawer/payload.js
+- src/modules/github/chat/payload.js
 
 ### 2. Repository context
 
@@ -29,7 +29,7 @@ Each request includes repository targeting context as a dedicated system message
 
 Primary implementation:
 
-- src/modules/github/chat-drawer/drawer.js
+- src/modules/github/chat/drawer.js
 
 ### 3. Editor context (Send tab content)
 
@@ -44,8 +44,8 @@ This context is designed to support dynamic proposal targeting by tab id/path an
 
 Primary implementation:
 
-- src/modules/github/chat-drawer/active-tab-context.js
-- src/modules/github/chat-drawer/drawer.js
+- src/modules/github/chat/active-tab-context.js
+- src/modules/github/chat/drawer.js
 
 ### 4. Tooling model
 
@@ -62,9 +62,9 @@ Contract:
 
 Primary implementation:
 
-- src/modules/github/chat-drawer/proposals.js
-- src/modules/github/chat-drawer/tab-target-resolver.js
-- src/modules/github/chat-drawer/drawer.js
+- src/modules/github/chat/proposals.js
+- src/modules/github/chat/tab-target-resolver.js
+- src/modules/github/chat/drawer.js
 
 ### 5. Apply and undo behavior
 
@@ -74,8 +74,8 @@ Primary implementation:
 
 Primary implementation:
 
-- src/modules/github/chat-drawer/drawer.js
-- src/modules/github/chat-drawer/tab-scoped-undo-state.js
+- src/modules/github/chat/drawer.js
+- src/modules/github/chat/tab-scoped-undo-state.js
 
 ### 6. Payload size controls and summary strategy
 
@@ -88,7 +88,7 @@ The payload builder includes bounded-conversation controls:
 
 Primary implementation:
 
-- src/modules/github/chat-drawer/payload.js
+- src/modules/github/chat/payload.js
 
 ### 7. Fallback and transport behavior
 
@@ -98,7 +98,7 @@ Primary implementation:
 
 Primary implementation:
 
-- src/modules/github/chat-drawer/drawer.js
+- src/modules/github/chat/drawer.js
 - src/modules/github/api/chat.js
 
 ## Why this approach

src/app.js

@@ -51,8 +51,9 @@ import { persistClosedPrContextRecords } from './modules/app-core/pr-context-rec
 import { createPrContextStateChangeHandler } from './modules/app-core/pr-context-state-change-handler.js'
 import { createWorkspaceContextStatusController } from './modules/app-core/workspace-context-status-controller.js'
 import { createWorkspaceRecordAppliedHandler } from './modules/app-core/workspace-record-applied-handler.js'
+import { createGitHubChatWorkspaceActions } from './modules/app-core/github-chat-workspace-actions.js'
 import { createDiagnosticsUiController } from './modules/diagnostics/diagnostics-ui.js'
-import { createGitHubChatDrawer } from './modules/github/chat-drawer/drawer.js'
+import { createGitHubChatDrawer } from './modules/github/chat/drawer.js'
 import { createGitHubByotControls } from './modules/github/byot-controls.js'
 import {
   formatActivePrReference,
@@ -1041,6 +1042,18 @@ const onPrContextStateChange = createPrContextStateChangeHandler({
   editedIndicatorVisibilityController,
 })
 
+const githubChatWorkspaceActions = createGitHubChatWorkspaceActions({
+  getActiveWorkspaceTab,
+  isStyleWorkspaceTab,
+  getCssSource: () => getCssSource(),
+  getJsxSource: () => getJsxSource(),
+  workspaceTabsState,
+  getDirtyStateForTabChange,
+  loadWorkspaceTabIntoEditor,
+  renderWorkspaceTabs,
+  queueWorkspaceSave: () => queueWorkspaceSave(),
+})
+
 const githubWorkflows = createGitHubWorkflowsSetup({
   factories: {
     createGitHubPrEditorSyncController,
@@ -1218,71 +1231,7 @@ const githubWorkflows = createGitHubWorkflowsSetup({
     confirmAction: options => confirmAction(options),
     setStatus,
     showAppToast,
-    getActiveWorkspaceTabContext: () => {
-      const activeTab = getActiveWorkspaceTab()
-      if (!activeTab) {
-        return null
-      }
-
-      const isStylesTab = isStyleWorkspaceTab(activeTab)
-
-      return {
-        id: activeTab.id,
-        name: activeTab.name,
-        path: activeTab.path,
-        language: activeTab.language,
-        content: isStylesTab ? getCssSource() : getJsxSource(),
-        isActive: true,
-      }
-    },
-    getWorkspaceTabContexts: () => {
-      const activeTabId = workspaceTabsState.getActiveTabId()
-      return workspaceTabsState.getTabs().map(tab => {
-        const isActive = tab.id === activeTabId
-        const isStylesTab = isStyleWorkspaceTab(tab)
-        return {
-          id: tab.id,
-          name: tab.name,
-          path: tab.path,
-          language: tab.language,
-          isActive,
-          content: isActive
-            ? isStylesTab
-              ? getCssSource()
-              : getJsxSource()
-            : tab.content,
-        }
-      })
-    },
-    applyWorkspaceTabContent: ({ tabId, content }) => {
-      const tab = workspaceTabsState.getTab(tabId)
-      if (!tab || typeof content !== 'string') {
-        return null
-      }
-
-      const updatedTab = workspaceTabsState.upsertTab(
-        {
-          ...tab,
-          content,
-          isDirty: getDirtyStateForTabChange(tab, content),
-          lastModified: Date.now(),
-          isActive: tab.isActive,
-        },
-        { emitReason: 'chatApplyTabContent' },
-      )
-
-      if (!updatedTab) {
-        return null
-      }
-
-      if (updatedTab.isActive) {
-        loadWorkspaceTabIntoEditor(updatedTab)
-      }
-
-      renderWorkspaceTabs()
-      queueWorkspaceSave()
-      return updatedTab
-    },
+    ...githubChatWorkspaceActions,
     scheduleRender: () => {
       if (
         autoRenderToggle?.checked &&

src/modules/app-core/github-chat-workspace-actions.js

@@ -0,0 +1,85 @@
+const createGitHubChatWorkspaceActions = ({
+  getActiveWorkspaceTab,
+  isStyleWorkspaceTab,
+  getCssSource,
+  getJsxSource,
+  workspaceTabsState,
+  getDirtyStateForTabChange,
+  loadWorkspaceTabIntoEditor,
+  renderWorkspaceTabs,
+  queueWorkspaceSave,
+}) => {
+  const getActiveWorkspaceTabContext = () => {
+    const activeTab = getActiveWorkspaceTab()
+    if (!activeTab) {
+      return null
+    }
+
+    const isStylesTab = isStyleWorkspaceTab(activeTab)
+
+    return {
+      id: activeTab.id,
+      name: activeTab.name,
+      path: activeTab.path,
+      language: activeTab.language,
+      content: isStylesTab ? getCssSource() : getJsxSource(),
+      isActive: true,
+    }
+  }
+
+  const getWorkspaceTabContexts = () => {
+    const activeTabId = workspaceTabsState.getActiveTabId()
+
+    return workspaceTabsState.getTabs().map(tab => {
+      const isActive = tab.id === activeTabId
+      const isStylesTab = isStyleWorkspaceTab(tab)
+
+      return {
+        id: tab.id,
+        name: tab.name,
+        path: tab.path,
+        language: tab.language,
+        isActive,
+        content: isActive ? (isStylesTab ? getCssSource() : getJsxSource()) : tab.content,
+      }
+    })
+  }
+
+  const applyWorkspaceTabContent = ({ tabId, content }) => {
+    const tab = workspaceTabsState.getTab(tabId)
+    if (!tab || typeof content !== 'string') {
+      return null
+    }
+
+    const updatedTab = workspaceTabsState.upsertTab(
+      {
+        ...tab,
+        content,
+        isDirty: getDirtyStateForTabChange(tab, content),
+        lastModified: Date.now(),
+        isActive: tab.isActive,
+      },
+      { emitReason: 'chatApplyTabContent' },
+    )
+
+    if (!updatedTab) {
+      return null
+    }
+
+    if (updatedTab.isActive) {
+      loadWorkspaceTabIntoEditor(updatedTab)
+    }
+
+    renderWorkspaceTabs()
+    queueWorkspaceSave()
+    return updatedTab
+  }
+
+  return {
+    getActiveWorkspaceTabContext,
+    getWorkspaceTabContexts,
+    applyWorkspaceTabContent,
+  }
+}
+
+export { createGitHubChatWorkspaceActions }

…github/chat-drawer/active-tab-context.js …odules/github/chat/active-tab-context.jssrc/modules/github/chat-drawer/active-tab-context.js renamed to src/modules/github/chat/active-tab-context.js src/modules/github/chat-drawer/active-tab-context.js renamed to src/modules/github/chat/active-tab-context.js

@@ -12,7 +12,7 @@ import {
   toModelId,
   toRepositoryLabel,
   toRepositoryUrl,
-} from './chat-utils.js'
+} from './utils.js'
 import {
   buildActiveTabEditorContext,
   normalizeWorkspaceTabContext,

src/modules/github/chat-drawer/drawer.js src/modules/github/chat/drawer.jssrc/modules/github/chat-drawer/drawer.js renamed to src/modules/github/chat/drawer.js

@@ -1,4 +1,4 @@
-import { toChatText } from './chat-utils.js'
+import { toChatText } from './utils.js'
 
 const chatByteBudget = 120_000
 const chatMaxSummaryChars = 3_600

src/modules/github/chat-drawer/payload.js src/modules/github/chat/payload.jssrc/modules/github/chat-drawer/payload.js renamed to src/modules/github/chat/payload.js

@@ -1,4 +1,4 @@
-import { toChatText } from './chat-utils.js'
+import { toChatText } from './utils.js'
 
 export const editorProposalTools = [
   {