test: Phase 2 — full hook coverage + pre-push test gate

[Dayron-Glez]

Summary

Closes #141

This PR completes Phase 2 of the testing strategy by adding .fn.ts + test coverage for all remaining 15 custom hooks, and introduces a pre-push hook so tests run automatically before any push to the remote.

---

What changed

🧪 Test coverage — 15 hooks added

Each hook follows the same .fn.ts pattern established in Phase 1 (#138):

• use-<name>.fn.ts — the real, callable function (imported by tests only, never by the app)
• __tests__/use-<name>.test.ts(x) — Vitest test file importing from .fn.ts

Group	Hooks	Technique
State & timing	useLocalStorage, useDebounce, useInterval	vi.useFakeTimers(), localStorage
Browser APIs	useClipboard, useOnlineStatus, useMediaQuery	navigator mocks, window events
DOM interactions	useClickOutside, useHover, useEventListener, useKeyPress, useLongPress	DOM events, userEvent, fake timers
Layout / scroll	useWindowSize, useScrollPosition	vi.stubGlobal, resize/scroll events
Observer APIs	useIntersectionObserver	IntersectionObserver class mock
Async	useFetch	fetch mock, waitFor

Total: 94 tests across 18 test files — all passing.

🔒 Pre-push hook

Added .husky/pre-push to run the full test suite before every push:

# .husky/pre-push
bun run test:run

Why pre-push and not pre-commit?

• Pre-commit only runs lint-staged (ESLint + Prettier) — fast, doesn't block atomic commits
• Pre-push runs tests once, right before code reaches the remote — catches regressions without slowing down local work
• Atomic commit workflows (where one commit intentionally breaks something to fix it in the next) remain unblocked locally

Skipping when needed:

git push --no-verify

Use this when you need to push a WIP branch or a series of commits that aren't all green yet.

⚠️ ESLint disable comments

Some .fn.ts files include targeted eslint-disable-next-line comments for patterns that are intentional but flagged by the project's ESLint rules:

File	Rule disabled	Reason
use-media-query.fn.ts	react-hooks/set-state-in-effect	Sync state with current media query value on mount
use-fetch.fn.ts	react-hooks/set-state-in-effect	Reset to loading state before each fetch
use-long-press.fn.ts	react-hooks/refs (×3)	Sync refs with latest callbacks during render to avoid stale closures

These are the same well-known patterns used in the original hook implementations — the disable comments make the intent explicit.

---

Test plan

bun run test:run     # 94 tests, 18 files — all green
bunx tsc --noEmit    # no TypeScript errors

[vercel]

@Dayron-Glez is attempting to deploy a commit to the drbarzaga's projects Team on Vercel.

A member of the Team first needs to authorize it.

[vercel]

The latest updates on your projects. Learn more about Vercel for GitHub.

Project	Deployment	Actions	Updated (UTC)
react-dojo	Ready	Preview, Comment	May 16, 2026 8:21pm

[drbarzaga]

LGTM! Thanks so much @Dayron-Glez for this contribution 🤝