fix(sports): nudge concepts (not cite when taught) + surface major off-field news #277

Merged
joshtronic merged 1 commit from fix/sports-digest-tone-and-news into master 2026-06-27 14:28:42 +00:00
Collaborator

Two pieces of reader feedback on the daily digest.

1. It mansplains the curriculum. Actual line from an email: "A shutout (taught June 15 — the pitching staff holds the opponent scoreless)." The taught-dates are an internal recency signal for the model — they must never reach the reader — and a recently-taught concept should just be used, not re-explained. The directive now forbids any "(taught June 15)" / "(introduced last week)" citation ("reads like a professor citing his own syllabus"), says to use recent concepts as if the reader knows them (they do), and to nudge facts in, not announce the curriculum.

2. It skipped major off-field news (two MLB firings the day before). The curation rule was framed around game results; big off-field stories — firings/hirings, blockbuster trades, suspensions, a star injury/retirement, scandals — are now explicitly first-class (the ESPN headlines carry them; one is often THE story of the day; lead with it).

Directive-only change; check-sync green. No reviewer.

Two pieces of reader feedback on the daily digest. **1. It mansplains the curriculum.** Actual line from an email: *"A shutout (taught June 15 — the pitching staff holds the opponent scoreless)."* The taught-dates are an internal recency signal for the model — they must never reach the reader — and a recently-taught concept should just be *used*, not re-explained. The directive now forbids any "(taught June 15)" / "(introduced last week)" citation ("reads like a professor citing his own syllabus"), says to use recent concepts as if the reader knows them (they do), and to **nudge facts in, not announce the curriculum**. **2. It skipped major off-field news** (two MLB firings the day before). The curation rule was framed around game results; big off-field stories — firings/hirings, blockbuster trades, suspensions, a star injury/retirement, scandals — are now explicitly **first-class** (the ESPN headlines carry them; one is often THE story of the day; lead with it). Directive-only change; check-sync green. No reviewer.
fix(sports): nudge concepts (don't cite when taught) + surface major off-field news
All checks were successful
Lint / check-sync (pull_request) Successful in 5s
Lint / check-sync (push) Successful in 4s
076f13ee47
Two pieces of reader feedback on the daily digest:

1. It mansplained the curriculum -- 'A shutout (taught June 15 -- the pitching
   staff holds the opponent scoreless)'. The taught-dates are internal recency
   signal for the model; they must NEVER reach the reader, and a recently-taught
   concept should just be USED naturally, not re-explained. Directive now: never
   write '(taught June 15)'/'(introduced last week)'/any when-you-learned-this
   citation (reads like a professor citing his own syllabus); use recent concepts
   as if the reader knows them; nudge facts in, don't announce the curriculum.

2. It skipped major off-field news (two MLB firings the prior day). The curation
   rule was framed around game results; big off-field stories (firings/hirings,
   blockbuster trades, suspensions, a star's injury/retirement, scandals) are now
   explicitly first-class -- the ESPN headlines carry them, one is often THE story
   of the day, lead with it instead of letting a game-by-game frame bury it.

Directive-only; check-sync green.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_011KXPafSYJPY6XkWHHk1Lfs
Author
Collaborator

🤖 Review — APPROVE (automated)

CI for 076f13ee: success

Directive-only edit to bin/lib/sports-digest-directive.md: (1) elevates off-field news (firings/trades/suspensions/injuries) to first-class in the curation rule, and (2) rewrites the Curriculum section to forbid surfacing taught-dates to the reader and to use recent concepts implicitly. CI is green; verdict APPROVE.

  • Scope: single file, ~27 lines, two tightly related prose changes both about digest content. No code, no unrelated drive-bys. In contract.
  • Honest description: matches the diff exactly — the off-field-news addition and the curriculum rewrite are both present; it is genuinely directive-only. No fabricated checklist items.
  • CI / green branch: success, and the author's "check-sync green" claim is consistent with that. No code paths to unit-test here; a prompt-text change needs none.
  • Security / CI config: none touched; no injection/secret surface. .forgejo/.github untouched.

Non-blocking observations (no action required):

  • The retained example "the Conn Smythe, the NHL's whole-postseason MVP" is an appositive that some would read as a parenthetical, sitting slightly in tension with the new "never a parenthetical lesson" rule. The intent (no heavy date-bearing lessons; light woven refreshers OK) is clear enough that the model should disambiguate, but worth a glance if you want the example to unambiguously model the new rule.
  • The hard-wrapped token "when-you-\n learned-this" will render with a space mid-token if this markdown is ever displayed; as an LLM prompt it's harmless.

Clean, low-risk change as described.


Independent review by the harness on claude-opus-4-8. The human reviewer is requested once Igor has reviewed; a human still merges.

### 🤖 Review — `APPROVE` _(automated)_ CI for `076f13ee`: **success** Directive-only edit to `bin/lib/sports-digest-directive.md`: (1) elevates off-field news (firings/trades/suspensions/injuries) to first-class in the curation rule, and (2) rewrites the Curriculum section to forbid surfacing taught-dates to the reader and to use recent concepts implicitly. CI is green; verdict APPROVE. - **Scope**: single file, ~27 lines, two tightly related prose changes both about digest content. No code, no unrelated drive-bys. In contract. - **Honest description**: matches the diff exactly — the off-field-news addition and the curriculum rewrite are both present; it is genuinely directive-only. No fabricated checklist items. - **CI / green branch**: `success`, and the author's "check-sync green" claim is consistent with that. No code paths to unit-test here; a prompt-text change needs none. - **Security / CI config**: none touched; no injection/secret surface. `.forgejo`/`.github` untouched. Non-blocking observations (no action required): - The retained example "the Conn Smythe, the NHL's whole-postseason MVP" is an appositive that some would read as a parenthetical, sitting slightly in tension with the new "never a parenthetical lesson" rule. The intent (no heavy date-bearing lessons; light woven refreshers OK) is clear enough that the model should disambiguate, but worth a glance if you want the example to unambiguously model the new rule. - The hard-wrapped token "when-you-\n learned-this" will render with a space mid-token if this markdown is ever displayed; as an LLM prompt it's harmless. Clean, low-risk change as described. --- <sub>Independent review by the harness on `claude-opus-4-8`. The human reviewer is requested once Igor has reviewed; a human still merges.</sub> <!-- review sha=076f13ee4726216efc0384f78f6908b3901d381e verdict=APPROVE ci=success -->
igor added spent time 2026-06-27 13:58:32 +00:00
32 seconds
joshtronic approved these changes 2026-06-27 14:28:38 +00:00
joshtronic deleted branch fix/sports-digest-tone-and-news 2026-06-27 14:28:42 +00:00
Sign in to join this conversation.
No reviewers
No milestone
No assignees
2 participants
Notifications
Total time spent: 32 seconds
igor
32 seconds
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set

Reference
joshtronic/igor!277
No description provided.