3 Commits

Author	SHA1	Message	Date
jakevin	810547c9a9	refactor: eliminate any types in core (non-clis) files (#886) ... Replace explicit `any` with `unknown` + narrowing or concrete types across all core src/ files (non-`clis/**`). Core drops from ~60 `any` occurrences to a handful of documented, unavoidable boundaries. Mechanical error-handler cleanup (uses getErrorMessage() from errors.ts): - cli.ts, cascade.ts, download/*.ts, external.ts, plugin.ts, doctor.ts — catch (err: any) → catch (err) + getErrorMessage(err) Pipeline steps — typed params with per-step interfaces: - pipeline/steps/intercept.ts — InterceptParams, signature uses unknown - pipeline/steps/tap.ts — TapParams - pipeline/steps/download.ts — DownloadParams + DownloadedItem (ytdlp_args is now coerced via String(v) for defence-in-depth) Probe / boundary typing: - cascade.ts — FetchProbeResponse interface; also fixes a latent bug where result.success could be assigned undefined (masked by any) by wrapping with !!(…) Browser-side injected scripts — structural types at the TS boundary (types stripped by tsc emit before .toString() runs, runtime unchanged): - scripts/store.ts — PiniaStore / VuexModule / VueApp - scripts/framework.ts — VueAppEl / FrameworkWindow Runtime detection: - runtime-detect.ts — BunGlobal interface; getRuntimeVersion reads Bun into a local to avoid non-null assertions. Test files — precise structural casts replacing `as any`: - browser.test.ts — `{ _state: string }` cast for private state; full DaemonStatus shape for the getDaemonHealth mock - browser/dom-helpers.test.ts — globalThis as Record<string, unknown> - browser/cdp.test.ts — (...args: unknown[]) in mock handlers - runtime-detect.test.ts — matches runtime-detect.ts BunGlobal - output.test.ts — logSpy.mock.calls typed with unknown[] - engine.test.ts, snapshotFormatter.test.ts, pipeline/executor.test.ts — narrow structural casts / removed stale any casts Verification: - npx tsc --noEmit: clean - npx vitest run (excluding e2e/smoke): 1415 passed, 1 skipped	2026-04-08 23:08:23 +08:00
jakevin	f88e7569e9	refactor: src cleanup — deduplicate errors, cache VM, extract BasePage, remove Playwright MCP legacy (#667) ... * refactor: deduplicate transient error checks, cache VM contexts, expose tab ID - Extract shared isTransientBrowserError() into browser/errors.ts, replacing duplicated string-matching lists in daemon-client.ts and pipeline/executor.ts - Cache compiled vm.Script objects in template.ts with LRU eviction (max 256), avoiding per-invocation VM context creation in pipeline loops - Add getActiveTabId() to IPage interface and Page class for tab state inspection * refactor: extract BasePage to deduplicate DOM helpers across Page and CDPPage Both Page (daemon-backed) and CDPPage (direct CDP) had ~200 lines of identical DOM helper implementations (click, type, scroll, wait, snapshot, interceptor, etc). Extract shared logic into abstract BasePage class. Subclasses now only implement transport-specific methods. * refactor: rename mcp.ts to bridge.ts and clean up Playwright MCP references The file browser/mcp.ts contained BrowserBridge (daemon session manager), not MCP functionality. Renamed to bridge.ts for clarity. Also removed all stale "Playwright MCP" references from comments and variable names across the codebase — Playwright was removed long ago.	2026-04-02 00:51:15 +08:00
jackwener	fbec2f6f5d	feat(snapshot): filter contentinfo subtrees, bilibili ad URLs, boilerplate buttons ... - Add contentinfo to subtree-level noise filtering (biggest single win) - Reuters: 51% → 62%, Google: 57% → 70%, Netflix: 48% → 60% - Add cm.bilibili.com/cm/api/fees/ ad URL pattern - Add 广告 keyword to ad detection - Add back-to-top / 回到顶部 boilerplate button filtering - Unify ad/boilerplate/contentinfo into single subtree-skip mechanism - Add vitest config and comprehensive test suite (33 tests) - Fixture tests skip gracefully when snapshot files are absent Bump to v0.7.1	2026-03-17 01:26:49 +08:00