From 386428118f58950c0e2f24a9eca799be58c91bdc Mon Sep 17 00:00:00 2001
From: Claude <noreply@anthropic.com>
Date: Wed, 5 Nov 2025 10:21:50 +0000
Subject: [PATCH 1/4] Add tall climbs only filter to BoardSesh

This commit adds a new filter toggle that shows only climbs using holds
available exclusively on the largest board size.

Changes:
- Add tallClimbsOnly boolean to SearchRequest type
- Update filter builder with SQL logic to identify climbs extending beyond
  smaller size boundaries (checking edgeTop, edgeBottom, edgeLeft, edgeRight)
- Add isLargestSize flag to BoardDetails to determine board size hierarchy
- Add UI toggle in BasicSearchForm (visible only on largest size)
- Update URL parameter serialization and analytics tracking
- Fix test mocks to include new tallClimbsOnly property

The filter helps users identify climbs that require the full height/width
of their board configuration.
---
 .../[set_ids]/[angle]/list/layout-client.tsx  |  2 +-
 .../hooks/use-queue-data-fetching.test.tsx    |  3 +-
 .../queue-control/__tests__/reducer.test.ts   |  6 ++-
 .../ui-searchparams-provider.tsx              |  1 +
 .../search-drawer/basic-search-form.tsx       | 20 +++++++-
 app/components/search-drawer/search-form.tsx  |  2 +-
 app/lib/data/queries.ts                       | 24 +++++++++-
 .../db/queries/climbs/create-climb-filters.ts | 47 ++++++++++++++++++-
 app/lib/types.ts                              |  2 +
 app/lib/url-utils.ts                          |  6 +++
 10 files changed, 104 insertions(+), 9 deletions(-)

