chore: drop the em dashes this release added to prose and test titles

The notification categories' contract description, which lands in the
published API document, and a handful of new code comments and test
titles used em dashes; the house style keeps them out of new prose.

src/components/dashboard/__tests__/medication-intake-quick-add.test.tsx

@@ -408,7 +408,7 @@ describe("submit invalidation — the quick-add is a capture-picker surface, not
   });
 
   it("forces the inactive Today hero + dashboard reads to refetch after submit", async () => {
-    // The helper the submit path calls — asserted directly, same shape as
+    // The helper the submit path calls, asserted directly, same shape as
     // the take-all-due suite: the bundle fans out (N keys), then the two
     // daily reads are invalidated with `refetchType: "inactive"`, which is
     // what actually refetches them while unmounted.

src/components/documents/document-picker-sheet.tsx

@@ -2,9 +2,9 @@
 
 /**
  * The vault document picker sheet: a `ResponsiveSheet` (bottom sheet on
- * phones) with a debounced search over the person's own vault — the same
- * recall path as the vault page, one page (50) of matches with a line that
- * says when more exist, the search input as the way to an older document —
+ * phones) with a debounced search over the person's own vault (the same
+ * recall path as the vault page, one page of 50 matches with a line that
+ * says when more exist, the search input as the way to an older document)
  * and one toggle row per document.
  *
  * Two pickers used to carry this whole body as copies: linking documents to