Compare commits

..

No commits in common. "9a8110a7cb4cd306ee343b684bc429c84bfd8fb5" and "9eae2573fd4bff5db6e7f543aeaee73e3b2830cc" have entirely different histories.

16 changed files with 67 additions and 68 deletions

View File

@ -15,16 +15,17 @@ export default defineConfig(
},
{
rules: {
"no-empty": "off", // Temporarily disabled until codebase conformant
"no-empty-pattern": "off", // Temporarily disabled until codebase conformant
"no-undef": "off", // Temporarily disabled until codebase conformant
"@typescript-eslint/no-empty-object-type": [
"error",
{
// Allow empty extending interfaces because there's no real reason not to, and it makes it obvious where to put extra attributes in the future
allowInterfaces: 'with-single-extends',
},
],
"no-useless-escape": "off", // Temporarily disabled until codebase conformant
"no-case-declarations": "off", // Temporarily disabled until codebase conformant
"prefer-const": "off", // Temporarily disabled until codebase conformant
"@typescript-eslint/ban-ts-comment": "off", // Temporarily disabled until codebase conformant
"@typescript-eslint/no-empty-object-type": "off", // Temporarily disabled until codebase conformant
"@typescript-eslint/no-explicit-any": "off", // Temporarily disabled until codebase conformant
"@typescript-eslint/no-require-imports": "off", // Temporarily disabled until codebase conformant
"@typescript-eslint/no-unused-expressions": "off", // Temporarily disabled until codebase conformant
"@typescript-eslint/no-unused-vars": "off", // Temporarily disabled until codebase conformant
},
}

View File

