Highlight CO after “contracting officer” in chapters (#77)

* Link glossary acronyms even after the full phrase

CO was already an alias of Contracting Officer, but once-per-term
deduping skipped the acronym after “contracting officer” linked first.
Allow one phrase link and one acronym link per term so CO highlights
in chapters like early-exchange-lawful-limited.

* Clarify chapter glossary link dedupe comment

Match the phrase-plus-acronym linking behavior in ChapterView.

---------

Co-authored-by: Cursor Agent <cursoragent@cursor.com>

Author: jsolly

src/components/app/ChapterView.svelte

@@ -34,22 +34,22 @@
 		return questionsForChapter(action.chapterId).length > 0;
 	});
 
-	/** One pass per chapter so each glossary term links at most once. */
+	/** One pass per chapter: each term links once as a phrase and once as an acronym. */
 	let linkedCopy = $derived.by(() => {
 		const current = chapter;
 		if (!current) return null;
-		const linkedIds = new Set<string>();
+		const linkedKeys = new Set<string>();
 		return {
-			intro: segmentGlossaryText(current.intro, linkedIds),
+			intro: segmentGlossaryText(current.intro, linkedKeys),
 			pieces: current.pieces.map((piece) => ({
 				id: piece.id,
-				teach: segmentGlossaryText(piece.teach, linkedIds),
+				teach: segmentGlossaryText(piece.teach, linkedKeys),
 				watchFor: piece.watchFor
-					? segmentGlossaryText(piece.watchFor, linkedIds)
+					? segmentGlossaryText(piece.watchFor, linkedKeys)
 					: null,
 			})),
 			closing: current.closing
-				? segmentGlossaryText(current.closing, linkedIds)
+				? segmentGlossaryText(current.closing, linkedKeys)
 				: null,
 		};
 	});

src/lib/far/link-glossary-terms.ts

@@ -111,10 +111,12 @@ function collectHits(
 }
 
 /**
- * Split plain text into text/term segments. Mutates `linkedIds` so each term id
- * is linked at most once across a chapter (pass the same Set to every field).
+ * Split plain text into text/term segments. Mutates `linkedKeys` so each term
+ * is linked at most once per surface class across a chapter (phrase once and
+ * acronym once — e.g. “contracting officer” and later “CO”). Pass the same
+ * Set to every field in a chapter.
  */
-export function segmentGlossaryText(text: string, linkedIds: Set<string>): TextSegment[] {
+export function segmentGlossaryText(text: string, linkedKeys: Set<string>): TextSegment[] {
 	if (!text) return [];
 
 	const { insensitive, sensitive, insensitiveLookup, sensitiveLookup } = matchEngine();
@@ -128,13 +130,14 @@ export function segmentGlossaryText(text: string, linkedIds: Set<string>): TextS
 
 	for (const hit of hits) {
 		if (hit.index < lastIndex) continue;
-		if (linkedIds.has(hit.termId)) continue;
+		const key = `${hit.termId}:${isAcronymLabel(hit.text) ? "acronym" : "phrase"}`;
+		if (linkedKeys.has(key)) continue;
 
 		if (hit.index > lastIndex) {
 			segments.push({ kind: "text", text: text.slice(lastIndex, hit.index) });
 		}
 
-		linkedIds.add(hit.termId);
+		linkedKeys.add(key);
 		segments.push({ kind: "term", text: hit.text, termId: hit.termId });
 		lastIndex = hit.index + hit.text.length;
 	}