Compare commits

..

No commits in common. "8fb5f54e9d54cf0cd2cc2bcb4f35199bbbc41873" and "9c49cf95841e0befc6b883d46e73c7c1829c7bfb" have entirely different histories.

7 changed files with 92 additions and 124 deletions

View File

@ -38,7 +38,7 @@ const FileHistoryGroup: React.FC<FileHistoryGroupProps> = ({
<Collapse in={isExpanded}>
<Box ml="md" mt="xs" mb="sm">
<Group align="center" mb="sm">
<Text size="xs" fw={600} c="dimmed">
<Text size="xs" fw={600} c="dimmed" tt="uppercase">
{t('fileManager.fileHistory', 'File History')} ({sortedHistory.length})
</Text>
</Group>

View File

@ -53,6 +53,55 @@ const FileInfoCard: React.FC<FileInfoCardProps> = ({
</Group>
<Divider />
{/* Standard PDF Metadata */}
{currentFile?.pdfMetadata?.title && (
<>
<Group justify="space-between" py="xs">
<Text size="sm" c="dimmed">{t('fileManager.title', 'Title')}</Text>
<Text size="sm" fw={500} style={{ maxWidth: '60%', textAlign: 'right' }} truncate>
{currentFile.pdfMetadata.title}
</Text>
</Group>
<Divider />
</>
)}
{currentFile?.pdfMetadata?.author && (
<>
<Group justify="space-between" py="xs">
<Text size="sm" c="dimmed">{t('fileManager.author', 'Author')}</Text>
<Text size="sm" fw={500} style={{ maxWidth: '60%', textAlign: 'right' }} truncate>
{currentFile.pdfMetadata.author}
</Text>
</Group>
<Divider />
</>
)}
{currentFile?.pdfMetadata?.subject && (
<>
<Group justify="space-between" py="xs">
<Text size="sm" c="dimmed">{t('fileManager.subject', 'Subject')}</Text>
<Text size="sm" fw={500} style={{ maxWidth: '60%', textAlign: 'right' }} truncate>
{currentFile.pdfMetadata.subject}
</Text>
</Group>
<Divider />
</>
)}
{currentFile?.pdfMetadata?.creationDate && (
<>
<Group justify="space-between" py="xs">
<Text size="sm" c="dimmed">{t('fileManager.created', 'Created')}</Text>
<Text size="sm" fw={500}>
{new Date(currentFile.pdfMetadata.creationDate).toLocaleDateString()}
</Text>
</Group>
<Divider />
</>
)}
<Group justify="space-between" py="xs">
<Text size="sm" c="dimmed">{t('fileManager.lastModified', 'Last Modified')}</Text>
<Text size="sm" fw={500}>

View File

@ -6,12 +6,12 @@ import { ToolOperationHook } from './useToolOperation';
import { BaseParametersHook } from './useBaseParameters';
import { StirlingFile } from '../../../types/fileContext';
interface BaseToolReturn<TParams, TParamsHook extends BaseParametersHook<TParams>> {
interface BaseToolReturn<TParams> {
// File management
selectedFiles: StirlingFile[];
// Tool-specific hooks
params: TParamsHook;
params: BaseParametersHook<TParams>;
operation: ToolOperationHook<TParams>;
// Endpoint validation
@ -33,13 +33,13 @@ interface BaseToolReturn<TParams, TParamsHook extends BaseParametersHook<TParams
/**
* Base tool hook for tool components. Manages standard behaviour for tools.
*/
export function useBaseTool<TParams, TParamsHook extends BaseParametersHook<TParams>>(
export function useBaseTool<TParams>(
toolName: string,
useParams: () => TParamsHook,
useParams: () => BaseParametersHook<TParams>,
useOperation: () => ToolOperationHook<TParams>,
props: BaseToolProps,
options?: { minFiles?: number }
): BaseToolReturn<TParams, TParamsHook> {
): BaseToolReturn<TParams> {
const minFiles = options?.minFiles ?? 1;
const { onPreviewFile, onComplete, onError } = props;

View File

@ -87,33 +87,17 @@ class IndexedDBManager {
request.onupgradeneeded = (event) => {
const db = request.result;
const oldVersion = event.oldVersion;
const transaction = request.transaction;
console.log(`Upgrading ${config.name} from v${oldVersion} to v${config.version}`);
// Create or update object stores
config.stores.forEach(storeConfig => {
let store: IDBObjectStore | undefined;
if (db.objectStoreNames.contains(storeConfig.name)) {
// Store exists - get reference for migration
// Store exists - for now, just continue (could add migration logic here)
console.log(`Object store '${storeConfig.name}' already exists`);
store = transaction?.objectStore(storeConfig.name);
return;
}
// Add new indexes if they don't exist
if (storeConfig.indexes && store) {
storeConfig.indexes.forEach(indexConfig => {
if (!store?.indexNames.contains(indexConfig.name)) {
store?.createIndex(
indexConfig.name,
indexConfig.keyPath,
{ unique: indexConfig.unique }
);
console.log(`Created index '${indexConfig.name}' on '${storeConfig.name}'`);
}
});
}
} else {
// Create new object store
const options: IDBObjectStoreParameters = {};
if (storeConfig.keyPath) {
@ -123,13 +107,13 @@ class IndexedDBManager {
options.autoIncrement = storeConfig.autoIncrement;
}
store = db.createObjectStore(storeConfig.name, options);
const store = db.createObjectStore(storeConfig.name, options);
console.log(`Created object store '${storeConfig.name}'`);
// Create indexes
if (storeConfig.indexes) {
storeConfig.indexes.forEach(indexConfig => {
store?.createIndex(
store.createIndex(
indexConfig.name,
indexConfig.keyPath,
{ unique: indexConfig.unique }
@ -137,85 +121,11 @@ class IndexedDBManager {
console.log(`Created index '${indexConfig.name}' on '${storeConfig.name}'`);
});
}
}
// Perform data migration for files database
if (config.name === 'stirling-pdf-files' && storeConfig.name === 'files' && store) {
this.migrateFileHistoryFields(store, oldVersion);
}
});
};
});
}
/**
* Migrate existing file records to include new file history fields
*/
private migrateFileHistoryFields(store: IDBObjectStore, oldVersion: number): void {
// Only migrate if upgrading from a version before file history was added (version < 3)
if (oldVersion >= 3) {
return;
}
console.log('Starting file history migration for existing records...');
const cursor = store.openCursor();
let migratedCount = 0;
cursor.onsuccess = (event) => {
const cursor = (event.target as IDBRequest).result;
if (cursor) {
const record = cursor.value;
let needsUpdate = false;
// Add missing file history fields with sensible defaults
if (record.isLeaf === undefined) {
record.isLeaf = true; // Existing files are unprocessed, should appear in recent files
needsUpdate = true;
}
if (record.versionNumber === undefined) {
record.versionNumber = 1; // Existing files are first version
needsUpdate = true;
}
if (record.originalFileId === undefined) {
record.originalFileId = record.id; // Existing files are their own root
needsUpdate = true;
}
if (record.parentFileId === undefined) {
record.parentFileId = undefined; // No parent for existing files
needsUpdate = true;
}
if (record.toolHistory === undefined) {
record.toolHistory = []; // No history for existing files
needsUpdate = true;
}
// Update the record if any fields were missing
if (needsUpdate) {
try {
cursor.update(record);
migratedCount++;
} catch (error) {
console.error('Failed to migrate record:', record.id, error);
}
}
cursor.continue();
} else {
// Migration complete
console.log(`File history migration completed. Migrated ${migratedCount} records.`);
}
};
cursor.onerror = (event) => {
console.error('File history migration failed:', (event.target as IDBRequest).error);
};
}
/**
* Get database connection (must be already opened)
*/

View File

@ -28,8 +28,8 @@ export interface BaseFileMetadata {
createdAt?: number; // When file was added to system
// File history tracking
isLeaf: boolean; // True if this file hasn't been processed yet
originalFileId: string; // Root file ID for grouping versions
isLeaf?: boolean; // True if this file hasn't been processed yet
originalFileId?: string; // Root file ID for grouping versions
versionNumber: number; // Version number in chain
parentFileId?: FileId; // Immediate parent file ID
toolHistory?: Array<{
@ -37,4 +37,14 @@ export interface BaseFileMetadata {
timestamp: number;
}>; // Tool chain for history tracking
// Standard PDF document metadata
pdfMetadata?: {
title?: string;
author?: string;
subject?: string;
creator?: string;
producer?: string;
creationDate?: Date;
modificationDate?: Date;
};
}

View File

@ -169,7 +169,6 @@ export function createNewStirlingFileStub(
size: file.size,
type: file.type,
lastModified: file.lastModified,
originalFileId: fileId,
quickKey: createQuickKey(file),
createdAt: Date.now(),
isLeaf: true, // New files are leaf nodes by default