@ -224,7 +224,7 @@ function getLicenseUrl(licenseType) {
// Handle complex SPDX expressions like "(MIT AND Zlib)" or "(MIT OR CC0-1.0)"
if (licenseType.includes('AND') || licenseType.includes('OR')) {
// Extract the first license from compound expressions for URL
const match = licenseType.match(/\(?\s*([A-Za-z0-9\-.]+)/);
const match = licenseType.match(/\(?\s*([A-Za-z0-9\-\.]+)/);
if (match && licenseUrls[match[1]]) {
return licenseUrls[match[1]];
}

View File

@ -17,7 +17,8 @@ import {
getActiveNavButton,
} from './quickAccessBar/QuickAccessBar';
const QuickAccessBar = forwardRef<HTMLDivElement>((_, ref) => {
const QuickAccessBar = forwardRef<HTMLDivElement>(({
}, ref) => {
const { t } = useTranslation();
const { isRainbowMode } = useRainbowThemeContext();
const { openFilesModal, isFilesModalOpen } = useFilesModalContext();

View File

@ -82,8 +82,8 @@ export function adjustFontSizeToFit(
return () => {
cancelAnimationFrame(raf);
try { ro.disconnect(); } catch { /* Ignore errors */ }
try { mo.disconnect(); } catch { /* Ignore errors */ }
try { ro.disconnect(); } catch {}
try { mo.disconnect(); } catch {}
};
}

View File

@ -3,6 +3,8 @@ import { Stack, Text, Divider, Card, Group } from '@mantine/core';
import { useTranslation } from 'react-i18next';
import { useSuggestedTools } from '../../../hooks/useSuggestedTools';
export interface SuggestedToolsSectionProps {}
export function SuggestedToolsSection(): React.ReactElement {
const { t } = useTranslation();
const suggestedTools = useSuggestedTools();

View File

@ -126,7 +126,7 @@ const ToolSearch = ({
key={id}
variant="subtle"
onClick={() => {
onToolSelect?.(id);
onToolSelect && onToolSelect(id);
setDropdownOpen(false);
}}
leftSection={<div style={{ color: "var(--tools-text-and-icon-color)" }}>{tool.icon}</div>}

View File

@ -6,7 +6,7 @@ import {
StirlingFileStub,
FileContextAction,
FileContextState,
toStirlingFileStub,
toWorkbenchFile,
createFileId,
createQuickKey
} from '../../types/fileContext';
@ -163,7 +163,7 @@ export async function addFiles(
}
// Create record with immediate thumbnail and page metadata
const record = toStirlingFileStub(file, fileId);
const record = toWorkbenchFile(file, fileId);
if (thumbnail) {
record.thumbnailUrl = thumbnail;
// Track blob URLs for cleanup (images return blob URLs that need revocation)
@ -205,7 +205,7 @@ export async function addFiles(
const fileId = createFileId();
filesRef.current.set(fileId, file);
const record = toStirlingFileStub(file, fileId);
const record = toWorkbenchFile(file, fileId);
if (thumbnail) {
record.thumbnailUrl = thumbnail;
// Track blob URLs for cleanup (images return blob URLs that need revocation)
@ -254,7 +254,7 @@ export async function addFiles(
filesRef.current.set(fileId, file);
const record = toStirlingFileStub(file, fileId);
const record = toWorkbenchFile(file, fileId);
// Generate processedFile metadata for stored files
let pageCount: number = 1;
@ -347,7 +347,7 @@ async function processFilesIntoRecords(
if (DEBUG) console.warn(`📄 Failed to generate thumbnail for file ${file.name}:`, error);
}
const record = toStirlingFileStub(file, fileId);
const record = toWorkbenchFile(file, fileId);
if (thumbnail) {
record.thumbnailUrl = thumbnail;
}

View File

@ -347,9 +347,9 @@ describe('useConvertParameters - Auto Detection & Smart Conversion', () => {
const malformedFiles: Array<{name: string}> = [
{ name: 'valid.pdf' },
// @ts-expect-error - Testing runtime resilience
// @ts-ignore - Testing runtime resilience
{ name: null },
// @ts-expect-error - Testing runtime resilience
// @ts-ignore
{ name: undefined }
];

View File

@ -172,7 +172,7 @@ export const useToolOperation = <TParams>(
const validRegularFiles = extractFiles(validFiles);
switch (config.toolType) {
case ToolType.singleFile: {
case ToolType.singleFile:
// Individual file processing - separate API call per file
const apiCallsConfig: ApiCallsConfig<TParams> = {
endpoint: config.endpoint,
@ -188,9 +188,8 @@ export const useToolOperation = <TParams>(
actions.setStatus
);
break;
}
case ToolType.multiFile: {
case ToolType.multiFile:
// Multi-file processing - single API call with all files
actions.setStatus('Processing files...');
const formData = config.buildFormData(params, validRegularFiles);
@ -218,7 +217,6 @@ export const useToolOperation = <TParams>(
}
}
break;
}
case ToolType.custom:
actions.setStatus('Processing files...');
@ -252,7 +250,7 @@ export const useToolOperation = <TParams>(
inputFileIds.push(fileId);
inputStirlingFileStubs.push(record);
} else {
console.warn(`No file stub found for file: ${file.name}`);
console.warn(`No file record found for file: ${file.name}`);
}
}
@ -321,7 +319,6 @@ export const useToolOperation = <TParams>(
// Undo the consume operation
await undoConsumeFiles(inputFiles, inputStirlingFileStubs, outputFileIds);
// Clear results and operation tracking
resetResults();
lastOperationRef.current = null;

View File

@ -35,11 +35,8 @@ function updatePosthogConsent(){
return;
}
const optIn = (window.CookieConsent as any).acceptedCategory('analytics');
if (optIn) {
posthog.opt_in_capturing();
} else {
posthog.opt_out_capturing();
}
optIn?
posthog.opt_in_capturing() : posthog.opt_out_capturing();
console.log("Updated analytics consent: ", optIn? "opted in" : "opted out");
}

View File

@ -460,14 +460,13 @@ export class EnhancedPDFProcessingService {
case 'failed':
this.metrics.failedFiles++;
break;
case 'cacheHit': {
case 'cacheHit':
// Update cache hit rate
const totalAttempts = this.metrics.totalFiles + 1;
this.metrics.cacheHitRate = (this.metrics.cacheHitRate * this.metrics.totalFiles + 1) / totalAttempts;
break;
}
}
}
/**
* Get processing metrics
@ -521,7 +520,10 @@ export class EnhancedPDFProcessingService {
this.notifyListeners();
// Force memory cleanup hint
setTimeout(() => window.gc?.(), 100);
if (typeof window !== 'undefined' && window.gc) {
let gc = window.gc;
setTimeout(() => gc(), 100);
}
}
/**

View File

@ -148,17 +148,15 @@ export class FileAnalyzer {
case 'immediate_full':
return pageCount * baseTime;
case 'priority_pages': {
case 'priority_pages':
// Estimate time for priority pages (first 10)
const priorityPages = Math.min(pageCount, 10);
return priorityPages * baseTime;
}
case 'progressive_chunked': {
case 'progressive_chunked':
// Estimate time for first chunk (20 pages)
const firstChunk = Math.min(pageCount, 20);
return firstChunk * baseTime;
}
default:
return pageCount * baseTime;

View File

@ -92,6 +92,8 @@ class IndexedDBManager {
// Create or update object stores
config.stores.forEach(storeConfig => {
let store: IDBObjectStore;
if (db.objectStoreNames.contains(storeConfig.name)) {
// Store exists - for now, just continue (could add migration logic here)
console.log(`Object store '${storeConfig.name}' already exists`);
@ -107,7 +109,7 @@ class IndexedDBManager {
options.autoIncrement = storeConfig.autoIncrement;
}
const store = db.createObjectStore(storeConfig.name, options);
store = db.createObjectStore(storeConfig.name, options);
console.log(`Created object store '${storeConfig.name}'`);
// Create indexes

View File

@ -94,7 +94,6 @@ class PDFWorkerManager {
try {
loadingTask.destroy();
} catch (destroyError) {
// Ignore errors
}
}
throw error;
@ -167,7 +166,6 @@ class PDFWorkerManager {
try {
pdf.destroy();
} catch (error) {
// Ignore errors
}
});

View File

@ -150,10 +150,7 @@ export function isFileObject(obj: any): obj is File | StirlingFile {
export function toStirlingFileStub(
file: File,
id?: FileId
): StirlingFileStub {
export function toWorkbenchFile(file: File, id?: FileId): StirlingFileStub {
const fileId = id || createFileId();
return {
id: fileId,
@ -353,3 +350,6 @@ export interface FileContextActionsValue {
actions: FileContextActions;
dispatch: (action: FileContextAction) => void;
}
// TODO: URL parameter types will be redesigned for new routing system

View File

@ -1,6 +1,7 @@
// Base parameter interfaces for reusable patterns
// Base interface that all tool parameters should extend
// Provides a foundation for adding common properties across all tools
// Examples of future additions: userId, sessionId, commonFlags, etc.
export type BaseParameters = object
export interface BaseParameters {
// Base interface that all tool parameters should extend
// Provides a foundation for adding common properties across all tools
// Examples of future additions: userId, sessionId, commonFlags, etc.
}