fix(build): single-brace JSX for schoolUrl, migrate images.domains to remotePatterns

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

nextjs-app/components/HomeView.tsx

@@ -317,7 +317,7 @@ function CompactSchoolItem({ school, onAddToCompare, isInCompare }: CompactSchoo
     <div className={styles.compactItem}>
       <div className={styles.compactItemContent}>
         <div className={styles.compactItemHeader}>
-          <a href={{schoolUrl(school.urn, school.school_name)}} className={styles.compactItemName}>
+          <a href={schoolUrl(school.urn, school.school_name)} className={styles.compactItemName}>
             {school.school_name}
           </a>
           {school.distance !== undefined && school.distance !== null && (
@@ -353,7 +353,7 @@ function CompactSchoolItem({ school, onAddToCompare, isInCompare }: CompactSchoo
         >
           {isInCompare ? '✓ Comparing' : '+ Compare'}
         </button>
-        <a href={{schoolUrl(school.urn, school.school_name)}} className="btn btn-tertiary btn-sm">
+        <a href={schoolUrl(school.urn, school.school_name)} className="btn btn-tertiary btn-sm">
           View
         </a>
       </div>

nextjs-app/components/RankingsView.tsx

@@ -267,7 +267,7 @@ export function RankingsView({
                         )}
                       </td>
                       <td className={styles.schoolCell}>
-                        <a href={{schoolUrl(ranking.urn, ranking.school_name)}} className={styles.schoolLink}>
+                        <a href={schoolUrl(ranking.urn, ranking.school_name)} className={styles.schoolLink}>
                           {ranking.school_name}
                         </a>
                       </td>
@@ -277,7 +277,7 @@ export function RankingsView({
                         <strong>{displayValue}</strong>
                       </td>
                       <td className={styles.actionCell}>
-                        <a href={{schoolUrl(ranking.urn, ranking.school_name)}} className="btn btn-tertiary btn-sm">View</a>
+                        <a href={schoolUrl(ranking.urn, ranking.school_name)} className="btn btn-tertiary btn-sm">View</a>
                         <button
                           onClick={() => handleAddToCompare(ranking)}
                           disabled={alreadyInComparison}

nextjs-app/components/SchoolCard.tsx

@@ -25,7 +25,7 @@ export function SchoolCard({ school, onAddToCompare, onRemoveFromCompare, showDi
     <div className={`${styles.card} ${isInCompare ? styles.cardInCompare : ''}`}>
       <div className={styles.header}>
         <h3 className={styles.title}>
-          <Link href={{schoolUrl(school.urn, school.school_name)}}>
+          <Link href={schoolUrl(school.urn, school.school_name)}>
             {school.school_name}
           </Link>
         </h3>
@@ -146,7 +146,7 @@ export function SchoolCard({ school, onAddToCompare, onRemoveFromCompare, showDi
       )}
 
       <div className={styles.actions}>
-        <Link href={{schoolUrl(school.urn, school.school_name)}} className="btn btn-primary">
+        <Link href={schoolUrl(school.urn, school.school_name)} className="btn btn-primary">
           View Details
         </Link>
         {onAddToCompare && (

nextjs-app/components/SchoolRow.tsx

@@ -61,7 +61,7 @@ export function SchoolRow({
 
         {/* Line 1: School name + Ofsted badge */}
         <div className={styles.line1}>
-          <a href={{schoolUrl(school.urn, school.school_name)}} className={styles.schoolName}>
+          <a href={schoolUrl(school.urn, school.school_name)} className={styles.schoolName}>
             {school.school_name}
           </a>
           {school.ofsted_grade && (
@@ -155,7 +155,7 @@ export function SchoolRow({
 
       {/* Right: actions, vertically centred */}
       <div className={styles.rowActions}>
-        <a href={{schoolUrl(school.urn, school.school_name)}} className="btn btn-tertiary btn-sm">
+        <a href={schoolUrl(school.urn, school.school_name)} className="btn btn-tertiary btn-sm">
           View
         </a>
         {(onAddToCompare || onRemoveFromCompare) && (

nextjs-app/components/SecondarySchoolRow.tsx

@@ -73,7 +73,7 @@ export function SecondarySchoolRow({
 
         {/* Line 1: School name + Ofsted badge */}
         <div className={styles.line1}>
-          <a href={{schoolUrl(school.urn, school.school_name)}} className={styles.schoolName}>
+          <a href={schoolUrl(school.urn, school.school_name)} className={styles.schoolName}>
             {school.school_name}
           </a>
           {school.ofsted_grade && (
@@ -155,7 +155,7 @@ export function SecondarySchoolRow({
 
       {/* Right: actions */}
       <div className={styles.rowActions}>
-        <a href={{schoolUrl(school.urn, school.school_name)}} className="btn btn-tertiary btn-sm">
+        <a href={schoolUrl(school.urn, school.school_name)} className="btn btn-tertiary btn-sm">
           View
         </a>
         {(onAddToCompare || onRemoveFromCompare) && (

nextjs-app/next.config.js

@@ -16,12 +16,10 @@ const nextConfig = {
 
   // Image optimization
   images: {
-    domains: [
-      'tile.openstreetmap.org',
-      'a.tile.openstreetmap.org',
-      'b.tile.openstreetmap.org',
-      'c.tile.openstreetmap.org',
-      'cdnjs.cloudflare.com',
+    remotePatterns: [
+      { protocol: 'https', hostname: '*.tile.openstreetmap.org' },
+      { protocol: 'https', hostname: 'tile.openstreetmap.org' },
+      { protocol: 'https', hostname: 'cdnjs.cloudflare.com' },
     ],
     formats: ['image/avif', 'image/webp'],
     minimumCacheTTL: 60,