feat: /database page redirect invalid database to the first one existing

[duyet]

Summary by Sourcery

This pull request introduces a feature that redirects users to the first existing database if the specified database is invalid on the /database page. Additionally, it enhances the database query by changing the join type from left join to inner join for improved accuracy in fetching database names and table counts.

• New Features:
  • Added a redirect to the first existing database if the current database is invalid on the /database page.
• Enhancements:
  • Updated the database query to use an inner join instead of a left join for fetching database names and table counts.

[vercel]

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name	Status	Preview	Comments	Updated (UTC)
clickhouse-monitoring	✅ Ready (Inspect)	Visit Preview	💬 Add feedback	Jul 19, 2024 3:29pm

[sourcery-ai]

Reviewer's Guide by Sourcery

This pull request introduces a feature to redirect users to the first existing database if the specified database is invalid. The changes include modifying the database fetching logic to include caching settings, handling errors more robustly, and updating the database query to use an inner join instead of a left join.

File-Level Changes

Files	Changes
app/database/[database]/breadcrumb.tsx app/database/queries.ts	Implemented a redirect to the first existing database if the current database is invalid and optimized the database query.

---

Tips

• Trigger a new Sourcery review by commenting @sourcery-ai review on the pull request.
• Continue your discussion with Sourcery by replying directly to review comments.
• You can change your review settings at any time by accessing your dashboard:
  • Enable or disable the Sourcery-generated pull request summary or reviewer's guide;
  • Change the review language;
• You can always contact us if you have any questions or feedback.

[sourcery-ai]

Hey @duyet - I've reviewed your changes and they look great!

Here's what I looked at during the review

• 🟡 General issues: 1 issue found
• 🟢 Security: all looks good
• 🟢 Testing: all looks good
• 🟢 Complexity: all looks good
• 🟢 Documentation: all looks good

---

Sourcery is free for open source - if you like our reviews please consider sharing them ✨

• X
• Mastodon
• LinkedIn
• Facebook

_{Help me be more useful! Please click 👍 or 👎 on each comment to tell me if it was helpful.}

[sourcery-ai]

suggestion: Consider using Array.prototype.some for better readability.

Using Array.prototype.some can make the intent clearer when checking if any database matches the current database name.

Suggested change

	if (!databases.find((db) => db.name === database)) {
	if (!databases.some((db) => db.name === database)) {