fix(api): stabilize DOCX to PDF conversion and cleanup

- Isolate concurrent LibreOffice runs with per-job profile directories
  (soffice -env:UserInstallation) and per-job temp folders
- Poll for generated PDF and verify stable file size before reading
- Consolidate temp artifacts under docstore/tmp and clean up via rm -r
- Add headless/nologo flags and improve error handling

ui(accessibility):
- ConfirmDialog exposes proper dialog roles
- DocumentFolder toggle adds type, aria-expanded, aria-controls, and title;
  associate content panel with an id
- HTMLViewer wraps content in .html-container for HTML/TXT views

test: add comprehensive Playwright specs and helpers
- Accessibility, API health, deletion flows, folders, navigation, playback,
  and upload scenarios
- Add sample.md and unsupported.xyz; update sample.pdf; extend helpers

ci: run Playwright workflow on version1.0.0 branch

Author: richardr1126

.github/workflows/playwright.yml

@@ -1,7 +1,7 @@
 name: Playwright Tests
 on:
   push:
-    branches: [ main, master ]
+    branches: [ main, master, version1.0.0 ]
   pull_request:
     branches: [ main, master ]
 jobs:

src/app/api/documents/docx-to-pdf/route.ts

@@ -1,26 +1,40 @@
 import { NextRequest, NextResponse } from 'next/server';
-import { writeFile, mkdir, unlink, readFile } from 'fs/promises';
+import { writeFile, mkdir, readFile, readdir, rm, stat } from 'fs/promises';
 import { spawn } from 'child_process';
 import path from 'path';
 import { existsSync } from 'fs';
 import { randomUUID } from 'crypto';
+import { pathToFileURL } from 'url';
 
-const TEMP_DIR = path.join(process.cwd(), 'temp');
+const DOCSTORE_DIR = path.join(process.cwd(), 'docstore');
+const TEMP_DIR = path.join(DOCSTORE_DIR, 'tmp');
 
 async function ensureTempDir() {
+  if (!existsSync(DOCSTORE_DIR)) {
+    await mkdir(DOCSTORE_DIR, { recursive: true });
+  }
   if (!existsSync(TEMP_DIR)) {
     await mkdir(TEMP_DIR, { recursive: true });
   }
 }
 
