Refactor #Changes: placeholder replacement to conditionally include file contents based on whether the prompt mode is 'no-context'

Author: robertpiosik

packages/vscode/src/views/panel/backend/message-handlers/handle-copy-prompt.ts

@@ -103,9 +103,14 @@ export const handle_copy_prompt = async (params: {
     let post_context_instructions = instructions
 
     if (instructions.includes('#Changes:')) {
+      const is_no_context_web_mode =
+        params.panel_provider.mode == MODE.WEB &&
+        params.panel_provider.web_prompt_type == 'no-context'
+
       pre_context_instructions = await replace_changes_placeholder({
         instruction: pre_context_instructions,
-        workspace_provider: params.panel_provider.workspace_provider
+        workspace_provider: params.panel_provider.workspace_provider,
+        is_no_context_web_mode
       })
       post_context_instructions = await replace_changes_placeholder({
         instruction: post_context_instructions,

packages/vscode/src/views/panel/backend/message-handlers/handle-preview-preset.ts

@@ -10,6 +10,7 @@ import {
 } from '@/views/panel/backend/utils/replace-git-placeholders'
 import { Preset } from '@shared/types/preset'
 import { apply_preset_affixes_to_instruction } from '@/utils/apply-preset-affixes'
+import { MODE } from '@/views/panel/types/main-view-mode'
 import { dictionary } from '@shared/constants/dictionary'
 
 export const handle_preview_preset = async (
@@ -98,9 +99,14 @@ export const handle_preview_preset = async (
     let post_context_instructions = instructions
 
     if (pre_context_instructions.includes('#Changes:')) {
+      const is_no_context_web_mode =
+        panel_provider.mode == MODE.WEB &&
+        panel_provider.web_prompt_type == 'no-context'
+
       pre_context_instructions = await replace_changes_placeholder({
         instruction: pre_context_instructions,
-        workspace_provider: panel_provider.workspace_provider
+        workspace_provider: panel_provider.workspace_provider,
+        is_no_context_web_mode
       })
       post_context_instructions = await replace_changes_placeholder({
         instruction: post_context_instructions,

packages/vscode/src/views/panel/backend/message-handlers/handle-send-prompt.ts

@@ -179,9 +179,14 @@ export const handle_send_prompt = async (params: {
         let pre_context_instructions = instructions
         let post_context_instructions = instructions
         if (pre_context_instructions.includes('#Changes:')) {
+          const is_no_context_web_mode =
+            params.panel_provider.mode == MODE.WEB &&
+            params.panel_provider.web_prompt_type == 'no-context'
+
           pre_context_instructions = await replace_changes_placeholder({
             instruction: pre_context_instructions,
-            workspace_provider: params.panel_provider.workspace_provider
+            workspace_provider: params.panel_provider.workspace_provider,
+            is_no_context_web_mode
           })
           post_context_instructions = await replace_changes_placeholder({
             instruction: post_context_instructions,

packages/vscode/src/views/panel/backend/utils/replace-git-placeholders.ts

@@ -10,7 +10,8 @@ import { dictionary } from '@shared/constants/dictionary'
 const build_changes_xml = (
   diff: string,
   cwd: string,
-  workspace_provider: WorkspaceProvider
+  workspace_provider: WorkspaceProvider,
+  is_no_context_web_mode?: boolean
 ): string => {
   // Split diff into per-file sections. Each section starts with 'diff --git '.
   const file_diffs = diff.split(/^diff --git /m).filter((d) => d.trim() != '')
@@ -52,7 +53,7 @@ const build_changes_xml = (
       changes_content += `<diff>\n<![CDATA[\n${full_file_diff}\n]]>\n</diff>\n`
 
       const absolute_path = path.join(cwd, file_path)
-      if (checked_files.has(absolute_path)) {
+      if (!is_no_context_web_mode && checked_files.has(absolute_path)) {
         const workspace_root =
           workspace_provider.get_workspace_root_for_file(absolute_path)
         let display_path: string
@@ -101,7 +102,8 @@ const build_changes_xml = (
 export const replace_changes_placeholder = async (params: {
   instruction: string
   after_context?: boolean
-  workspace_provider: WorkspaceProvider
+  workspace_provider: WorkspaceProvider,
+  is_no_context_web_mode?: boolean
 }): Promise<string> => {
   const matches = params.instruction.match(
     /#Changes:([^\s,;:.!?]+(?:\/[^\s,;:.!?]+)?)/
@@ -181,7 +183,8 @@ export const replace_changes_placeholder = async (params: {
       const replacement_text = build_changes_xml(
         diff,
         target_folder.uri.fsPath,
-        params.workspace_provider
+        params.workspace_provider,
+        params.is_no_context_web_mode
       )
       return params.instruction.replace(
         new RegExp(`#Changes:${branch_spec}`, 'g'),
@@ -249,7 +252,8 @@ export const replace_changes_placeholder = async (params: {
       const replacement_text = build_changes_xml(
         diff,
         repository.rootUri.fsPath,
-        params.workspace_provider
+        params.workspace_provider,
+        params.is_no_context_web_mode
       )
       return params.instruction.replace(
         new RegExp(`#Changes:${branch_name}`, 'g'),