small review fixes

Author: mpcgrid

apps/webapp/app/components/code/CodeBlock.tsx

@@ -458,7 +458,6 @@ type HighlightCodeProps = {
   preClassName?: string;
   isWrapped: boolean;
   searchTerm?: string;
-  containerRef?: React.RefObject<HTMLDivElement>;
 };
 
 function HighlightCode({
@@ -472,7 +471,6 @@ function HighlightCode({
   preClassName,
   isWrapped,
   searchTerm,
-  containerRef,
 }: HighlightCodeProps) {
   const [isLoaded, setIsLoaded] = useState(false);
 
@@ -502,7 +500,7 @@ function HighlightCode({
 
   if (!isLoaded) {
     return (
-      <div dir="ltr" className={containerClasses} ref={containerRef}>
+      <div dir="ltr" className={containerClasses}>
         <pre className={preClasses}>{code}</pre>
       </div>
     );
@@ -517,7 +515,7 @@ function HighlightCode({
         getLineProps,
         getTokenProps,
       }) => (
-        <div dir="ltr" className={containerClasses} ref={containerRef}>
+        <div dir="ltr" className={containerClasses}>
           <pre className={cn(preClasses, inheritedClassName)} style={inheritedStyle} dir="ltr">
             {tokens
               .map((line, index) => {

apps/webapp/app/components/logs/LogsTable.tsx

@@ -204,12 +204,7 @@ export function LogsTable({
       {/* Infinite scroll trigger */}
       {hasMore && logs.length > 0 && (
         <div ref={loadMoreRef} className="flex items-center justify-center py-12">
-          <div
-            className={cn(
-              "flex items-center gap-2",
-              !showLoadMoreSpinner && "invisible"
-            )}
-          >
+          <div className={cn("flex items-center gap-2", !showLoadMoreSpinner && "invisible")}>
             <Spinner /> <span className="text-text-dimmed">Loading more…</span>
           </div>
         </div>
@@ -224,8 +219,9 @@ export function LogsTable({
                 LeadingIcon={ArrowPathIcon}
                 variant="tertiary/small"
                 onClick={onCheckForMore}
+                disabled={isLoadingMore}
               >
-                Check for new logs
+                {isLoadingMore ? "Checking…" : "Check for new logs"}
               </Button>
             )}
           </div>

apps/webapp/app/routes/resources.timezone.ts

@@ -9,14 +9,27 @@ const schema = z.object({
   timezone: z.string().min(1).max(100),
 });
 
+// Cache the supported timezones to avoid repeated calls
+const supportedTimezones = new Set(Intl.supportedValuesOf("timeZone"));
+
 export async function action({ request }: ActionFunctionArgs) {
-  const data = await request.json();
+  let data: unknown;
+  try {
+    data = await request.json();
+  } catch {
+    return json({ success: false, error: "Invalid JSON" }, { status: 400 });
+  }
+
   const result = schema.safeParse(data);
 
   if (!result.success) {
     return json({ success: false, error: "Invalid timezone" }, { status: 400 });
   }
 
+  if (!supportedTimezones.has(result.data.timezone)) {
+    return json({ success: false, error: "Invalid timezone" }, { status: 400 });
+  }
+
   const session = await setTimezonePreference(result.data.timezone, request);
 
   return json(