@tscircuit/fake-snippets 0.0.82 → 0.0.84

package/fake-snippets-api/lib/db/db-client.ts

@@ -16,6 +16,8 @@ import {
   type PackageFile,
   type PackageRelease,
   packageReleaseSchema,
+  type AiReview,
+  aiReviewSchema,
   type Session,
   type Snippet,
   databaseSchema,
@@ -1338,4 +1340,42 @@ const initializer = combine(databaseSchema.parse({}), (set, get) => ({
       ),
     }))
   },
+
+  addAiReview: (review: Omit<AiReview, "ai_review_id">): AiReview => {
+    const base = aiReviewSchema.omit({ ai_review_id: true }).parse(review)
+    const newReview = {
+      ai_review_id: crypto.randomUUID(),
+      ...base,
+    }
+    set((state) => ({
+      aiReviews: [...state.aiReviews, newReview],
+      idCounter: state.idCounter + 1,
+    }))
+    return newReview
+  },
+  updateAiReview: (
+    aiReviewId: string,
+    updates: Partial<AiReview>,
+  ): AiReview | undefined => {
+    let updated: AiReview | undefined
+    set((state) => {
+      const index = state.aiReviews.findIndex(
+        (ar) => ar.ai_review_id === aiReviewId,
+      )
+      if (index === -1) return state
+      const aiReviews = [...state.aiReviews]
+      aiReviews[index] = { ...aiReviews[index], ...updates }
+      updated = aiReviews[index]
+      return { ...state, aiReviews }
+    })
+    return updated
+  },
+  getAiReviewById: (aiReviewId: string): AiReview | undefined => {
+    const state = get()
+    return state.aiReviews.find((ar) => ar.ai_review_id === aiReviewId)
+  },
+  listAiReviews: (): AiReview[] => {
+    const state = get()
+    return state.aiReviews
+  },
 }))

package/fake-snippets-api/lib/db/schema.ts

@@ -138,6 +138,17 @@ export const orderQuoteSchema = z.object({
 })
 export type OrderQuote = z.infer<typeof orderQuoteSchema>
 
