Remove data! assertions in API client; delete unused mock data

Commit notes

Replace the error-check-then-data! pattern with a combined (error || !data) guard across all lib/api modules (100 occurrences, 10 files) so a response with neither error nor data fails loudly instead of crashing downstream. Delete lib/mock-data.ts: its only export was never imported.

[private session redacted]