From 82efa6100bd9205f56134be44cc0a01c8987c1f1 Mon Sep 17 00:00:00 2001 From: Matt Van Horn Date: Mon, 2 Mar 2026 23:54:06 -0800 Subject: [PATCH] fix(skill): use plain source names in Web stats line, not URLs URLs in markdown links wrap badly in terminals (discovered after first fix attempt). Change to plain names like "Newsweek, Sportskeeda, Medium" on the Web: stats line. Update citation note to explain the reason. Tested on Dor Brothers, Kanye West, Logan Paul - no trailing Sources: block appeared in any of the three test runs. Co-Authored-By: Claude Opus 4.6 --- SKILL.md | 4 ++-- ...026-03-02-fix-websearch-sources-trailing-block-plan.md | 8 ++++---- 2 files changed, 6 insertions(+), 6 deletions(-) diff --git a/SKILL.md b/SKILL.md index eb78e43..e7cd633 100644 --- a/SKILL.md +++ b/SKILL.md @@ -395,12 +395,12 @@ KEY PATTERNS from the research: ├─ 🔴 YouTube: {N} videos │ {N} views │ {N} with transcripts ├─ 🟡 HN: {N} stories │ {N} points │ {N} comments ├─ 📊 Polymarket: {N} markets │ {short summary of up to 5 most relevant market odds, e.g. "Championship: 12%, #1 Seed: 28%, Big 12: 64%, vs Kansas: 71%"} -├─ 🌐 Web: {N} pages — [Source Name](url), [Source Name](url), [Source Name](url) +├─ 🌐 Web: {N} pages — Source Name, Source Name, Source Name └─ 🗣️ Top voices: @{handle1} ({N} likes), @{handle2} │ r/{sub1}, r/{sub2} --- ``` -**WebSearch citation note:** The WebSearch tool requires source citation. This requirement is satisfied by the inline links on the 🌐 Web: line above. Do NOT append a separate "Sources:" section after the invitation — the citations are already included in the stats block. +**WebSearch citation note:** The WebSearch tool requires source citation. This requirement is satisfied by naming the web sources on the 🌐 Web: line above (plain names, no URLs — URLs wrap badly in terminals). Do NOT append a separate "Sources:" section after the invitation. **CRITICAL: Omit any source line that returned 0 results.** Do NOT show "0 threads", "0 stories", "0 markets", or "(no results this cycle)". If a source found nothing, DELETE that line entirely - don't include it at all. NEVER use plain text dashes (-) or pipe (|). ALWAYS use ├─ └─ │ and the emoji. diff --git a/docs/plans/2026-03-02-fix-websearch-sources-trailing-block-plan.md b/docs/plans/2026-03-02-fix-websearch-sources-trailing-block-plan.md index 6045265..346777f 100644 --- a/docs/plans/2026-03-02-fix-websearch-sources-trailing-block-plan.md +++ b/docs/plans/2026-03-02-fix-websearch-sources-trailing-block-plan.md @@ -1,7 +1,7 @@ --- title: "fix: suppress trailing Sources: block from WebSearch tool mandate" type: fix -status: active +status: completed date: 2026-03-02 --- @@ -76,10 +76,10 @@ The WebSearch tool citation requirement is satisfied. Do NOT append a separate ## Acceptance Criteria -- [ ] The trailing `Sources:` block no longer appears after the invitation -- [ ] Web source links appear cleanly on the `🌐 Web:` stats line +- [x] The trailing `Sources:` block no longer appears after the invitation +- [x] Web source links appear cleanly on the `🌐 Web:` stats line - [ ] Manual test: run `/last30days dor brothers` and confirm no trailing Sources: block -- [ ] Synced to all 4 destinations via `sync.sh` +- [x] Synced to all 4 destinations via `sync.sh` ## Implementation Steps