gitoriaLog in with ident

tracker

All repositories: gitoria

ReadmeCodePull requestsReleasesTicketsSettings
Commit8f1d45428f1d4542tracker#40: superseded collections — a TMDB collection timeline whose titles are all in one curated timeline is hidden (supersededBy; kept: own page + editor finder), set by the collection seed when it makes one and by the curated build (lifted when the cover is gone); partly covered ones join that franchise; no second widget (First Contact: only Star Trek — Prime); gate franchiseseed 26/0, browser 365/0, kinds 32/0, franchises 53/0, check-theme 0; real copy 24 supersededmre8f1d4542/reports/010-tracker-adult-posters.md

4.9 KB

  1. # w054: tracker, adult titles hidden + adult-flag backfill (mission 010 (old 054))
  2. The work is in loreana:/media/STORAGE/projects/tracker.worldapi.org, commit **`c171227`** on main. Nothing was pushed and nothing was deployed.
  3. **What was built**
  4. - Every title has `shows.adult`: `true`, `false`, or `null` for unknown.
  5. - The homepage rows, `/movies`, `/shows` and the search (our database and "Fetch from web") show only titles where `adult == false`.
  6. - Unknown titles are hidden too.
  7. - The web search sends `include_adult=false` and also checks each result's own `adult` field.
  8. - `/my/*` is not filtered, so an adult title the user follows stays there.
  9. - The daily sync and the search import also store TMDB's adult flag.
  10. - **The backfill** runs inside the app (`project.hl backfillTick` calls `tmdbsync.hl backfillTitle`):
  11. - It handles one title per step, on the same clock as the daily sync. While the daily sync runs, the backfill waits.
  12. - Per title it makes 1 TMDB details request (with external_ids) and downloads the poster if the file is missing. After each API request it pauses 260 ms.
  13. - A 404 marks the title as `adultCheck` and it is not asked again. Passing failures are retried.
  14. - After a restart it continues with the titles that are still unknown.
  15. - It writes a log line every 500 titles and a done line at the end.
  16. - `TRACKER_BACKFILL=0` switches it off.
  17. **Gate**
  18. `TRACKER_GATE_SHOTS=/tmp/w054/shots TRACKER_GATE_PORT=8720 TRACKER_GATE_IDENT_PORT=8721 TRACKER_GATE_CHROME=8722-8726 TRACKER_GATE_TMDB_PORT=8727 node tests/browser.mjs` → last line **`184 passed, 0 failed`** (167 existing checks + 17 new).
  19. - Proven by the new checks:
  20. - Titles with an unknown flag and adult titles are hidden.
  21. - Non-adult titles are shown.
  22. - A followed adult title is still on `/my/movies` and its page opens.
  23. - The web search hides TMDB's adult result.
  24. - The resume works: stopped after 6 titles, the restart asks only the other 8.
  25. - Pages stayed under 1.2 s during the backfill (slowest 634 ms).
  26. - The lists and the search update without a restart.
  27. - A third start has nothing to do.
  28. - Screenshot `/tmp/w054/shots/adult-home-1280.png` (I looked at it).
  29. **Real data** (copy of the live `storage/`, real TMDB; the copy was deleted afterwards)
  30. | | |
  31. |---|---|
  32. | Run time | 9,453 titles in **98.9 min** (5935 s) |
  33. | Adult | **3,450** (3,448 movies, 2 series) |
  34. | Not adult | **5,866** |
  35. | Unknown left | **137**, all TMDB 404, stay hidden |
  36. | Posters | **8,740** downloaded (126 → 8,866 files, 459 MB) |
  37. | Errors | **140**: 137 × 404, 2 poster 404s, 1 TLS failure that was retried and then worked |
  38. - A restart afterwards logs "nothing to do".
  39. - After the run, `/movies` shows 4,242 movies and `/shows` 1,624 series.
  40. - I checked every link on `/`, on all 245 list pages and in 10 searches (6,008 links, including "sex", "porn", "xxx"). All of them are titles marked `adult == false`.
  41. - Page load times during the run, signed out, measured with curl about every 5 s:
  42. | page | average | max |
  43. |---|---|---|
  44. | `/` | 0.156 s | 4.87 s |
  45. | `/movies` | 0.059 s | 1.67 s |
  46. | `/shows` | 0.045 s | 1.54 s |
  47. | `/search/the` | 0.075 s | 1.33 s |
  48. - 14 of 4,620 requests took more than 1 s. Each one happened during a slow TMDB or poster answer (13 steps over 1.5 s, the slowest 5.8 s).
  49. - Screenshots on Loreana: `/tmp/w054/real/{home,movies,shows,search-the}{,-bottom}-{390,1280}.png`. I looked at them and saw no adult title or poster. Logs are in `/tmp/w054/real-*.log`.
  50. **Open points**
  51. 1. Deploy: the live app starts the backfill immediately. It takes about 100 min and adds about 450 MB of posters to `storage/`, which also goes into the nightly backup. At the first start the public lists are empty, and they fill up while the backfill runs.
  52. 2. A slow TMDB answer blocks all pages while it lasts, because hl:fetch is synchronous. The worst seen was 4.9 s. Fixing it needs a non-blocking fetch in Hybriel or a shorter timeout (it is 20 s now).
  53. 3. Posters of adult titles are downloaded too. Skipping them would save about 170 MB.
  54. 4. **Hybriel issue:** I could not hide the show page of an adult title from people who don't follow it.
  55. - Attempt: `found = showRow != null && !adultHiddenFor(showRow, me)`, where `adultHiddenFor` is a static that reaches mpackdb.
  56. - Result: after any session face on the page (for example `showToggleEpisode`), the browser re-derives `found` and then `rows`, and `rows` fails with "'watchesOfUser' … realm 'client' cannot bind". Copying the value into a plain member (`hideAdult`) first did not help.
  57. - So I reverted it. The page of an adult title opens for anyone who has the link, but it appears in no list and no search.
  58. 5. 137 titles that TMDB no longer has stay hidden.
  59. 6. Not mine, seen in passing: signed out at 390 px, the header buttons wrap to two lines. I also tarred the live `storage/` twice (read-only both times): the first `ssh` inside `ssh loreana bash -s` swallowed the rest of the script, so I re-ran it with `< /dev/null`. This is noted in STATUS.

