Skip to content

Commit b790520

Browse files
committed
Rename all references and commands related to "code review" to "code preview" for consistent terminology across the application
1 parent 22bf56e commit b790520

9 files changed

Lines changed: 18 additions & 17 deletions

File tree

‎packages/vscode/src/commands/apply-chat-response-command/apply-chat-response-command.ts‎

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -10,7 +10,7 @@ import { code_review_promise_resolve } from './utils/preview/preview'
1010
import { get_diff_stats } from './utils/preview/diff-utils'
1111
import { create_safe_path } from '@/utils/path-sanitizer'
1212
import {
13-
handle_code_review_and_cleanup,
13+
preview_handler,
1414
ongoing_review_cleanup_promise
1515
} from './utils/preview-handler'
1616
import { process_chat_response, CommandArgs } from './response-processor'
@@ -147,7 +147,7 @@ export const apply_chat_response_command = (
147147
})
148148
}
149149

150-
const changes_accepted = await handle_code_review_and_cleanup({
150+
const changes_accepted = await preview_handler({
151151
original_states: review_data.original_states,
152152
chat_response: review_data.chat_response,
153153
panel_provider,

packages/vscode/src/commands/apply-chat-response-command/handlers/restore-review-handler.ts renamed to packages/vscode/src/commands/apply-chat-response-command/handlers/restore-preview-handler.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -8,7 +8,7 @@ import { OriginalFileState } from '../types/original-file-state'
88
import { FileInPreview } from '@shared/types/file-in-preview'
99
import { remove_directory_if_empty } from '../utils/file-operations'
1010

11-
export const handle_restore_review = async (
11+
export const handle_restore_preview = async (
1212
files: FileInPreview[]
1313
): Promise<{ success: boolean; original_states?: OriginalFileState[] }> => {
1414
Logger.info({

‎packages/vscode/src/commands/apply-chat-response-command/response-processor.ts‎

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,7 @@
11
import * as vscode from 'vscode'
22
import * as fs from 'fs'
33
import { OriginalFileState } from './types/original-file-state'
4-
import { handle_restore_review } from './handlers/restore-review-handler'
4+
import { handle_restore_preview } from './handlers/restore-preview-handler'
55
import { parse_response, FileItem, DiffItem } from './utils/clipboard-parser'
66
import { create_safe_path } from '@/utils/path-sanitizer'
77
import { dictionary } from '@shared/constants/dictionary'
@@ -44,7 +44,7 @@ export const process_chat_response = async (
4444
panel_provider: PanelProvider
4545
): Promise<PreviewData | null> => {
4646
if (args?.files_with_content) {
47-
const result = await handle_restore_review(args.files_with_content)
47+
const result = await handle_restore_preview(args.files_with_content)
4848
if (result.success && result.original_states) {
4949
const augmented_states = result.original_states.map((state) => {
5050
const file_in_review = args?.files_with_content!.find(

‎packages/vscode/src/commands/apply-chat-response-command/utils/preview-handler.ts‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -8,7 +8,7 @@ import { update_undo_button_state } from './state-manager'
88

99
export let ongoing_review_cleanup_promise: Promise<void> | null = null
1010

11-
export const handle_code_review_and_cleanup = async (params: {
11+
export const preview_handler = async (params: {
1212
original_states: OriginalFileState[]
1313
chat_response: string
1414
panel_provider: PanelProvider

‎packages/vscode/src/commands/apply-chat-response-command/utils/preview/preview.ts‎

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -51,7 +51,7 @@ export const preview = async (params: {
5151

5252
if (params.panel_provider) {
5353
params.panel_provider.send_message({
54-
command: 'CODE_REVIEW_STARTED',
54+
command: 'CODE_PREVIEW_STARTED',
5555
files: prepared_files.map((p) => p.reviewable_file),
5656
raw_instructions: params.raw_instructions
5757
})
@@ -174,7 +174,7 @@ export const preview = async (params: {
174174

175175
if (params.panel_provider) {
176176
params.panel_provider.cancel_all_intelligent_updates()
177-
params.panel_provider.send_message({ command: 'CODE_REVIEW_FINISHED' })
177+
params.panel_provider.send_message({ command: 'CODE_PREVIEW_FINISHED' })
178178
}
179179
}
180180
}

‎packages/vscode/src/views/panel/frontend/Main/MainView/MainView.module.scss‎

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -24,6 +24,8 @@
2424
border-top-right-radius: var(--border-radius-5px);
2525
border: 1px solid var(--vscode-commandCenter-inactiveBorder);
2626
border-bottom: 0px;
27+
padding-bottom: 5px;
28+
margin-bottom: -5px;
2729
overflow: hidden;
2830

2931
&__item {

‎packages/vscode/src/views/panel/frontend/Main/MainView/components/Header/Header.module.scss‎

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -25,7 +25,6 @@
2525
padding: 0 var(--padding);
2626
margin: 0 calc(var(--padding) * -1);
2727
height: 20px;
28-
color: var(--vscode-icon-foreground);
2928
white-space: nowrap;
3029

3130
&:focus {

‎packages/vscode/src/views/panel/frontend/hooks/use-panel.ts‎

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -129,7 +129,7 @@ export const use_panel = (vscode: any) => {
129129
set_has_active_editor(message.has_active_editor)
130130
} else if (message.command == 'EDITOR_SELECTION_CHANGED') {
131131
set_has_active_selection(message.has_selection)
132-
} else if (message.command == 'CODE_REVIEW_STARTED') {
132+
} else if (message.command == 'CODE_PREVIEW_STARTED') {
133133
set_files_to_review(message.files)
134134
set_raw_instructions(message.raw_instructions)
135135
} else if (message.command == 'UPDATE_FILE_IN_REVIEW') {
@@ -197,7 +197,7 @@ export const use_panel = (vscode: any) => {
197197

198198
return new_history
199199
})
200-
} else if (message.command == 'CODE_REVIEW_FINISHED') {
200+
} else if (message.command == 'CODE_PREVIEW_FINISHED') {
201201
set_files_to_review(undefined)
202202
set_raw_instructions(undefined)
203203
} else if (message.command == 'WORKSPACE_STATE') {

‎packages/vscode/src/views/panel/types/messages.ts‎

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -495,14 +495,14 @@ export interface CanUndoChangedMessage extends BaseMessage {
495495
can_undo: boolean
496496
}
497497

498-
export interface CodeReviewStartedMessage extends BaseMessage {
499-
command: 'CODE_REVIEW_STARTED'
498+
export interface CodePreviewStartedMessage extends BaseMessage {
499+
command: 'CODE_PREVIEW_STARTED'
500500
files: FileInPreview[]
501501
raw_instructions?: string
502502
}
503503

504-
export interface CodeReviewFinishedMessage extends BaseMessage {
505-
command: 'CODE_REVIEW_FINISHED'
504+
export interface CodePreviewFinishedMessage extends BaseMessage {
505+
command: 'CODE_PREVIEW_FINISHED'
506506
}
507507

508508
export interface WorkspaceStateMessage extends BaseMessage {
@@ -608,8 +608,8 @@ export type BackendMessage =
608608
| ApiModeMessage
609609
| VersionMessage
610610
| CanUndoChangedMessage
611-
| CodeReviewStartedMessage
612-
| CodeReviewFinishedMessage
611+
| CodePreviewStartedMessage
612+
| CodePreviewFinishedMessage
613613
| WorkspaceStateMessage
614614
| SelectedPresetOrGroupChangedMessage
615615
| SelectedConfigurationChangedMessage

0 commit comments

Comments
 (0)