humanspeak / svelte-markdown / 29040595303

94%

main: 95%

push

github

chore(lint): lint test files and enable type-aware promise rules (#352)

* chore(lint): lint test files; enable type-aware promise rules

- Drop '**/*.test.ts' from top-level eslint ignores; add a test override
  relaxing any/unused-vars while keeping correctness rules on
- Switch to ts.configs.recommendedTypeChecked with parserOptions.projectService
- Enable no-floating-promises and no-misused-promises; disable the
  no-unsafe-* family and a few others during migration
- Fix surfaced violations: void fire-and-forget promises in Mermaid effects,
  timer-driven streaming loops, and walkTokens iteration

Guard snapshot for plan 009.

* docs(guard): checkpoint 1 for plan 009; amend plan to Trunk-only lint

Verdict: DRIFTING (seven recommendedTypeChecked rules blanket-disabled,
contrary to the plan's own maintenance note) + PLAN AMENDED.

- Replace plan 009's four `pnpm exec eslint .` calls with trunk equivalents;
  raw ESLint is forbidden by CLAUDE.md
- Strengthen Step 3 to prove no-floating-promises fires in .ts, .test.ts and
  .svelte; strengthen done criteria with `trunk check --all` and a rationale
  requirement for any rule set to 'off'
- Planned at stays 939f154 (tooling correction, no source-baseline change)

* fix(lint): restore await-thenable; drop no-op awaits on Playwright locators

Guard checkpoint 1 flagged await-thenable as blanket-disabled despite being
promise-correctness, which is this plan's stated purpose.

- Re-enable @typescript-eslint/await-thenable as an error
- Suppress its single false positive at parse-and-cache.ts:128 with an inline
  trunk-ignore (Marked types walkTokens as void-returning, hiding the real
  promises async callbacks put in the result array)
- Fix the 4 real violations it caught: `await page.locator(...)` and
  `await page.getByRole(...)` await a non-thenable Locator

Guard snapshot for plan 009 final.

* docs(guard): final checkpoint and close-out report for plan 009 — PASS

- Checkpoint 2: corrective work veri... (continued)

843 of 940 branches covered (89.68%)

Branch coverage included in aggregate %.

12 of 12 new or added lines in 5 files covered. (100.0%)

1393 of 1432 relevant lines covered (97.28%)

4714.53 hits per line