mirror of
https://github.com/vas3k/TaxHacker.git
synced 2026-07-03 10:52:28 +08:00
* feat: initial email impl * feat: IMAP email ingest (builds on the scaffold) (#100) * chore: add imap-simple, mailparser, vitest * feat: AES-256-GCM helpers for email credentials * feat: extract ingestUnsortedFile helper, reuse in upload action * chore: gitignore .worktrees/ * feat: email-sync types and pure attachment/search filters * feat: imap-simple + mailparser client wrapper * feat: email sync orchestration with UID watermark + status persistence * feat: encrypt email credentials at rest, add UID/addedAt fields * feat: real IMAP test-connection, scoped sync-now, thin cron entry * docs: update email app README to match real IMAP/encryption/UID behavior * fix: nest SINCE search criteria and guard missing addedAt for first-run sync * fix: show last-sync time and error detail from sync in server card * fix: skip storage recompute when no attachments ingested Avoids an ENOENT crash on first sync when the user's uploads dir does not exist yet and nothing was ingested; this was also masking the real per-server error. Adds regression tests for the guard. * feat: configurable initial-grab window (fetch-since date) First sync is bounded by a user-chosen 'Fetch emails since' date instead of the server's addedAt; blank = entire mailbox (IMAP ALL). The UID watermark takes over after the first run. * fix: add missing @langchain/core dependency @langchain/core is only a peer dep of the @langchain/* packages and was not installed on a clean npm install, breaking the build (e.g. /unsorted via ai/analyze). * fix: harden email sync — UID dedup guard, locked status write, graceful decrypt, scrypt memo Addresses review findings: skip messages at/below the UID watermark (defends against the IMAP `n:*` re-fetch quirk); lock the app_data row with SELECT FOR UPDATE so concurrent cron/manual syncs can't clobber each other; return a friendly error when a stored password can't be decrypted (e.g. after BETTER_AUTH_SECRET rotation) and document the coupling; memoize the scrypt-derived key. * feat: enforce per-server syncInterval on cron; skip non-Buffer attachments The cron now honors each server's syncInterval (manual Sync Now bypasses the throttle), so the configured interval is no longer ignored. Attachments whose parsed content is not a Buffer are skipped instead of throwing on .length. Adds throttle regression tests. * refactor: remove dead lastProcessedMessageId field; clarify cron throttle in README lastProcessedMessageId was superseded by the lastProcessedUid watermark and never read; dropped from the type and form state. README now describes the per-server interval as an app-level throttle (manual Sync Now bypasses). * feat(email): UI-selectable sync frequency + working cron heartbeat Replace the per-server sync-interval number input with a dropdown of presets (15m/30m/hourly/6h/12h/daily). Switch the stored unit from hours to minutes and update the throttle accordingly. Make the cron actually run: heartbeat now fires every 5 minutes as the resolution floor while each mailbox's UI frequency gates real fetches. Propagate env into cron jobs via /etc/cron.env (cron strips the environment) and add BETTER_AUTH_SECRET to the email-sync service in the dev/build compose files so stored passwords can be decrypted. * fix(email): reset Add Server dialog to provider selection on close Radix's onOpenChange only toggled isOpen, so closing the dialog via Esc, overlay click, or the X left the step/selectedProvider state intact. Reopening then jumped straight to the previous provider's config form instead of the provider-selection screen. Route every close through handleClose() to reset the step. --------- Co-authored-by: Evgenii Burmakin <Freika@users.noreply.github.com>
53 lines
1.6 KiB
TypeScript
53 lines
1.6 KiB
TypeScript
import { afterAll, describe, expect, it, vi } from "vitest"
|
|
import { mkdtemp, readFile, rm } from "fs/promises"
|
|
import { tmpdir } from "os"
|
|
import path from "path"
|
|
|
|
const tmpRoot = await mkdtemp(path.join(tmpdir(), "th-uploads-"))
|
|
process.env.UPLOAD_PATH = tmpRoot
|
|
process.env.SELF_HOSTED_MODE = "true"
|
|
|
|
const created: any[] = []
|
|
vi.mock("./config", () => ({
|
|
default: {
|
|
upload: { images: { maxWidth: 1920, maxHeight: 1080, quality: 80 } },
|
|
selfHosted: { isEnabled: true },
|
|
},
|
|
}))
|
|
vi.mock("@/models/files", () => ({
|
|
createFile: vi.fn(async (userId: string, data: any) => {
|
|
const row = { ...data, userId }
|
|
created.push(row)
|
|
return row
|
|
}),
|
|
}))
|
|
|
|
const { ingestUnsortedFile } = await import("./uploads")
|
|
|
|
const user = { id: "user-1", email: "u@example.com", storageUsed: 0, storageLimit: -1 } as any
|
|
|
|
afterAll(async () => {
|
|
await rm(tmpRoot, { recursive: true, force: true })
|
|
})
|
|
|
|
describe("ingestUnsortedFile", () => {
|
|
it("writes the buffer under the user's unsorted dir and creates an unsorted File row", async () => {
|
|
const buffer = Buffer.from("%PDF-1.4 fake invoice")
|
|
const file = await ingestUnsortedFile(user, {
|
|
buffer,
|
|
filename: "invoice.pdf",
|
|
mimetype: "application/pdf",
|
|
metadata: { source: "email" },
|
|
})
|
|
|
|
expect(file.filename).toBe("invoice.pdf")
|
|
expect(file.mimetype).toBe("application/pdf")
|
|
expect(file.path).toMatch(/^unsorted\/.+\.pdf$/)
|
|
expect((file.metadata as any).source).toBe("email")
|
|
|
|
const onDisk = await readFile(path.join(tmpRoot, user.email, file.path))
|
|
expect(onDisk.equals(buffer)).toBe(true)
|
|
expect(created).toHaveLength(1)
|
|
})
|
|
})
|