fix: repair cross-package test failures and typecheck errors

[stooit]

Summary

Fixes all failing tests and eliminates all tsc --noEmit errors across the monorepo. Bugs spanned three packages and one config file. No test files were modified and no dependencies were added.

Result: bun run test → 13 pass / 0 fail; npx tsc --noEmit → clean (exit 0).

Fixes

Area	Bug	Fix
apps/web/src/lib/api.ts	Imported useThrottle from @e2e/utils, which was renamed to useDebounce (TS2305; broke both api module tests)	Import useDebounce and re-export it as useSearchDebounce; consolidate imports
packages/ui/src/components/Button/Button.tsx	aria-label was destructured but never applied to the <button> (accessibility test failed)	Apply aria-label; for icon-only buttons with no explicit label, fall back to string children (else a generic name) and warn in dev — WCAG 2.2 SC 4.1.2. Text buttons still omit the attribute so visible labels aren't clobbered
packages/utils/src/format/date.ts	Produced zero-padded day 01/03/2024; test expects unpadded day-before-month	Use en-AU with unpadded day → 1/03/2024 (4-digit year, DD/MM ordering preserved)
tsconfig.json	Cannot find module 'bun:test' for all test files	Add "types": ["bun-types"] (already a devDependency); make DOM lib explicit; also typecheck apps/*/test (previously an untypechecked gap)
packages/ui/src/components/DataTable/DataTable.tsx	No actual bug — all DataTable tests already passed	Behaviour-neutral refactor to a functional setSortDir updater + removed a misleading "stale closure" comment. No failing test depended on this

Verification

• bun run test → 13 pass / 0 fail (the single console line during the run is the intentional dev-mode WCAG warning from the icon-only-without-label test)
• npx tsc --noEmit → no output, exit 0

Assumptions / notes

• Date format: the test comment states the desired output is 1/03/2024 (unpadded day), so formatDate uses an unpadded day rather than the conventional zero-padded 01/03/2024. Chose to honour the test's stated intent.
• Button fallback: the icon-only-without-label test requires a non-null aria-label, so a generic "button" fallback is used when no label and no string children are available. A dev-mode console.warn is emitted to drive callers to supply a real label; consider a lint rule for genuine 4.1.2 conformance.
• DataTable: the task hinted at a stale-closure bug here, but investigation confirmed the tests already passed and the closure was not actually stale. The change made is a defensive refactor only — no behaviour change.
• tsconfig: tsc --noEmit was already failing on main due to the missing bun-types; this PR also fixes that pre-existing break.

🤖 Generated with QuantCode