diff --git a/app/[board_name]/[layout_id]/[size_id]/[set_ids]/[angle]/list/layout-client.tsx b/app/[board_name]/[layout_id]/[size_id]/[set_ids]/[angle]/list/layout-client.tsx
index 8afbe60..bc92d0e 100644
--- a/app/[board_name]/[layout_id]/[size_id]/[set_ids]/[angle]/list/layout-client.tsx
+++ b/app/[board_name]/[layout_id]/[size_id]/[set_ids]/[angle]/list/layout-client.tsx
@@ -44,7 +44,7 @@ const TabsWrapper: React.FC<{ boardDetails: BoardDetails }> = ({ boardDetails })
       children: (
         <div style={{ height: '100%', display: 'flex', flexDirection: 'column' }}>
           <div style={{ flex: 1, overflow: 'auto' }}>
-            <BasicSearchForm />
+            <BasicSearchForm boardDetails={boardDetails} />
           </div>
           <SearchResultsFooter />
         </div>
diff --git a/app/components/queue-control/__tests__/hooks/use-queue-data-fetching.test.tsx b/app/components/queue-control/__tests__/hooks/use-queue-data-fetching.test.tsx
index a589129..7cb3fbf 100644
--- a/app/components/queue-control/__tests__/hooks/use-queue-data-fetching.test.tsx
+++ b/app/components/queue-control/__tests__/hooks/use-queue-data-fetching.test.tsx
@@ -81,7 +81,8 @@ const mockSearchParams: SearchRequestPagination = {
   hideAttempted: false,
   hideCompleted: false,
   showOnlyAttempted: false,
-  showOnlyCompleted: false
+  showOnlyCompleted: false,
+  tallClimbsOnly: false
 };
 
 const mockParsedParams: ParsedBoardRouteParameters = {
diff --git a/app/components/queue-control/__tests__/reducer.test.ts b/app/components/queue-control/__tests__/reducer.test.ts
index 34b11b5..22da6d3 100644
--- a/app/components/queue-control/__tests__/reducer.test.ts
+++ b/app/components/queue-control/__tests__/reducer.test.ts
@@ -47,7 +47,8 @@ const mockSearchParams: SearchRequestPagination = {
   hideAttempted: false,
   hideCompleted: false,
   showOnlyAttempted: false,
-  showOnlyCompleted: false
+  showOnlyCompleted: false,
+  tallClimbsOnly: false
 };
 
 const initialState: QueueState = {
@@ -292,7 +293,8 @@ describe('queueReducer', () => {
         hideAttempted: false,
         hideCompleted: false,
         showOnlyAttempted: false,
-        showOnlyCompleted: false
+        showOnlyCompleted: false,
+        tallClimbsOnly: false
       };
 
       const action: QueueAction = {
diff --git a/app/components/queue-control/ui-searchparams-provider.tsx b/app/components/queue-control/ui-searchparams-provider.tsx
index 9b4d2ef..456fce9 100644
--- a/app/components/queue-control/ui-searchparams-provider.tsx
+++ b/app/components/queue-control/ui-searchparams-provider.tsx
@@ -41,6 +41,7 @@ export const UISearchParamsProvider: React.FC<{ children: React.ReactNode }> = (
     if (uiSearchParams.hideCompleted) activeFilters.push('hideCompleted');
     if (uiSearchParams.showOnlyAttempted) activeFilters.push('showOnlyAttempted');
     if (uiSearchParams.showOnlyCompleted) activeFilters.push('showOnlyCompleted');
+    if (uiSearchParams.tallClimbsOnly) activeFilters.push('tallClimbsOnly');
 
     if (activeFilters.length > 0) {
       track('Climb Search Performed', {
diff --git a/app/components/search-drawer/basic-search-form.tsx b/app/components/search-drawer/basic-search-form.tsx
index e648135..4678fec 100644
--- a/app/components/search-drawer/basic-search-form.tsx
+++ b/app/components/search-drawer/basic-search-form.tsx
@@ -7,15 +7,21 @@ import { useUISearchParams } from '@/app/components/queue-control/ui-searchparam
 import { useBoardProvider } from '@/app/components/board-provider/board-provider-context';
 import SearchClimbNameInput from './search-climb-name-input';
 import SetterNameSelect from './setter-name-select';
+import { BoardDetails } from '@/app/lib/types';
 
 const { Title } = Typography;
 
-const BasicSearchForm: React.FC = () => {
+interface BasicSearchFormProps {
+  boardDetails: BoardDetails;
+}
+
+const BasicSearchForm: React.FC<BasicSearchFormProps> = ({ boardDetails }) => {
   const { uiSearchParams, updateFilters } = useUISearchParams();
   const { token, user_id } = useBoardProvider();
   const grades = TENSION_KILTER_GRADES;
-  
+
   const isLoggedIn = token && user_id;
+  const isLargestSize = boardDetails.isLargestSize ?? false;
 
   const handleGradeChange = (type: 'min' | 'max', value: number | undefined) => {
     if (type === 'min') {
@@ -178,6 +184,16 @@ const BasicSearchForm: React.FC = () => {
         />
       </Form.Item>
 
+      {isLargestSize && (
+        <Form.Item label="Tall Climbs Only" valuePropName="checked">
+          <Switch
+            style={{ float: 'right' }}
+            checked={uiSearchParams.tallClimbsOnly}
+            onChange={(checked) => updateFilters({ tallClimbsOnly: checked })}
+          />
+        </Form.Item>
+      )}
+
       <Form.Item label="Grade Accuracy">
         <Select
           value={uiSearchParams.gradeAccuracy}
diff --git a/app/components/search-drawer/search-form.tsx b/app/components/search-drawer/search-form.tsx
index 58b8d38..5dd2d12 100644
--- a/app/components/search-drawer/search-form.tsx
+++ b/app/components/search-drawer/search-form.tsx
@@ -14,7 +14,7 @@ const SearchForm: React.FC<SearchFormProps> = ({ boardDetails }) => {
     {
       key: 'filters',
       label: 'Search',
-      children: <BasicSearchForm />,
+      children: <BasicSearchForm boardDetails={boardDetails} />,
     },
     {
       key: 'holds',
diff --git a/app/lib/data/queries.ts b/app/lib/data/queries.ts
index 42d8648..823b665 100644
--- a/app/lib/data/queries.ts
+++ b/app/lib/data/queries.ts
@@ -104,7 +104,7 @@ export const getBoardDetails = async ({
       })),
   );
 
-  // Fetch names for slug-based URLs
+  // Fetch names for slug-based URLs and all sizes to determine if current is largest
   const [layouts, sizes, sets] = await Promise.all([
     getLayouts(board_name),
     getSizes(board_name, layout_id),
@@ -115,6 +115,27 @@ export const getBoardDetails = async ({
   const size = sizes.find((s) => s.id === size_id);
   const selectedSets = sets.filter((s) => set_ids.includes(s.id));
 
+  // Determine if current size is the largest by checking all size dimensions
+  // Get all sizes for this layout to compare
+  const allSizesForLayoutResult = await sql`
+    SELECT edge_left, edge_right, edge_bottom, edge_top
+    FROM ${sql.unsafe(getTableName(board_name, 'product_sizes'))} ps
+    INNER JOIN ${sql.unsafe(getTableName(board_name, 'layouts'))} layouts ON ps.product_id = layouts.product_id
+    WHERE layouts.id = ${layout_id}
+  `;
+
+  const allSizesForLayout = allSizesForLayoutResult as ProductSizeRow[];
+  const currentEdges = sizeDimensions[0];
+  const isLargestSize = !allSizesForLayout.some((otherSize) => {
+    // Check if any other size has larger boundaries
+    return (
+      otherSize.edge_top > currentEdges.edge_top ||
+      otherSize.edge_bottom < currentEdges.edge_bottom ||
+      otherSize.edge_left < currentEdges.edge_left ||
+      otherSize.edge_right > currentEdges.edge_right
+    );
+  });
+
   return {
     images_to_holds: imagesToHolds,
     holdsData,
@@ -130,6 +151,7 @@ export const getBoardDetails = async ({
     set_ids,
     ledPlacements: Object.fromEntries(ledPlacements.map(({ id, position }) => [id, position])),
     supportsMirroring: board_name === 'tension' && layout_id !== 11,
+    isLargestSize,
     // Added for slug-based URLs
     layout_name: layout?.name,
     size_name: size?.name,
diff --git a/app/lib/db/queries/climbs/create-climb-filters.ts b/app/lib/db/queries/climbs/create-climb-filters.ts
index 9e067e7..fb577aa 100644
--- a/app/lib/db/queries/climbs/create-climb-filters.ts
+++ b/app/lib/db/queries/climbs/create-climb-filters.ts
@@ -145,6 +145,50 @@ export const createClimbFilters = (
     }
   }
 
+  // Tall climbs filter condition
+  // A climb is "tall" if it extends beyond the boundaries of smaller sizes
+  const tallClimbsConditions: SQL[] = [];
+  if (searchParams.tallClimbsOnly) {
+    // Filter climbs that extend beyond the maximum edgeTop of smaller sizes
+    // This subquery finds the max edgeTop of all sizes smaller than the current size
+    tallClimbsConditions.push(
+      sql`(
+        ${tables.climbs.edgeTop} > (
+          SELECT COALESCE(MAX(ps2.edge_top), 0)
+          FROM ${tables.productSizes} ps2
+          INNER JOIN ${tables.layouts} layouts2 ON ps2.product_id = layouts2.product_id
+          WHERE layouts2.id = ${params.layout_id}
+          AND ps2.id != ${params.size_id}
+          AND ps2.edge_top < ${sizeTable.edgeTop}
+        )
+        OR ${tables.climbs.edgeBottom} < (
+          SELECT COALESCE(MIN(ps2.edge_bottom), 999)
+          FROM ${tables.productSizes} ps2
+          INNER JOIN ${tables.layouts} layouts2 ON ps2.product_id = layouts2.product_id
+          WHERE layouts2.id = ${params.layout_id}
+          AND ps2.id != ${params.size_id}
+          AND ps2.edge_bottom > ${sizeTable.edgeBottom}
+        )
+        OR ${tables.climbs.edgeLeft} < (
+          SELECT COALESCE(MIN(ps2.edge_left), 999)
+          FROM ${tables.productSizes} ps2
+          INNER JOIN ${tables.layouts} layouts2 ON ps2.product_id = layouts2.product_id
+          WHERE layouts2.id = ${params.layout_id}
+          AND ps2.id != ${params.size_id}
+          AND ps2.edge_left > ${sizeTable.edgeLeft}
+        )
+        OR ${tables.climbs.edgeRight} > (
+          SELECT COALESCE(MAX(ps2.edge_right), 0)
+          FROM ${tables.productSizes} ps2
+          INNER JOIN ${tables.layouts} layouts2 ON ps2.product_id = layouts2.product_id
+          WHERE layouts2.id = ${params.layout_id}
+          AND ps2.id != ${params.size_id}
+          AND ps2.edge_right < ${sizeTable.edgeRight}
+        )
+      )`
+    );
+  }
+
   // User-specific logbook data selectors
   const getUserLogbookSelects = () => {
     const ascentsTable = getTableName(params.board_name, 'ascents');
@@ -193,7 +237,7 @@ export const createClimbFilters = (
 
   return {
     // Helper function to get all climb filtering conditions
-    getClimbWhereConditions: () => [...baseConditions, ...nameCondition, ...setterNameCondition, ...holdConditions, ...personalProgressConditions],
+    getClimbWhereConditions: () => [...baseConditions, ...nameCondition, ...setterNameCondition, ...holdConditions, ...personalProgressConditions, ...tallClimbsConditions],
 
     // Size-specific conditions
     getSizeConditions: () => sizeConditions,
@@ -227,6 +271,7 @@ export const createClimbFilters = (
     holdConditions,
     sizeConditions,
     personalProgressConditions,
+    tallClimbsConditions,
     anyHolds,
     notHolds,
   };
diff --git a/app/lib/types.ts b/app/lib/types.ts
index e1782fd..2309d86 100644
--- a/app/lib/types.ts
+++ b/app/lib/types.ts
@@ -93,6 +93,7 @@ export type SearchRequest = {
   hideCompleted: boolean;
   showOnlyAttempted: boolean;
   showOnlyCompleted: boolean;
+  tallClimbsOnly: boolean;
   [key: `hold_${number}`]: HoldFilterValue; // Allow dynamic hold keys directly in the search params
 };
 
@@ -214,6 +215,7 @@ export type BoardDetails = {
   set_ids: SetIdList;
   ledPlacements: LedPlacements;
   supportsMirroring?: boolean;
+  isLargestSize?: boolean;
   // Added for slug-based URLs
   layout_name?: string;
   size_name?: string;
diff --git a/app/lib/url-utils.ts b/app/lib/url-utils.ts
index 532790c..fa9ec06 100644
--- a/app/lib/url-utils.ts
+++ b/app/lib/url-utils.ts
@@ -54,6 +54,7 @@ export const searchParamsToUrlParams = ({
   hideCompleted,
   showOnlyAttempted,
   showOnlyCompleted,
+  tallClimbsOnly,
   page,
   pageSize,
 }: SearchRequestPagination): URLSearchParams => {
@@ -111,6 +112,9 @@ export const searchParamsToUrlParams = ({
   if (showOnlyCompleted !== DEFAULT_SEARCH_PARAMS.showOnlyCompleted) {
     params.showOnlyCompleted = showOnlyCompleted.toString();
   }
+  if (tallClimbsOnly !== DEFAULT_SEARCH_PARAMS.tallClimbsOnly) {
+    params.tallClimbsOnly = tallClimbsOnly.toString();
+  }
 
   // Add holds filter entries only if they exist
   if (holdsFilter && Object.keys(holdsFilter).length > 0) {
@@ -138,6 +142,7 @@ export const DEFAULT_SEARCH_PARAMS: SearchRequestPagination = {
   hideCompleted: false,
   showOnlyAttempted: false,
   showOnlyCompleted: false,
+  tallClimbsOnly: false,
   page: 0,
   pageSize: PAGE_LIMIT,
 };
@@ -168,6 +173,7 @@ export const urlParamsToSearchParams = (urlParams: URLSearchParams): SearchReque
     hideCompleted: urlParams.get('hideCompleted') === 'true',
     showOnlyAttempted: urlParams.get('showOnlyAttempted') === 'true',
     showOnlyCompleted: urlParams.get('showOnlyCompleted') === 'true',
+    tallClimbsOnly: urlParams.get('tallClimbsOnly') === 'true',
     page: Number(urlParams.get('page') ?? DEFAULT_SEARCH_PARAMS.page),
     pageSize: Number(urlParams.get('pageSize') ?? DEFAULT_SEARCH_PARAMS.pageSize),
   };

From fb6256c3879485d3318dfdf716ae060512bd46a1 Mon Sep 17 00:00:00 2001
From: Claude <noreply@anthropic.com>
Date: Wed, 5 Nov 2025 10:22:37 +0000
Subject: [PATCH 2/4] Update package-lock.json from npm install

---
 package-lock.json | 168 ++++++++++++++++++++++++++++++++--------------
 1 file changed, 119 insertions(+), 49 deletions(-)

diff --git a/package-lock.json b/package-lock.json
index 87f6d10..b293c1a 100644
--- a/package-lock.json
+++ b/package-lock.json
@@ -82,7 +82,6 @@
     },
     "node_modules/@ampproject/remapping": {
       "version": "2.3.0",
-      "dev": true,
       "license": "Apache-2.0",
       "dependencies": {
         "@jridgewell/gen-mapping": "^0.3.5",
@@ -459,6 +458,71 @@
       }
     },
     "node_modules/@auth/core": {
+      "version": "0.34.2",
+      "resolved": "https://registry.npmjs.org/@auth/core/-/core-0.34.2.tgz",
+      "integrity": "sha512-KywHKRgLiF3l7PLyL73fjLSIBe1YNcA6sMeew4yMP6cfCWGXZrkkXd32AjRi1hlJ9nvovUBGZHvbn+LijO6ZeQ==",
+      "license": "ISC",
+      "optional": true,
+      "peer": true,
+      "dependencies": {
+        "@panva/hkdf": "^1.1.1",
+        "@types/cookie": "0.6.0",
+        "cookie": "0.6.0",
+        "jose": "^5.1.3",
+        "oauth4webapi": "^2.10.4",
+        "preact": "10.11.3",
+        "preact-render-to-string": "5.2.3"
+      },
+      "peerDependencies": {
+        "@simplewebauthn/browser": "^9.0.1",
+        "@simplewebauthn/server": "^9.0.2",
+        "nodemailer": "^6.8.0"
+      },
+      "peerDependenciesMeta": {
+        "@simplewebauthn/browser": {
+          "optional": true
+        },
+        "@simplewebauthn/server": {
+          "optional": true
+        },
+        "nodemailer": {
+          "optional": true
+        }
+      }
+    },
+    "node_modules/@auth/core/node_modules/cookie": {
+      "version": "0.6.0",
+      "resolved": "https://registry.npmjs.org/cookie/-/cookie-0.6.0.tgz",
+      "integrity": "sha512-U71cyTamuh1CRNCfpGY6to28lxvNwPG4Guz/EVjgf3Jmzv0vlDp1atT9eS5dDjMYHucpHbWns6Lwf3BKz6svdw==",
+      "license": "MIT",
+      "optional": true,
+      "peer": true,
+      "engines": {
+        "node": ">= 0.6"
+      }
+    },
+    "node_modules/@auth/core/node_modules/preact": {
+      "version": "10.11.3",
+      "resolved": "https://registry.npmjs.org/preact/-/preact-10.11.3.tgz",
+      "integrity": "sha512-eY93IVpod/zG3uMF22Unl8h9KkrcKIRs2EGar8hwLZZDU1lkjph303V9HZBwufh2s736U6VXuhD109LYqPoffg==",
+      "license": "MIT",
+      "optional": true,
+      "peer": true,
+      "funding": {
+        "type": "opencollective",
+        "url": "https://opencollective.com/preact"
+      }
+    },
+    "node_modules/@auth/drizzle-adapter": {
+      "version": "1.10.0",
+      "resolved": "https://registry.npmjs.org/@auth/drizzle-adapter/-/drizzle-adapter-1.10.0.tgz",
+      "integrity": "sha512-3MKsdAINTfvV4QKev8PFMNG93HJEUHh9sggDXnmUmriFogRf8qLvgqnPsTlfUyWcLwTzzrrYjeu8CGM+4IxHwQ==",
+      "license": "ISC",
+      "dependencies": {
+        "@auth/core": "0.40.0"
+      }
+    },
+    "node_modules/@auth/drizzle-adapter/node_modules/@auth/core": {
       "version": "0.40.0",
       "resolved": "https://registry.npmjs.org/@auth/core/-/core-0.40.0.tgz",
       "integrity": "sha512-n53uJE0RH5SqZ7N1xZoMKekbHfQgjd0sAEyUbE+IYJnmuQkbvuZnXItCU7d+i7Fj8VGOgqvNO7Mw4YfBTlZeQw==",
@@ -487,13 +551,31 @@
         }
       }
     },
-    "node_modules/@auth/drizzle-adapter": {
-      "version": "1.10.0",
-      "resolved": "https://registry.npmjs.org/@auth/drizzle-adapter/-/drizzle-adapter-1.10.0.tgz",
-      "integrity": "sha512-3MKsdAINTfvV4QKev8PFMNG93HJEUHh9sggDXnmUmriFogRf8qLvgqnPsTlfUyWcLwTzzrrYjeu8CGM+4IxHwQ==",
-      "license": "ISC",
-      "dependencies": {
-        "@auth/core": "0.40.0"
+    "node_modules/@auth/drizzle-adapter/node_modules/jose": {
+      "version": "6.1.0",
+      "resolved": "https://registry.npmjs.org/jose/-/jose-6.1.0.tgz",
+      "integrity": "sha512-TTQJyoEoKcC1lscpVDCSsVgYzUDg/0Bt3WE//WiTPK6uOCQC2KZS4MpugbMWt/zyjkopgZoXhZuCi00gLudfUA==",
+      "license": "MIT",
+      "funding": {
+        "url": "https://github.com/sponsors/panva"
+      }
+    },
+    "node_modules/@auth/drizzle-adapter/node_modules/oauth4webapi": {
+      "version": "3.8.2",
+      "resolved": "https://registry.npmjs.org/oauth4webapi/-/oauth4webapi-3.8.2.tgz",
+      "integrity": "sha512-FzZZ+bht5X0FKe7Mwz3DAVAmlH1BV5blSak/lHMBKz0/EBMhX6B10GlQYI51+oRp8ObJaX0g6pXrAxZh5s8rjw==",
+      "license": "MIT",
+      "funding": {
+        "url": "https://github.com/sponsors/panva"
+      }
+    },
+    "node_modules/@auth/drizzle-adapter/node_modules/preact-render-to-string": {
+      "version": "6.5.11",
+      "resolved": "https://registry.npmjs.org/preact-render-to-string/-/preact-render-to-string-6.5.11.tgz",
+      "integrity": "sha512-ubnauqoGczeGISiOh6RjX0/cdaF8v/oDXIjO85XALCQjwQP+SB4RDXXtvZ6yTYSjG+PC1QRP2AhPgCEsM2EvUw==",
+      "license": "MIT",
+      "peerDependencies": {
+        "preact": ">=10"
       }
     },
     "node_modules/@babel/code-frame": {
@@ -510,7 +592,6 @@
     },
     "node_modules/@babel/compat-data": {
       "version": "7.28.0",
-      "dev": true,
       "license": "MIT",
       "engines": {
         "node": ">=6.9.0"
@@ -518,7 +599,6 @@
     },
     "node_modules/@babel/core": {
       "version": "7.28.0",
-      "dev": true,
       "license": "MIT",
       "dependencies": {
         "@ampproject/remapping": "^2.2.0",
@@ -547,12 +627,10 @@
     },
     "node_modules/@babel/core/node_modules/convert-source-map": {
       "version": "2.0.0",
-      "dev": true,
       "license": "MIT"
     },
     "node_modules/@babel/core/node_modules/json5": {
       "version": "2.2.3",
-      "dev": true,
       "license": "MIT",
       "bin": {
         "json5": "lib/cli.js"
@@ -563,7 +641,6 @@
     },
     "node_modules/@babel/core/node_modules/semver": {
       "version": "6.3.1",
-      "dev": true,
       "license": "ISC",
       "bin": {
         "semver": "bin/semver.js"
@@ -585,7 +662,6 @@
     },
     "node_modules/@babel/helper-compilation-targets": {
       "version": "7.27.2",
-      "dev": true,
       "license": "MIT",
       "dependencies": {
         "@babel/compat-data": "^7.27.2",
@@ -600,7 +676,6 @@
     },
     "node_modules/@babel/helper-compilation-targets/node_modules/semver": {
       "version": "6.3.1",
-      "dev": true,
       "license": "ISC",
       "bin": {
         "semver": "bin/semver.js"
@@ -626,7 +701,6 @@
     },
     "node_modules/@babel/helper-module-transforms": {
       "version": "7.27.3",
-      "dev": true,
       "license": "MIT",
       "dependencies": {
         "@babel/helper-module-imports": "^7.27.1",
@@ -664,7 +738,6 @@
     },
     "node_modules/@babel/helper-validator-option": {
       "version": "7.27.1",
-      "dev": true,
       "license": "MIT",
       "engines": {
         "node": ">=6.9.0"
@@ -672,7 +745,6 @@
     },
     "node_modules/@babel/helpers": {
       "version": "7.27.6",
-      "dev": true,
       "license": "MIT",
       "dependencies": {
         "@babel/template": "^7.27.2",
@@ -2190,7 +2262,7 @@
       "version": "0.6.0",
       "resolved": "https://registry.npmjs.org/@types/cookie/-/cookie-0.6.0.tgz",
       "integrity": "sha512-4Kh9a6B2bQciAhf7FSuMRRkUWecJgJu9nPnx3yzpsfXX/c50REIqpHY4C82bXP90qrLtXtkDxTZosYO3UpOwlA==",
-      "dev": true,
+      "devOptional": true,
       "license": "MIT"
     },
     "node_modules/@types/d3-scale": {
@@ -2247,12 +2319,12 @@
     },
     "node_modules/@types/prop-types": {
       "version": "15.7.15",
-      "dev": true,
+      "devOptional": true,
       "license": "MIT"
     },
     "node_modules/@types/react": {
       "version": "18.3.23",
-      "dev": true,
+      "devOptional": true,
       "license": "MIT",
       "dependencies": {
         "@types/prop-types": "*",
@@ -3201,7 +3273,6 @@
     },
     "node_modules/browserslist": {
       "version": "4.25.1",
-      "dev": true,
       "funding": [
         {
           "type": "opencollective",
@@ -3236,7 +3307,7 @@
     },
     "node_modules/bufferutil": {
       "version": "4.0.9",
-      "dev": true,
+      "devOptional": true,
       "hasInstallScript": true,
       "license": "MIT",
       "dependencies": {
@@ -3941,7 +4012,6 @@
     },
     "node_modules/electron-to-chromium": {
       "version": "1.5.183",
-      "dev": true,
       "license": "ISC"
     },
     "node_modules/emoji-regex": {
@@ -4191,7 +4261,6 @@
     },
     "node_modules/escalade": {
       "version": "3.2.0",
-      "dev": true,
       "license": "MIT",
       "engines": {
         "node": ">=6"
@@ -4913,7 +4982,6 @@
     },
     "node_modules/gensync": {
       "version": "1.0.0-beta.2",
-      "dev": true,
       "license": "MIT",
       "engines": {
         "node": ">=6.9.0"
@@ -5786,10 +5854,12 @@
       }
     },
     "node_modules/jose": {
-      "version": "6.0.12",
-      "resolved": "https://registry.npmjs.org/jose/-/jose-6.0.12.tgz",
-      "integrity": "sha512-T8xypXs8CpmiIi78k0E+Lk7T2zlK4zDyg+o1CZ4AkOHgDg98ogdP2BeZ61lTFKFyoEwJ9RgAgN+SdM3iPgNonQ==",
+      "version": "5.10.0",
+      "resolved": "https://registry.npmjs.org/jose/-/jose-5.10.0.tgz",
+      "integrity": "sha512-s+3Al/p9g32Iq+oqXxkW//7jk2Vig6FF1CFqzVXoTUXt2qz89YWbL+OwS17NFYEvxC35n0FKeGO2LGYSxeM2Gg==",
       "license": "MIT",
+      "optional": true,
+      "peer": true,
       "funding": {
         "url": "https://github.com/sponsors/panva"
       }
@@ -6004,7 +6074,6 @@
     },
     "node_modules/lru-cache": {
       "version": "5.1.1",
-      "dev": true,
       "license": "ISC",
       "dependencies": {
         "yallist": "^3.0.2"
@@ -6272,12 +6341,6 @@
         "preact": ">=10"
       }
     },
-    "node_modules/next-auth/node_modules/pretty-format": {
-      "version": "3.8.0",
-      "resolved": "https://registry.npmjs.org/pretty-format/-/pretty-format-3.8.0.tgz",
-      "integrity": "sha512-WuxUnVtlWL1OfZFQFuqvnvs6MiAGk9UNsBostyBOB0Is9wb5uRESevA6rnl/rkksXaGX3GzZhPup5d6Vp1nFew==",
-      "license": "MIT"
-    },
     "node_modules/next-auth/node_modules/uuid": {
       "version": "8.3.2",
       "resolved": "https://registry.npmjs.org/uuid/-/uuid-8.3.2.tgz",
@@ -6289,7 +6352,7 @@
     },
     "node_modules/node-gyp-build": {
       "version": "4.8.4",
-      "dev": true,
+      "devOptional": true,
       "license": "MIT",
       "bin": {
         "node-gyp-build": "bin.js",
@@ -6299,7 +6362,6 @@
     },
     "node_modules/node-releases": {
       "version": "2.0.19",
-      "dev": true,
       "license": "MIT"
     },
     "node_modules/nwsapi": {
@@ -6314,10 +6376,12 @@
       "license": "MIT"
     },
     "node_modules/oauth4webapi": {
-      "version": "3.6.2",
-      "resolved": "https://registry.npmjs.org/oauth4webapi/-/oauth4webapi-3.6.2.tgz",
-      "integrity": "sha512-hwWLiyBYuqhVdcIUJMJVKdEvz+DCweOcbSfqDyIv9PuUwrNfqrzfHP2bypZgZdbYOS67QYqnAnvZa2BJwBBrHw==",
+      "version": "2.17.0",
+      "resolved": "https://registry.npmjs.org/oauth4webapi/-/oauth4webapi-2.17.0.tgz",
+      "integrity": "sha512-lbC0Z7uzAFNFyzEYRIC+pkSVvDHJTbEW+dYlSBAlCYDe6RxUkJ26bClhk8ocBZip1wfI9uKTe0fm4Ib4RHn6uQ==",
       "license": "MIT",
+      "optional": true,
+      "peer": true,
       "funding": {
         "url": "https://github.com/sponsors/panva"
       }
@@ -6939,10 +7003,15 @@
       }
     },
     "node_modules/preact-render-to-string": {
-      "version": "6.5.11",
-      "resolved": "https://registry.npmjs.org/preact-render-to-string/-/preact-render-to-string-6.5.11.tgz",
-      "integrity": "sha512-ubnauqoGczeGISiOh6RjX0/cdaF8v/oDXIjO85XALCQjwQP+SB4RDXXtvZ6yTYSjG+PC1QRP2AhPgCEsM2EvUw==",
+      "version": "5.2.3",
+      "resolved": "https://registry.npmjs.org/preact-render-to-string/-/preact-render-to-string-5.2.3.tgz",
+      "integrity": "sha512-aPDxUn5o3GhWdtJtW0svRC2SS/l8D9MAgo2+AWml+BhDImb27ALf04Q2d+AHqUUOc6RdSXFIBVa2gxzgMKgtZA==",
       "license": "MIT",
+      "optional": true,
+      "peer": true,
+      "dependencies": {
+        "pretty-format": "^3.8.0"
+      },
       "peerDependencies": {
         "preact": ">=10"
       }
@@ -6980,6 +7049,12 @@
         "node": ">=6.0.0"
       }
     },
+    "node_modules/pretty-format": {
+      "version": "3.8.0",
+      "resolved": "https://registry.npmjs.org/pretty-format/-/pretty-format-3.8.0.tgz",
+      "integrity": "sha512-WuxUnVtlWL1OfZFQFuqvnvs6MiAGk9UNsBostyBOB0Is9wb5uRESevA6rnl/rkksXaGX3GzZhPup5d6Vp1nFew==",
+      "license": "MIT"
+    },
     "node_modules/prop-types": {
       "version": "15.8.1",
       "license": "MIT",
@@ -7564,7 +7639,6 @@
     },
     "node_modules/react": {
       "version": "18.3.1",
-      "dev": true,
       "license": "MIT",
       "dependencies": {
         "loose-envify": "^1.1.0"
@@ -7583,7 +7657,6 @@
     },
     "node_modules/react-dom": {
       "version": "18.3.1",
-      "dev": true,
       "license": "MIT",
       "dependencies": {
         "loose-envify": "^1.1.0",
@@ -7902,7 +7975,6 @@
     },
     "node_modules/scheduler": {
       "version": "0.23.2",
-      "dev": true,
       "license": "MIT",
       "dependencies": {
         "loose-envify": "^1.1.0"
@@ -8864,7 +8936,6 @@
     },
     "node_modules/update-browserslist-db": {
       "version": "1.1.3",
-      "dev": true,
       "funding": [
         {
           "type": "opencollective",
@@ -9425,7 +9496,6 @@
     },
     "node_modules/yallist": {
       "version": "3.1.1",
-      "dev": true,
       "license": "ISC"
     },
     "node_modules/yocto-queue": {

From bc8f7729879a87a54f9420febbb7e82c04accbd6 Mon Sep 17 00:00:00 2001
From: Claude <noreply@anthropic.com>
Date: Sun, 9 Nov 2025 08:12:52 +0000
Subject: [PATCH 3/4] Fix tall climbs filter logic

Fixed a critical bug in the tall climbs filter SQL query. The previous
implementation incorrectly filtered size comparisons by dimension, causing
it to miss climbs or incorrectly include them.

The Bug:
- Each subquery filtered sizes by the specific dimension being compared
  (e.g., only comparing against sizes with smaller edgeTop)
- This could return no sizes in some dimensions, causing COALESCE fallbacks
  (0 or 999) to match almost all or no climbs incorrectly
- On boards like Kilter home wall where the larger board extends downward,
  this failed to identify climbs using the extended bottom area

The Fix:
- Now queries ALL other sizes for the layout without dimension filtering
- Compares climb edges against MAX/MIN of all other sizes' edges
- Better COALESCE fallbacks prevent false positives when only one size exists
- Correctly identifies climbs extending beyond ALL other sizes in any direction

This ensures "tall climbs only" shows climbs that use holds available
exclusively on the largest board size.
---
 .../db/queries/climbs/create-climb-filters.ts  | 18 +++++++-----------
 1 file changed, 7 insertions(+), 11 deletions(-)

diff --git a/app/lib/db/queries/climbs/create-climb-filters.ts b/app/lib/db/queries/climbs/create-climb-filters.ts
index fb577aa..92a2181 100644
--- a/app/lib/db/queries/climbs/create-climb-filters.ts
+++ b/app/lib/db/queries/climbs/create-climb-filters.ts
@@ -146,44 +146,40 @@ export const createClimbFilters = (
   }
 
   // Tall climbs filter condition
-  // A climb is "tall" if it extends beyond the boundaries of smaller sizes
+  // A climb is "tall" if it uses holds beyond the boundaries of ALL other sizes
   const tallClimbsConditions: SQL[] = [];
   if (searchParams.tallClimbsOnly) {
-    // Filter climbs that extend beyond the maximum edgeTop of smaller sizes
-    // This subquery finds the max edgeTop of all sizes smaller than the current size
+    // Find climbs that extend beyond the boundaries of ALL other sizes for this layout
+    // This means the climb uses at least 1 hold only available on the current (largest) size
     tallClimbsConditions.push(
       sql`(
         ${tables.climbs.edgeTop} > (
-          SELECT COALESCE(MAX(ps2.edge_top), 0)
+          SELECT COALESCE(MAX(ps2.edge_top), ${sizeTable.edgeTop} - 1)
           FROM ${tables.productSizes} ps2
           INNER JOIN ${tables.layouts} layouts2 ON ps2.product_id = layouts2.product_id
           WHERE layouts2.id = ${params.layout_id}
           AND ps2.id != ${params.size_id}
-          AND ps2.edge_top < ${sizeTable.edgeTop}
         )
         OR ${tables.climbs.edgeBottom} < (
-          SELECT COALESCE(MIN(ps2.edge_bottom), 999)
+          SELECT COALESCE(MIN(ps2.edge_bottom), ${sizeTable.edgeBottom} + 1)
           FROM ${tables.productSizes} ps2
           INNER JOIN ${tables.layouts} layouts2 ON ps2.product_id = layouts2.product_id
           WHERE layouts2.id = ${params.layout_id}
           AND ps2.id != ${params.size_id}
-          AND ps2.edge_bottom > ${sizeTable.edgeBottom}
         )
         OR ${tables.climbs.edgeLeft} < (
-          SELECT COALESCE(MIN(ps2.edge_left), 999)
+          SELECT COALESCE(MIN(ps2.edge_left), ${sizeTable.edgeLeft} + 1)
           FROM ${tables.productSizes} ps2
           INNER JOIN ${tables.layouts} layouts2 ON ps2.product_id = layouts2.product_id
           WHERE layouts2.id = ${params.layout_id}
           AND ps2.id != ${params.size_id}
-          AND ps2.edge_left > ${sizeTable.edgeLeft}
         )
         OR ${tables.climbs.edgeRight} > (
-          SELECT COALESCE(MAX(ps2.edge_right), 0)
+          SELECT COALESCE(MAX(ps2.edge_right), ${sizeTable.edgeRight} - 1)
           FROM ${tables.productSizes} ps2
           INNER JOIN ${tables.layouts} layouts2 ON ps2.product_id = layouts2.product_id
           WHERE layouts2.id = ${params.layout_id}
           AND ps2.id != ${params.size_id}
-          AND ps2.edge_right < ${sizeTable.edgeRight}
         )
       )`
     );

From 052f0c2289fe5fcff92b1e9225e956da386eaa55 Mon Sep 17 00:00:00 2001
From: Claude <noreply@anthropic.com>
Date: Sun, 9 Nov 2025 08:32:21 +0000
Subject: [PATCH 4/4] Fix tall climbs filter SQL syntax and 404 error on empty
 results

Fixed two issues with the tall climbs filter:

1. SQL Syntax Error:
   - Removed invalid arithmetic operations in SQL template literals
   - Changed COALESCE fallbacks from column arithmetic (edgeTop - 1) to
     static values (999/-999)
   - Fallbacks now correctly ensure no matches when only one size exists
   - MAX queries use 999 so > comparisons always fail
   - MIN queries use -999 so < comparisons always fail

2. Empty Results 404 Error:
   - Modified page.tsx to distinguish between invalid routes and empty
     filter results
   - Now only returns 404 when no climbs exist AND no filters are active
   - Empty filtered results display correctly instead of showing 404
   - Improves UX when filters legitimately return no matches

The filter now works correctly to show climbs using holds available
exclusively on the largest board size.
---
 .../[size_id]/[set_ids]/[angle]/list/page.tsx | 20 ++++++++++++++++++-
 .../db/queries/climbs/create-climb-filters.ts |  9 +++++----
 2 files changed, 24 insertions(+), 5 deletions(-)

diff --git a/app/[board_name]/[layout_id]/[size_id]/[set_ids]/[angle]/list/page.tsx b/app/[board_name]/[layout_id]/[size_id]/[set_ids]/[angle]/list/page.tsx
index 4760f33..6ebbe8f 100644
--- a/app/[board_name]/[layout_id]/[size_id]/[set_ids]/[angle]/list/page.tsx
+++ b/app/[board_name]/[layout_id]/[size_id]/[set_ids]/[angle]/list/page.tsx
@@ -76,7 +76,25 @@ export default async function DynamicResultsPage(props: {
       getBoardDetails(parsedParams),
     ]);
 
-    if (!fetchedResults || fetchedResults.climbs.length === 0) {
+    // Only show 404 if there are no climbs at all AND no filters are applied
+    // If filters are active and return 0 results, that's a valid empty state
+    const hasActiveFilters =
+      searchParamsObject.name ||
+      searchParamsObject.minGrade ||
+      searchParamsObject.maxGrade ||
+      searchParamsObject.minAscents ||
+      searchParamsObject.minRating ||
+      searchParamsObject.onlyClassics ||
+      searchParamsObject.tallClimbsOnly ||
+      searchParamsObject.gradeAccuracy ||
+      searchParamsObject.settername.length > 0 ||
+      Object.keys(searchParamsObject.holdsFilter).length > 0 ||
+      searchParamsObject.hideAttempted ||
+      searchParamsObject.hideCompleted ||
+      searchParamsObject.showOnlyAttempted ||
+      searchParamsObject.showOnlyCompleted;
+
+    if (!fetchedResults || (fetchedResults.climbs.length === 0 && !hasActiveFilters)) {
       notFound();
     }
 
diff --git a/app/lib/db/queries/climbs/create-climb-filters.ts b/app/lib/db/queries/climbs/create-climb-filters.ts
index 92a2181..fe3426b 100644
--- a/app/lib/db/queries/climbs/create-climb-filters.ts
+++ b/app/lib/db/queries/climbs/create-climb-filters.ts
@@ -151,31 +151,32 @@ export const createClimbFilters = (
   if (searchParams.tallClimbsOnly) {
     // Find climbs that extend beyond the boundaries of ALL other sizes for this layout
     // This means the climb uses at least 1 hold only available on the current (largest) size
+    // COALESCE fallbacks ensure no matches if there's only one size (which is correct - no "tall-only" climbs exist)
     tallClimbsConditions.push(
       sql`(
         ${tables.climbs.edgeTop} > (
-          SELECT COALESCE(MAX(ps2.edge_top), ${sizeTable.edgeTop} - 1)
+          SELECT COALESCE(MAX(ps2.edge_top), 999)
           FROM ${tables.productSizes} ps2
           INNER JOIN ${tables.layouts} layouts2 ON ps2.product_id = layouts2.product_id
           WHERE layouts2.id = ${params.layout_id}
           AND ps2.id != ${params.size_id}
         )
         OR ${tables.climbs.edgeBottom} < (
-          SELECT COALESCE(MIN(ps2.edge_bottom), ${sizeTable.edgeBottom} + 1)
+          SELECT COALESCE(MIN(ps2.edge_bottom), -999)
           FROM ${tables.productSizes} ps2
           INNER JOIN ${tables.layouts} layouts2 ON ps2.product_id = layouts2.product_id
           WHERE layouts2.id = ${params.layout_id}
           AND ps2.id != ${params.size_id}
         )
         OR ${tables.climbs.edgeLeft} < (
-          SELECT COALESCE(MIN(ps2.edge_left), ${sizeTable.edgeLeft} + 1)
+          SELECT COALESCE(MIN(ps2.edge_left), -999)
           FROM ${tables.productSizes} ps2
           INNER JOIN ${tables.layouts} layouts2 ON ps2.product_id = layouts2.product_id
           WHERE layouts2.id = ${params.layout_id}
           AND ps2.id != ${params.size_id}
         )
         OR ${tables.climbs.edgeRight} > (
-          SELECT COALESCE(MAX(ps2.edge_right), ${sizeTable.edgeRight} - 1)
+          SELECT COALESCE(MAX(ps2.edge_right), 999)
           FROM ${tables.productSizes} ps2
           INNER JOIN ${tables.layouts} layouts2 ON ps2.product_id = layouts2.product_id
           WHERE layouts2.id = ${params.layout_id}