Add utility to filter prompts containing only symbols and apply it to commit message flows

├── apps
│   └── editor
│       ├── package.json
│       └── src
│           ├── commands
│           │   ├── apply-response-command
│           │   │   └── utils
│           │   │       └── preview-handler.ts
│           │   ├── code-at-cursor-command
│           │   │   └── code-at-cursor-commands.ts
│           │   └── generate-commit-message-commands
│           │       ├── actions
│           │       │   ├── get-prompt-data.ts
│           │       │   └── run-generate-action.ts
│           │       ├── generate-commit-message-commands.ts
│           │       └── index.ts
│           └── utils
│               ├── commit-message-details.ts
│               └── git-repository-utils.ts
└── packages
    └── shared
        └── src
            └── utils
                └── simplify-prompt-symbols
                    ├── index.ts
                    ├── simplify-prompt-symbols.spec.ts
                    └── simplify-prompt-symbols.ts

Author: robertpiosik

apps/editor/src/commands/generate-commit-message-commands/actions/run-generate-action.ts

@@ -22,6 +22,7 @@ import {
 } from '@/constants/state-keys'
 import { get_prompt_data } from './get-prompt-data'
 import { display_token_count } from '@shared/utils/display-token-count'
+import { is_prompt_empty_or_only_symbols } from '@shared/utils/simplify-prompt-symbols'
 import { show_configurations_quick_pick } from '@/utils/show-configurations-quick-pick'
 import { WebSocketManager } from '@/services/websocket-manager'
 import { get_response_preview_promise_resolve } from '@/commands/apply-response-command/utils/preview'
@@ -612,10 +613,10 @@ export const run_generate_action = async (params: {
       )
 
     const relevant_prompts = all_relevant_prompts.filter(
-      (p) => p.prompt.trim() != ''
+      (p) => !is_prompt_empty_or_only_symbols({ prompt: p.prompt })
     )
-    const empty_prompts = all_relevant_prompts.filter(
-      (p) => p.prompt.trim() == ''
+    const empty_prompts = all_relevant_prompts.filter((p) =>
+      is_prompt_empty_or_only_symbols({ prompt: p.prompt })
     )
 
     const get_tree_text_if_applicable = async (

apps/editor/src/utils/commit-message-details.ts

@@ -1,7 +1,10 @@
 import * as fs from 'fs'
 import * as path from 'path'
 import * as vscode from 'vscode'
-import { simplify_prompt_symbols } from '@shared/utils/simplify-prompt-symbols'
+import {
+  simplify_prompt_symbols,
+  is_prompt_empty_or_only_symbols
+} from '@shared/utils/simplify-prompt-symbols'
 import { normalize_path } from './normalize-path'
 
 export namespace CommitMessageDetails {
@@ -91,10 +94,13 @@ export namespace CommitMessageDetails {
     const selected_files_to_add = params.selected_files.map(normalize)
 
     const existing = all_prompts[params.workspace_root].find((p) => {
-      if (simplified_prompt !== '') {
+      if (!is_prompt_empty_or_only_symbols({ prompt: simplified_prompt })) {
         return p.prompt == simplified_prompt
       }
-      return p.prompt == '' && p.files.some((f) => files_to_add.includes(f))
+      return (
+        is_prompt_empty_or_only_symbols({ prompt: p.prompt }) &&
+        p.files.some((f) => files_to_add.includes(f))
+      )
     })
 
     if (existing) {

apps/editor/src/utils/git-repository-utils.ts

@@ -8,6 +8,7 @@ import {
 } from '@/constants/values'
 import { PromptBuilder } from './prompt-builder'
 import { display_token_count } from '@shared/utils/display-token-count'
+import { is_prompt_empty_or_only_symbols } from '@shared/utils/simplify-prompt-symbols'
 import { WorkspaceProvider } from '@/context/providers/workspace/workspace-provider'
 import { WebSocketManager } from '@/services/websocket-manager'
 import { search_files } from '@/features/search-files'
@@ -254,7 +255,7 @@ export const prepare_staged_changes = async (params: {
           const normalized_path = normalize_path(relative_path)
           const file_prompts = prompts_for_repo.filter(
             (p) =>
-              p.prompt.trim() !== '' &&
+              !is_prompt_empty_or_only_symbols({ prompt: p.prompt }) &&
               (p.files.includes(normalized_path) ||
                 p.files.includes(change.uri.fsPath))
           )

packages/shared/src/utils/simplify-prompt-symbols/simplify-prompt-symbols.spec.ts

@@ -1,4 +1,7 @@
-import { simplify_prompt_symbols } from './simplify-prompt-symbols'
+import {
+  simplify_prompt_symbols,
+  is_prompt_empty_or_only_symbols
+} from './simplify-prompt-symbols'
 
 describe('simplify_prompt_symbols', () => {
   it('should return plain text as is', () => {
@@ -57,3 +60,29 @@ describe('simplify_prompt_symbols', () => {
     )
   })
 })
+
+describe('is_prompt_empty_or_only_symbols', () => {
+  it('should return true for empty string', () => {
+    expect(is_prompt_empty_or_only_symbols({ prompt: '' })).toBe(true)
+  })
+
+  it('should return true for whitespace string', () => {
+    expect(is_prompt_empty_or_only_symbols({ prompt: '   ' })).toBe(true)
+  })
+
+  it('should return true for only one symbol', () => {
+    expect(is_prompt_empty_or_only_symbols({ prompt: '[Fragment]' })).toBe(true)
+  })
+
+  it('should return true for multiple symbols with whitespace', () => {
+    expect(
+      is_prompt_empty_or_only_symbols({ prompt: '[Fragment]  [Website] ' })
+    ).toBe(true)
+  })
+
+  it('should return false if there is actual text', () => {
+    expect(
+      is_prompt_empty_or_only_symbols({ prompt: 'Fix this [Fragment]' })
+    ).toBe(false)
+  })
+})

packages/shared/src/utils/simplify-prompt-symbols/simplify-prompt-symbols.ts

@@ -29,3 +29,22 @@ export const simplify_prompt_symbols = (params: { prompt: string }): string => {
 
   return simplified
 }
+
+export const is_prompt_empty_or_only_symbols = (params: {
+  prompt: string
+}): boolean => {
+  if (!params.prompt) return true
+
+  const stripped = params.prompt
+    .replace(/\[Fragment\]/g, '')
+    .replace(/\[Changes\]/g, '')
+    .replace(/\[Saved context\]/g, '')
+    .replace(/\[Commit\]/g, '')
+    .replace(/\[Skill\]/g, '')
+    .replace(/\[Image\]/g, '')
+    .replace(/\[Pasted text\]/g, '')
+    .replace(/\[Website\]/g, '')
+    .trim()
+
+  return stripped === ''
+}