- |
+ |
Detailed File Information:
{job.files_detailed.map((file: Filename) => (
@@ -156,7 +211,7 @@ export const history = new Elysia().use(userService).get(
xmlns="http://www.w3.org/2000/svg"
viewBox="0 0 20 20"
fill="currentColor"
- class={`mx-2 inline-block h-4 w-4 text-neutral-500`}
+ class={`mx-2 inline-block size-4 text-neutral-500`}
>
{`
document.addEventListener('DOMContentLoaded', () => {
+ // Expand/collapse job details
const toggles = document.querySelectorAll('.job-details-toggle');
toggles.forEach(toggle => {
toggle.addEventListener('click', function() {
const jobId = this.dataset.jobId;
const detailsRow = document.getElementById(\`details-\${jobId}\`);
- // The arrow SVG itself has the ID arrow-\${jobId}
const arrow = document.getElementById(\`arrow-\${jobId}\`);
if (detailsRow && arrow) {
detailsRow.classList.toggle("hidden");
if (detailsRow.classList.contains("hidden")) {
- // Right-facing arrow (collapsed)
arrow.innerHTML = '';
} else {
- // Down-facing arrow (expanded)
arrow.innerHTML = '';
}
}
});
});
+
+ // Checkbox management
+ const selectAllCheckbox = document.getElementById('select-all');
+ const jobCheckboxes = document.querySelectorAll('[data-checkbox-type="job"]');
+ const deleteSelectedBtn = document.getElementById('delete-selected-btn');
+ const deleteSelectedContainer = document.getElementById('delete-selected-container');
+ const selectedCountSpan = document.getElementById('selected-count');
+
+ function updateDeleteButton() {
+ const checkedBoxes = Array.from(jobCheckboxes).filter(cb => cb.checked);
+ if (checkedBoxes.length > 0) {
+ deleteSelectedBtn.disabled = false;
+ selectedCountSpan.textContent = checkedBoxes.length;
+ } else {
+ deleteSelectedBtn.disabled = true;
+ selectedCountSpan.textContent = '0';
+ }
+ }
+
+ selectAllCheckbox?.addEventListener('change', function() {
+ jobCheckboxes.forEach(checkbox => {
+ checkbox.checked = this.checked;
+ });
+ updateDeleteButton();
+ });
+
+ jobCheckboxes.forEach(checkbox => {
+ checkbox.addEventListener('change', function() {
+ const allChecked = Array.from(jobCheckboxes).every(cb => cb.checked);
+ const someChecked = Array.from(jobCheckboxes).some(cb => cb.checked);
+ if (selectAllCheckbox) {
+ selectAllCheckbox.checked = allChecked;
+ selectAllCheckbox.indeterminate = someChecked && !allChecked;
+ }
+ updateDeleteButton();
+ });
+ });
+
+ deleteSelectedBtn?.addEventListener('click', async function() {
+ const checkedBoxes = Array.from(jobCheckboxes).filter(cb => cb.checked);
+ const jobIds = checkedBoxes.map(cb => cb.dataset.jobId);
+
+ if (jobIds.length === 0) return;
+
+ const confirmed = confirm(\`Are you sure you want to delete \${jobIds.length} job(s)? This action cannot be undone.\`);
+ if (!confirmed) return;
+
+ try {
+ const response = await fetch('${WEBROOT}/delete-multiple', {
+ method: 'POST',
+ headers: {
+ 'Content-Type': 'application/json',
+ },
+ body: JSON.stringify({ jobIds }),
+ });
+
+ if (!response.ok) {
+ throw new Error(\`HTTP error! status: \${response.status}\`);
+ }
+
+ const result = await response.json();
+
+ if (result.success || result.deleted > 0) {
+ alert(\`Successfully deleted \${result.deleted} job(s).\${result.failed > 0 ? \` Failed to delete \${result.failed} job(s).\` : ''}\`);
+ window.location.reload();
+ } else {
+ alert('Failed to delete jobs. Please try again.');
+ }
+ } catch (error) {
+ console.error('Error deleting jobs:', error);
+ alert('An error occurred while deleting jobs. Please try again.');
+ }
+ });
});
`}
diff --git a/src/pages/listConverters.tsx b/src/pages/listConverters.tsx
index 71343b8..2e3cc77 100644
--- a/src/pages/listConverters.tsx
+++ b/src/pages/listConverters.tsx
@@ -22,7 +22,7 @@ export const listConverters = new Elysia().use(userService).get(
Converters
Results
+
|
@@ -72,7 +78,7 @@ function ResultsArticle({
|
@@ -80,7 +86,7 @@ function ResultsArticle({
|
@@ -163,7 +169,7 @@ export const results = new Elysia()
sm:px-4
`}
>
-
+
@@ -199,7 +205,7 @@ export const results = new Elysia()
.as(Filename)
.all(params.jobId);
- return ;
+ return ;
},
{ auth: true },
);
diff --git a/src/pages/root.tsx b/src/pages/root.tsx
index f2a83de..bf1a73c 100644
--- a/src/pages/root.tsx
+++ b/src/pages/root.tsx
@@ -126,8 +126,9 @@ export const root = new Elysia().use(userService).get(
@@ -135,8 +136,8 @@ export const root = new Elysia().use(userService).get(
@@ -190,7 +191,7 @@ export const root = new Elysia().use(userService).get(
// https://stackoverflow.com/questions/121499/when-a-blur-event-occurs-how-can-i-find-out-which-element-focus-went-to#comment82388679_33325953
tabindex={0}
class={`
- target rounded bg-neutral-700 p-1 text-base
+ target rounded-sm bg-neutral-700 p-1 text-base
hover:bg-neutral-600
`}
data-value={`${target},${converter}`}
diff --git a/src/pages/upload.tsx b/src/pages/upload.tsx
index eeaae1c..0c00009 100644
--- a/src/pages/upload.tsx
+++ b/src/pages/upload.tsx
@@ -3,6 +3,7 @@ import db from "../db/db";
import { WEBROOT } from "../helpers/env";
import { uploadsDir } from "../index";
import { userService } from "./user";
+import sanitize from "sanitize-filename";
export const upload = new Elysia().use(userService).post(
"/upload",
@@ -24,10 +25,12 @@ export const upload = new Elysia().use(userService).post(
if (body?.file) {
if (Array.isArray(body.file)) {
for (const file of body.file) {
- await Bun.write(`${userUploadsDir}${file.name}`, file);
+ const santizedFileName = sanitize(file.name);
+ await Bun.write(`${userUploadsDir}${santizedFileName}`, file);
}
} else {
- await Bun.write(`${userUploadsDir}${body.file["name"]}`, body.file);
+ const santizedFileName = sanitize(body.file["name"]);
+ await Bun.write(`${userUploadsDir}${santizedFileName}`, body.file);
}
}
diff --git a/tests/converters/dasel.test.ts b/tests/converters/dasel.test.ts
new file mode 100644
index 0000000..b08bed9
--- /dev/null
+++ b/tests/converters/dasel.test.ts
@@ -0,0 +1,72 @@
+import fs from "fs";
+import { beforeEach, afterEach, expect, test, describe } from "bun:test";
+import { convert } from "../../src/converters/dasel";
+import type { ExecFileFn } from "../../src/converters/types";
+
+const originalWriteFile = fs.writeFile;
+
+describe("convert", () => {
+ let mockExecFile: ExecFileFn;
+
+ beforeEach(() => {
+ // mock fs.writeFile
+ // @ts-expect-error: property __promisify__ is missing
+ fs.writeFile = (path, data, cb) => cb(null);
+ // mock execFile
+ mockExecFile = (cmd, args, callback) => callback(null, "output-data", "");
+ });
+
+ afterEach(() => {
+ // reset fs.writeFile
+ fs.writeFile = originalWriteFile;
+ });
+
+ test("should call dasel with correct arguments and write output", async () => {
+ let calledArgs: Parameters = ["", [], () => {}];
+ mockExecFile = (cmd, args, callback) => {
+ calledArgs = [cmd, args, callback];
+ callback(null, "output-data", "");
+ };
+
+ let writeFileCalled = false;
+ // @ts-expect-error: property __promisify__ is missing
+ fs.writeFile = (path, data, cb) => {
+ writeFileCalled = true;
+ expect(path).toBe("output.json");
+ expect(data).toBe("output-data");
+ // @ts-expect-error: could not be callable with null
+ cb(null);
+ };
+
+ const result = await convert(
+ "input.yaml",
+ "yaml",
+ "json",
+ "output.json",
+ undefined,
+ mockExecFile,
+ );
+
+ expect(calledArgs[0]).toBe("dasel");
+ expect(calledArgs[1]).toEqual(["--file", "input.yaml", "--read", "yaml", "--write", "json"]);
+ expect(writeFileCalled).toBe(true);
+ expect(result).toBe("Done");
+ });
+
+ test("should reject if execFile returns an error", async () => {
+ mockExecFile = (cmd, args, callback) => callback(new Error("fail"), "", "");
+ await expect(
+ convert("input.yaml", "yaml", "json", "output.json", undefined, mockExecFile),
+ ).rejects.toMatch(/error: Error: fail/);
+ });
+
+ test("should reject if writeFile fails", async () => {
+ // @ts-expect-error: property __promisify__ is missing
+ fs.writeFile = (path, data, cb) => cb(new Error("write fail"));
+ await expect(
+ convert("input.yaml", "yaml", "json", "output.json", undefined, (cmd, args, cb) =>
+ cb(null, "output-data", ""),
+ ),
+ ).rejects.toMatch(/Failed to write output/);
+ });
+});
diff --git a/tests/converters/libreoffice.test.ts b/tests/converters/libreoffice.test.ts
new file mode 100644
index 0000000..5d34d45
--- /dev/null
+++ b/tests/converters/libreoffice.test.ts
@@ -0,0 +1,168 @@
+import { afterEach, beforeEach, expect, test } from "bun:test";
+import { convert } from "../../src/converters/libreoffice";
+import type { ExecFileFn } from "../../src/converters/types";
+
+function requireDefined(value: T, msg: string): NonNullable {
+ if (value === undefined || value === null) throw new Error(msg);
+ return value as NonNullable;
+}
+
+// --- capture/inspect execFile calls -----------------------------------------
+type Call = { cmd: string; args: string[] };
+let calls: Call[] = [];
+
+let behavior:
+ | { kind: "success"; stdout?: string; stderr?: string }
+ | { kind: "error"; message?: string; stderr?: string } = { kind: "success" };
+
+const mockExecFile: ExecFileFn = (cmd, args, cb) => {
+ calls.push({ cmd, args });
+ if (behavior.kind === "error") {
+ cb(new Error(behavior.message ?? "mock failure"), "", behavior.stderr ?? "");
+ } else {
+ cb(null, behavior.stdout ?? "ok", behavior.stderr ?? "");
+ }
+ // We don't return a real ChildProcess in tests.
+ return undefined;
+};
+
+// --- capture console output (no terminal noise) ------------------------------
+let logs: string[] = [];
+let errors: string[] = [];
+
+const originalLog = console.log;
+const originalError = console.error;
+
+// Use Console["log"] for typing; avoids explicit `any`
+const makeSink =
+ (sink: string[]): Console["log"] =>
+ (...data) => {
+ sink.push(data.map(String).join(" "));
+ };
+
+beforeEach(() => {
+ calls = [];
+ behavior = { kind: "success" };
+
+ logs = [];
+ errors = [];
+ console.log = makeSink(logs);
+ console.error = makeSink(errors);
+});
+
+afterEach(() => {
+ console.log = originalLog;
+ console.error = originalError;
+});
+
+// --- core behavior -----------------------------------------------------------
+test("invokes soffice with --headless and outdir derived from targetPath", async () => {
+ await convert("in.docx", "docx", "odt", "out/out.odt", undefined, mockExecFile);
+
+ const { cmd, args } = requireDefined(calls[0], "Expected at least one execFile call");
+ expect(cmd).toBe("soffice");
+ expect(args).toEqual([
+ "--headless",
+ "--infilter=MS Word 2007 XML",
+ "--convert-to",
+ "odt:writer8",
+ "--outdir",
+ "out",
+ "in.docx",
+ ]);
+});
+
+test("uses only outFilter when input has no filter (e.g., pdf -> txt)", async () => {
+ await convert("in.pdf", "pdf", "txt", "out/out.txt", undefined, mockExecFile);
+
+ const { args } = requireDefined(calls[0], "Expected at least one execFile call");
+
+ expect(args).toEqual([
+ "--headless",
+ "--infilter=writer_pdf_import",
+ "--convert-to",
+ "txt:Text",
+ "--outdir",
+ "out",
+ "in.pdf",
+ ]);
+});
+
+test("uses only infilter when convertTo has no out filter (e.g., docx -> pdf)", async () => {
+ await convert("in.docx", "docx", "pdf", "out/out.pdf", undefined, mockExecFile);
+
+ const { args } = requireDefined(calls[0], "Expected at least one execFile call");
+
+ // If docx has an infilter, it should be present
+ expect(args).toEqual(["--headless", "--convert-to", "pdf", "--outdir", "out", "in.docx"]);
+
+ const i = args.indexOf("--convert-to");
+ expect(i).toBeGreaterThanOrEqual(0);
+ expect(args[i + 1]).toBe("pdf");
+ expect(args.slice(-2)).toEqual(["out", "in.docx"]);
+});
+
+test("strips leading './' from outdir", async () => {
+ await convert("in.txt", "txt", "docx", "./out/out.docx", undefined, mockExecFile);
+
+ const { args } = requireDefined(calls[0], "Expected at least one execFile call");
+
+ const outDirIdx = args.indexOf("--outdir");
+ expect(outDirIdx).toBeGreaterThanOrEqual(0);
+ expect(args[outDirIdx + 1]).toBe("out");
+});
+
+// --- promise settlement ------------------------------------------------------
+test("resolves with 'Done' when execFile succeeds", async () => {
+ behavior = { kind: "success", stdout: "fine", stderr: "" };
+ await expect(
+ convert("in.txt", "txt", "docx", "out/out.docx", undefined, mockExecFile),
+ ).resolves.toBe("Done");
+});
+
+test("rejects when execFile returns an error", async () => {
+ behavior = { kind: "error", message: "convert failed", stderr: "oops" };
+ await expect(
+ convert("in.txt", "txt", "docx", "out/out.docx", undefined, mockExecFile),
+ ).rejects.toMatch(/error: Error: convert failed/);
+});
+
+// --- logging behavior --------------------------------------------------------
+test("logs stdout when present", async () => {
+ behavior = { kind: "success", stdout: "hello", stderr: "" };
+
+ await convert("in.txt", "txt", "docx", "out/out.docx", undefined, mockExecFile);
+
+ expect(logs).toContain("stdout: hello");
+ expect(errors).toHaveLength(0);
+});
+
+test("logs stderr when present", async () => {
+ behavior = { kind: "success", stdout: "", stderr: "uh-oh" };
+
+ await convert("in.txt", "txt", "docx", "out/out.docx", undefined, mockExecFile);
+
+ expect(errors).toContain("stderr: uh-oh");
+ // When stdout is empty, no stdout log
+ expect(logs.find((l) => l.startsWith("stdout:"))).toBeUndefined();
+});
+
+test("logs both stdout and stderr when both are present", async () => {
+ behavior = { kind: "success", stdout: "alpha", stderr: "beta" };
+
+ await convert("in.txt", "txt", "docx", "out/out.docx", undefined, mockExecFile);
+
+ expect(logs).toContain("stdout: alpha");
+ expect(errors).toContain("stderr: beta");
+});
+
+test("logs stderr on exec error as well", async () => {
+ behavior = { kind: "error", message: "boom", stderr: "EPIPE" };
+
+ expect(convert("in.txt", "txt", "docx", "out/out.docx", undefined, mockExecFile)).rejects.toMatch(
+ /error: Error: boom/,
+ );
+
+ // The callback still provided stderr; your implementation logs it before settling
+ expect(errors).toContain("stderr: EPIPE");
+});
diff --git a/tests/converters/pandoc.test.ts b/tests/converters/pandoc.test.ts
new file mode 100644
index 0000000..c910d6c
--- /dev/null
+++ b/tests/converters/pandoc.test.ts
@@ -0,0 +1,66 @@
+import { beforeEach, expect, test, describe } from "bun:test";
+import { convert } from "../../src/converters/pandoc";
+import type { ExecFileFn } from "../../src/converters/types";
+
+describe("convert", () => {
+ let mockExecFile: ExecFileFn;
+
+ beforeEach(() => {
+ mockExecFile = (cmd, args, callback) => callback(null, "output-data", "");
+ });
+
+ test("should call pandoc with correct arguments (normal)", async () => {
+ let calledArgs: Parameters = ["", [], () => {}];
+ mockExecFile = (cmd, args, callback) => {
+ calledArgs = [cmd, args, callback];
+ callback(null, "output-data", "");
+ };
+
+ const result = await convert(
+ "input.md",
+ "markdown",
+ "html",
+ "output.html",
+ undefined,
+ mockExecFile,
+ );
+
+ expect(calledArgs[0]).toBe("pandoc");
+ expect(calledArgs[1]).toEqual([
+ "input.md",
+ "-f",
+ "markdown",
+ "-t",
+ "html",
+ "-o",
+ "output.html",
+ ]);
+ expect(result).toBe("Done");
+ });
+
+ test("should add xelatex argument for pdf/latex", async () => {
+ let calledArgs: Parameters = ["", [], () => {}];
+ mockExecFile = (cmd, args, callback) => {
+ calledArgs = [cmd, args, callback];
+ callback(null, "output-data", "");
+ };
+
+ await convert("input.md", "markdown", "pdf", "output.pdf", undefined, mockExecFile);
+
+ expect(calledArgs[1][0]).toBe("--pdf-engine=xelatex");
+ expect(calledArgs[1]).toContain("input.md");
+ expect(calledArgs[1]).toContain("-f");
+ expect(calledArgs[1]).toContain("markdown");
+ expect(calledArgs[1]).toContain("-t");
+ expect(calledArgs[1]).toContain("pdf");
+ expect(calledArgs[1]).toContain("-o");
+ expect(calledArgs[1]).toContain("output.pdf");
+ });
+
+ test("should reject if execFile returns an error", async () => {
+ mockExecFile = (cmd, args, callback) => callback(new Error("fail"), "", "");
+ await expect(
+ convert("input.md", "markdown", "html", "output.html", undefined, mockExecFile),
+ ).rejects.toMatch(/error: Error: fail/);
+ });
+});
diff --git a/tests/converters/vcf.test.ts b/tests/converters/vcf.test.ts
new file mode 100644
index 0000000..181dd71
--- /dev/null
+++ b/tests/converters/vcf.test.ts
@@ -0,0 +1,98 @@
+import { expect, test, describe } from "bun:test";
+import { convert, parseVCF, toCSV } from "../../src/converters/vcf";
+
+describe("parseVCF", () => {
+ test("should parse a simple VCF card", () => {
+ const vcfData = `BEGIN:VCARD
+VERSION:3.0
+FN:John Doe
+N:Doe;John;;;
+TEL:+123456789
+EMAIL:john@example.com
+ORG:Example Corp
+END:VCARD`;
+
+ const result = parseVCF(vcfData);
+
+ expect(result).toEqual([
+ {
+ "Full Name": "John Doe",
+ "Last Name": "Doe",
+ "First Name": "John",
+ Phone: "+123456789",
+ Email: "john@example.com",
+ Organization: "Example Corp",
+ },
+ ]);
+ });
+
+ test("should handle multiple cards", () => {
+ const vcfData = `BEGIN:VCARD
+FN:John Doe
+END:VCARD
+BEGIN:VCARD
+FN:Jane Smith
+END:VCARD`;
+
+ const result = parseVCF(vcfData);
+
+ expect(result).toEqual([{ "Full Name": "John Doe" }, { "Full Name": "Jane Smith" }]);
+ });
+
+ test("should parse VCF with TYPE parameters", () => {
+ const vcfData = `BEGIN:VCARD
+VERSION:3.0
+FN:John Doe
+N:Doe;John;;;
+TEL;TYPE=WORK,VOICE:(111) 555-1212
+EMAIL;TYPE=PREF,INTERNET:john.doe@example.com
+END:VCARD`;
+
+ const result = parseVCF(vcfData);
+
+ expect(result).toEqual([
+ {
+ "Full Name": "John Doe",
+ "Last Name": "Doe",
+ "First Name": "John",
+ Phone: "(111) 555-1212",
+ Email: "john.doe@example.com",
+ },
+ ]);
+ });
+});
+
+describe("toCSV", () => {
+ test("should convert contacts to CSV", () => {
+ const contacts = [
+ {
+ "Full Name": "John Doe",
+ Phone: "+123",
+ Email: "john@example.com",
+ },
+ ];
+
+ const result = toCSV(contacts);
+
+ expect(result).toBe('Full Name,Phone,Email\n"John Doe","+123","john@example.com"');
+ });
+
+ test("should escape quotes", () => {
+ const contacts = [{ "Full Name": 'John "Johnny" Doe' }];
+
+ const result = toCSV(contacts);
+
+ expect(result).toBe('Full Name\n"John ""Johnny"" Doe"');
+ });
+
+ test("should handle empty data", () => {
+ const result = toCSV([]);
+ expect(result).toBe("");
+ });
+});
+
+describe("convert", () => {
+ test("should be a function", () => {
+ expect(typeof convert).toBe("function");
+ });
+});
diff --git a/tests/converters/vtracer.test.ts b/tests/converters/vtracer.test.ts
new file mode 100644
index 0000000..365748e
--- /dev/null
+++ b/tests/converters/vtracer.test.ts
@@ -0,0 +1,58 @@
+import { beforeEach, expect, test, describe } from "bun:test";
+import { convert } from "../../src/converters/vtracer";
+import type { ExecFileFn } from "../../src/converters/types";
+
+describe("convert", () => {
+ let mockExecFile: ExecFileFn;
+
+ beforeEach(() => {
+ mockExecFile = (cmd, args, callback) => callback(null, "output-data", "");
+ });
+
+ test("should call vtracer with correct arguments (minimal)", async () => {
+ let calledArgs: Parameters = ["", [], () => {}];
+ mockExecFile = (cmd, args, callback) => {
+ calledArgs = [cmd, args, callback];
+ callback(null, "output-data", "");
+ };
+
+ const result = await convert("input.png", "png", "svg", "output.svg", undefined, mockExecFile);
+
+ expect(calledArgs[0]).toBe("vtracer");
+ expect(calledArgs[1]).toEqual(["--input", "input.png", "--output", "output.svg"]);
+ expect(result).toBe("Done");
+ });
+
+ test("should add options as arguments", async () => {
+ let calledArgs: Parameters = ["", [], () => {}];
+ mockExecFile = (cmd, args, callback) => {
+ calledArgs = [cmd, args, callback];
+ callback(null, "output-data", "");
+ };
+
+ const options = {
+ colormode: "color",
+ hierarchical: "true",
+ filter_speckle: 5,
+ path_precision: 0.8,
+ };
+
+ await convert("input.png", "png", "svg", "output.svg", options, mockExecFile);
+
+ expect(calledArgs[1]).toContain("--colormode");
+ expect(calledArgs[1]).toContain("color");
+ expect(calledArgs[1]).toContain("--hierarchical");
+ expect(calledArgs[1]).toContain("true");
+ expect(calledArgs[1]).toContain("--filter_speckle");
+ expect(calledArgs[1]).toContain("5");
+ expect(calledArgs[1]).toContain("--path_precision");
+ expect(calledArgs[1]).toContain("0.8");
+ });
+
+ test("should reject if execFile returns an error", async () => {
+ mockExecFile = (cmd, args, callback) => callback(new Error("fail"), "", "stderr output");
+ await expect(
+ convert("input.png", "png", "svg", "output.svg", undefined, mockExecFile),
+ ).rejects.toMatch(/error: Error: fail\nstderr: stderr output/);
+ });
+});
diff --git a/tsconfig.eslint.json b/tsconfig.eslint.json
new file mode 100644
index 0000000..acecbf1
--- /dev/null
+++ b/tsconfig.eslint.json
@@ -0,0 +1,7 @@
+{
+ "compilerOptions": {
+ "noEmit": true
+ },
+ "extends": "./tsconfig.json",
+ "include": ["src", "tests", "eslint.config.ts", "package.json"]
+}
diff --git a/tsconfig.json b/tsconfig.json
index ac50753..cb524d9 100644
--- a/tsconfig.json
+++ b/tsconfig.json
@@ -29,5 +29,6 @@
"esModuleInterop": true
// "noImplicitReturns": true
},
- "include": ["src", "tests", "package.json"]
+ "include": ["src", "package.json"],
+ "exclude": ["dist", "node_modules"]
}
| |