Skip to content

Troubleshoot and fix app loading and timeout issues #105

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

Troubleshoot and fix app loading and timeout issues #105

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
50 changes: 36 additions & 14 deletions src/lib/db.ts
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,9 +1,19 @@
import { createClient } from "@libsql/client";
import { createClient, type Client } from "@libsql/client";

const client = createClient({
url: process.env.TURSO_DATABASE_URL!,
authToken: process.env.TURSO_AUTH_TOKEN,
});
let client: Client | null = null;

function getClient(): Client {
if (!client) {
if (!process.env.TURSO_DATABASE_URL) {
throw new Error("TURSO_DATABASE_URL is not set");
}
client = createClient({
url: process.env.TURSO_DATABASE_URL,
authToken: process.env.TURSO_AUTH_TOKEN,
});
}
return client;
}

interface Release {
version: string;
Expand All @@ -16,14 +26,25 @@
}

export async function getAnalyzedReleases(): Promise<Release[]> {
const result = await client.execute(
"SELECT * FROM releases ORDER BY analyzed_at DESC",
);
// TODO(serhalp) Verify the right turso typing pattern
return result.rows.map((row) => ({
...row,
relevantPRs: JSON.parse(row.relevantPRs as string),
})) as unknown[] as Release[];
try {
const dbClient = getClient();
const timeoutPromise = new Promise<never>((_, reject) => {
setTimeout(() => reject(new Error("Database query timeout")), 5000);
});

Check failure on line 34 in src/lib/db.ts

View workflow job for this annotation

GitHub Actions / test 

Delete `····`

Check failure on line 34 in src/lib/db.ts

View workflow job for this annotation

GitHub Actions / test 

Delete `····`
const result = await Promise.race([
dbClient.execute("SELECT * FROM releases ORDER BY analyzed_at DESC"),
timeoutPromise,
]);

Check failure on line 39 in src/lib/db.ts

View workflow job for this annotation

GitHub Actions / test 

Delete `····`

Check failure on line 39 in src/lib/db.ts

View workflow job for this annotation

GitHub Actions / test 

Delete `····`
return result.rows.map((row) => ({
...row,
relevantPRs: JSON.parse(row.relevantPRs as string),
})) as unknown[] as Release[];
} catch (error) {
console.error("Error fetching releases:", error);
return [];
}
}

export async function insertAnalyzedRelease(
Expand All @@ -34,7 +55,8 @@
releaseLink: string,
relevantPRs: string[],
): Promise<void> {
await client.execute({
const dbClient = getClient();
await dbClient.execute({
sql: "INSERT INTO releases (version, summary, score, relevance, releaseLink, relevantPRs, analyzed_at) VALUES (?, ?, ?, ?, ?, ?, ?)",
args: [
version,
Expand Down
115 changes: 62 additions & 53 deletions src/pages/index.astro
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -40,63 +40,72 @@
</div>
<div class="space-y-12">
{
releases.map((release) => (
<div
class={`card bg-base-200 shadow-xl transition-all hover:shadow-2xl ${
release.score > 60 ? "border-l-8 border-error" : ""
}`}
>
<div class="card-body">
<h2 class="card-title text-2xl">
<a
href={release.releaseLink}
target="_blank"
rel="noopener noreferrer"
class="hover:underline cursor-pointer"
>
{release.version}
</a>
</h2>
<div class="flex justify-between items-center">
<div
class="badge badge-lg"
style={`background-color: hsl(${120 - release.score * 1.2}, 100%, 50%); color: ${release.score > 50 ? "white" : "black"};`}
>
Impact Score: {release.score}
releases.length === 0 ? (
<div class="card bg-base-200 shadow-xl">
<div class="card-body text-center">
<h2 class="card-title text-2xl justify-center">No releases yet</h2>

Check failure on line 46 in src/pages/index.astro

View workflow job for this annotation

GitHub Actions / test 

Replace `No·releases·yet` with `⏎················No·releases·yet⏎··············`

Check failure on line 46 in src/pages/index.astro

View workflow job for this annotation

GitHub Actions / test 

Replace `No·releases·yet` with `⏎················No·releases·yet⏎··············`
<p class="text-base-content">Check back soon for Next.js release analysis!</p>

Check failure on line 47 in src/pages/index.astro

View workflow job for this annotation

GitHub Actions / test 

Replace `Check·back·soon·for·Next.js·release·analysis!` with `⏎················Check·back·soon·for·Next.js·release·analysis!⏎··············`

Check failure on line 47 in src/pages/index.astro

View workflow job for this annotation

GitHub Actions / test 

Replace `Check·back·soon·for·Next.js·release·analysis!` with `⏎················Check·back·soon·for·Next.js·release·analysis!⏎··············`
</div>
</div>
) : (
releases.map((release) => (
<div
class={`card bg-base-200 shadow-xl transition-all hover:shadow-2xl ${
release.score > 60 ? "border-l-8 border-error" : ""
}`}
>
<div class="card-body">
<h2 class="card-title text-2xl">
<a
href={release.releaseLink}
target="_blank"
rel="noopener noreferrer"
class="hover:underline cursor-pointer"
>
{release.version}
</a>
</h2>
<div class="flex justify-between items-center">
<div
class="badge badge-lg"
style={`background-color: hsl(${120 - release.score * 1.2}, 100%, 50%); color: ${release.score > 50 ? "white" : "black"};`}
>
Impact Score: {release.score}
</div>
<div class="text-sm opacity-70">
{formatDistanceToNow(new Date(release.analyzed_at))} ago
</div>
</div>
<div class="text-sm opacity-70">
{formatDistanceToNow(new Date(release.analyzed_at))} ago
<p class="mt-4 text-base-content">{release.summary}</p>
<div class="mt-6">
<h3 class="font-semibold text-lg mb-2">
Relevance to Netlify:
</h3>
<p class="text-base-content">{release.relevance}</p>
</div>
{release.relevantPRs.length > 0 && (
<div class="mt-4">
<h3 class="font-semibold text-lg mb-2">Relevant PRs:</h3>
<ul class="list-disc list-inside">
{release.relevantPRs.map((pr) => (
<li>
<a
href={pr}
target="_blank"
rel="noopener noreferrer"
class="hover:underline"
>
#{pr.split("/").pop()}
</a>
</li>
))}
</ul>
</div>
)}
</div>
<p class="mt-4 text-base-content">{release.summary}</p>
<div class="mt-6">
<h3 class="font-semibold text-lg mb-2">
Relevance to Netlify:
</h3>
<p class="text-base-content">{release.relevance}</p>
</div>
{release.relevantPRs.length > 0 && (
<div class="mt-4">
<h3 class="font-semibold text-lg mb-2">Relevant PRs:</h3>
<ul class="list-disc list-inside">
{release.relevantPRs.map((pr) => (
<li>
<a
href={pr}
target="_blank"
rel="noopener noreferrer"
class="hover:underline"
>
#{pr.split("/").pop()}
</a>
</li>
))}
</ul>
</div>
)}
</div>
</div>
))
))
)
}
</div>
</main>
Expand Down
Loading