remove iscompleted and completedmessages

This commit is contained in:
Connor Yoh 2025-08-12 17:15:27 +01:00
parent bb94334406
commit 63c4d98fda
10 changed files with 5 additions and 36 deletions

View File

@ -17,10 +17,8 @@ export interface ToolStepProps {
title: string; title: string;
isVisible?: boolean; isVisible?: boolean;
isCollapsed?: boolean; isCollapsed?: boolean;
isCompleted?: boolean;
onCollapsedClick?: () => void; onCollapsedClick?: () => void;
children?: React.ReactNode; children?: React.ReactNode;
completedMessage?: string;
helpText?: string; helpText?: string;
showNumber?: boolean; showNumber?: boolean;
_stepNumber?: number; // Internal prop set by ToolStepContainer _stepNumber?: number; // Internal prop set by ToolStepContainer
@ -70,10 +68,8 @@ const ToolStep = ({
title, title,
isVisible = true, isVisible = true,
isCollapsed = false, isCollapsed = false,
isCompleted = false,
onCollapsedClick, onCollapsedClick,
children, children,
completedMessage,
helpText, helpText,
showNumber, showNumber,
_stepNumber, _stepNumber,
@ -135,20 +131,7 @@ const ToolStep = ({
)} )}
</Flex> </Flex>
{isCollapsed ? ( {!isCollapsed && (
<div>
{isCompleted && completedMessage && (
<Text size="sm" c="green">
{completedMessage}
{onCollapsedClick && (
<Text span c="dimmed" size="xs" ml="sm">
(click to change)
</Text>
)}
</Text>
)}
</div>
) : (
<Stack gap="md" pl="md"> <Stack gap="md" pl="md">
{helpText && ( {helpText && (
<Text size="sm" c="dimmed"> <Text size="sm" c="dimmed">

View File

@ -19,9 +19,7 @@ export function createFilesToolStep(
return createStep("Files", { return createStep("Files", {
isVisible: true, isVisible: true,
isCollapsed: props.isCollapsed, isCollapsed: props.isCollapsed,
isCompleted: hasFiles, onCollapsedClick: props.onCollapsedClick
onCollapsedClick: props.onCollapsedClick,
completedMessage: undefined
}, ( }, (
<FileStatusIndicator <FileStatusIndicator
selectedFiles={props.selectedFiles} selectedFiles={props.selectedFiles}

View File

@ -15,9 +15,7 @@ export interface MiddleStepConfig {
title: string; title: string;
isVisible?: boolean; isVisible?: boolean;
isCollapsed?: boolean; isCollapsed?: boolean;
isCompleted?: boolean;
onCollapsedClick?: () => void; onCollapsedClick?: () => void;
completedMessage?: string;
content: React.ReactNode; content: React.ReactNode;
tooltip?: { tooltip?: {
content?: React.ReactNode; content?: React.ReactNode;
@ -75,9 +73,7 @@ export function createToolFlow(config: ToolFlowConfig) {
steps.create(stepConfig.title, { steps.create(stepConfig.title, {
isVisible: stepConfig.isVisible, isVisible: stepConfig.isVisible,
isCollapsed: stepConfig.isCollapsed, isCollapsed: stepConfig.isCollapsed,
isCompleted: stepConfig.isCompleted,
onCollapsedClick: stepConfig.onCollapsedClick, onCollapsedClick: stepConfig.onCollapsedClick,
completedMessage: stepConfig.completedMessage,
tooltip: stepConfig.tooltip tooltip: stepConfig.tooltip
}, stepConfig.content) }, stepConfig.content)
)} )}

View File

@ -37,7 +37,7 @@ function stripExt(name: string): string {
return i > 0 ? name.slice(0, i) : name; return i > 0 ? name.slice(0, i) : name;
} }
// Signature must be (file, params) // Signature must be (file, params) to match useToolApiCalls interface
const buildFormData = (file: File, parameters: OCRParameters): FormData => { const buildFormData = (file: File, parameters: OCRParameters): FormData => {
const formData = new FormData(); const formData = new FormData();
formData.append('fileInput', file); formData.append('fileInput', file);
@ -99,7 +99,7 @@ export const useOCROperation = () => {
const ocrConfig: ToolOperationConfig<OCRParameters> = { const ocrConfig: ToolOperationConfig<OCRParameters> = {
operationType: 'ocr', operationType: 'ocr',
endpoint: '/api/v1/misc/ocr-pdf', endpoint: '/api/v1/misc/ocr-pdf',
buildFormData: buildFormData as any /* FIX ME */, buildFormData,
filePrefix: 'ocr_', filePrefix: 'ocr_',
multiFileEndpoint: false, // Process files individually multiFileEndpoint: false, // Process files individually
responseHandler, // use shared flow responseHandler, // use shared flow

View File

@ -170,7 +170,7 @@ export const useToolOperation = <TParams = void>(
// Individual file processing - separate API call per file // Individual file processing - separate API call per file
const apiCallsConfig: ApiCallsConfig<TParams> = { const apiCallsConfig: ApiCallsConfig<TParams> = {
endpoint: config.endpoint, endpoint: config.endpoint,
buildFormData: (file: File, params: TParams) => (config.buildFormData as (params: TParams, file: File) => FormData /* FIX ME */)(params, file), buildFormData: config.buildFormData as (file: File, params: TParams) => FormData,
filePrefix: config.filePrefix, filePrefix: config.filePrefix,
responseHandler: config.responseHandler responseHandler: config.responseHandler
}; };

View File

@ -74,9 +74,7 @@ const Compress = ({ onPreviewFile, onComplete, onError }: BaseToolProps) => {
steps: [{ steps: [{
title: "Settings", title: "Settings",
isCollapsed: settingsCollapsed, isCollapsed: settingsCollapsed,
isCompleted: hasResults,
onCollapsedClick: settingsCollapsed ? handleSettingsReset : undefined, onCollapsedClick: settingsCollapsed ? handleSettingsReset : undefined,
completedMessage: t("compress.header", "Compression completed"),
tooltip: compressTips, tooltip: compressTips,
content: ( content: (
<CompressSettings <CompressSettings

View File

@ -112,7 +112,6 @@ const Convert = ({ onPreviewFile, onComplete, onError }: BaseToolProps) => {
steps: [{ steps: [{
title: t("convert.settings", "Settings"), title: t("convert.settings", "Settings"),
isCollapsed: settingsCollapsed, isCollapsed: settingsCollapsed,
isCompleted: settingsCollapsed,
onCollapsedClick: settingsCollapsed ? handleSettingsReset : undefined, onCollapsedClick: settingsCollapsed ? handleSettingsReset : undefined,
content: ( content: (
<ConvertSettings <ConvertSettings

View File

@ -94,7 +94,6 @@ const OCR = ({ onPreviewFile, onComplete, onError }: BaseToolProps) => {
{ {
title: "Settings", title: "Settings",
isCollapsed: !hasFiles || settingsCollapsed, isCollapsed: !hasFiles || settingsCollapsed,
isCompleted: hasFiles && hasValidSettings,
onCollapsedClick: hasResults ? handleSettingsReset : () => { onCollapsedClick: hasResults ? handleSettingsReset : () => {
if (!hasFiles) return; // Only allow if files are selected if (!hasFiles) return; // Only allow if files are selected
setExpandedStep(expandedStep === 'settings' ? null : 'settings'); setExpandedStep(expandedStep === 'settings' ? null : 'settings');
@ -111,7 +110,6 @@ const OCR = ({ onPreviewFile, onComplete, onError }: BaseToolProps) => {
{ {
title: "Advanced", title: "Advanced",
isCollapsed: expandedStep !== 'advanced', isCollapsed: expandedStep !== 'advanced',
isCompleted: hasFiles && hasResults,
onCollapsedClick: hasResults ? handleSettingsReset : () => { onCollapsedClick: hasResults ? handleSettingsReset : () => {
if (!hasFiles) return; // Only allow if files are selected if (!hasFiles) return; // Only allow if files are selected
setExpandedStep(expandedStep === 'advanced' ? null : 'advanced'); setExpandedStep(expandedStep === 'advanced' ? null : 'advanced');

View File

@ -74,7 +74,6 @@ const Split = ({ onPreviewFile, onComplete, onError }: BaseToolProps) => {
steps: [{ steps: [{
title: "Settings", title: "Settings",
isCollapsed: settingsCollapsed, isCollapsed: settingsCollapsed,
isCompleted: hasResults,
onCollapsedClick: hasResults ? handleSettingsReset : undefined, onCollapsedClick: hasResults ? handleSettingsReset : undefined,
content: ( content: (
<SplitSettings <SplitSettings

View File

@ -15,9 +15,7 @@ export interface ToolStepConfig {
type: ToolStepType; type: ToolStepType;
title: string; title: string;
isVisible: boolean; isVisible: boolean;
isCompleted: boolean;
isCollapsed?: boolean; isCollapsed?: boolean;
completedMessage?: string;
onCollapsedClick?: () => void; onCollapsedClick?: () => void;
} }