fix: separate raised citation markers with a space, not a comma #23

2026-10-07T08:23:59Z

adrianbonpin commented

2026-10-07 08:23:59 +00:00

Follow-up to #22, from a screenshot of the comma version.

The comma sits at the baseline, so between two raised digits it drops away from them and reads as a stray mark on the line. A space separates them without introducing a character that lives at the wrong height.

[1][2] and [1, 2] both still arrive as two raised digits, still in the order the model wrote them, and a run containing a number we were never shown is still left exactly as written.

Worth noting from the same screenshot: the model writes its citations descending, so [2][1] is what it actually sends for a sentence resting on both. The order is deliberately preserved rather than sorted, so the two sources read in the order the model ranked them. Sorting ascending would be a one-line change if that is preferred instead.

Follow-up to #22, from a screenshot of the comma version. The comma sits at the baseline, so between two raised digits it drops away from them and reads as a stray mark on the line. A space separates them without introducing a character that lives at the wrong height. `[1][2]` and `[1, 2]` both still arrive as two raised digits, still in the order the model wrote them, and a run containing a number we were never shown is still left exactly as written. Worth noting from the same screenshot: the model writes its citations descending, so `[2][1]` is what it actually sends for a sentence resting on both. The order is deliberately preserved rather than sorted, so the two sources read in the order the model ranked them. Sorting ascending would be a one-line change if that is preferred instead.

adrianbonpin added 1 commit 2026-10-07 08:24:02 +00:00

fix: separate raised citation markers with a space, not a comma 7caab85cb5

The comma sits at the baseline, so between two raised digits it drops away from
them and reads as a stray mark on the line. A space separates them without
introducing a character that lives at the wrong height.

[1][2] and [1, 2] both still arrive as two raised digits, still in the order the
model wrote them, and a run containing a number we were never shown is still
left exactly as written.

adrianbonpin merged commit ade6145425 into prod

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

adrianbonpin referenced this pull request

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

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

Sign in to join this conversation.

1 Participants

Notifications

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: adrianbonpin/lam-agent-vicinae#23