chg(properties): simpler query, use properties in frontmatter, invalidatations

[synoet]

• Removes the commplex caching logic we have setup
• Makes frontmatter of blocks use a tanstack query instead of calling service directly
• issues proper invalidations when updating a property from the frontmatter

[github-actions]

Preview: https://synoet-properties-query-invali-bb49ty.preview.macro.com/app (3050798)

[claude]

---

Code review

No issues found. Checked for bugs and CLAUDE.md compliance.

---

[claude]

Code review

I found debug console.log statements that should be removed before merging.

Issue: Debug console.log statements in production code

Two debug console.log statements were left in the new query file:

File: js/app/packages/queries/properties/entity.ts

1. Line 42 - console.log(queryKey); in the bulkIncludesEntityPredicate function
2. Line 59 - console.log('invalidate'); in the invalidatePropertiesForEntity function

Why this is an issue:

According to CLAUDE.md#L20, the guideline mentions using console.trace for debugging, but these console.log statements appear to be leftover debug code that will pollute the browser console in production.

Suggested fix:

Remove both console.log statements:

// Line 42 - Remove this line
function bulkIncludesEntityPredicate(queryKey: QueryKey, entityId: string) {
  console.log(queryKey); // <-- Remove
  return (
    queryKey.includes('properties') &&
    queryKey.includes('bulk') &&
    queryKey.some(
      (subKey) => Array.isArray(subKey) && subKey.includes(entityId)
    )
  );
}

// Line 59 - Remove this line
export function invalidatePropertiesForEntity(
  entityType: EntityType,
  entityId: string
) {
  console.log('invalidate'); // <-- Remove
  queryClient.invalidateQueries({
    queryKey: propertiesKeys.entity({ entityType, entityId }).queryKey,
  });
  // ...
}

---

Otherwise, the PR looks good! The migration to TanStack Query is well-structured and follows the codebase patterns.