add validation filtering in getStaticPaths for OpenProcessing sketches

- Add basic validation (visualID, type check) in sketch getStaticPaths
- Remove unnecessary isNaN check from SketchLayout
- Prevent invalid sketch pages from being generated during build

Author: junseok44

src/layouts/SketchLayout.astro

@@ -24,10 +24,6 @@ const { sketchId, authorName } = Astro.props;
 
 const sketchIdNumber = Number(sketchId);
 
-if (isNaN(sketchIdNumber)) {
-  console.error(`Invalid sketch ID: ${sketchId}`);
-}
-
 const { title, createdOn, instructions } = await getSketch(sketchIdNumber);
 
 const currentLocale = getCurrentLocale(Astro.url.pathname);

src/pages/[locale]/sketches/[...slug].astro

@@ -7,7 +7,7 @@ export async function getStaticPaths() {
   const sketches = await getCurationSketches();
   const entries = await Promise.all(
     nonDefaultSupportedLocales.map(async (locale) => {
-      return sketches.map((sketch) => ({
+      return sketches.filter(sketch => sketch.visualID && typeof sketch.visualID === 'number').map((sketch) => ({
         // Even though slug gets converted to string at runtime,
         // TypeScript infers it as number from sketch.visualID, so we explicitly convert to string.
         params: { locale, slug: String(sketch.visualID) },

src/pages/sketches/[...slug].astro

@@ -4,7 +4,7 @@ import { getCurationSketches } from "@src/api/OpenProcessing";
 
 export async function getStaticPaths() {
   const sketches = await getCurationSketches();
-  return sketches.map((sketch) => ({
+  return sketches.filter(sketch => sketch.visualID && typeof sketch.visualID === 'number').map((sketch) => ({
     // Even though slug gets converted to string at runtime,
     // TypeScript infers it as number from sketch.visualID, so we explicitly convert to string.
     params: { slug: String(sketch.visualID) },