@poodle64/librarian 2026.9.7 → 2026.9.8

package/dist/citations.d.ts

@@ -21,7 +21,13 @@ export interface Citation {
 /** Inline `[n]` markers, in order of first appearance.
  *
  * Deliberately not a global "[digits]" match: `[1](…)` is a markdown link and
- * `[1]: …` a link definition, and both appear in real answers. */
+ * `[1]: …` a link definition, and both appear in real answers.
+ *
+ * A following `[` is NOT excluded, and that is the point: `[1][2]` is how an
+ * answer cites two sources for one claim, and a lookahead that rejected it
+ * dropped the FIRST of the pair — `matchAll` resumes after the failed attempt
+ * rather than backtracking, so `[1]` was never seen at all and rendered as
+ * dead text beside a live `[2]`. */
 export declare function citationMarkers(markdown: string): number[];
 export interface SplitAnswer {
     /** The prose, with any trailing Sources block removed. */

package/dist/citations.js

@@ -11,10 +11,16 @@
 /** Inline `[n]` markers, in order of first appearance.
  *
  * Deliberately not a global "[digits]" match: `[1](…)` is a markdown link and
- * `[1]: …` a link definition, and both appear in real answers. */
+ * `[1]: …` a link definition, and both appear in real answers.
+ *
+ * A following `[` is NOT excluded, and that is the point: `[1][2]` is how an
+ * answer cites two sources for one claim, and a lookahead that rejected it
+ * dropped the FIRST of the pair — `matchAll` resumes after the failed attempt
+ * rather than backtracking, so `[1]` was never seen at all and rendered as
+ * dead text beside a live `[2]`. */
 export function citationMarkers(markdown) {
     const seen = [];
-    for (const match of markdown.matchAll(/\[(\d{1,3})\](?![(:[])/g)) {
+    for (const match of markdown.matchAll(/\[(\d{1,3})\](?![(:])/g)) {
         const n = Number(match[1]);
         if (!seen.includes(n))
             seen.push(n);
@@ -33,11 +39,18 @@ export function splitSources(markdown) {
     const heading = markdown.match(SOURCES_HEADING);
     if (!heading || heading.index === undefined)
         return { body: markdown, citations: [] };
-    const block = markdown.slice(heading.index + heading[0].length);
+    // The block runs to the NEXT heading, not to the end of the answer. Cutting
+    // to the end loses anything Milton wrote after his sources — a closing note,
+    // a caveat — and loses it silently, which is the worst way to lose it.
+    const after = markdown.slice(heading.index + heading[0].length);
+    const next = after.search(/^[ \t]{0,3}#{1,6}[ \t]/m);
+    const block = next === -1 ? after : after.slice(0, next);
+    const tail = next === -1 ? '' : after.slice(next).trim();
     const citations = parseSourceLines(block);
     if (citations.length === 0)
         return { body: markdown, citations: [] };
-    return { body: markdown.slice(0, heading.index).trimEnd(), citations };
+    const before = markdown.slice(0, heading.index).trimEnd();
+    return { body: tail ? `${before}\n\n${tail}` : before, citations };
 }
 /** One list item per source; anything else in the block is ignored. */
 function parseSourceLines(block) {

package/dist/components/agent-transcript/agent-transcript.svelte

@@ -15,7 +15,8 @@
 		readerQuestion,
 		segment,
 		type Block,
-		type Outcome
+		type Outcome,
+		type TextBlock
 	} from '../../transcript.svelte';
 	import {
 		citationMarkers,
@@ -62,31 +63,34 @@
 	const segments = $derived(segment(blocks));
 	const lastIndex = $derived(segments.at(-1)?.index ?? -1);
 
-	// The prose block a Sources list may be hiding at the end of is the last
-	// one; splitting every text segment would eat a "## Sources" heading a
-	// reader legitimately asked about mid-answer.
+	// An answer can arrive as SEVERAL text blocks with tool calls between them,
+	// so the Sources block is at the end of the LAST one. It is split off that
+	// block alone: splitting the joined answer and substituting the result back
+	// into one segment would print every earlier block twice.
+	//
+	// And only once the answer has settled — mid-stream the "## Sources" heading
+	// may be all that has arrived, with the list still to come.
+	const texts = $derived(segments.filter((s): s is TextBlock => s.kind === 'text'));
+	const lastTextIndex = $derived(texts.at(-1)?.index ?? -1);
+	const split = $derived(
+		running ? { body: '', citations: [] } : splitSources(texts.at(-1)?.text ?? '')
+	);
+	const sources = $derived(resolveCitations(citations, split.citations));
+
+	/** The answer as a reader would paste it: every block, Sources stripped. */
 	const answer = $derived(
-		segments
-			.filter((s) => s.kind === 'text')
-			.map((s) => s.text)
+		texts
+			.map((s) => (s.index === lastTextIndex && split.citations.length > 0 ? split.body : s.text))
 			.join('\n\n')
 	);
-	const split = $derived(running ? { body: '', citations: [] } : splitSources(answer));
-	const sources = $derived(resolveCitations(citations, split.citations));
+
 	// Only a number the prose actually marks becomes a chip; a source list may
 	// legitimately carry an entry the answer never points at inline.
 	const numbers = $derived(
 		new Set(citationMarkers(answer).filter((n) => sources.some((c) => c.n === n)))
 	);
 
-	function prose(text: string, isLast: boolean): string {
-		// Only the last text segment can carry the trailing Sources block, and
-		// only once the answer has settled — mid-stream the heading may be all
-		// that has arrived.
-		return isLast && split.citations.length > 0 ? split.body : text;
-	}
-
-	const hasAnswer = $derived(segments.some((s) => s.kind === 'text'));
+	const hasAnswer = $derived(texts.length > 0);
 	// A failed turn settles too, and "Ask again" is the one thing a reader wants
 	// from it — there is just nothing to copy.
 	const settled = $derived(!running && (hasAnswer || Boolean(outcome?.error)));
@@ -124,7 +128,9 @@
 		{:else}
 			<div class="max-w-[72ch] min-w-0">
 				<Markdown
-					content={prose(seg.text, seg.index === lastIndex)}
+					content={seg.index === lastTextIndex && split.citations.length > 0
+						? split.body
+						: seg.text}
 					streaming={running && seg.index === lastIndex}
 					{collectionNames}
 					citationNumbers={numbers}

package/dist/components/composer/composer.svelte

@@ -101,8 +101,11 @@
 	});
 
 	// Focus comes back once Milton is done, so a reader can keep going without
-	// reaching for the mouse. Focusing at send is pointless — the box is
-	// disabled a frame later — so the transition out of `running` is the hook.
+	// reaching for the mouse. It has to be the transition OUT of `running`: a
+	// disabled element cannot hold focus, so the focus call in `submit()` below
+	// survives only for a host that never sets `running` — measured, focus lands
+	// on `body` the frame the box disables and comes back here.
+	//
 	// Plain, not `$state`: read and written in the same effect, which would
 	// otherwise re-trigger itself forever.
 	let wasRunning = false;

package/dist/components/conversation/conversation.svelte

@@ -113,6 +113,7 @@
 			onscroll={scrolled}
 			role="log"
 			aria-live="polite"
+			aria-busy={running}
 			aria-label="Conversation with Milton"
 			class="min-h-0 flex-1 overflow-y-auto overscroll-contain"
 		>

package/dist/components/document-pane/document-pane.svelte

@@ -112,8 +112,12 @@
 <aside
 	style="--pane-width: {width}px"
 	aria-label="Source document"
-	class="bg-surface-1 border-border fixed inset-x-0 bottom-0 z-40 flex h-[80svh] flex-col rounded-t-2xl border-t shadow-2xl lg:static lg:h-auto lg:w-[var(--pane-width)] lg:shrink-0 lg:rounded-none lg:border-t-0 lg:border-l lg:shadow-none"
+	class="bg-surface-1 border-border fixed inset-x-0 bottom-0 z-40 flex h-[80svh] flex-col rounded-t-2xl border-t shadow-2xl lg:relative lg:h-auto lg:w-[var(--pane-width)] lg:shrink-0 lg:rounded-none lg:border-t-0 lg:border-l lg:shadow-none"
 >
+	<!-- The drag handle. `lg:relative` on the pane above, not `lg:static`, is
+	     what this depends on: an absolutely-positioned child needs a positioned
+	     ancestor, and without one the handle lands somewhere else entirely and
+	     the pane is silently not resizable. Measured — the drag moved nothing. -->
 	<!-- svelte-ignore a11y_no_static_element_interactions -- a pointer-only
 	     affordance for a width that has a keyboard-independent default; the
 	     pane is fully usable without ever touching it. -->

package/package.json

@@ -1,6 +1,6 @@
 {
 	"name": "@poodle64/librarian",
-	"version": "2026.9.7",
+	"version": "2026.9.8",
 	"description": "Milton's conversation surface as a consumable Svelte 5 package: the stream client, transcript state and chat components (transcript, composer, markdown) every household app renders instead of rebuilding.",
 	"type": "module",
 	"license": "MIT",