+export const aiReviewSchema = z.object({
+  ai_review_id: z.string().uuid(),
+  ai_review_text: z.string().nullable(),
+  start_processing_at: z.string().datetime().nullable(),
+  finished_processing_at: z.string().datetime().nullable(),
+  processing_error: z.any().nullable(),
+  created_at: z.string().datetime(),
+  display_status: z.enum(["pending", "completed", "failed"]),
+})
+export type AiReview = z.infer<typeof aiReviewSchema>
+
 // TODO: Remove this schema after migration to accountPackages is complete
 export const accountSnippetSchema = z.object({
   account_id: z.string(),
@@ -203,7 +214,11 @@ export const packageReleaseSchema = z.object({
   circuit_json_build_is_stale: z.boolean().default(false),
 
   // AI Review
-  ai_review_text: z.string().nullable().default(null),
+  ai_review_text: z.string().nullable().default(null).optional(),
+  ai_review_started_at: z.string().datetime().nullable().optional(),
+  ai_review_completed_at: z.string().datetime().nullable().optional(),
+  ai_review_error: z.any().optional().nullable(),
+  ai_review_logs: z.array(z.any()).optional().nullable(),
   ai_review_requested: z.boolean().default(false),
 })
 export type PackageRelease = z.infer<typeof packageReleaseSchema>
@@ -310,5 +325,6 @@ export const databaseSchema = z.object({
   jlcpcbOrderState: z.array(jlcpcbOrderStateSchema).default([]),
   jlcpcbOrderStepRuns: z.array(jlcpcbOrderStepRunSchema).default([]),
   orderQuotes: z.array(orderQuoteSchema).default([]),
+  aiReviews: z.array(aiReviewSchema).default([]),
 })
 export type DatabaseSchema = z.infer<typeof databaseSchema>

package/fake-snippets-api/lib/public-mapping/public-map-package-release.ts

@@ -4,11 +4,13 @@ export const publicMapPackageRelease = (
   internal_package_release: ZT.PackageRelease,
   options: {
     include_logs?: boolean
+    include_ai_review?: boolean
   } = {
     include_logs: false,
+    include_ai_review: false,
   },
 ): ZT.PackageRelease => {
-  return {
+  const result = {
     ...internal_package_release,
     created_at: internal_package_release.created_at,
     circuit_json_build_error_last_updated_at:
@@ -21,4 +23,15 @@ export const publicMapPackageRelease = (
       ? internal_package_release.circuit_json_build_logs
       : [],
   }
+
+  // Only include AI review fields when include_ai_review is true
+  if (!options.include_ai_review) {
+    delete result.ai_review_text
+    delete result.ai_review_started_at
+    delete result.ai_review_completed_at
+    delete result.ai_review_error
+    delete result.ai_review_logs
+  }
+
+  return result
 }

package/fake-snippets-api/routes/api/_fake/ai_reviews/process_review.ts

@@ -0,0 +1,31 @@
+import { withRouteSpec } from "fake-snippets-api/lib/middleware/with-winter-spec"
+import { z } from "zod"
+import { aiReviewSchema } from "fake-snippets-api/lib/db/schema"
+
+export default withRouteSpec({
+  methods: ["POST"],
+  auth: "session",
+  jsonBody: z.object({
+    ai_review_id: z.string(),
+  }),
+  jsonResponse: z.object({
+    ai_review: aiReviewSchema,
+  }),
+})(async (req, ctx) => {
+  const { ai_review_id } = req.jsonBody
+  const existing = ctx.db.getAiReviewById(ai_review_id)
+  if (!existing) {
+    return ctx.error(404, {
+      error_code: "ai_review_not_found",
+      message: "AI review not found",
+    })
+  }
+  const now = new Date().toISOString()
+  const updated = ctx.db.updateAiReview(ai_review_id, {
+    ai_review_text: "Placeholder AI Review",
+    start_processing_at: existing.start_processing_at ?? now,
+    finished_processing_at: now,
+    display_status: "completed",
+  })!
+  return ctx.json({ ai_review: updated })
+})

package/fake-snippets-api/routes/api/ai_reviews/create.ts

@@ -0,0 +1,22 @@
+import { withRouteSpec } from "fake-snippets-api/lib/middleware/with-winter-spec"
+import { z } from "zod"
+import { aiReviewSchema } from "fake-snippets-api/lib/db/schema"
+
+export default withRouteSpec({
+  methods: ["POST"],
+  auth: "session",
+  jsonResponse: z.object({
+    ai_review: aiReviewSchema,
+  }),
+})(async (req, ctx) => {
+  const ai_review = ctx.db.addAiReview({
+    ai_review_text: null,
+    start_processing_at: null,
+    finished_processing_at: null,
+    processing_error: null,
+    created_at: new Date().toISOString(),
+    display_status: "pending",
+  })
+
+  return ctx.json({ ai_review })
+})

package/fake-snippets-api/routes/api/ai_reviews/get.ts

@@ -0,0 +1,24 @@
+import { withRouteSpec } from "fake-snippets-api/lib/middleware/with-winter-spec"
+import { z } from "zod"
+import { aiReviewSchema } from "fake-snippets-api/lib/db/schema"
+
+export default withRouteSpec({
+  methods: ["GET"],
+  auth: "session",
+  queryParams: z.object({
+    ai_review_id: z.string(),
+  }),
+  jsonResponse: z.object({
+    ai_review: aiReviewSchema,
+  }),
+})(async (req, ctx) => {
+  const { ai_review_id } = req.query
+  const ai_review = ctx.db.getAiReviewById(ai_review_id)
+  if (!ai_review) {
+    return ctx.error(404, {
+      error_code: "ai_review_not_found",
+      message: "AI review not found",
+    })
+  }
+  return ctx.json({ ai_review })
+})

package/fake-snippets-api/routes/api/ai_reviews/list.ts

@@ -0,0 +1,14 @@
+import { withRouteSpec } from "fake-snippets-api/lib/middleware/with-winter-spec"
+import { z } from "zod"
+import { aiReviewSchema } from "fake-snippets-api/lib/db/schema"
+
+export default withRouteSpec({
+  methods: ["GET"],
+  auth: "session",
+  jsonResponse: z.object({
+    ai_reviews: z.array(aiReviewSchema),
+  }),
+})(async (req, ctx) => {
+  const ai_reviews = ctx.db.listAiReviews()
+  return ctx.json({ ai_reviews })
+})

package/fake-snippets-api/routes/api/package_releases/get.ts

@@ -8,6 +8,7 @@ export default withRouteSpec({
   auth: "none",
   commonParams: z.object({
     include_logs: z.boolean().optional().default(false),
+    include_ai_review: z.boolean().optional().default(false),
   }),
   jsonBody: z.object({
     package_release_id: z.string().optional(),
@@ -58,7 +59,9 @@ export default withRouteSpec({
 
     return ctx.json({
       ok: true,
-      package_release: publicMapPackageRelease(packageReleases[0]),
+      package_release: publicMapPackageRelease(packageReleases[0], {
+        include_ai_review: req.commonParams?.include_ai_review,
+      }),
     })
   }
 
@@ -82,7 +85,9 @@ export default withRouteSpec({
 
     return ctx.json({
       ok: true,
-      package_release: publicMapPackageRelease(packageReleases[0]),
+      package_release: publicMapPackageRelease(packageReleases[0], {
+        include_ai_review: req.commonParams?.include_ai_review,
+      }),
     })
   }
 
@@ -102,7 +107,9 @@ export default withRouteSpec({
 
     return ctx.json({
       ok: true,
-      package_release: publicMapPackageRelease(pkgRelease),
+      package_release: publicMapPackageRelease(pkgRelease, {
+        include_ai_review: req.commonParams?.include_ai_review,
+      }),
     })
   }
 
@@ -120,6 +127,7 @@ export default withRouteSpec({
     ok: true,
     package_release: publicMapPackageRelease(foundRelease, {
       include_logs: req.commonParams?.include_logs,
+      include_ai_review: req.commonParams?.include_ai_review,
     }),
   })
 })

package/fake-snippets-api/routes/api/package_releases/list.ts

@@ -6,6 +6,9 @@ import { z } from "zod"
 export default withRouteSpec({
   methods: ["POST"],
   auth: "none",
+  commonParams: z.object({
+    include_ai_review: z.boolean().optional().default(false),
+  }),
   jsonBody: z
     .object({
       package_id: z.string().optional(),
@@ -72,6 +75,10 @@ export default withRouteSpec({
 
   return ctx.json({
     ok: true,
-    package_releases: releases.map((pr) => publicMapPackageRelease(pr)),
+    package_releases: releases.map((pr) =>
+      publicMapPackageRelease(pr, {
+        include_ai_review: req.commonParams?.include_ai_review,
+      }),
+    ),
   })
 })

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@tscircuit/fake-snippets",
-  "version": "0.0.82",
+  "version": "0.0.84",
   "type": "module",
   "repository": {
     "type": "git",
@@ -86,7 +86,7 @@
     "circuit-json-to-bom-csv": "^0.0.6",
     "circuit-json-to-gerber": "^0.0.21",
     "circuit-json-to-pnp-csv": "^0.0.6",
-    "circuit-json-to-readable-netlist": "^0.0.8",
+    "circuit-json-to-readable-netlist": "^0.0.13",
     "circuit-json-to-tscircuit": "^0.0.4",
     "class-variance-authority": "^0.7.1",
     "clsx": "^2.1.1",
@@ -145,8 +145,9 @@
     "@playwright/test": "^1.48.0",
     "@tailwindcss/typography": "^0.5.16",
     "@tscircuit/core": "^0.0.433",
+    "@tscircuit/eval": "^0.0.227",
     "@tscircuit/prompt-benchmarks": "^0.0.28",
-    "@tscircuit/runframe": "^0.0.562",
+    "@tscircuit/runframe": "^0.0.578",
     "@types/babel__standalone": "^7.1.7",
     "@types/bun": "^1.1.10",
     "@types/country-list": "^2.1.4",

package/src/App.tsx

@@ -1,5 +1,4 @@
 import { ComponentType, Suspense, lazy } from "react"
-import { Toaster } from "@/components/ui/toaster"
 import { Route, Switch } from "wouter"
 import "./components/CmdKMenu"
 import { ContextProviders } from "./ContextProviders"
@@ -129,7 +128,6 @@ function App() {
             <Route component={lazyImport(() => import("@/pages/404"))} />
           </Switch>
         </Suspense>
-        <Toaster />
       </ErrorBoundary>
     </ContextProviders>
   )

package/src/ContextProviders.tsx

@@ -64,7 +64,7 @@ export const ContextProviders = ({ children }: any) => {
       <HelmetProvider>
         <PostHogIdentifier />
         {children}
-        <Toaster />
+        <Toaster position="bottom-right" />
       </HelmetProvider>
     </QueryClientProvider>
   )

package/src/components/Header2.tsx

@@ -15,18 +15,15 @@ const SearchButtonComponent = () => {
   return (
     <div className="relative">
       {isExpanded ? (
-        <div className="flex items-center gap-2">
-          <div className="w-32 bg-white">
-            <SearchComponent autofocus />
+        <div className="flex items-center gap-2 ml-8">
+          <div className="absolute -top-4 right-3 bg-white">
+            <SearchComponent
+              autofocus
+              closeOnClick={() => {
+                setIsExpanded(false)
+              }}
+            />
           </div>
-          {/* <Button
-            variant="ghost"
-            size="icon"
-            onClick={() => setIsExpanded(false)}
-            className="h-8 w-8"
-          >
-            <X className="h-4 w-4" />
-          </Button> */}
         </div>
       ) : (
         <>
@@ -58,13 +55,6 @@ export const Header2 = () => {
   const isLoggedIn = useGlobalStore((state) => Boolean(state.session))
   return (
     <>
-      {/* <div className="absolute left-0 top-0 z-[9999999]">
-        <div className="hidden xl:block">xl</div>
-        <div className="hidden lg:block xl:hidden">lg</div>
-        <div className="hidden md:block lg:hidden">md</div>
-        <div className="hidden sm:block md:hidden">sm</div>
-        <div className="hidden xs:block sm:hidden">xs</div>
-      </div> */}
       <header className="sticky top-0 z-50 w-full border-b bg-background/95 backdrop-blur supports-[backdrop-filter]:bg-background/60">
         <div className="container mx-auto flex h-16 items-center justify-between px-2 md:px-6">
           <div className="flex items-center gap-2">

package/src/components/PackageBuildsPage/package-build-header.tsx

@@ -1,13 +1,15 @@
 import { Button } from "@/components/ui/button"
 import { useCurrentPackageRelease } from "@/hooks/use-current-package-release"
 import { useRebuildPackageReleaseMutation } from "@/hooks/use-rebuild-package-release-mutation"
-import { Github, RefreshCw } from "lucide-react"
+import { Github, RefreshCw, RotateCcw } from "lucide-react"
 import { useParams } from "wouter"
 import { DownloadButtonAndMenu } from "../DownloadButtonAndMenu"
 
 export function PackageBuildHeader() {
   const { author, packageName } = useParams()
-  const { packageRelease } = useCurrentPackageRelease({ refetchInterval: 2000 })
+  const { packageRelease, refetch, isFetching } = useCurrentPackageRelease({
+    include_logs: true,
+  })
   const { mutate: rebuildPackage, isLoading } =
     useRebuildPackageReleaseMutation()
 
@@ -54,6 +56,16 @@ export function PackageBuildHeader() {
             <RefreshCw className="w-3 h-3 sm:w-4 sm:h-4 mr-1 sm:mr-2" />
             {isLoading ? "Rebuilding..." : "Rebuild"}
           </Button>
+          <Button
+            variant="outline"
+            size="icon"
+            aria-label="Reload logs"
+            className="border-gray-300 bg-white hover:bg-gray-50"
+            onClick={() => refetch()}
+            disabled={isFetching}
+          >
+            <RotateCcw className="w-3 h-3 sm:w-4 sm:h-4" />
+          </Button>
           <DownloadButtonAndMenu
             snippetUnscopedName={`${author}/${packageName}`}
           />

package/src/components/SearchComponent.tsx

@@ -6,10 +6,12 @@ import { useQuery } from "react-query"
 import { Alert } from "./ui/alert"
 import { useSnippetsBaseApiUrl } from "@/hooks/use-snippets-base-api-url"
 import { PrefetchPageLink } from "./PrefetchPageLink"
+import { CircuitBoard } from "lucide-react"
 
 interface SearchComponentProps {
   onResultsFetched?: (results: any[]) => void
   autofocus?: boolean
+  closeOnClick?: () => void
 }
 
 const LinkWithNewTabHandling = ({
@@ -45,13 +47,14 @@ const LinkWithNewTabHandling = ({
 const SearchComponent: React.FC<SearchComponentProps> = ({
   onResultsFetched,
   autofocus = false,
+  closeOnClick,
 }) => {
   const [searchQuery, setSearchQuery] = useState("")
   const [showResults, setShowResults] = useState(false)
   const axios = useAxios()
   const resultsRef = useRef<HTMLDivElement>(null)
   const inputRef = useRef<HTMLInputElement>(null)
-  const [location] = useLocation()
+  const [location, setLocation] = useLocation()
   const snippetsBaseApiUrl = useSnippetsBaseApiUrl()
 
   const { data: searchResults, isLoading } = useQuery(
@@ -71,7 +74,9 @@ const SearchComponent: React.FC<SearchComponentProps> = ({
 
   const handleSearch = (e: React.FormEvent) => {
     e.preventDefault()
-    setShowResults(!!searchQuery)
+    if (searchQuery.trim()) {
+      setLocation(`/search?q=${encodeURIComponent(searchQuery.trim())}`)
+    }
   }
 
   // Focus input on mount
@@ -91,18 +96,31 @@ const SearchComponent: React.FC<SearchComponentProps> = ({
       }
     }
 
+    const handleEscapeKey = (event: KeyboardEvent) => {
+      if (event.key === "Escape") {
+        setShowResults(false)
+        if (closeOnClick) {
+          closeOnClick()
+        }
+      }
+    }
+
     document.addEventListener("mousedown", handleClickOutside)
+    document.addEventListener("keydown", handleEscapeKey)
     return () => {
       document.removeEventListener("mousedown", handleClickOutside)
+      document.removeEventListener("keydown", handleEscapeKey)
     }
-  }, [])
+  }, [closeOnClick])
 
   const shouldOpenInNewTab = location === "/editor" || location === "/ai"
   const shouldOpenInEditor = location === "/editor" || location === "/ai"
 
   return (
-    <form onSubmit={handleSearch} className="relative">
+    <form onSubmit={handleSearch} autoComplete="off" className="relative w-44">
       <Input
+        autoComplete="off"
+        spellCheck={false}
         ref={inputRef}
         type="search"
         placeholder="Search"
@@ -112,12 +130,20 @@ const SearchComponent: React.FC<SearchComponentProps> = ({
           setSearchQuery(e.target.value)
           setShowResults(!!e.target.value)
         }}
+        onKeyDown={(e) => {
+          if (e.key === "Backspace" && !searchQuery && closeOnClick) {
+            closeOnClick()
+          }
+        }}
         aria-label="Search packages"
         role="searchbox"
       />
       {isLoading && (
-        <div className="absolute top-full left-0 right-0 mt-2 bg-white shadow-lg rounded-md z-10 p-2 flex items-center justify-center space-x-2">
-          <span className="text-gray-500 text-sm">Loading...</span>
+        <div className="absolute top-full w-lg left-0 right-0 mt-1 bg-white shadow-lg rounded-lg border w-80 grid place-items-center py-4 z-10 p-3">
+          <div className="flex items-center space-x-2">
+            <div className="w-4 h-4 border-2 border-blue-500 border-t-transparent rounded-full animate-spin"></div>
+            <span className="text-gray-600 text-sm">Searching...</span>
+          </div>
         </div>
       )}
 
@@ -139,12 +165,24 @@ const SearchComponent: React.FC<SearchComponentProps> = ({
                     shouldOpenInNewTab={shouldOpenInNewTab}
                     className="flex"
                   >
-                    <div className="w-12 h-12 overflow-hidden mr-2 flex-shrink-0 rounded-sm">
+                    <div className="w-12 h-12 overflow-hidden mr-2 flex-shrink-0 rounded-sm bg-gray-50 border flex items-center justify-center">
                       <img
                         src={`${snippetsBaseApiUrl}/snippets/images/${pkg.name}/pcb.svg`}
                         alt={`PCB preview for ${pkg.name}`}
                         className="w-12 h-12 object-contain p-1 scale-[4] rotate-45"
+                        onError={(e) => {
+                          e.currentTarget.style.display = "none"
+                          e.currentTarget.nextElementSibling?.classList.remove(
+                            "hidden",
+                          )
+                          e.currentTarget.nextElementSibling?.classList.add(
+                            "flex",
+                          )
+                        }}
                       />
+                      <div className="w-12 h-12 hidden items-center justify-center">
+                        <CircuitBoard className="w-6 h-6 text-gray-300" />
+                      </div>
                     </div>
                     <div className="flex-grow">
                       <div className="font-medium text-blue-600 break-words text-xs">
@@ -161,7 +199,7 @@ const SearchComponent: React.FC<SearchComponentProps> = ({
               ))}
             </ul>
           ) : (
-            <Alert variant="default" className="p-4">
+            <Alert variant="default" className="p-4 text-center">
               No results found for "{searchQuery}"
             </Alert>
           )}

package/src/components/ViewPackagePage/hooks/use-toast.tsx

@@ -0,0 +1,70 @@
+"use client"
+import toastLibrary, { Toaster, type Toast } from "react-hot-toast"
+import React from "react"
+
+export interface ToasterToast {
+  title?: React.ReactNode
+  description?: React.ReactNode
+  variant?: "default" | "destructive"
+  duration?: number
+}
+
+function ToastContent({
+  title,
+  description,
+  variant,
+  t,
+}: ToasterToast & { t: Toast }) {
+  return (
+    <div
+      className={`rounded-md border p-4 shadow-lg transition-all ${
+        t.visible
+          ? "animate-in fade-in slide-in-from-top-full"
+          : "animate-out fade-out slide-out-to-right-full"
+      } ${
+        variant === "destructive"
+          ? "border-red-500 bg-red-500 text-slate-50"
+          : "border-slate-200 bg-white text-slate-950 dark:bg-slate-950 dark:text-slate-50"
+      }`}
+    >
+      {title && <div className="text-sm font-semibold">{title}</div>}
+      {description && <div className="text-sm opacity-90">{description}</div>}
+    </div>
+  )
+}
+
+const toast = ({
+  duration,
+  description,
+  variant = "default",
+  title,
+}: ToasterToast) => {
+  if (description) {
+    return toastLibrary.custom(
+      (t) => (
+        <ToastContent
+          title={title}
+          description={description}
+          variant={variant}
+          t={t}
+        />
+      ),
+      { duration },
+    )
+  }
+
+  if (variant === "destructive") {
+    return toastLibrary.error(<>{title}</>, { duration })
+  }
+
+  return toastLibrary(<>{title}</>, { duration })
+}
+
+function useToast() {
+  return {
+    toast,
+    dismiss: toastLibrary.dismiss,
+  }
+}
+
+export { useToast, toast, Toaster }

package/src/components/ViewSnippetHeader.tsx

@@ -63,18 +63,21 @@ export default function ViewSnippetHeader() {
           onSuccess?.(forkedSnippet)
         },
         onError: (error: any) => {
-          // Check if the error message contains 'already exists'
-          if (error.message?.includes("already forked")) {
+          const message =
+            error?.data?.error?.message ||
+            error.message ||
+            "Failed to fork snippet. Please try again."
+          if (message.includes("already forked")) {
             toast({
               title: "Snippet already exists",
-              description: error.message,
-              variant: "destructive", // You can style this variant differently
+              description: message,
+              variant: "destructive",
             })
           } else {
             toast({
               title: "Error",
-              description: "Failed to fork snippet. Please try again.",
-              variant: "destructive", // Use destructive variant for errors
+              description: message,
+              variant: "destructive",
             })
           }
           console.error("Error forking snippet:", error)