-async function convertDocxToPdf(inputPath: string, outputDir: string): Promise<void> {
+async function convertDocxToPdf(inputPath: string, outputDir: string, profileDir?: string): Promise<void> {
   return new Promise((resolve, reject) => {
-    const process = spawn('soffice', [
+    const args: string[] = [];
+    if (profileDir) {
+      // Ensure a per-job profile to isolate concurrent soffice instances
+      // Note: mkdir is async; we prepare the directory before calling this in POST, but safe to include here too
+      // (we avoid awaiting here; POST ensures creation)
+      args.push(`-env:UserInstallation=${pathToFileURL(profileDir).toString()}`);
+    }
+    args.push(
       '--headless',
+      '--nologo',
       '--convert-to', 'pdf',
       '--outdir', outputDir,
       inputPath
-    ]);
+    );
+    const process = spawn('soffice', args);
 
     process.on('error', (error) => {
       reject(error);
@@ -36,6 +50,29 @@ async function convertDocxToPdf(inputPath: string, outputDir: string): Promise<v
   });
 }
 
+async function waitForPdfReady(dir: string, timeoutMs = 20000, intervalMs = 100): Promise<string> {
+  const end = Date.now() + timeoutMs;
+  while (Date.now() < end) {
+    const files = await readdir(dir);
+    const pdf = files.find(f => f.toLowerCase().endsWith('.pdf'));
+    if (pdf) {
+      const pdfPath = path.join(dir, pdf);
+      try {
+        const first = await stat(pdfPath);
+        await new Promise((res) => setTimeout(res, intervalMs));
+        const second = await stat(pdfPath);
+        if (second.size > 0 && second.size === first.size) {
+          return pdfPath;
+        }
+      } catch {
+        // If stat fails (transient), continue polling
+      }
+    }
+    await new Promise((res) => setTimeout(res, intervalMs));
+  }
+  throw new Error(`PDF not ready in ${dir} after ${timeoutMs}ms`);
+}
+
 export async function POST(req: NextRequest) {
   try {
     await ensureTempDir();
@@ -59,24 +96,25 @@ export async function POST(req: NextRequest) {
 
     const buffer = Buffer.from(await file.arrayBuffer());
     const tempId = randomUUID();
-    const inputPath = path.join(TEMP_DIR, `${tempId}.docx`);
-    const outputPath = path.join(TEMP_DIR, `${tempId}.pdf`);
+    const jobDir = path.join(TEMP_DIR, tempId);
+    await mkdir(jobDir, { recursive: true });
+    const profileDir = path.join(jobDir, 'lo-profile');
+    await mkdir(profileDir, { recursive: true });
+    const inputPath = path.join(jobDir, 'input.docx');
 
     // Write the uploaded file
     await writeFile(inputPath, buffer);
 
     try {
       // Convert the file
-      await convertDocxToPdf(inputPath, TEMP_DIR);
+      await convertDocxToPdf(inputPath, jobDir, profileDir);
 
       // Return the PDF file
-      const pdfContent = await readFile(outputPath);
-      
+      const pdfPath = await waitForPdfReady(jobDir);
+      const pdfContent = await readFile(pdfPath);
+
       // Clean up temp files
-      await Promise.all([
-        unlink(inputPath),
-        unlink(outputPath)
-      ]).catch(console.error);
+      await rm(jobDir, { recursive: true, force: true }).catch(console.error);
 
       return new NextResponse(pdfContent, {
         headers: {
@@ -86,11 +124,7 @@ export async function POST(req: NextRequest) {
       });
     } catch (error) {
       // Clean up temp files on error
-      await Promise.all([
-        unlink(inputPath),
-        unlink(outputPath)
-      ]).catch(console.error);
-      
+      await rm(jobDir, { recursive: true, force: true }).catch(console.error);
       throw error;
     }
   } catch (error) {

src/components/ConfirmDialog.tsx

@@ -33,6 +33,7 @@ export function ConfirmDialog({
     <Transition appear show={isOpen} as={Fragment}>
       <Dialog 
         as="div" 
+        role={undefined}
         className="relative z-50" 
         onClose={onClose}
         onKeyDown={handleKeyDown}
@@ -60,7 +61,7 @@ export function ConfirmDialog({
               leaveFrom="opacity-100 scale-100"
               leaveTo="opacity-0 scale-95"
             >
-              <DialogPanel className="w-full max-w-md transform rounded-2xl bg-base p-6 text-left align-middle shadow-xl transition-all">
+              <DialogPanel role='dialog' className="w-full max-w-md transform rounded-2xl bg-base p-6 text-left align-middle shadow-xl transition-all">
                 <DialogTitle
                   as="h3"
                   className="text-lg font-semibold leading-6 text-foreground"

src/components/HTMLViewer.tsx

@@ -24,7 +24,7 @@ export function HTMLViewer({ className = '' }: HTMLViewerProps) {
     return (
       <div className={`flex flex-col h-full ${className}`} ref={containerRef}>
         <div className="flex-1 overflow-auto">
-          <div className={`min-w-full px-4 py-4 ${isTxtFile ? 'whitespace-pre-wrap font-mono text-sm' : 'prose prose-base'}`}>
+          <div className={`html-container min-w-full px-4 py-4 ${isTxtFile ? 'whitespace-pre-wrap font-mono text-sm' : 'prose prose-base'}`}>
             {isTxtFile ? (
               currDocData
             ) : (

src/components/doclist/DocumentFolder.tsx

@@ -70,11 +70,15 @@ export function DocumentFolder({
             <div className="flex items-center">
               <h3 className="text-sm px-1 font-semibold leading-tight">{folder.name}</h3>
                 <Button
-                onClick={() => onToggleCollapse(folder.id)}
-                className="ml-0.5 transform transition-transform duration-200 ease-in-out hover:scale-[1.08] hover:text-accent"
-                aria-label={isCollapsed ? "Expand folder" : "Collapse folder"}
+                  type="button"
+                  onClick={() => onToggleCollapse(folder.id)}
+                  className="ml-0.5 p-1 inline-flex items-center justify-center transform transition-transform duration-200 ease-in-out hover:scale-[1.08] hover:text-accent"
+                  aria-label={isCollapsed ? "Expand folder" : "Collapse folder"}
+                  aria-expanded={!isCollapsed}
+                  aria-controls={`folder-panel-${folder.id}`}
+                  title={isCollapsed ? "Expand folder" : "Collapse folder"}
                 >
-                <ChevronIcon className={`transform transition-transform duration-300 ease-in-out ${isCollapsed ? '-rotate-180' : ''}`} />
+                  <ChevronIcon className={`w-4 h-4 transform transition-transform duration-300 ease-in-out ${isCollapsed ? '-rotate-180' : ''}`} />
                 </Button>
             </div>
             <Button
@@ -98,7 +102,7 @@ export function DocumentFolder({
               leaveFrom="transform scale-y-100 opacity-100 max-h-[1000px]"
               leaveTo="transform scale-y-0 opacity-0 max-h-0"
             >
-              <div className="space-y-1 origin-top">
+              <div id={`folder-panel-${folder.id}`} className="space-y-1 origin-top">
                 {sortedDocuments.map(doc => (
                   <DocumentListItem
                     key={`${doc.type}-${doc.id}`}

tests/accessibility.spec.ts

@@ -0,0 +1,88 @@
+import { test, expect } from '@playwright/test';
+import {
+  setupTest,
+  uploadFiles,
+  ensureDocumentsListed,
+  uploadAndDisplay,
+  expectProcessingTransition,
+} from './helpers';
+
+test.describe('Accessibility smoke', () => {
+  test.beforeEach(async ({ page }) => {
+    await setupTest(page);
+  });
+
+  test('dropzone input and hint text are accessible', async ({ page }) => {
+    // Input is present and visible
+    await expect(page.locator('input[type="file"]')).toBeVisible({ timeout: 10000 });
+
+    // Hint text present (supports compact or default variants)
+    await expect(
+      page.getByText(/Drop your file\(s\) here|Drop files or click|Drop your file\(s\) here, or click to select/i)
+    ).toBeVisible();
+  });
+
+  test('document links have roles and accessible names', async ({ page }) => {
+    await uploadFiles(page, 'sample.pdf', 'sample.epub', 'sample.txt');
+    await ensureDocumentsListed(page, ['sample.pdf', 'sample.epub', 'sample.txt']);
+
+    await expect(page.getByRole('link', { name: /sample\.pdf/i })).toBeVisible();
+    await expect(page.getByRole('link', { name: /sample\.epub/i })).toBeVisible();
+    await expect(page.getByRole('link', { name: /sample\.txt/i })).toBeVisible();
+  });
+
+  test('ConfirmDialog exposes role=dialog with title and actions', async ({ page }) => {
+    await uploadFiles(page, 'sample.pdf');
+    await ensureDocumentsListed(page, ['sample.pdf']);
+
+    // Open the confirm dialog by clicking the row delete button
+    await page.getByRole('button', { name: 'Delete document' }).first().click();
+
+    // Title and dialog role visible
+    const heading = page.getByRole('heading', { name: 'Delete Document' });
+    await expect(heading).toBeVisible({ timeout: 10000 });
+    const dialog = heading.locator('xpath=ancestor::*[@role="dialog"][1]');
+    await expect(dialog).toBeVisible();
+
+    // Has a destructive action (Delete)
+    await expect(dialog.getByRole('button', { name: 'Delete' })).toBeVisible();
+
+    // Close with Escape to avoid deleting test data
+    await page.keyboard.press('Escape');
+  });
+
+  test('TTS controls expose aria labels and are keyboard focusable', async ({ page }) => {
+    await uploadAndDisplay(page, 'sample.pdf');
+
+    // TTS bar present
+    const ttsbar = page.locator('[data-app-ttsbar]');
+    await expect(ttsbar).toBeVisible();
+
+    // Verify control labels
+    const backBtn = page.getByRole('button', { name: 'Skip backward' });
+    const playBtn = page.getByRole('button', { name: 'Play' });
+    const fwdBtn = page.getByRole('button', { name: 'Skip forward' });
+
+    await expect(backBtn).toBeVisible();
+    await expect(playBtn).toBeVisible();
+    await expect(fwdBtn).toBeVisible();
+
+    // Keyboard focus checks
+    await page.focus('button[aria-label="Skip backward"]');
+    await expect(backBtn).toBeFocused();
+
+    await page.focus('button[aria-label="Play"]');
+    await expect(playBtn).toBeFocused();
+
+    await page.focus('button[aria-label="Skip forward"]');
+    await expect(fwdBtn).toBeFocused();
+
+    // Toggle play and verify aria-label swap to Pause, then back to Play
+    await playBtn.click();
+    await expectProcessingTransition(page);
+    await expect(page.getByRole('button', { name: 'Pause' })).toBeVisible();
+
+    await page.getByRole('button', { name: 'Pause' }).click();
+    await expect(page.getByRole('button', { name: 'Play' })).toBeVisible({ timeout: 10000 });
+  });
+});

tests/api.spec.ts

@@ -0,0 +1,20 @@
+import { test, expect } from '@playwright/test';
+
+test.describe('API health checks', () => {
+  test('GET /api/tts/voices returns 200 and a non-empty voices array', async ({ request }) => {
+    const res = await request.get('/api/tts/voices');
+    expect(res.ok()).toBeTruthy();
+    const json = await res.json();
+    expect(Array.isArray(json.voices)).toBeTruthy();
+    expect(json.voices.length).toBeGreaterThan(0);
+  });
+
+  test('GET /api/audio/convert/chapters returns 200 with exists flag and chapters array', async ({ request }) => {
+    const bookId = `healthcheck-${Date.now()}`;
+    const res = await request.get(`/api/audio/convert/chapters?bookId=${bookId}`);
+    expect(res.ok()).toBeTruthy();
+    const json = await res.json();
+    expect(json).toHaveProperty('exists');
+    expect(Array.isArray(json.chapters)).toBeTruthy();
+  });
+});

tests/delete.spec.ts

@@ -0,0 +1,48 @@
+import { test, expect } from '@playwright/test';
+import { setupTest, uploadFile, expectDocumentListed, expectNoDocumentLink, deleteDocumentByName, deleteAllLocalDocuments, ensureDocumentsListed } from './helpers';
+
+test.describe('Document deletion flow', () => {
+  test.beforeEach(async ({ page }) => {
+    await setupTest(page);
+  });
+
+  test('deletes a document and updates list', async ({ page }) => {
+    // Upload two documents
+    await uploadFile(page, 'sample.pdf');
+    await uploadFile(page, 'sample.txt');
+
+    // Verify both appear
+    await expectDocumentListed(page, 'sample.pdf');
+    await expectDocumentListed(page, 'sample.txt');
+
+    // Delete the TXT document via row action
+    await deleteDocumentByName(page, 'sample.txt');
+
+    // Assert the TXT document is removed, PDF remains
+    await expectNoDocumentLink(page, 'sample.txt');
+    await expectDocumentListed(page, 'sample.pdf');
+
+    // Optional: summary exists (best-effort)
+    const summary = page.locator('[data-doc-summary]');
+    await expect(summary).toBeVisible();
+  });
+
+  test('deletes all local documents from Settings modal', async ({ page }) => {
+    // Upload multiple docs (PDF + EPUB)
+    await uploadFile(page, 'sample.pdf');
+    await uploadFile(page, 'sample.epub');
+
+    // Verify both appear
+    await ensureDocumentsListed(page, ['sample.pdf', 'sample.epub']);
+
+    // Delete all local documents via Settings
+    await deleteAllLocalDocuments(page);
+
+    // Assert both documents are removed
+    await expectNoDocumentLink(page, 'sample.pdf');
+    await expectNoDocumentLink(page, 'sample.epub');
+
+    // Uploader should be visible when no docs remain
+    await expect(page.locator('input[type=file]')).toBeVisible({ timeout: 10000 });
+  });
+});

tests/files/sample.md

@@ -0,0 +1,20 @@
+# Sample Markdown
+
+This is a test document with basic Markdown elements.
+
+## Section One
+
+- Item 1
+- Item 2
+
+Visit [OpenAI](https://www.openai.com) for more information.
+
+### Subsection
+
+Bold text: **strong**; Italic text: _emphasis_.
+
+Code:
+
+```js
+console.log('hello markdown');
+```