chore: gracefully handle future invalid versions

Relates to #110

Author: WarningImHack3r

src/lib/server/github-cache.ts

@@ -693,13 +693,19 @@ export class GitHubCache {
 		const { data: changelogResult } = await this.#octokit.rest.repos.getContent({
 			owner,
 			repo,
-			ref:
-				owner === "sveltejs" &&
-				repo === "prettier-plugin-svelte" && // this repo is a bit of a mess (https://github.com/sveltejs/prettier-plugin-svelte/issues/497)
-				tags[0] &&
-				semver.major(repository.metadataFromTag(tags[0].name)[1]) === 3
-					? "version-3" // a temporary fix to get the changelog from the right branch while v4 isn't out yet
-					: undefined,
+			ref: (() => {
+				try {
+					return owner === "sveltejs" &&
+						repo === "prettier-plugin-svelte" && // this repo is a bit of a mess (https://github.com/sveltejs/prettier-plugin-svelte/issues/497)
+						tags[0] &&
+						semver.major(repository.metadataFromTag(tags[0].name)[1]) === 3
+						? "version-3" // a temporary fix to get the changelog from the right branch while v4 isn't out yet
+						: undefined;
+				} catch {
+					// handle oopsies for invalid versions returned from `metadataFromTag` (or others)
+					return undefined;
+				}
+			})(),
 			path: "CHANGELOG.md"
 		});
 

src/routes/package/releases.ts

@@ -78,7 +78,20 @@ export async function getPackageReleases(
 						console.warn(`Empty release tag name: ${JSON.stringify(release)}`);
 						return false;
 					}
-					const [name] = repo.metadataFromTag(release.tag_name);
+					const [name, version] = repo.metadataFromTag(release.tag_name);
+					if (semver.valid(version) === null) {
+						posthog?.captureException(new Error("Invalid version"), undefined, {
+							repo: `${repo.repoOwner}/${repo.repoName}`,
+							packageName,
+							tag: release.tag_name,
+							parsedName: name,
+							parsedVersion: version
+						});
+						console.warn(
+							`Invalid version from \`metadataFromTag\` "${version}" gotten from ${release.tag_name}`
+						);
+						return false;
+					}
 					return (
 						(repo.dataFilter?.(release) ?? true) &&
 						repo.pkg.name.localeCompare(name, undefined, { sensitivity: "base" }) === 0