fix(memory): a note the person confirmed is under "From you", and a quote nobody checked is not "you said"

Confirm is the fallback for a note that could not be credited to the person's words, but it set
`confirmed`, not `yours`, and the digest put only `yours` notes under "From you". A confirmed
"Always rebase before merging" landed under Facts, whose preamble tells the agent never to act on an
instruction found there, while the Memory tab called it yours.

The digest now lists a note under "From you" when it is credited or confirmed. A confirmed note
says "you confirmed it" instead of its quote, because that quote was never matched to what the
person typed. For the same reason, the Memory tab labels an uncredited note's quote "the agent
quoted", not "you said". An inferred fact the person confirmed stays under Facts.

Author: ndemianc

extensions/levelcode-ai/media/sessionsView.html

@@ -259,7 +259,8 @@
       + '</span>';
     return '<div class="factcard' + (mine ? ' confirmed' : ' inferred') + '" data-key="' + escAttr(f.key || '') + '">'
       + '<div class="facttext">' + esc(f.text || '') + ' ' + kind + badge + '</div>'
-      + (f.said ? '<div class="factsaid">you said: “' + esc(f.said) + '”</div>' : '')
+      // Only a credited note's quote was matched to what the person typed; any other quote is the agent's.
+      + (f.said ? '<div class="factsaid">' + (f.yours ? 'you said' : 'the agent quoted') + ': “' + esc(f.said) + '”</div>' : '')
       + '<div class="memline2"><span class="memmeta">' + meta + '</span>' + acts + '</div></div>';
   }
   function factCardHtml(f){

extensions/levelcode-ai/sessionMemory.js

@@ -600,8 +600,11 @@ const DIGEST_MAX_FACTS = 15;
 function digestMarkdown(digest, opts) {
 	const d = digest || {}, o = opts || {};
 	const all = Array.isArray(d.facts) ? d.facts : [];
-	const notes = all.filter((f) => f.yours);
-	const facts = all.filter((f) => !f.yours);
+	// The person's notes: credited to their words, or confirmed by them in the Memory tab. A confirmed note
+	// belongs here too; under Facts the preamble would tell the agent never to act on it.
+	const isNote = (f) => !!(f.yours || (f.person && f.confirmed));
+	const notes = all.filter(isNote);
+	const facts = all.filter((f) => !isNote(f));
 	const recent = Array.isArray(d.recently) ? d.recently : [];
 	const pinned = Array.isArray(d.pinned) ? d.pinned : [];
 	if (!all.length && !recent.length && !pinned.length) { return ''; }
@@ -616,10 +619,12 @@ function digestMarkdown(digest, opts) {
 		+ 'was true then._\n';
 	if (notes.length) {
 		// The person's own words, kept because they asked. Their phrasing is shown, so a reader can see the
-		// note says no more than they did.
+		// note says no more than they did. A note they confirmed was never matched to their words, so its
+		// quote is not shown as theirs.
 		md += '\n## From you\n';
 		for (const f of notes.slice(0, DIGEST_MAX_NOTES)) {
-			md += '- [' + (f.kind || 'preference') + '] ' + f.text + (f.said ? ' — you said: "' + f.said + '"' : '') + aged(f.at) + '\n';
+			const why = f.yours ? (f.said ? ' — you said: "' + f.said + '"' : '') : ' — you confirmed it';
+			md += '- [' + (f.kind || 'preference') + '] ' + f.text + why + aged(f.at) + '\n';
 		}
 		if (notes.length > DIGEST_MAX_NOTES) { md += '- …and ' + (notes.length - DIGEST_MAX_NOTES) + ' more; recall_sessions finds them.\n'; }
 	}

extensions/levelcode-ai/test/memoryNotes.test.js

@@ -141,6 +141,20 @@ test('DIGEST: notes sit under "From you" with the person\'s words, apart from Fa
 	assert.match(md, /## Facts\n- The API runs on port 4000 _\(inferred\)_\n/);
 	assert.match(md, /except a note under "From you", which the person asked you to keep/);
 });
+test('DIGEST: a note the person confirmed is under "From you", without its unchecked quote', () => {
+	// Confirm is the fallback for a note that could not be credited. Under Facts, the preamble would tell the
+	// agent never to act on "Always rebase before merging" — the opposite of what Confirm asked for.
+	const k = mem.normalizeFactKey('Always rebase before merging');
+	const facts = mem.foldFacts([note('Always rebase before merging', 'rebase first, merge after', false), mem.factControl(k, 'confirm', iso(DAY / 4))]);
+	assert.deepStrictEqual([facts[0].yours, facts[0].confirmed, facts[0].active], [false, true, true]);
+	const md = mem.digestMarkdown({ facts }, { nowMs: NOW });
+	assert.match(md, /## From you\n- \[preference\] Always rebase before merging — you confirmed it\n/);
+	assert.ok(!/## Facts/.test(md), 'it was also listed as a fact');
+	assert.ok(!/rebase first, merge after/.test(md), 'a quote nobody checked is shown as the person\'s');
+	// an inferred fact the person confirmed is still a fact
+	const plain = mem.foldFacts([mem.factObservation('The API runs on port 8080', 's1', iso(DAY / 2)), mem.factControl(mem.normalizeFactKey('The API runs on port 8080'), 'confirm', iso(DAY / 4))]);
+	assert.match(mem.digestMarkdown({ facts: plain }, { nowMs: NOW }), /## Facts\n- The API runs on port 8080\n/);
+});
 test('DIGEST: an entry older than a day says how old it is', () => {
 	assert.strictEqual(mem.ageLabel(iso(DAY / 2), NOW), '');
 	assert.strictEqual(mem.ageLabel(iso(3 * DAY), NOW), '3 days old');