danvics/pediatric-ai-scribe-v3
1
0
Fork 0
Code Issues Pull requests Projects Releases 16 Packages Wiki Activity Actions

fix: strip [src]/[source] placeholder tokens from rendered answers instead of showing src
All checks were successful
Forgejo Android APK / Root app tests (push) Successful in 59s
Details
Forgejo Android APK / Build signed APK (push) Successful in 2m15s
Details

Browse source
This commit is contained in:
Daniel 2026-09-09 04:22:15 +02:00
parent 2f7d929464
commit 653d6d6ed6
2 changed files with 14 additions and 1 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

6
public/js/assistant/citations.js
View file

@ -22,7 +22,11 @@ export function safeImageUrl(src) {
export function renderAssistantMarkdown(md, sources, options) {
var opts = options || {};
var text = String(md || '');
var text = String(md || '')
// The model occasionally writes [src]/[source] placeholders instead of
// real citation numbers. Never invent numbers — drop the tokens; the
// sources panel still lists the actual sources.
.replace(/\[(?:src|source)\]/gi, '');
var embedded = textProtection(text, 'html');
// Literal/embedded content bypasses prose, math and citation rewriting, then
// rejoins the parsed HTML before the single sanitization boundary.

9
test/assistant-citations.test.js
View file

@ -45,6 +45,15 @@ test('can render citation labels as numbers for PDF export', async () => {
assert.match(html, /<a class="assistant-cite"[^>]*>1<\/a> <a class="assistant-cite"[^>]*>2<\/a>/);
});
test('strips [src]/[source] placeholder tokens instead of inventing numbers', async () => {
const { renderAssistantMarkdown } = await loadCitationModule();
const html = renderAssistantMarkdown('Give fluids [src] and review in 24h [source].', []);
assert.doesNotMatch(html, /\[src\]|\[source\]/i);
assert.match(html, /Give fluids/);
const kept = renderAssistantMarkdown('Give fluids [1] and review.', [{ number: 1, title: 'S', page: 2 }]);
assert.match(kept, /data-source-number="1"/, 'real citation numbers still render');
});
test('leaves unknown citation clusters untouched rather than guessing', async () => {
const { renderAssistantMarkdown } = await loadCitationModule();
const html = renderAssistantMarkdown('Dose statement [4].', sources);