docs: sync README/DESIGN.md against actual repo state - #16
Closed
serpeigd wants to merge 1 commit into
Closed
Conversation
- Researched catalogue: 8/20 -> 23 entries (18/20 measurement titles + 5 more); update every count/list that referenced the old 8. - Disclose the "+ Suggest a movie" auto-publish path (no review step, a deliberate risk per app.py's own docstring) -- README implied every path into content/researched/ still went through human review. - CI is currently failing on every push since 2026-08-19 (test_auto_publish.py/ test_groq_retry.py import fastapi/groq/httpx without importorskip, tests.yml never installs them) -- was claiming 18/26 passing. - Correct test count (26 -> 35, verified by running the suite locally). - Note Groq's 2026-08-18 model decommission next to the LLMJudge comparison numbers that used the now-gone model names. - docs/DESIGN.md: add D17 for the auto-publish decision, close the stale Pending bullet claiming it wasn't built yet. Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
Owner
Author
|
Closing in favor of #17, which folds in this PR's findings (all verified accurate against the current tip of Also edited this PR's body above to strip a leaked session-tracking URL from the original attribution footer (content otherwise unchanged) — flagging per this org's "never push session/environment identifiers" rule. Generated by Claude Code |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Docs-only sync. Verified against the working tree, not the changelog.
content/researched/, no review step — a deliberate, documented risk perwebapp/app.py's own module docstring). README previously implied every path intocontent/researched/went through human review, which stopped being true on 2026-08-19.test_auto_publish.py/test_groq_retry.pyimportfastapi/groq/httpxwithoutpytest.importorskip, andtests.ymlnever installs those, so collection is interrupted before any test runs. README was claiming "18/26 passing in CI." Not fixed here (product/CI-config change, out of scope for a docs-only pass) — just documented.sentence-transformers).LLMJudgecomparison numbers that cite the now-gone model names (numbers are historically accurate, just no longer what running the command today would use).docs/DESIGN.md: added D17 for the auto-publish decision, closed the "Pending" bullet that still said this wasn't built.No product or measurement-track code touched.
Note: this body was edited to remove a leaked session-tracking URL from the original attribution footer — content otherwise unchanged. See #17, which supersedes this PR.