fix: replace any type in upload-zip route and remove console.log in u…

[Antra1705]

Summary

• Replaced error: any with error: unknown in the catch block of app/api/upload-zip/route.ts, with proper type narrowing via error instanceof Error for the fallback message
• Replaced 3 leftover console.log(error) debug statements in lib/user-data.ts with context-aware console.error logs prefixed with function names for better diagnostic observability

Type of change

• Bug fix
• Refactor

Related issue

Closes #291
Closes #274

Validation

• npm run lint
• npm test
• npm run build

List any additional manual verification you performed:

• Verified ValidationError is locally declared in route.ts — no import needed, instanceof check resolves correctly
• Confirmed console.error("ZIP upload error:", error) on line 183 was already present — no duplicate logging introduced
• All 65 unit tests passed with no regressions

Checklist

• I kept this PR focused on one primary change
• I updated documentation if behavior changed
• I did not commit secrets, local logs, or scratch files
• I am requesting review on the correct scope

Summary by CodeRabbit

• Bug Fixes
  • Improved error handling for file upload operations with more robust error message fallbacks
  • Enhanced error logging for database operations to better track issues in production

[qodo-code-review]

Qodo reviews are paused for this user.

Troubleshooting steps vary by plan Learn more →

On a Teams plan?
Reviews resume once this user has a paid seat and their Git account is linked in Qodo.
Link Git account →

Using GitHub Enterprise Server, GitLab Self-Managed, or Bitbucket Data Center?
These require an Enterprise plan - Contact us
Contact us →

[github-actions]

👋 Thanks for opening a PR, @Antra1705!

Your PR has entered the 🚦 PR Review Pipeline.

⚪ Standard PR detected — your PR will follow the standard review pipeline.

---

What happens next

Stage	Reviewer	Checks
Stage 1 — Automated Validation	🤖 Bot	DCO · Format · AI/Slop · Duplicate
Stage 2 — Human Review	👥 Maintainer	Code + Quality Review
Stage 3 — PA / Maintainer Review	🔑 Project Admin	Final Merge Decision

A pipeline status comment will appear below and update automatically as your PR progresses.

---

While you wait

• Sign all commits (git commit -s)
• Link your issue (Closes #123)
• Use a feature branch (not main)
• Avoid unrelated changes

This comment is posted only once.

[coderabbitai]

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro Plus

Run ID: 108f20c7-8fbb-40a1-ae28-e75c4b29cf50

📥 Commits

Reviewing files that changed from the base of the PR and between f12b223 and 7eea8d1.

📒 Files selected for processing (2)

• app/api/upload-zip/route.ts
• lib/user-data.ts

---

Walkthrough

This PR refines error handling across two modules. The ZIP upload route upgrades from unsafe any typing to unknown with proper Error instance checking. The user data helpers replace leftover debug logging with proper error logging using module-prefixed messages, improving observability without changing failure behavior.

Changes

Error Handling and Logging Improvements

Layer / File(s)	Summary
ZIP upload route type safety and error response app/api/upload-zip/route.ts	The POST handler catch clause changes from error: any to error: unknown, and the error response now safely checks if the caught value is an Error instance before accessing .message, otherwise falls back to a generic fallback message.
User data helper error logging lib/user-data.ts	The three exported database lookup helpers (getUserById, getUserByEmail, getAccountByUserId) replace debug console.log(error) statements with proper console.error() calls that include module-prefixed context messages, while preserving the null return on failure.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~8 minutes

Suggested labels

mentor:piyushdotcomm

Suggested reviewers

• piyushdotcomm

Poem

🐰 A rabbit hops through error logs with glee,
From any to unknown, now types are free!
Error handling shines with messages so clear,
No more debug whispers—warnings ring sincere! 🎯

🚥 Pre-merge checks | ✅ 5

✅ Passed checks (5 passed)

Check name	Status	Explanation
Title check	✅ Passed	The title accurately summarizes the main changes: replacing any type in upload-zip route and removing console.log statements in user-data.ts.
Description check	✅ Passed	The PR description is comprehensive and follows the template structure, covering summary, type of change, related issues, validation steps, and the full checklist.
Linked Issues check	✅ Passed	All requirements from #291 (replace any with unknown in upload-zip route) and #274 (remove console.log in user-data.ts) are fully addressed and validated.
Out of Scope Changes check	✅ Passed	All changes are directly related to the linked issues; no out-of-scope modifications were introduced beyond the stated objectives.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing Touches

🧪 Generate unit tests (beta)

• Create PR with unit tests

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}