@krx3d/tizentube2 1.15.810 → 1.15.820

package/.github/scripts/build-release-notes.sh

@@ -0,0 +1,113 @@
+#!/usr/bin/env bash
+# Build the release body for a TizenTube Standalone release.
+#
+# Usage: build-release-notes.sh <release_sha> <release_version> <app_id> <app_name> <out_file>
+#
+# The changelog is derived from the merge/squash commits between the previous
+# published release and this build, grouped by conventional-commit prefix.
+# Version-bump commits made by CI carry no PR number and drop out on their own.
+set -euo pipefail
+
+RELEASE_SHA="$1"
+RELEASE_VERSION="$2"
+APP_ID="$3"
+APP_NAME="$4"
+OUT="$5"
+
+REPO="${GITHUB_REPOSITORY:-}"
+
+# Previous release = the newest published release that isn't the one being cut.
+# Falls back to the newest version tag, then to "no previous release".
+PREV_TAG=""
+if [ -n "$REPO" ] && command -v gh >/dev/null 2>&1; then
+  PREV_TAG=$(gh api "repos/${REPO}/releases" --jq \
+    "[.[] | select(.draft == false) | .tag_name] | map(select(. != \"${RELEASE_VERSION}\")) | .[0] // empty" \
+    2>/dev/null || true)
+fi
+# Reject anything that isn't an ancestor of this build: version-string order is
+# not history order (a v2.x standalone tag can sort above the current v1.15.x
+# line and would drag in every commit since the fork point).
+if [ -n "$PREV_TAG" ] && ! git merge-base --is-ancestor "${PREV_TAG}^{commit}" "$RELEASE_SHA" 2>/dev/null; then
+  echo "Ignoring ${PREV_TAG}: not an ancestor of ${RELEASE_SHA}" >&2
+  PREV_TAG=""
+fi
+if [ -z "$PREV_TAG" ]; then
+  PREV_TAG=$(git describe --tags --abbrev=0 "${RELEASE_SHA}^" 2>/dev/null || true)
+  [ "$PREV_TAG" = "$RELEASE_VERSION" ] && PREV_TAG=""
+fi
+
+if [ -n "$PREV_TAG" ] && git rev-parse -q --verify "${PREV_TAG}^{commit}" >/dev/null; then
+  RANGE="${PREV_TAG}..${RELEASE_SHA}"
+else
+  # First release, or the previous tag isn't in this clone — describe the last
+  # stretch of history rather than the entire project.
+  RANGE="${RELEASE_SHA}~50..${RELEASE_SHA}"
+  git rev-parse -q --verify "${RELEASE_SHA}~50" >/dev/null || RANGE="$RELEASE_SHA"
+fi
+
+# --- collect (pr_number, title) pairs -----------------------------------------
+# GitHub merge commits:  "Merge pull request #N from owner/branch" + body=title
+# GitHub squash commits: "title (#N)"
+declare -a FEATURES=() FIXES=() PORTS=() OTHER=()
+
+while IFS=$'\x1f' read -r -d $'\x1e' subject body; do
+  subject="${subject#$'\n'}"
+  pr=""
+  title=""
+  if [[ "$subject" =~ ^Merge\ pull\ request\ \#([0-9]+) ]]; then
+    pr="${BASH_REMATCH[1]}"
+    title=$(printf '%s' "$body" | sed '/^[[:space:]]*$/d' | head -n1)
+  elif [[ "$subject" =~ ^(.*)\ \(\#([0-9]+)\)$ ]]; then
+    title="${BASH_REMATCH[1]}"
+    pr="${BASH_REMATCH[2]}"
+  fi
+  [ -z "$pr" ] && continue
+  [ -z "$title" ] && title="$subject"
+
+  entry="- ${title} (#${pr})"
+  shopt -s nocasematch
+  if [[ "$title" == Port\ upstream* ]]; then
+    PORTS+=("$entry")
+  elif [[ "$title" =~ ^feat(\(.*\))?: ]]; then
+    FEATURES+=("$entry")
+  elif [[ "$title" =~ ^fix(\(.*\))?: ]]; then
+    FIXES+=("$entry")
+  else
+    OTHER+=("$entry")
+  fi
+  shopt -u nocasematch
+done < <(git log --first-parent --pretty=format:'%s%x1f%b%x1e' "$RANGE")
+
+# --- write the body -----------------------------------------------------------
+{
+  echo "TizenTube Standalone build from commit \`${RELEASE_SHA}\`, version-synced with the userscript."
+  echo "TBI_METADATA: {\"appId\":\"${APP_ID}\",\"appName\":\"${APP_NAME}\"}"
+  echo
+
+  section() {
+    local heading="$1"; shift
+    [ "$#" -eq 0 ] && return 0
+    echo "### ${heading}"
+    printf '%s\n' "$@"
+    echo
+  }
+
+  if [ ${#FEATURES[@]} -eq 0 ] && [ ${#FIXES[@]} -eq 0 ] && [ ${#PORTS[@]} -eq 0 ] && [ ${#OTHER[@]} -eq 0 ]; then
+    echo "No pull requests were merged since \`${PREV_TAG:-the previous build}\`."
+  else
+    echo "## What's Changed"
+    echo
+    section "Features" ${FEATURES[@]+"${FEATURES[@]}"}
+    section "Fixes" ${FIXES[@]+"${FIXES[@]}"}
+    section "Ported from upstream" ${PORTS[@]+"${PORTS[@]}"}
+    section "Other" ${OTHER[@]+"${OTHER[@]}"}
+  fi
+
+  if [ -n "$PREV_TAG" ] && [ -n "$REPO" ]; then
+    echo "**Full changelog:** https://github.com/${REPO}/compare/${PREV_TAG}...${RELEASE_VERSION}"
+  fi
+} > "$OUT"
+
+echo "Wrote release notes to $OUT:"
+echo "---"
+cat "$OUT"

package/.github/workflows/build-standalone-release.yaml

@@ -286,13 +286,20 @@ jobs:
           name: TizenTube-Standalone-${{ env.RELEASE_SHA }}.wgt
           path: standalone/release/TizenTube.wgt
 
+      # Changelog is built from the PRs merged since the previous published
+      # release; the build/TBI_METADATA header stays first so the TizenBrew
+      # Installer still finds it.
+      - name: Build release notes
+        env:
+          GH_TOKEN: ${{ github.token }}
+        run: |
+          bash .github/scripts/build-release-notes.sh             "${{ env.RELEASE_SHA }}"             "${{ env.RELEASE_VERSION }}"             "${{ env.APP_ID }}"             "${{ env.APP_NAME }}"             "$RUNNER_TEMP/release-notes.md"
+
       - name: Release TizenTube Standalone
         uses: softprops/action-gh-release@v1
         with:
           tag_name: ${{ env.RELEASE_VERSION }}
           target_commitish: ${{ env.RELEASE_SHA }}
-          body: |
-            TizenTube Standalone build from commit `${{ env.RELEASE_SHA }}`, version-synced with the userscript.
-            TBI_METADATA: {"appId":"${{ env.APP_ID }}","appName":"${{ env.APP_NAME }}"}
+          body_path: ${{ runner.temp }}/release-notes.md
           files: |
             standalone/release/*

package/.github/workflows/upstream-sync.yml

@@ -0,0 +1,138 @@
+# Watch reisxd/TizenTube for new commits and open a PR that carries them.
+#
+# This fork has rewritten large parts of mods/ (adblock.js, hideWatched.js,
+# settings.js in particular), so most upstream commits touching those files
+# WILL conflict. The workflow does not pretend otherwise: it always produces a
+# branch that merges cleanly into main, and when a file conflicted it keeps the
+# fork's version and says so in the PR body, so the upstream change can be
+# hand-ported afterwards.
+name: Upstream sync
+
+on:
+  schedule:
+    # 04:20 UTC daily — outside the usual release window.
+    - cron: '20 4 * * *'
+  workflow_dispatch:
+
+concurrency:
+  group: upstream-sync
+  cancel-in-progress: false
+
+env:
+  UPSTREAM_REPO: https://github.com/reisxd/TizenTube.git
+  UPSTREAM_BRANCH: main
+  SYNC_BRANCH: sync/upstream
+
+jobs:
+  sync:
+    runs-on: ubuntu-latest
+    permissions:
+      contents: write
+      pull-requests: write
+    steps:
+      - name: Clone repo with full history
+        uses: actions/checkout@v5
+        with:
+          fetch-depth: 0
+
+      - name: Configure git identity
+        run: |
+          git config user.name  'github-actions[bot]'
+          git config user.email 'github-actions[bot]@users.noreply.github.com'
+
+      - name: Fetch upstream
+        run: |
+          set -euo pipefail
+          git remote add upstream "$UPSTREAM_REPO" 2>/dev/null || git remote set-url upstream "$UPSTREAM_REPO"
+          git fetch upstream "$UPSTREAM_BRANCH" --no-tags
+
+      - name: Check for new upstream commits
+        id: check
+        run: |
+          set -euo pipefail
+          COUNT=$(git rev-list --count "origin/main..upstream/${UPSTREAM_BRANCH}")
+          echo "count=$COUNT" >> "$GITHUB_OUTPUT"
+          echo "Upstream commits not in main: $COUNT"
+
+      - name: Merge upstream, keeping the fork's side of any conflict
+        id: merge
+        if: steps.check.outputs.count != '0'
+        run: |
+          set -euo pipefail
+          git checkout -B "$SYNC_BRANCH" origin/main
+
+          CONFLICTS=""
+          if ! git merge --no-edit "upstream/${UPSTREAM_BRANCH}"; then
+            # Keep this fork's version of every conflicted file. The point of
+            # the PR is to surface the change for review, not to guess at a
+            # resolution — committing conflict markers would only produce a
+            # branch that cannot build.
+            CONFLICTS=$(git diff --name-only --diff-filter=U)
+            echo "Conflicted files kept at fork version:"
+            echo "$CONFLICTS"
+            while IFS= read -r f; do
+              [ -z "$f" ] && continue
+              git checkout --ours -- "$f" 2>/dev/null || git rm -q -- "$f"
+              git add -- "$f" 2>/dev/null || true
+            done <<< "$CONFLICTS"
+            git commit --no-edit
+          fi
+
+          {
+            echo 'conflicts<<TT_EOF'
+            echo "$CONFLICTS"
+            echo 'TT_EOF'
+          } >> "$GITHUB_OUTPUT"
+
+          if git diff --quiet origin/main -- .; then
+            echo "changed=false" >> "$GITHUB_OUTPUT"
+          else
+            echo "changed=true" >> "$GITHUB_OUTPUT"
+            git push --force-with-lease origin "$SYNC_BRANCH"
+          fi
+
+      - name: Open or update the sync PR
+        if: steps.check.outputs.count != '0' && steps.merge.outputs.changed == 'true'
+        env:
+          GH_TOKEN: ${{ github.token }}
+          CONFLICTS: ${{ steps.merge.outputs.conflicts }}
+          NEW_COMMITS: ${{ steps.check.outputs.count }}
+        run: |
+          set -euo pipefail
+
+          {
+            echo "Automated sync with [reisxd/TizenTube](https://github.com/reisxd/TizenTube) — **${NEW_COMMITS}** new upstream commit(s)."
+            echo
+
+            if [ -n "${CONFLICTS//[[:space:]]/}" ]; then
+              echo "## :warning: Needs hand-porting"
+              echo
+              echo "These files conflicted and were **kept at this fork's version**, so the upstream changes to them are *not* in this PR. Merging as-is is safe but incomplete — port them by hand before or after merging:"
+              echo
+              while IFS= read -r f; do
+                [ -z "$f" ] && continue
+                echo "- \`$f\`"
+              done <<< "$CONFLICTS"
+              echo
+            else
+              echo "Merged cleanly — no conflicts."
+              echo
+            fi
+
+            echo "## Upstream commits"
+            echo
+            git log --reverse --no-merges --pretty=format:'- [`%h`](https://github.com/reisxd/TizenTube/commit/%H) %s' "origin/main..upstream/${UPSTREAM_BRANCH}"
+            echo
+            echo
+            echo "_Opened automatically by \`.github/workflows/upstream-sync.yml\`._"
+          } > /tmp/pr-body.md
+
+          EXISTING=$(gh pr list --head "$SYNC_BRANCH" --state open --json number --jq '.[0].number // empty')
+          TITLE="Sync with upstream (${NEW_COMMITS} new commit(s))"
+
+          if [ -n "$EXISTING" ]; then
+            gh pr edit "$EXISTING" --title "$TITLE" --body-file /tmp/pr-body.md
+            echo "Updated PR #$EXISTING"
+          else
+            gh pr create --base main --head "$SYNC_BRANCH" --title "$TITLE" --body-file /tmp/pr-body.md
+          fi

package/AGENTS.md

@@ -788,6 +788,72 @@ install-blocked mode), and the user's hardware is an unconfirmed fit (has
 a Samsung soundbar, not confirmed Q-Symphony-compatible). Ask before
 implementing if this comes back up.
 
+## Hard-won lessons (read before touching `mods/features/adblock.js`)
+
+These cost real debugging time and are not obvious from the code. They are
+recorded here rather than in any one contributor's notes so a fork inherits
+them.
+
+### There are two response-handling paths, and they are not shared code
+
+YouTube delivers responses in two shapes, and `adblock.js` has a separate
+handler for each:
+
+- **array-root** → `processResponsePayload(payload, detectedPage)`
+- **object-root** → the branch inside the patched `JSON.parse`
+
+A filter added to one path and not the other works on some surfaces and
+silently does nothing on others. This has bitten repeatedly: the array-root
+grid handler was missing `filterShortsFromItems` and
+`filterMembersOnlyFromItems`; `noteContinuationBatch` had to be hooked at
+*both* continuation sites; `hideRelatedVideos` is called from both watch-next
+sites.
+
+When adding anything per-item or per-shelf, put the logic in its own module and
+call it from both sites — `mods/features/sidebarChannelButton.js` is the shape
+to copy — rather than duplicating the body. Grep for the sibling call site
+before assuming one hook is enough.
+
+### `JSON.parse` is wrapped by several modules, and order matters
+
+`playlistContinue.js`, `playlistBatchCollect.js`, `adblock.js` and
+`customGuideAction.js` each wrap `JSON.parse`, in that order (set by the import
+order in `userScript.js`). Every wrapper calls through to whatever it captured
+as the original, so they compose — but an early `return` added to any of them
+silently swallows every wrapper installed before it.
+
+`captionStylePersistence.js` independently wraps `resolveCommand` alongside
+`resolveCommand.js`'s own patch. Same rule applies.
+
+### A playlist continuation response must contain at least one item
+
+YouTube's refill loop stops dead if a continuation comes back empty — loading
+never resumes, even on scroll. `hideVideo` therefore deliberately keeps one
+"helper" tile per continuation batch (`__ttKeepOneForContinuation`).
+
+This was learned the hard way: PR #678 capped helpers at one per visit and
+returned `[]` for the rest, and playlist loading stalled after two batches on
+device. It was reverted in #679. If you change the helper logic, test a long,
+mostly-watched playlist and confirm it still loads past batch 2.
+
+### Tizen 5.0 has no usable Polymer element APIs
+
+`yt-virtual-list` positions its rows by transform from its own data model, and
+the Polymer APIs you would reach for to force a re-render are `undefined` on
+Tizen 5.0. Every DOM-level attempt to collapse the blank slot left behind by a
+removed helper tile failed for this reason — rows are recycled and carry their
+previous inline styles, and helper tiles carry no video-id attribute to target
+(the id exists only inside a thumbnail's `background-image` URL).
+
+Blank helper slots are an accepted limitation. Don't re-litigate it at the DOM
+layer without new evidence.
+
+### Verifying that a change actually reached the bundle
+
+`dist/userScript.js` is minified and Babel-transpiled: identifier names are
+mangled, string literals survive. Grep for a distinctive **string literal**,
+never a function name, and use `grep -a` — the bundle is detected as binary.
+
 ## Conventions this repo has established (follow these)
 
 - New app identities (Tizen `package`/app id) must be unique, not reused
@@ -818,3 +884,15 @@ implementing if this comes back up.
   mistake). If you need a per-dependency try/catch wrapper for
   diagnostics, wrap each literal `require('x')` call individually rather
   than passing the module name through a shared helper function.
+- Never run the build by hand and commit the generated output (`dist/*`,
+  `standalone/service/dist/*`) alongside a source change. CI rebuilds and
+  commits it on version bump; doing it manually produces merge conflicts on
+  generated files for everyone branching off `main`. Build locally to check a
+  change compiles, then `git checkout -- dist/` before committing.
+- Work on a branch and open a PR; don't push to `main`. Don't push further
+  commits onto a branch whose PR is already merged — the commits end up
+  orphaned. Check merge state before pushing to an existing branch.
+- When porting an upstream commit, read it rather than applying it: upstream
+  code has shipped with operator-precedence bugs, dead branches behind early
+  returns, and renames that would reset this fork's stored settings. Fix them
+  in the port and say so in the PR.

package/README.md

@@ -54,12 +54,98 @@ If a run failed because of a bug in the workflow file itself and that's since be
 
 # Features
 
-- Ad Blocker
-- [SponsorBlock](https://sponsor.ajay.app/) Support
-- Picture-in-Picture Mode
-- [DeArrow](https://dearrow.ajay.app/) Support
-- Customizable Themes (Custom Coloring)
-- More to come, if you [request](https://github.com/reisxd/TizenTube/issues/new) it!
+Everything below is toggleable from the in-app settings menu (**green button** on
+the remote, or `G` / `F2` / `2` when testing in Chrome). This fork carries a
+number of features that upstream TizenTube does not — those are marked
+**(fork)**.
+
+Each row is one settings-menu entry; where an entry has its own sub-menu, its
+options are described in the row rather than listed separately.
+
+## Blocking and filtering
+
+| Feature | What it does |
+| --- | --- |
+| Ad Block | Removes video ads, ad slots, and the masthead/banner ad on Home |
+| [SponsorBlock](https://sponsor.ajay.app/) | Skips sponsor, intro, outro, self-promo, interaction, filler, preview and non-music segments. Each category can be set to auto-skip, skip manually, or ignore, with optional toasts. Highlights (jump to the video's main point) can be enabled separately |
+| Hide End Screen Cards | Removes the suggested-video cards overlaid at the end of a video |
+| "Includes paid promotion" overlay | Toggle the paid-promotion banner |
+| Hide Members-Only Videos **(fork)** | Filters videos you can't watch without a channel membership out of shelves and grids |
+| Hide Channel Shelves **(fork)** | Removes whole channel-recommendation shelves from feeds |
+| Hide Feedback Surveys **(fork)** | Removes YouTube's in-feed "help us improve" survey cards |
+| Hide Special Playlists **(fork)** | Hides Liked Videos and/or Watch Later from the Library and Playlists pages |
+| Enable Shorts | Off by default — Shorts shelves and Shorts tiles are stripped from every surface |
+
+## Watched-video handling
+
+| Feature | What it does |
+| --- | --- |
+| Hide Watched Videos | Hides videos you've already watched. Configurable watched-percentage threshold, and per-page control over where it applies (Home, Search, Subscriptions, Channel pages, Library, individual playlists, History, Music, Gaming, More, Watch) |
+| Playlist Batch Load **(fork)** | Loads a playlist's later batches up front instead of only as you scroll, so hide-watched can act on the whole playlist rather than the first ~30 items. The number of batches fetched is capped by a configurable limit |
+
+## Video player
+
+| Feature | What it does |
+| --- | --- |
+| Preferred Video Quality | Locks playback to a chosen quality (or the next best available) instead of letting YouTube pick |
+| Preferred Video Codec | Prefer VP9, AV1 or AVC1 when the video offers a choice |
+| Auto Frame Rate | Matches the TV's output frame rate to the video, with a configurable pause duration while it switches (Tizen only) |
+| Spoof Viewport Resolution **(fork)** | Reports a different screen resolution to YouTube. Useful when the TV reports a lower resolution to the browser than it can actually decode. Requires an app restart |
+| Speed Settings Increments | Sets the step size used by the playback-speed control |
+| Picture-in-Picture / Mini Player | Both available from the player options popup; the two buttons can be swapped |
+| Screen Off | Blanks the screen while audio keeps playing — for using the TV as a music player |
+| Hide Related Videos in Player | Removes the related/suggested-videos rail that slides in over the player, so a nudge on the D-pad during playback doesn't cover the video with recommendations |
+| Number Keys Jump to Percentage | Pressing `1`–`9` jumps to that percentage of the video, `0` jumps to the start |
+| Reload Player After TV Wakes **(fork)** | Rebuilds the player after standby, fixing the frozen first frame |
+| Player UI patching | Optional Previous/Next, Super Thanks, Ask, and Speed Controls buttons |
+
+## Thumbnails and titles
+
+| Feature | What it does |
+| --- | --- |
+| [DeArrow](https://dearrow.ajay.app/) | Community-sourced, non-clickbait titles, with optional DeArrow thumbnails |
+| High Quality Thumbnails | Upgrades tile thumbnails to `hqdefault`/`sddefault` |
+| Video Previews | Hover/focus previews on tiles |
+| Disable Enlarged Thumbnails | Stops the focused tile from scaling up |
+| Enable Shrinked Thumbnails | Shrinks unfocused tiles instead |
+
+## Interface
+
+| Feature | What it does |
+| --- | --- |
+| Customizable Themes | Custom focus-container and route colouring (**red button**) |
+| Clock | On-screen clock, 12/24-hour, optional seconds, and an option to hide it while a video is playing |
+| Screen Dimming | Dims the screen after a configurable idle timeout, at a configurable opacity |
+| Disable Sidebar Contents | Choose which sidebar entries to hide |
+| Disable Channels on Sidebar | Removes subscribed-channel entries from the sidebar |
+| Launch To on Startup | Choose which page the app opens to |
+| Reload Home on Startup | Forces a fresh Home feed on launch |
+| Library Tabs Buttons to Hide **(fork)** | Hides individual Library tabs (Music, Movies & Shows, Podcasts, My Videos, History, Watch Later, Playlists) |
+| Sort Subscriptions Alphabetically | Alphabetical instead of YouTube's own ordering |
+| Long Press Actions | Long-press a tile for Play, Watch Later, Save to Playlist and Add to Queue |
+| Who's Watching Menu | Control whether the profile picker appears, including on app exit, and whether it stays permanently enabled |
+| "Are you still watching?" prompt | Toggle YouTube's idle-playback interruption |
+| Show Guest Sign In Reminder | Toggle the prompt asking a signed-out viewer to sign in |
+| Show TT Welcome Message | Toggle the TizenTube toast shown on launch |
+| Fix UI | Layout corrections for TVs that render the YouTube TV UI incorrectly |
+
+## Subtitles
+
+| Feature | What it does |
+| --- | --- |
+| Show Local Subtitle | Surfaces subtitles in your own language |
+| Show Hidden Subtitles | Exposes tracks YouTube hides from the picker |
+| Remember Caption Style **(fork)** | Persists caption font/size/colour settings across sessions and restarts |
+
+## Maintenance and diagnostics
+
+| Feature | What it does |
+| --- | --- |
+| Updater | Checks for TizenTube updates, optionally on startup |
+| Debug Console **(fork)** | On-screen log console (**yellow button**), with configurable corner position and height |
+| Remote Log Server **(fork)** | Streams logs to a PC receiver for on-device debugging, with a built-in connection test |
+
+Missing something? [Request it](https://github.com/reisxd/TizenTube/issues/new).
 
 # Tampermonkey local debugging helpers (Windows + Chrome)