fix: order a citation run ascending, and name each page once #24

2026-10-07T08:29:29Z

adrianbonpin commented

2026-10-07 08:29:29 +00:00

Follow-up to #23, closing out the citations work.

A model lists sources by rank rather than by number: a real answer sent [2][1] for a sentence resting on both, and once the digits sit next to each other that reads as "two, one". A run is now a set of sources ordered ascending, so those same two sources draw as the first two the turn found.

Naming each page once falls out of the same step: [3][1][3] draws as ¹ ³, not ¹ ³ ³.

The numbers themselves are still never renumbered. [3] remains the third source the turn found, whether or not the answer cited the two before it, so the marker and the list under the answer still agree.

633 tests, 2 new. The order of a run is now pinned by a test built from the descending order the model actually sends.

Follow-up to #23, closing out the citations work. A model lists sources by rank rather than by number: a real answer sent `[2][1]` for a sentence resting on both, and once the digits sit next to each other that reads as "two, one". A run is now a set of sources ordered ascending, so those same two sources draw as the first two the turn found. Naming each page once falls out of the same step: `[3][1][3]` draws as `¹ ³`, not `¹ ³ ³`. The numbers themselves are still never renumbered. `[3]` remains the third source the turn found, whether or not the answer cited the two before it, so the marker and the list under the answer still agree. 633 tests, 2 new. The order of a run is now pinned by a test built from the descending order the model actually sends.

adrianbonpin added 1 commit 2026-10-07 08:29:30 +00:00

fix: order a citation run ascending, and name each page once 3f86a968ae

A model lists sources by rank rather than by number: a real answer sent [2][1] for
a sentence resting on both, and once the digits sit next to each other that reads
as "two, one". A run is now a set of sources ordered ascending, so the same run
draws as the first two sources rather than as a puzzle.

The numbers themselves are still never renumbered. [3] remains the third source
the turn found, whether or not the answer cited the two before it, so the list and
the marker still agree.

adrianbonpin merged commit 34c4383428 into prod

2026-10-07 08:29:36 +00:00

Sign in to join this conversation.

1 Participants

Notifications

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: adrianbonpin/lam-agent-vicinae#24