Remove redundant trim/toLowerCase in scoreQueryMatch

All callers already pass pre-normalized (trimmed and lowercased) inputs,
so the internal .trim().toLowerCase() calls were redundant O(n) string
operations allocating unnecessary copies on every candidate per keystroke.

Replace with a destructured read and a JSDoc contract stating that inputs
must be pre-normalized.

Author: cursoragent

packages/shared/src/searchRanking.ts

@@ -77,6 +77,12 @@ function findBoundaryMatchIndex(
   return bestIndex;
 }
 
+/**
+ * Scores how well `value` matches `query` using tiered match strategies.
+ *
+ * **Expects pre-normalized inputs**: both `value` and `query` must already be
+ * trimmed and lowercased (e.g. via {@link normalizeSearchQuery}).
+ */
 export function scoreQueryMatch(input: {
   value: string;
   query: string;
@@ -87,8 +93,7 @@ export function scoreQueryMatch(input: {
   fuzzyBase?: number;
   boundaryMarkers?: readonly string[];
 }): number | null {
-  const value = input.value.trim().toLowerCase();
-  const query = input.query.trim().toLowerCase();
+  const { value, query } = input;
 
   if (!value || !query) {
     return null;