Branches

Latest commits

  • 8f1d4542tracker#40: superseded collections — a TMDB collection timeline whose titles are all in one curated timeline is hidden (supersededBy; kept: own page + editor finder), set by the collection seed when it makes one and by the curated build (lifted when the cover is gone); partly covered ones join that franchise; no second widget (First Contact: only Star Trek — Prime); gate franchiseseed 26/0, browser 365/0, kinds 32/0, franchises 53/0, check-theme 0; real copy 24 supersededmre
  • 7d4b293dtracker#40: "Franchises" in the main menu (desktop header after People, phone sidebar) → /franchises, marked on franchise and timeline pages; gates 365/0, 32/0, 53/0, 24/0, check-theme 0mre
  • 93dfb0batracker#40 (mission 034): the curated franchises — data/franchises.json (17 franchises, 31 timelines, 285 TMDB titles, movies + series, in-universe/release order, 12 TMDB collections attached); lib/franchiseseed.hl + jobs.hl franchiseSeedTick (last start job, imports missing titles via details.hl importWithCredits = the search's Add, one per step paced, then one build; franchiseseed.db: editor changes win, the creator's same-name franchise adopted / timeline left alone, 404 remembered, resumable, idempotent); timeline heads 'N titles · in-universe order' (orderKind) and wrap on a phone; series pages show the widget; new gate tests/franchiseseed.mjs (5th in deploy.sh), the others run with TRACKER_FRANCHISE_SEED=0; gates 365/0, 32/0, 52/0, 24/0, check-theme 0; real copy 196 imported, 0 failed, 7 min, restart unchanged=31mre
  • 96ba683adeploy.sh: a gate without a 'passed,' line (check-theme) no longer ends the scriptmre
  • eb3b9205tracker: report 031mre
  • 9b5d2e89tracker mission 031: README (What it does, Test: four gates + the #32 checks, Files: theme/, new pages), STATUS (real copy, A/B load, how to repeat, open points), LOGmre
  • 39950e4ctracker#32 (mission 031): the WorldAPI theme (theme/ vendored verbatim from layouts.worldapi.org 85b5654; styles.hl inherits it: accent green-dark, type colours 1-6; own base/header rules, row lines, genre-pill and inverted-button frames removed, the season foldable keeps its line; check-theme 21 -> 0, 4th deploy gate; main actions class primary) and the #32 header (theme AppHeader/MainMenu/UserMenu/Sidebar/ContentFirst: desktop brand, search, Series|Shows|Movies|Genres|People, user icon with Unwatched..Settings, Logout; signed out the ident selector, phone the iD icon dropdown; phone menu in the sidebar overlay; marked entry by :has); /find -> /search/<q>, /genres, /people(/<letter>), /settings; main { ContentFirst { slot } } works around the hl:web one-line slot bug; gates 365/0, 32/0, 52/0, check-theme 0mre
  • a386dc92tracker: reports 029 + 030mre
  • 71e0fd7dtracker missions 029 + 030: README (What it does, Files, gate count), STATUS (real-copy numbers, how to repeat, open points), LOGmre
  • d36ea6eatracker#34 + #35 (mission 030): Follow directly under the poster, as wide as the poster (show.hl, styles.hl); the status pill next to a series' title — TVmaze's status (new tvmazeStatus, stored by the sync's TVmaze merge) else TMDB's, TVmaze Ended + TMDB Canceled = Canceled, inverted (filled, dark text, no border), green running / yellow pending / red canceled / muted ended (shows.hl statusOf); the daily delta asks TVmaze's status of an unfollowed series TVmaze's change list names (dailysync.hl syncRunStep, sync.hl syncTvmazeStatus); the status backfill after the details repair (backfill.hl, jobs.hl statusTick; resumable, 550 ms per TVmaze request); gates 354/0, 32/0, 52/0mre
  • 7d7d4487tracker#33 (mission 029): reduced titles — every title TMDB's details never went through this app (no detailsAt, no tmdbSync) is incomplete (shows.hl isIncomplete; the old tracker's migrated rows passed #26's test: 5,697 non-adult on the live copy, 691 series without seasons); the repair job does the visibly reduced first (shows.hl missingParts), the page completes one on open; a title TMDB has no poster for (The Remaining) shows the placeholder; tools/count-incomplete.hl; gate fixtures stand for synced titles (tmdbSync), tests/seed-reduced.hl + #33 checks; gates 347/0, 32/0, 52/0mre
  • 661c2592tracker: report 028mre
  • 27c916fatracker mission 028: README ("Code order", the new file map), STATUS (counts before/after, tests, how to repeat, open), LOGmre
  • d924f398tracker mission 028: comments name the new files (sync.hl, dailysync.hl, backfill.hl, credits.hl, jobs.hl, images.hl …); tools/ref-params.py + tools/lambda-audit.py also scan lib/ (they globbed the root only), lambda-audit counts a plain `x = p` alias like `let x = p`mre
  • 2e89b968tracker mission 028 (code order) 5/5 let: `let` only where a variable is reassigned — 667 never-reassigned lets became plain declarations (project.hl, lib/, components/, tools/, tests/); kept: 264 in loop bodies (a plain declaration there is 'Cannot reassign' on the 2nd pass), 234 reassigned, 27 whose name is also a member/outer/free name (a plain write would rebind it); tools/let-audit.py decides and fixes (README 'Code order'); tests/realdata-m028.{sh,mjs} = the page-output diff on a real copy; gates 342/0, 32/0, 52/0, real-copy pages identicalmre
  • 54796ff2tracker mission 028 (code order) 4/5 thin faces + last copies: the show page's check/follow faces call lib/watches.hl toggleWatched / toggleSeasonWatched (seasonAllWatched moved there) and lib/follows.hl toggleFollowed; both logins (header selector face, /login/callback) share lib/users.hl userOfCode; todayStr/listOf copies in components and the export readers copied into tools/migrate.hl + tools/old-short-ids.hl now once (lib/util.hl, lib/export.hl); gates 342/0, 32/0, 52/0; old-short-ids output byte-identical, migrate output identicalmre
  • 06b078e3tracker mission 028 (code order) 3/5 project.hl is the map: config, routes, wiring and a feature → file index (914 → 258 lines); the background jobs (daily sync run, backfills, details repair, credits job, merge, short ids, collection seed) moved unchanged into lib/jobs.hl (a class: their state is reassigned every step, a static cannot be; one instance made after the server), the login callback into lib/users.hl, poster/photo serving into lib/images.hl, the /shows/<slug> rule into lib/shows.hl showsMovedPath; route handlers are thin wrappers; gates 342/0, 32/0, 52/0, real-copy pages identicalmre
  • 94716fd2tracker mission 028 (code order) 2/5 util + topics: lib/util.hl holds envOr, storageDir, postersDir, profilesDir, newId, hexDigits, todayStr, dateOr, textOr, hasId, listOr, firstOf, sortDesc once (were copied into up to 5 files); tmdbsync.hl split into tmdb.hl (TMDB/TVmaze requests), sync.hl (one title's sync), sync-helpers.hl, backfill.hl; details.hl split into details.hl, credits.hl, credits-helpers.hl (isIncomplete to shows.hl); search-helpers.hl (words, query, ranking, slugs); collections.hl (the TMDB collection seed, out of franchises.hl); deltasync.hl renamed dailysync.hl; no behaviour change: gates 342/0, 32/0, 52/0, real-copy pages identicalmre
  • 186079b0tracker mission 028 (code order) 1/5 move: every root .hl except project.hl into lib/ (styles.hl into components/), import paths only; gates 342/0, 32/0, 52/0; real-copy pages identicalmre
  • 4f47f181tracker: report 027mre