refactor(metadata): collapse FilterAttr.coerce no-op ternary; fix stale desugar comment

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>

Author: dmealing

server/typescript/packages/metadata/src/loader/validation-passes.ts

@@ -338,7 +338,7 @@ function checkFilterClauses(
       );
       continue;
     }
-    // After parse-time desugaring (normalizeFilterAttr), every non-composition field clause
+    // After parse-time desugaring (FilterAttr.desugar), every non-composition field clause
     // is canonical { op: value } — a bare scalar should not reach here; the object guard is defensive.
     if (typeof clause === "object" && clause !== null && !Array.isArray(clause)) {
       for (const op of Object.keys(clause)) {

server/typescript/packages/metadata/src/meta/meta-attr-filter.ts

@@ -22,11 +22,9 @@ export class FilterAttr extends MetaAttr {
   }
 
   override coerce(raw: unknown): AttrValue {
-    // Object stored verbatim; a non-object (e.g. a legacy JSON string) is
-    // returned as-is so validateValue rejects it.
-    return typeof raw === "object" && raw !== null && !Array.isArray(raw)
-      ? (raw as AttrValue)
-      : (raw as AttrValue);
+    // Stored verbatim: an object is the canonical shape; a non-object (e.g. a
+    // legacy JSON string) is passed through unchanged so validateValue rejects it.
+    return raw as AttrValue;
   }
 
   override desugar(value: AttrValue): AttrValue {