chore: address review feedback on predicted rating logging (#11 )

- align diagnostic endpoint date windows with getNextPDGAUpdateDate - pass pdgaNumber to initial target-rating calculation for log attribution - avoid double log when lazy-recompute returns 0 - log full error object in admin endpoint catch
chore: add diagnostic logging for predicted rating lifecycle (#11 )
2026-05-25 22:02:42 +02:00 · 2026-05-25 21:55:53 +02:00 · 2026-05-25 09:18:47 +00:00 · 2026-05-25 11:18:35 +02:00 · 2026-05-25 11:12:01 +02:00 · 2026-05-25 11:01:53 +02:00
26 changed files with 1371 additions and 579 deletions
@@ -0,0 +1,91 @@
+name: Release
+
+# Auto-tagging baserat på conventional commits sedan senaste taggen.
+#   feat: → minor bump  (1.2.9 → 1.3.0)
+#   fix:  → patch bump  (1.2.9 → 1.2.10)
+#   "!:" eller "BREAKING CHANGE" → major bump (1.2.9 → 2.0.0)
+# Andra prefix (chore:, docs:, ci:, refactor:, style:, test:) bumpar inte.
+# Skippas tyst när inga releaseable commits hittas — t.ex. när workflow:n själv pushar version-commiten.
+# Push av tag triggar deploy.yml (PACKAGES_TOKEN är en PAT så cross-workflow-triggers funkar).
+
+on:
+  push:
+    branches: [main]
+
+jobs:
+  release:
+    runs-on: ubuntu-latest
+    permissions:
+      contents: write
+    steps:
+      - uses: actions/checkout@v4
+        with:
+          fetch-depth: 0
+          token: ${{ secrets.PACKAGES_TOKEN }}
+
+      - name: Configure git
+        run: |
+          git config user.name "Release Bot"
+          git config user.email "noreply@shcizo.se"
+
+      - name: Determine version bump
+        id: bump
+        run: |
+          set -euo pipefail
+
+          LAST_TAG=$(git describe --tags --abbrev=0 --match 'v*' 2>/dev/null || echo "")
+          if [ -z "$LAST_TAG" ]; then
+            RANGE=""
+            echo "No previous v* tag found — scanning all commits."
+          else
+            RANGE="${LAST_TAG}..HEAD"
+            echo "Scanning commits since ${LAST_TAG}."
+          fi
+
+          BUMP="none"
+          while IFS= read -r subject; do
+            [ -z "$subject" ] && continue
+            if echo "$subject" | grep -qE "^[a-z]+(\([^)]+\))?!:|BREAKING[ -]CHANGE"; then
+              BUMP="major"
+              break
+            elif echo "$subject" | grep -qE "^feat(\([^)]+\))?:"; then
+              [ "$BUMP" != "major" ] && BUMP="minor"
+            elif echo "$subject" | grep -qE "^fix(\([^)]+\))?:"; then
+              [ "$BUMP" = "none" ] && BUMP="patch"
+            fi
+          done < <(git log ${RANGE} --no-merges --pretty=format:%s)
+
+          echo "Determined bump: ${BUMP}"
+          echo "bump=${BUMP}" >> "$GITHUB_OUTPUT"
+
+      - name: Compute new version
+        if: steps.bump.outputs.bump != 'none'
+        id: version
+        run: |
+          set -euo pipefail
+          CURRENT=$(node -p "require('./package.json').version")
+          IFS=. read -r MAJ MIN PAT <<< "$CURRENT"
+          case "${{ steps.bump.outputs.bump }}" in
+            major) NEW="$((MAJ+1)).0.0" ;;
+            minor) NEW="${MAJ}.$((MIN+1)).0" ;;
+            patch) NEW="${MAJ}.${MIN}.$((PAT+1))" ;;
+          esac
+          echo "Bumping ${CURRENT} → ${NEW}"
+          echo "new=${NEW}" >> "$GITHUB_OUTPUT"
+
+      - name: Update files, commit, tag, push
+        if: steps.bump.outputs.bump != 'none'
+        run: |
+          set -euo pipefail
+          NEW="${{ steps.version.outputs.new }}"
+
+          npm version "$NEW" --no-git-tag-version
+
+          git add package.json package-lock.json
+          git commit -m "$NEW"
+          git tag "v${NEW}"
+          git push origin main "v${NEW}"
+
+      - name: No-op summary
+        if: steps.bump.outputs.bump == 'none'
+        run: echo "No feat/fix/breaking commits since last tag — no release."
@@ -46,7 +46,7 @@ public/
 - **Logging:** Use `require('./logger')` (or relative path). Never use `console.log/error` in backend code. Use appropriate Pino levels: `debug` for verbose/diagnostic data, `info` for operational status, `warn` for retries/degraded state, `error` for failures, `fatal` for startup crashes.
 - **Frontend JS:** `console.error` is fine in `public/js/` — runs in browser, no Pino.
 - **Commits:** Conventional commits (`feat:`, `fix:`, `refactor:`, `chore:`, `ci:`).
- **Releases:** Manual version bump — edit `version` in `package.json` + `package-lock.json`, commit as `<version>`, tag `v<version>`, push commit + tag (`git push origin main v<version>`). Triggers `.gitea/workflows/deploy.yml` which (1) builds and pushes the image to `gitea.shcizo.se/shcizo/pdga-rating:<tag>` + `:latest`, then (2) calls `package-updater-action` against `updater.shcizo.se/update` to roll out the new image. Required secrets: `PACKAGES_TOKEN` (PAT with `write:package`, for registry auth — the auto-injected `GITEA_TOKEN` does not have effective registry access) and `UPDATER_API_KEY` (for the updater endpoint). The action repo `shcizo/package-updater-action` is referenced via full Gitea URL (`https://gitea.shcizo.se/...`) since `uses:` defaults to GitHub.
+- **Releases:** Automated via `.gitea/workflows/release.yml` on push to `main`. Workflow scans conventional commits sedan senaste `v*`-taggen och bumpar strikt semver: `feat:` → minor, `fix:` → patch, `!:`/`BREAKING CHANGE` → major. Andra prefix (`chore:`, `docs:`, `ci:`, `refactor:`, `style:`, `test:`) bumpar inte. Workflow:n uppdaterar `package.json` + `package-lock.json`, committar som `<version>`, taggar `v<version>` och pushar. Tag-pushen triggar `.gitea/workflows/deploy.yml` som (1) bygger + pushar image till `gitea.shcizo.se/shcizo/pdga-rating:<tag>` + `:latest`, sen (2) anropar `package-updater-action` mot `updater.shcizo.se/update`. Required secrets: `PACKAGES_TOKEN` (PAT med `write:package` + repo-push-scope — används av både release.yml för att pusha main/tag och deploy.yml för registry-auth; auto-injicerade `GITEA_TOKEN` triggar inte cross-workflows och saknar effective registry-access) och `UPDATER_API_KEY` (för updater-endpointen). Action-repot `shcizo/package-updater-action` refereras via full Gitea-URL (`https://gitea.shcizo.se/...`) eftersom `uses:` defaultar till GitHub. **Manuell tag** är fortfarande möjlig om release-workflow:n skippas eller behöver kringgås — pusha bara `v<version>`-tag direkt.
 - **Scraping:** Two strategies per entity: direct HTTP (fast, preferred) with Puppeteer fallback (stealth plugin for anti-bot). Rate limiting must be respected.
 - **Database:** Migrations run automatically on startup in `db.js`. Schema changes go there.
 - **Templates:** EJS with shared layout in `views/partials/`. Pages use HTMX for dynamic content loading.
@@ -1,12 +1,12 @@
 {
  "name": "pdga-ratings",
-  "version": "1.2.9",
+  "version": "1.4.0",
  "lockfileVersion": 3,
  "requires": true,
  "packages": {
    "": {
      "name": "pdga-ratings",
-      "version": "1.2.9",
+      "version": "1.4.0",
      "dependencies": {
        "ejs": "^4.0.1",
        "express": "^4.18.2",
@@ -1,6 +1,6 @@
 {
  "name": "pdga-ratings",
-  "version": "1.2.9",
+  "version": "1.4.0",
  "description": "PDGA rating scraper and display",
  "main": "server.js",
  "scripts": {
@@ -2,133 +2,461 @@
   Courses Page
   ═══════════════════════════════════════════════════ */

-/* ── Controls ─────────────────────────────────── */
+/* ── Action Card (tabs + inputs) ─────────────────── */

-.controls {
+.action-card {
+    background: var(--paper);
+    border: 1px solid var(--line-2);
+    border-radius: var(--radius);
+    box-shadow: var(--shadow-card);
+    margin-bottom: 16px;
+}
+
+.action-card-tabs {
    display: flex;
-    justify-content: flex-end;
-    margin-bottom: 20px;
+    border-bottom: 1px solid var(--line-2);
 }

-/* ── Search ───────────────────────────────────── */
-
-.search-results-info {
-    text-align: center;
-    margin: 10px 0;
-    color: var(--text-muted);
-    font-size: 13px;
+.action-tab {
+    padding: 12px 18px;
+    background: transparent;
+    border: 0;
+    color: var(--ink-2);
+    font: 600 14px/1.2 var(--font-sans);
+    cursor: pointer;
+    transition: color 120ms;
 }

-/* ── Layouts ──────────────────────────────────── */
-
-.layouts-container {
-    padding: 16px;
+.action-tab:hover {
+    color: var(--ink);
 }

-.layouts-container h4 {
-    margin: 0 0 12px 0;
-    font-size: 13px;
-    font-weight: 600;
-    text-transform: uppercase;
-    letter-spacing: 0.04em;
-    color: var(--text-secondary);
+.action-tab.is-active {
+    color: var(--ink);
+    box-shadow: inset 0 -2px 0 var(--accent);
 }

-.layout-item {
-    padding: 12px 14px;
-    margin: 4px 0;
-    background: var(--surface-1);
+.action-card-body {
+    padding: 16px 20px;
+}
+
+.action-pane[hidden] {
+    display: none;
+}
+
+.action-card-body input[type=text] {
+    width: 100%;
+    height: 40px;
+    padding: 0 14px;
+    border: 1px solid var(--line-2);
    border-radius: var(--radius-sm);
-    border: 1px solid var(--border);
+    background: var(--paper-2);
+    font: 14px/1.2 var(--font-sans);
+}
+
+.action-card-body input[type=text]:focus {
+    outline: none;
+    border-color: var(--accent);
+    box-shadow: 0 0 0 4px color-mix(in oklch, var(--accent) 14%, transparent);
+}
+
+.action-hint {
+    margin: 8px 0 0;
+    font-size: 11.5px;
+    color: var(--ink-3);
+}
+
+.tjing-search-row {
    display: flex;
-    justify-content: space-between;
+    gap: 10px;
    align-items: center;
-    transition: border-color var(--transition), box-shadow var(--transition);
 }

-.layout-item:hover {
-    border-color: var(--accent-border);
-    box-shadow: var(--shadow-sm);
+.tjing-search-row input[type=text] {
+    flex: 1;
 }

-.layout-name {
-    font-weight: 600;
-    color: var(--text-primary);
-    font-size: 14px;
-}
+/* ── Buttons ──────────────────────────────────────── */

-.layout-par {
-    color: var(--accent);
-    font-weight: 700;
-    font-size: 14px;
-    font-variant-numeric: tabular-nums;
+/* .btn-primary is defined in shared.css — no override needed here */
+
+.btn-pill {
+    padding: 6px 12px;
+    background: var(--accent);
+    color: #fff;
+    border: 0;
+    border-radius: 999px;
+    font: 600 12.5px/1 var(--font-sans);
+    cursor: pointer;
+    height: 28px;
    white-space: nowrap;
 }

+.btn-pill:disabled {
+    opacity: .6;
+    cursor: default;
+}
+
+/* ── Results bar ─────────────────────────────────── */
+
+.results-bar {
+    display: flex;
+    justify-content: space-between;
+    align-items: center;
+    padding: 12px 4px;
+    font-variant-numeric: tabular-nums;
+}
+
+.results-count {
+    color: var(--ink-2);
+    font-size: 13px;
+}
+
+.results-count strong {
+    color: var(--ink);
+    font-weight: 600;
+}
+
+.results-link {
+    color: var(--accent);
+    text-decoration: none;
+    font: 500 13.5px/1.2 var(--font-sans);
+}
+
+/* ── Course grid ─────────────────────────────────── */
+
+.course-grid {
+    background: var(--paper);
+    border: 1px solid var(--line-2);
+    border-radius: var(--radius);
+    box-shadow: var(--shadow-card);
+    overflow: hidden;
+}
+
+.course-row {
+    display: grid;
+    grid-template-columns: minmax(280px, 2fr) minmax(140px, 1fr) minmax(140px, 0.9fr) 72px;
+    align-items: center;
+    gap: 14px;
+    padding: 14px 20px;
+    border-bottom: 1px solid var(--line-2);
+    cursor: pointer;
+    transition: background 120ms;
+}
+
+.course-row:hover {
+    background: var(--paper-2);
+}
+
+.course-row.row-open {
+    background: var(--paper-2);
+    box-shadow: inset 3px 0 0 var(--accent);
+}
+
+.course-row[hidden],
+.expanded-content[hidden] {
+    display: none;
+}
+
+.course-row.row-open .icon-chev i {
+    transform: rotate(180deg);
+}
+
+.course-row--header {
+    height: 48px;
+    padding: 0 20px;
+    background: var(--paper-2);
+    border-bottom: 1px solid var(--line);
+    cursor: default;
+}
+
+.course-row--header:hover {
+    background: var(--paper-2);
+}
+
+.course-header-cell {
+    font-size: 11px;
+    font-weight: 600;
+    text-transform: uppercase;
+    letter-spacing: 0.08em;
+    color: var(--ink-3);
+}
+
+.course-cell {
+    display: flex;
+    flex-direction: column;
+    gap: 2px;
+}
+
+.course-name {
+    font-weight: 600;
+    font-size: 14px;
+    letter-spacing: -0.005em;
+    color: var(--ink);
+    overflow: hidden;
+    text-overflow: ellipsis;
+    white-space: nowrap;
+    display: block;
+}
+
+.course-row:hover .course-name,
+.course-row.row-open .course-name {
+    color: var(--accent);
+}
+
+.course-meta {
+    font-size: 11px;
+    color: var(--ink-3);
+}
+
+.course-city {
+    font-size: 14px;
+    color: var(--ink);
+}
+
+.course-updated {
+    font-family: var(--font-mono);
+    font-feature-settings: "tnum", "zero";
+    font-size: 12.5px;
+    color: var(--ink-3);
+}
+
+.course-actions {
+    display: flex;
+    gap: 4px;
+    justify-content: flex-end;
+}
+
+/* ── Expanded layout panel ───────────────────────── */
+
+.expanded-content {
+    display: none;
+}
+
+.expanded-content.is-open {
+    display: block;
+    animation: expandIn .2s ease;
+}
+
+.expanded-cell {
+    padding: 22px 28px 28px;
+    background: color-mix(in oklch, var(--accent) 4%, var(--paper-2));
+    border-bottom: 1px solid var(--line-2);
+}
+
+/* ── Layout list ─────────────────────────────────── */
+
+.layouts-header {
+    display: flex;
+    justify-content: space-between;
+    align-items: baseline;
+    margin-bottom: 12px;
+}
+
+.layouts-kicker {
+    font: 600 11px/1 var(--font-sans);
+    letter-spacing: 0.08em;
+    text-transform: uppercase;
+    color: var(--ink-3);
+}
+
+.layouts-count {
+    font-size: 12.5px;
+    color: var(--ink-3);
+}
+
+.layout-list {
+    list-style: none;
+    padding: 0;
+    margin: 0;
+    display: flex;
+    flex-direction: column;
+    gap: 8px;
+}
+
+.layout-card {
+    display: flex;
+    justify-content: space-between;
+    align-items: baseline;
+    gap: 14px;
+    padding: 12px 18px;
+    background: var(--paper);
+    border: 1px solid var(--line-2);
+    border-radius: var(--radius-sm);
+    transition: border-color 120ms, box-shadow 120ms;
+}
+
+.layout-card:hover {
+    border-color: var(--line);
+    box-shadow: var(--shadow-card);
+}
+
+.layout-card--inactive {
+    background: transparent;
+    border-style: dashed;
+    opacity: 0.65;
+}
+
+.layout-card--inactive .layout-name {
+    color: var(--ink-2);
+    font-weight: 500;
+}
+
+.layout-info {
+    display: flex;
+    align-items: baseline;
+    gap: 14px;
+    min-width: 0;
+}
+
+.layout-name {
+    font: 600 13.5px/1.2 var(--font-sans);
+    color: var(--ink);
+}
+
+.layout-last-played {
+    font: 12.5px/1 var(--font-mono);
+    color: var(--ink-3);
+}
+
+.layout-never-played {
+    font-size: 12.5px;
+    color: var(--down);
+}
+
+.layout-chips {
+    display: flex;
+    gap: 14px;
+    align-items: center;
+}
+
+.chip {
+    display: inline-flex;
+    align-items: center;
+    padding: 4px 10px;
+    border-radius: 8px;
+    font: 600 12.5px/1 var(--font-mono);
+    font-variant-numeric: tabular-nums;
+}
+
+.chip-par {
+    color: var(--accent);
+    background: color-mix(in oklch, var(--accent) 8%, transparent);
+}
+
+.chip-rating--green {
+    color: var(--rating-tier-high);
+    background: color-mix(in oklch, var(--rating-tier-high) 10%, transparent);
+}
+
+.chip-rating--amber {
+    color: var(--rating-tier-mid);
+    background: color-mix(in oklch, var(--rating-tier-mid) 10%, transparent);
+}
+
+.chip-rating--orange {
+    color: var(--rating-tier-low);
+    background: color-mix(in oklch, var(--rating-tier-low) 10%, transparent);
+}
+
+/* ── Inactive layouts collapsible ────────────────── */
+
+.inactive-layouts {
+    margin-top: 14px;
+    background: var(--paper-2);
+    border: 1px solid var(--line-2);
+    border-radius: var(--radius-sm);
+    overflow: hidden;
+}
+
+.inactive-toggle {
+    display: flex;
+    justify-content: space-between;
+    align-items: center;
+    width: 100%;
+    padding: 12px 16px;
+    background: transparent;
+    border: 0;
+    font: 500 13.5px/1.2 var(--font-sans);
+    color: var(--ink-2);
+    cursor: pointer;
+}
+
+.inactive-toggle .icon-chev {
+    transition: transform 180ms;
+    display: inline-block;
+}
+
+.inactive-toggle.is-open .icon-chev {
+    transform: rotate(180deg);
+}
+
+.inactive-layouts-body {
+    padding: 0 12px 12px;
+}
+
+.inactive-layouts-body[hidden] {
+    display: none;
+}
+
+
+/* ── Tjing results ───────────────────────────────── */
+
+#tjing-results {
+    margin-top: 12px;
+}
+
+.tjing-result {
+    display: flex;
+    justify-content: space-between;
+    align-items: center;
+    gap: 12px;
+    padding: 12px 14px;
+    border-bottom: 1px solid var(--line-2);
+}
+
+.tjing-result:last-child {
+    border-bottom: 0;
+}
+
+.tjing-result-info {
+    display: flex;
+    flex-direction: column;
+    gap: 2px;
+    min-width: 0;
+}
+
+.tjing-result-name {
+    font: 600 14px/1.3 var(--font-sans);
+    color: var(--ink);
+}
+
+.tjing-result-address {
+    font-size: 12.5px;
+    color: var(--ink-3);
+}
+
+.tjing-error {
+    color: var(--down);
+    font-size: 13px;
+    padding: 12px 0;
+}
+
+/* ── No-layouts message ──────────────────────────── */
+
 .no-layouts {
    text-align: center;
-    color: var(--text-muted);
+    color: var(--ink-3);
    font-style: italic;
    padding: 24px;
    font-size: 13px;
 }

-/* ── Inactive Layouts Accordion ───────────────── */
+/* ── Loading placeholder ─────────────────────────── */

-.inactive-layouts-accordion {
-    margin-top: 16px;
-    border: 1px solid var(--border);
-    border-radius: var(--radius-md);
-    background: var(--surface-2);
-    overflow: hidden;
-}
-
-.accordion-header {
-    padding: 12px 16px;
-    cursor: pointer;
-    user-select: none;
-    display: flex;
-    justify-content: space-between;
-    align-items: center;
-    background: var(--surface-3);
-    transition: background var(--transition);
-}
-
-.accordion-header:hover {
-    background: var(--border);
-}
-
-.accordion-header-text {
-    font-weight: 600;
-    color: var(--text-secondary);
+.loading {
+    text-align: center;
+    color: var(--ink-3);
    font-size: 13px;
+    padding: 24px;
 }
-
-.accordion-icon {
-    transition: transform 0.3s ease;
-    color: var(--text-muted);
-    font-size: 12px;
-}
-
-.accordion-icon.expanded {
-    transform: rotate(180deg);
-}
-
-.accordion-content {
-    max-height: 0;
-    overflow: hidden;
-    transition: max-height 0.3s ease-out;
-    padding: 0 12px;
-}
-
-.accordion-content.expanded {
-    max-height: 2000px;
-    padding: 12px;
-    transition: max-height 0.5s ease-in;
-}
-
-.layout-item.inactive {
-    opacity: 0.6;
-    border-style: dashed;
-}
-
@@ -64,13 +64,11 @@
        display: none;
    }

-    /* Hide search result info text on courses (mobile has section-head) */
-    #search-results-info { display: none; }
-
    /* ── Container ──────────────────────────────────── */

    .container {
-        padding: 10px 12px 80px;
+        --m-container-pad-bottom: 80px;
+        padding: 10px 12px var(--m-container-pad-bottom);
        gap: 12px;
    }

@@ -598,7 +596,7 @@
        /* Negative margin to break out of container padding */
        margin-left: -12px;
        margin-right: -12px;
-        margin-bottom: -80px;
+        margin-bottom: calc(-1 * var(--m-container-pad-bottom));
    }

    .mobile-add-bar input {
@@ -94,100 +94,6 @@
    box-shadow: var(--shadow-lg);
 }

-/* ── Tooltips ─────────────────────────────────── */
-
-.std-dev-tooltip {
-    position: absolute;
-    background: var(--navy-900);
-    color: var(--text-inverse);
-    padding: 6px 10px;
-    border-radius: var(--radius-sm);
-    font-size: 12px;
-    font-family: var(--font-mono);
-    pointer-events: none;
-    z-index: 10000;
-    display: none;
-    white-space: nowrap;
-    box-shadow: var(--shadow-lg);
-    font-weight: 400;
-}
-
-/* ── Debug Icon ───────────────────────────────── */
-
-.debug-icon:hover {
-    opacity: 1 !important;
-    color: var(--accent) !important;
-}
-
-/* ── Debug Modal ──────────────────────────────── */
-
-.debug-modal {
-    position: fixed;
-    top: 0;
-    left: 0;
-    width: 100%;
-    height: 100%;
-    background: rgba(15, 23, 42, 0.5);
-    backdrop-filter: blur(4px);
-    z-index: 10001;
-    display: none;
-    justify-content: center;
-    align-items: center;
-}
-
-.debug-content {
-    background: var(--surface-1);
-    border-radius: var(--radius-lg);
-    padding: 24px;
-    max-width: 640px;
-    width: 90%;
-    max-height: 80vh;
-    overflow-y: auto;
-    box-shadow: var(--shadow-overlay);
-    position: relative;
-}
-
-.debug-header {
-    font-weight: 700;
-    font-size: 16px;
-    margin-bottom: 16px;
-    color: var(--text-primary);
-    padding-bottom: 12px;
-    border-bottom: 1px solid var(--border);
-}
-
-.debug-log {
-    font-family: var(--font-mono);
-    background: var(--surface-2);
-    border: 1px solid var(--border);
-    border-radius: var(--radius-md);
-    padding: 16px;
-    font-size: 12px;
-    line-height: 1.6;
-    white-space: pre-line;
-    color: var(--text-primary);
-}
-
-.debug-close {
-    position: absolute;
-    top: 12px;
-    right: 16px;
-    font-size: 22px;
-    color: var(--text-muted);
-    cursor: pointer;
-    background: none;
-    border: none;
-    padding: 4px;
-    border-radius: var(--radius-sm);
-    transition: color var(--transition), background var(--transition);
-    line-height: 1;
-}
-
-.debug-close:hover {
-    color: var(--text-primary);
-    background: var(--surface-3);
-}
-
 /* ── Add Player Modal ─────────────────────────── */

 .modal {
@@ -34,6 +34,11 @@
    --font-sans: 'Plus Jakarta Sans', system-ui, sans-serif;
    --font-mono: 'JetBrains Mono', ui-monospace, monospace;

+    /* ── Rating tier tokens ───────────────────────── */
+    --rating-tier-high:   oklch(0.55 0.15 150);
+    --rating-tier-mid:    oklch(0.55 0.12 100);
+    --rating-tier-low:    oklch(0.55 0.10 50);
+
    /* legacy token aliases — remove as components migrate */
    --surface-0: var(--bg);
    --surface-1: var(--paper);
@@ -1,33 +1,97 @@
-function toggleAccordion(accordionId) {
-    const content = document.getElementById(accordionId);
-    const icon = document.getElementById(`${accordionId}-icon`);
+// ── Tab switching ──────────────────────────────────
+function initCourseTabs() {
+    const tabs = document.querySelectorAll('.action-tab');
+    tabs.forEach(function(tab) {
+        tab.addEventListener('click', function() {
+            tabs.forEach(function(t) {
+                t.classList.remove('is-active');
+                t.setAttribute('aria-selected', 'false');
+            });
+            tab.classList.add('is-active');
+            tab.setAttribute('aria-selected', 'true');

-    if (content.classList.contains('expanded')) {
-        content.classList.remove('expanded');
-        icon.classList.remove('expanded');
-    } else {
-        content.classList.add('expanded');
-        icon.classList.add('expanded');
-    }
+            document.querySelectorAll('.action-pane').forEach(function(pane) {
+                pane.hidden = true;
+                pane.classList.remove('is-active');
+            });
+
+            const targetId = 'tab-pane-' + tab.dataset.tab;
+            const pane = document.getElementById(targetId);
+            if (pane) {
+                pane.hidden = false;
+                pane.classList.add('is-active');
+            }
+        });
+    });
 }

+// ── Live filter ────────────────────────────────────
+function initCourseLiveFilter() {
+    const input = document.getElementById('course-filter-input');
+    if (!input) return;
+
+    input.addEventListener('input', function() {
+        const q = input.value.toLowerCase().trim();
+        const rows = document.querySelectorAll('.course-row');
+        let visible = 0;
+
+        rows.forEach(function(row) {
+            const name = row.dataset.courseName || '';
+            const city = row.dataset.courseCity || '';
+            const match = !q || name.includes(q) || city.includes(q);
+
+            row.hidden = !match;
+
+            // Keep the expanded content sibling in sync
+            const next = row.nextElementSibling;
+            if (next && next.classList.contains('expanded-content')) {
+                next.hidden = !match;
+            }
+
+            if (match) visible++;
+        });
+
+        const visibleEl = document.getElementById('visible-count');
+        if (visibleEl) visibleEl.textContent = visible;
+    });
+}
+
+// ── Count display ──────────────────────────────────
+function initCourseCounts() {
+    const grid = document.querySelector('.course-grid');
+    const total = grid ? parseInt(grid.dataset.totalCount || '0', 10) : 0;
+    const rows = document.querySelectorAll('.course-row');
+    let visible = 0;
+    rows.forEach(function(r) { if (!r.hidden) visible++; });
+
+    const totalEl = document.getElementById('total-count');
+    const visibleEl = document.getElementById('visible-count');
+    if (totalEl) totalEl.textContent = total;
+    if (visibleEl) visibleEl.textContent = visible || total;
+}
+
+// ── Course row expand/collapse ─────────────────────
 function toggleCourseLayouts(courseId) {
-    const layoutsRow = document.getElementById(`layouts-${courseId}`);
-    const layoutsContainer = document.getElementById(`layouts-container-${courseId}`);
+    const row = document.querySelector('.course-row[data-course-id="' + courseId + '"]');
+    const content = document.getElementById('course-layouts-' + courseId);
+    if (!row || !content) return;

-    if (layoutsRow.style.display === 'table-row') {
-        layoutsRow.style.display = 'none';
-        return;
+    const isOpen = content.classList.contains('is-open');
+
+    if (isOpen) {
+        content.classList.remove('is-open');
+        row.classList.remove('row-open');
+    } else {
+        content.classList.add('is-open');
+        row.classList.add('row-open');
+
+        // Lazy-load layouts on first expand
+        const cell = content.querySelector('.expanded-cell');
+        if (cell && cell.dataset.loaded !== 'true') {
+            cell.dataset.loaded = 'true';
+            htmx.ajax('GET', '/partials/course-layouts/' + courseId, { target: cell, swap: 'innerHTML' });
+        }
    }
-
-    layoutsRow.style.display = 'table-row';
-
-    if (layoutsContainer.dataset.loaded === 'true') {
-        return;
-    }
-
-    htmx.ajax('GET', `/partials/course-layouts/${courseId}`, {target: `#layouts-container-${courseId}`, swap: 'innerHTML'});
-    layoutsContainer.dataset.loaded = 'true';
 }

 // ── Mobile course card toggle ──────────────────────
@@ -68,10 +132,24 @@ function toggleMobileCourseLayouts(courseId) {
    }
 }

+// ── Inactive layouts toggle ────────────────────────
+function toggleInactiveLayouts(btn) {
+    const body = btn.nextElementSibling;
+    if (!body) return;
+
+    const isOpen = btn.classList.contains('is-open');
+    btn.classList.toggle('is-open', !isOpen);
+    btn.setAttribute('aria-expanded', String(!isOpen));
+    body.hidden = isOpen;
+}
+
+// ── Scrape courses ─────────────────────────────────
 async function scrapeCourses() {
    const btn = document.getElementById('scrape-courses-btn');
-    btn.disabled = true;
-    btn.textContent = 'Scraping...';
+    if (btn) {
+        btn.disabled = true;
+        btn.textContent = 'Scraping...';
+    }

    try {
        const response = await fetch('/api/scrape-courses', { method: 'POST' });
@@ -79,7 +157,7 @@ async function scrapeCourses() {

        if (data.success) {
            alert(data.message);
-            htmx.ajax('GET', '/partials/course-table', '#courses-table');
+            htmx.trigger(document.body, 'refresh');
        } else {
            alert('Failed to scrape courses');
        }
@@ -87,31 +165,33 @@ async function scrapeCourses() {
        console.error('Error scraping courses:', error);
        alert('Error scraping courses');
    } finally {
-        btn.disabled = false;
-        btn.textContent = 'Scrape Courses';
+        if (btn) {
+            btn.disabled = false;
+            btn.textContent = 'Scrape Courses';
+        }
    }
 }

-async function scrapeLayouts(courseId, courseName) {
-    const icon = document.querySelector(`#row-${courseId} .refresh-icon`);
-    icon.classList.add('spinning');
+// ── Scrape layouts for a course ────────────────────
+async function scrapeLayouts(courseId, btn) {
+    if (btn) btn.classList.add('spinning');

    try {
-        const response = await fetch(`/api/scrape-layouts/${courseId}`, { method: 'POST' });
+        const response = await fetch('/api/scrape-layouts/' + courseId, { method: 'POST' });
        const data = await response.json();

        if (response.status === 409) {
            alert(data.message || 'Scrape already in progress for this course. Please wait.');
        } else if (data.success) {
-            const layoutsContainer = document.getElementById(`layouts-container-${courseId}`);
-            layoutsContainer.dataset.loaded = 'false';
-
-            const layoutsRow = document.getElementById(`layouts-${courseId}`);
-            if (layoutsRow.style.display === 'table-row') {
-                htmx.ajax('GET', `/partials/course-layouts/${courseId}`, {target: `#layouts-container-${courseId}`, swap: 'innerHTML'});
-                layoutsContainer.dataset.loaded = 'true';
+            // Reload expanded layout content if currently open
+            const content = document.getElementById('course-layouts-' + courseId);
+            if (content && content.classList.contains('is-open')) {
+                const cell = content.querySelector('.expanded-cell');
+                if (cell) {
+                    cell.dataset.loaded = 'true';
+                    htmx.ajax('GET', '/partials/course-layouts/' + courseId, { target: cell, swap: 'innerHTML' });
+                }
            }
-
            alert(data.message);
        } else {
            alert('Failed to scrape layouts');
@@ -120,6 +200,141 @@ async function scrapeLayouts(courseId, courseName) {
        console.error('Error scraping layouts:', error);
        alert('Error scraping layouts');
    } finally {
-        icon.classList.remove('spinning');
+        if (btn) btn.classList.remove('spinning');
    }
 }
+
+// ── Tjing search ───────────────────────────────────
+async function searchTjing() {
+    const input = document.getElementById('tjing-search-input');
+    const btn = document.getElementById('tjing-search-btn');
+    const container = document.getElementById('tjing-results');
+    if (!input || !container) return;
+
+    const q = input.value.trim();
+    if (!q) return;
+
+    btn.disabled = true;
+
+    // Clear previous results safely
+    while (container.firstChild) {
+        container.removeChild(container.firstChild);
+    }
+
+    try {
+        const response = await fetch('/api/tjing/search?q=' + encodeURIComponent(q));
+        let data;
+        try {
+            data = await response.json();
+        } catch (e) {
+            const errP = document.createElement('p');
+            errP.className = 'tjing-error';
+            errP.textContent = 'Invalid response from server.';
+            container.appendChild(errP);
+            return;
+        }
+
+        if (!response.ok || data.error) {
+            const errP2 = document.createElement('p');
+            errP2.className = 'tjing-error';
+            errP2.textContent = 'Error: ' + (data.error || 'Search failed');
+            container.appendChild(errP2);
+            return;
+        }
+
+        const results = data.results || [];
+        if (results.length === 0) {
+            const noResults = document.createElement('p');
+            noResults.className = 'tjing-error';
+            noResults.textContent = 'No courses found on Tjing.';
+            container.appendChild(noResults);
+            return;
+        }
+
+        results.forEach(function(course) {
+            const item = document.createElement('div');
+            item.className = 'tjing-result';
+
+            const info = document.createElement('div');
+            info.className = 'tjing-result-info';
+
+            const nameSpan = document.createElement('span');
+            nameSpan.className = 'tjing-result-name';
+            nameSpan.textContent = course.name || '';
+
+            const addrSpan = document.createElement('span');
+            addrSpan.className = 'tjing-result-address';
+            addrSpan.textContent = course.address || '';
+
+            info.appendChild(nameSpan);
+            info.appendChild(addrSpan);
+
+            const importBtn = document.createElement('button');
+            importBtn.className = 'btn-pill';
+            importBtn.textContent = 'Import';
+            (function(id, b) {
+                b.addEventListener('click', function() { importFromTjing(id, b); });
+            })(course.id, importBtn);
+
+            item.appendChild(info);
+            item.appendChild(importBtn);
+            container.appendChild(item);
+        });
+    } catch (error) {
+        console.error('Error searching Tjing:', error);
+        const errFallback = document.createElement('p');
+        errFallback.className = 'tjing-error';
+        errFallback.textContent = 'Failed to search Tjing.';
+        container.appendChild(errFallback);
+    } finally {
+        btn.disabled = false;
+    }
+}
+
+// ── Tjing import ───────────────────────────────────
+async function importFromTjing(tjingId, btn) {
+    btn.disabled = true;
+    btn.textContent = 'Importing…';
+
+    try {
+        const response = await fetch('/api/tjing/import/' + encodeURIComponent(tjingId), { method: 'POST' });
+        let data;
+        try {
+            data = await response.json();
+        } catch (e) {
+            btn.textContent = 'Error';
+            btn.disabled = false;
+            return;
+        }
+
+        if (!response.ok || data.error) {
+            btn.textContent = 'Error: ' + (data.error || 'Import failed');
+            btn.disabled = false;
+            return;
+        }
+
+        btn.textContent = 'Imported ✓';
+        // Trigger table reload
+        htmx.trigger(document.body, 'refresh');
+    } catch (error) {
+        console.error('Error importing from Tjing:', error);
+        btn.textContent = 'Failed';
+        btn.disabled = false;
+    }
+}
+
+// ── Init ───────────────────────────────────────────
+function initAll() {
+    initCourseTabs();
+    initCourseLiveFilter();
+    initCourseCounts();
+}
+
+document.addEventListener('DOMContentLoaded', initAll);
+
+document.addEventListener('htmx:afterSwap', function(evt) {
+    if (evt.detail && evt.detail.target && evt.detail.target.id === 'course-table-region') {
+        initCourseLiveFilter();
+        initCourseCounts();
+    }
+});
@@ -1,4 +1,3 @@
-const cachedDebugInfo = {};
 let pendingPlayerData = null;
 let openPdgaNumber = null;

@@ -57,7 +56,6 @@ function setupAfterTableSwap() {
    document.body.addEventListener('htmx:afterSwap', function(event) {
        const target = event.detail.target;
        if (target.id === 'ratings-table') {
-            initRatingsTooltips();
            initChartsIn(target);
            return;
        }
@@ -67,31 +65,6 @@ function setupAfterTableSwap() {
    });
 }

-function initRatingsTooltips() {
-    document.querySelectorAll('.predicted-value').forEach(span => {
-        const pdgaNumber = span.dataset.pdga;
-        const stdDev = span.dataset.stddev;
-        const tooltip = document.getElementById(`tooltip-stddev-${pdgaNumber}`);
-
-        if (stdDev && tooltip) {
-            setupTooltip(span, tooltip, () => `Standard Deviation: \u00b1${stdDev}`);
-        }
-    });
-
-    document.querySelectorAll('.rating-value').forEach(span => {
-        const pdgaNumber = span.dataset.pdga;
-        const rating = parseInt(span.dataset.rating);
-        const stdDev = parseInt(span.dataset.stddev);
-        const tooltip = document.getElementById(`tooltip-rating-${pdgaNumber}`);
-
-        if (rating && stdDev && tooltip) {
-            const minRating = rating - stdDev;
-            const maxRating = rating + stdDev;
-            setupTooltip(span, tooltip, () => `Rating Range: ${minRating} - ${maxRating} (\u00b1${stdDev})`);
-        }
-    });
-}
-
 function togglePlayerHistory(pdgaNumber) {
    const historyRow = document.getElementById('history-' + pdgaNumber);
    const contentDiv = document.getElementById('history-content-' + pdgaNumber);
@@ -191,16 +164,6 @@ async function refreshPlayer(pdgaNumber) {
            if (ratingValue) {
                ratingValue.textContent = data.player.rating || 'N/A';
                ratingValue.dataset.rating = data.player.rating || '';
-
-                const stdDev = parseInt(ratingValue.dataset.stddev);
-                const rating = parseInt(data.player.rating);
-                const tooltip = document.getElementById(`tooltip-rating-${pdgaNumber}`);
-
-                if (rating && stdDev && tooltip) {
-                    const minRating = rating - stdDev;
-                    const maxRating = rating + stdDev;
-                    replaceWithTooltip(ratingValue, tooltip, () => `Rating Range: ${minRating} - ${maxRating} (\u00b1${stdDev})`);
-                }
            }

            const deltaMonthPill = ratingCell ? ratingCell.querySelector('.delta-pill') : null;
@@ -222,21 +185,12 @@ async function refreshRoundHistory(pdgaNumber) {
        }

        if (data.success) {
-            if (data.debugLog) {
-                cachedDebugInfo[pdgaNumber] = data.debugLog;
-            }
-
            const predictedCell = document.getElementById(`predicted-${pdgaNumber}`);
            if (predictedCell) {
                const predictedValue = predictedCell.querySelector('.predicted-value');
                if (predictedValue) {
                    predictedValue.textContent = data.predictedRating || 'N/A';
                    predictedValue.dataset.stddev = data.stdDev || '';
-
-                    const tooltip = document.getElementById(`tooltip-stddev-${pdgaNumber}`);
-                    if (data.stdDev && tooltip) {
-                        replaceWithTooltip(predictedValue, tooltip, () => `Standard Deviation: \u00b1${data.stdDev}`);
-                    }
                }
            }

@@ -245,16 +199,6 @@ async function refreshRoundHistory(pdgaNumber) {
            const ratingValue = ratingCell ? ratingCell.querySelector('.rating-value') : null;
            if (ratingValue && data.stdDev) {
                ratingValue.dataset.stddev = data.stdDev;
-
-                const rating = parseInt(ratingValue.dataset.rating);
-                const stdDev = parseInt(data.stdDev);
-                const ratingTooltip = document.getElementById(`tooltip-rating-${pdgaNumber}`);
-
-                if (rating && stdDev && ratingTooltip) {
-                    const minRating = rating - stdDev;
-                    const maxRating = rating + stdDev;
-                    replaceWithTooltip(ratingValue, ratingTooltip, () => `Rating Range: ${minRating} - ${maxRating} (\u00b1${stdDev})`);
-                }
            }
        }
    } catch (error) {
@@ -285,43 +229,6 @@ async function refreshRatingHistory(pdgaNumber) {
    }
 }

-async function showDebugInfo(pdgaNumber) {
-    const modal = document.getElementById('debug-modal');
-    const header = document.getElementById('debug-header');
-    const log = document.getElementById('debug-log');
-
-    const playerNameElement = document.querySelector(`#row-${pdgaNumber} .player-name a`);
-    const playerName = playerNameElement ? playerNameElement.textContent : `PDGA #${pdgaNumber}`;
-
-    header.textContent = `Prediction Calculation Details - ${playerName}`;
-    log.textContent = 'Loading calculation details...';
-    modal.style.display = 'flex';
-
-    try {
-        if (cachedDebugInfo[pdgaNumber]) {
-            log.textContent = cachedDebugInfo[pdgaNumber].join('\n');
-            return;
-        }
-
-        const response = await fetch(`/api/refresh-round-history/${pdgaNumber}`, { method: 'POST' });
-        const data = await response.json();
-
-        if (data.success && data.debugLog) {
-            cachedDebugInfo[pdgaNumber] = data.debugLog;
-            log.textContent = data.debugLog.join('\n');
-        } else {
-            log.textContent = 'No debug information available. Try refreshing the prediction first.';
-        }
-    } catch (error) {
-        console.error('Error fetching debug info:', error);
-        log.textContent = 'Error loading debug information. Please try again.';
-    }
-}
-
-function closeDebugModal(event) {
-    document.getElementById('debug-modal').style.display = 'none';
-}
-
 async function searchAndAddPlayer(event) {
    if (event) event.preventDefault();
    const input = document.getElementById('pdga-number-input');
@@ -763,7 +670,6 @@ async function refreshHistoryThenCalculate(pdgaNumber) {
            return;
        }

-        if (data.debugLog) cachedDebugInfo[pdgaNumber] = data.debugLog;
        const predictedCell = document.getElementById('predicted-' + pdgaNumber);
        if (predictedCell) {
            const predictedValue = predictedCell.querySelector('.predicted-value');
@@ -34,6 +34,8 @@ function initializeDatabase() {
          const hasLastRoundUpdate = columns.some(col => col.name === 'last_round_update');
          const hasPredictedRating = columns.some(col => col.name === 'predicted_rating');
          const hasStdDev = columns.some(col => col.name === 'std_dev');
+          const hasExcludedRoundsCount = columns.some(col => col.name === 'excluded_rounds_count');
+          const hasCutoffRating = columns.some(col => col.name === 'cutoff_rating');

          if (!hasLastRoundUpdate) {
            logger.info('Adding last_round_update column to players table...');
@@ -58,6 +60,22 @@ function initializeDatabase() {
              else logger.info('Successfully added std_dev column');
            });
          }
+
+          if (!hasExcludedRoundsCount) {
+            logger.info('Adding excluded_rounds_count column to players table...');
+            db.run(`ALTER TABLE players ADD COLUMN excluded_rounds_count INTEGER DEFAULT NULL`, (err) => {
+              if (err) logger.error('Error adding excluded_rounds_count column:', err.message);
+              else logger.info('Successfully added excluded_rounds_count column');
+            });
+          }
+
+          if (!hasCutoffRating) {
+            logger.info('Adding cutoff_rating column to players table...');
+            db.run(`ALTER TABLE players ADD COLUMN cutoff_rating INTEGER DEFAULT NULL`, (err) => {
+              if (err) logger.error('Error adding cutoff_rating column:', err.message);
+              else logger.info('Successfully added cutoff_rating column');
+            });
+          }
        });
      });

@@ -8,8 +8,14 @@ function saveCourseToDB(courseData) {
       ON CONFLICT(link) DO UPDATE SET name = excluded.name, city = excluded.city, last_updated = datetime('now')`,
      [courseData.name, courseData.link, courseData.city],
      function(err) {
-        if (err) reject(err);
-        else resolve(this.lastID);
+        if (err) return reject(err);
+        // node-sqlite3 leaves lastID = 0 when ON CONFLICT triggers an UPDATE.
+        // Fall back to a SELECT to get the real id in that case.
+        if (this.lastID !== 0) return resolve(this.lastID);
+        db.get('SELECT id FROM courses WHERE link = ?', [courseData.link], (err2, row) => {
+          if (err2) reject(err2);
+          else resolve(row ? row.id : 0);
+        });
      }
    );
  });
@@ -70,10 +76,33 @@ function updateLayoutRating(courseId, layoutName, par, meanRating, ratingCount,
  });
 }

+function getOrCreateLayout(courseId, name, par) {
+  return new Promise((resolve, reject) => {
+    db.get(
+      'SELECT id FROM layouts WHERE course_id = ? AND name = ? AND par = ?',
+      [courseId, name, par],
+      (err, row) => {
+        if (err) return reject(err);
+        if (row) return resolve(row.id);
+
+        db.run(
+          'INSERT INTO layouts (course_id, name, par) VALUES (?, ?, ?)',
+          [courseId, name, par],
+          function(err) {
+            if (err) reject(err);
+            else resolve(this.lastID);
+          }
+        );
+      }
+    );
+  });
+}
+
 module.exports = {
  saveCourseToDB,
  getAllCoursesFromDB,
  saveLayoutToDB,
  getLayoutsForCourse,
+  getOrCreateLayout,
  updateLayoutRating
 };
@@ -1,5 +1,6 @@
 const { db } = require('../db');
 const { parseDate } = require('../services/rating-calculator');
+const logger = require('../logger');

 function getPlayerFromDB(pdgaNumber) {
  return new Promise((resolve, reject) => {
@@ -16,6 +17,7 @@ function getPlayerFromDB(pdgaNumber) {

 function savePlayerToDB(playerData) {
  return new Promise((resolve, reject) => {
+    logger.info({ pdgaNumber: playerData.pdgaNumber, name: playerData.name, currentRating: playerData.rating, ratingChange: playerData.ratingChange, resetColumns: ['predicted_rating', 'std_dev', 'excluded_rounds_count', 'cutoff_rating', 'last_round_update'] }, 'INSERT OR REPLACE on players — derived columns will be reset to NULL');
    db.run(
      `INSERT OR REPLACE INTO players (pdga_number, name, current_rating, rating_change, last_updated)
       VALUES (?, ?, ?, ?, datetime('now'))`,
@@ -172,16 +174,28 @@ function saveRoundHistoryToDB(pdgaNumber, roundData, isIncremental = false) {
  });
 }

-function savePredictedRatingToDB(pdgaNumber, predictedRating, stdDev = null) {
+function savePredictedRatingToDB(pdgaNumber, predictedRating, stdDev = null, excludedRoundsCount = null, cutoffRating = null, callsite = 'unknown') {
  return new Promise((resolve, reject) => {
-    db.run(
-      'UPDATE players SET predicted_rating = ?, std_dev = ? WHERE pdga_number = ?',
-      [predictedRating, stdDev, pdgaNumber],
-      function(err) {
-        if (err) reject(err);
-        else resolve();
-      }
-    );
+    db.get('SELECT predicted_rating AS oldValue FROM players WHERE pdga_number = ?', [pdgaNumber], (selectErr, row) => {
+      if (selectErr) return reject(selectErr);
+      const oldValue = row ? row.oldValue : null;
+
+      db.run(
+        'UPDATE players SET predicted_rating = ?, std_dev = ?, excluded_rounds_count = ?, cutoff_rating = ? WHERE pdga_number = ?',
+        [predictedRating, stdDev, excludedRoundsCount, cutoffRating, pdgaNumber],
+        function(err) {
+          if (err) return reject(err);
+
+          const logData = { pdgaNumber, oldValue, newValue: predictedRating, callsite };
+          if (predictedRating === 0 && oldValue != null && oldValue > 0) {
+            logger.warn(logData, 'predicted rating overwritten with 0');
+          } else {
+            logger.info(logData, 'predicted rating saved');
+          }
+          resolve();
+        }
+      );
+    });
  });
 }

@@ -1,7 +1,8 @@
 const express = require('express');
 const router = express.Router();
 const { db } = require('../db');
-const { getAllCoursesFromDB, getLayoutsForCourse, updateLayoutRating } = require('../models/course');
+const { getAllCoursesFromDB, getLayoutsForCourse, updateLayoutRating, saveCourseToDB, getOrCreateLayout } = require('../models/course');
+const { searchTjingCourses, getTjingCourse } = require('../scrapers/tjing');
 const { launchBrowser } = require('../scrapers/browser');
 const { layoutEventCache, scrapeCourseDirectory, scrapeCourseLayouts, scrapeEventResults } = require('../scrapers/course-puppeteer');
 const logger = require('../logger');
@@ -11,19 +12,30 @@ const activeScrapes = new Map();

 router.get('/partials/course-table', async (req, res) => {
  try {
-    const allCourses = await getAllCoursesFromDB();
-    const query = req.query.q || '';
-    let courses = allCourses;
+    const oneYearAgo = new Date();
+    oneYearAgo.setDate(oneYearAgo.getDate() - 365);
+    const oneYearAgoStr = oneYearAgo.toISOString().slice(0, 10);

-    if (query) {
-      const q = query.toLowerCase();
-      courses = allCourses.filter(c =>
-        c.name.toLowerCase().includes(q) || c.city.toLowerCase().includes(q)
+    const allCourses = await new Promise((resolve, reject) => {
+      db.all(
+        `SELECT c.*,
+                COUNT(l.id) AS layoutCount,
+                SUM(CASE WHEN l.last_played >= ? THEN 1 ELSE 0 END) AS activeLayoutCount
+         FROM courses c
+         LEFT JOIN layouts l ON l.course_id = c.id
+         GROUP BY c.id
+         ORDER BY c.name ASC`,
+        [oneYearAgoStr],
+        (err, rows) => {
+          if (err) reject(err);
+          else resolve(rows);
+        }
      );
-    }
+    });

-    res.render('../partials/course-table', { courses, query, total: allCourses.length });
+    res.render('../partials/course-table', { courses: allCourses });
  } catch (error) {
+    logger.error({ err: error }, 'Error loading course table');
    res.status(500).send('<p>Error loading courses. Please try again.</p>');
  }
 });
@@ -34,7 +46,7 @@ router.get('/partials/course-layouts/:courseId', async (req, res) => {
    const layouts = await getLayoutsForCourse(courseId);
    res.render('../partials/course-layouts', { layouts, courseId });
  } catch (error) {
-    logger.error('Error loading course layouts:', error.message);
+    logger.error({ err: error }, 'Error loading course layouts');
    res.status(500).send('<div class="no-layouts">Error loading layouts</div>');
  }
 });
@@ -44,7 +56,7 @@ router.get('/api/courses', async (req, res) => {
    const courses = await getAllCoursesFromDB();
    res.json(courses);
  } catch (error) {
-    logger.error('Error fetching courses:', error.message);
+    logger.error({ err: error }, 'Error fetching courses');
    res.status(500).json({ error: 'Failed to fetch courses' });
  }
 });
@@ -55,7 +67,7 @@ router.get('/api/layouts/:courseId', async (req, res) => {
    const layouts = await getLayoutsForCourse(courseId);
    res.json(layouts);
  } catch (error) {
-    logger.error('Error fetching layouts:', error.message);
+    logger.error({ err: error }, 'Error fetching layouts');
    res.status(500).json({ error: 'Failed to fetch layouts' });
  }
 });
@@ -214,7 +226,7 @@ router.post('/api/scrape-layouts/:courseId', async (req, res) => {
              savedCount++;
            }
          } catch (err) {
-            logger.error(`  Error updating layout ${layoutDataResult.name}:`, err.message);
+            logger.error({ err, layoutName: layoutDataResult.name }, 'Error updating layout');
          }
        }
      }
@@ -348,7 +360,7 @@ router.post('/api/scrape-event-results/:courseId', async (req, res) => {
            savedCount++;
          }
        } catch (err) {
-          logger.error(`  Error updating layout ${ld.name}:`, err.message);
+          logger.error({ err, layoutName: ld.name }, 'Error updating layout');
        }
      }
    }
@@ -369,4 +381,53 @@ router.post('/api/scrape-event-results/:courseId', async (req, res) => {
  }
 });

+// Search Tjing for courses
+router.get('/api/tjing/search', async (req, res) => {
+  const { q } = req.query;
+  if (!q || q.trim().length === 0) {
+    return res.json({ results: [] });
+  }
+
+  const result = await searchTjingCourses(q.trim());
+  if (result.error) {
+    logger.warn({ q, err: result.error }, 'Tjing search error');
+    return res.status(502).json({ error: result.error });
+  }
+
+  res.json({ results: result.data });
+});
+
+// Import a course from Tjing
+router.post('/api/tjing/import/:tjingId', async (req, res) => {
+  const { tjingId } = req.params;
+
+  const result = await getTjingCourse(tjingId);
+  if (result.error) {
+    logger.warn({ tjingId, err: result.error }, 'Tjing import error');
+    return res.status(502).json({ error: result.error });
+  }
+
+  const courseData = result.data;
+
+  try {
+    const courseId = await saveCourseToDB({
+      name: courseData.name,
+      link: `https://tjing.se/courses/${tjingId}`,
+      city: courseData.address || ''
+    });
+
+    let layoutsImported = 0;
+    for (const layout of courseData.layouts) {
+      await getOrCreateLayout(courseId, layout.name, layout.par);
+      layoutsImported++;
+    }
+
+    logger.info({ courseId, name: courseData.name, layoutsImported }, 'Imported course from Tjing');
+    res.json({ courseId, layoutsImported });
+  } catch (err) {
+    logger.error({ err, tjingId }, 'Failed to save Tjing course to DB');
+    res.status(500).json({ error: 'Failed to save course to database' });
+  }
+});
+
 module.exports = router;
@@ -7,7 +7,7 @@ const { getOfficialRatingHistory, getOptimizedPlayerRounds } = require('../scrap
 const { launchBrowser } = require('../scrapers/browser');
 const { getPlayerDataFromDB, scrapePDGARating, getAllRatingsFromDB, refreshAllPlayersInDB, getPredictedRatingFromDB, formatDisplayDate } = require('../services/player-service');
 const { getTopbarLocals } = require('../services/topbar-service');
-const { calculatePredictedRating } = require('../services/rating-calculator');
+const { calculatePredictedRating, getNextPDGAUpdateDate } = require('../services/rating-calculator');
 const { calculateRequiredAverage } = require('../services/target-rating-calculator');
 const logger = require('../logger');

@@ -19,6 +19,7 @@ router.post('/api/refresh-all', async (req, res, next) => {
    return res.status(409).json({ error: 'Refresh already in progress' });
  }
  refreshInProgress = true;
+  logger.info({ pdgaNumber: 'all' }, 'refresh-all triggered — will invoke savePlayerToDB for all players (INSERT OR REPLACE wipes derived columns)');
  try {
    try {
      await refreshAllPlayersInDB();
@@ -253,7 +254,7 @@ router.post('/api/add-player', async (req, res) => {
 router.post('/api/refresh-player/:pdgaNumber', async (req, res) => {
  try {
    const { pdgaNumber } = req.params;
-    logger.info(`Manually refreshing player data for PDGA ${pdgaNumber}`);
+    logger.info({ pdgaNumber }, 'refresh-player triggered — will invoke savePlayerToDB (INSERT OR REPLACE wipes derived columns)');

    const html = await fetchPlayerDataHTTP(pdgaNumber);
    const playerData = parsePlayerData(html, pdgaNumber);
@@ -349,48 +350,45 @@ router.post('/api/refresh-round-history/:pdgaNumber', async (req, res) => {

    const isIncremental = !!sinceDate;

-    logger.info(`${isIncremental ? 'Incrementally updating' : 'Fully refreshing'} round history for PDGA ${pdgaNumber}${sinceDate ? ` since ${sinceDate.toDateString()}` : ''}`);
+    logger.info({ pdgaNumber, lastRoundUpdate, isIncremental }, 'refresh-round-history started');

    browser = await launchBrowser();
-    
+
    let officialHistory;
    try {
      officialHistory = await getOfficialRatingHistory(browser, pdgaNumber);
      if (officialHistory.length > 0) {
        await saveRatingHistoryToDB(pdgaNumber, officialHistory);
      }
+      logger.info({ pdgaNumber, step: 'official_history_scrape', success: officialHistory.length > 0, count: officialHistory.length }, 'official history scrape completed');
    } catch (historyError) {
-      logger.error('Failed to fetch official history:', historyError.message);
+      logger.warn({ pdgaNumber, step: 'official_history_scrape', success: false, error: historyError.message }, 'official history scrape failed');
      officialHistory = [];
    }
-    
+
    let allRounds = [];
    try {
-      logger.info(`Using optimized approach: /details + new tournaments only for PDGA ${pdgaNumber}...`);
      allRounds = await getOptimizedPlayerRounds(browser, pdgaNumber);
-      
+
      if (allRounds.length > 0) {
        const roundsForDB = allRounds.map(round => ({
          rating: round.rating,
          date: round.date,
          competition: round.competition
        }));
-        
+
        await saveRoundHistoryToDB(pdgaNumber, roundsForDB, false);
-        logger.info(`✓ Saved ${allRounds.length} rounds using optimized approach`);
-        
        await updateLastRoundUpdateDate(pdgaNumber);
-      } else {
-        logger.info('ℹ No rounds found');
      }
+      logger.info({ pdgaNumber, step: 'rounds_scrape_save', success: allRounds.length > 0, count: allRounds.length, lastRoundUpdateTouched: allRounds.length > 0 }, 'rounds scrape and save completed');
    } catch (detailsError) {
-      logger.error('Failed to fetch rounds using optimized approach:', detailsError.message);
+      logger.warn({ pdgaNumber, step: 'rounds_scrape_save', success: false, error: detailsError.message }, 'rounds scrape and save failed');
      allRounds = [];
    }
-    
+
    await browser.close();
    browser = null;
-    
+
    const dbRounds = await getRoundHistoryFromDB(pdgaNumber);
    const roundsForPrediction = dbRounds.map(round => ({
      rating: round.rating,
@@ -398,9 +396,10 @@ router.post('/api/refresh-round-history/:pdgaNumber', async (req, res) => {
      competition: round.competition_name
    }));

-    const result = calculatePredictedRating(roundsForPrediction);
+    const result = calculatePredictedRating(roundsForPrediction, { pdgaNumber, callsite: 'refresh-round-history' });
+    logger.info({ pdgaNumber, step: 'predicted_calc', rating: result.rating, stdDev: result.stdDev, excludedRoundsCount: result.excludedRoundsCount }, 'predicted rating calculation step completed');

-    await savePredictedRatingToDB(pdgaNumber, result.rating, result.stdDev);
+    await savePredictedRatingToDB(pdgaNumber, result.rating, result.stdDev, result.excludedRoundsCount, result.cutoffRating, 'refresh-round-history');

    const officialCount = allRounds.filter(r => r.source === 'official').length;
    const newCount = allRounds.filter(r => r.source === 'new').length;
@@ -409,7 +408,8 @@ router.post('/api/refresh-round-history/:pdgaNumber', async (req, res) => {
      success: true,
      predictedRating: result.rating,
      stdDev: result.stdDev,
-      debugLog: result.debugLog,
+      excludedRoundsCount: result.excludedRoundsCount,
+      cutoffRating: result.cutoffRating,
      totalRounds: roundsForPrediction.length,
      officialRounds: officialCount,
      newRounds: newCount,
@@ -417,10 +417,8 @@ router.post('/api/refresh-round-history/:pdgaNumber', async (req, res) => {
      message: `Used /details (${officialCount} rounds) + new tournaments (${newCount} rounds)`
    });
  } catch (error) {
-    logger.error(`=== Error refreshing round history for PDGA ${pdgaNumber} ===`);
-    logger.error('Error type:', error.constructor.name);
-    logger.error('Error message:', error.message);
-    
+    logger.error({ pdgaNumber, step: 'unknown_or_orchestration', errorType: error.constructor.name, errorMessage: error.message }, 'refresh-round-history failed');
+
    if (browser) {
      try {
        await browser.close();
@@ -428,14 +426,14 @@ router.post('/api/refresh-round-history/:pdgaNumber', async (req, res) => {
        logger.error('Error closing browser:', closeError.message);
      }
    }
-    
-    res.status(500).json({ 
+
+    res.status(500).json({
      error: 'Failed to refresh round history',
      details: error.message,
      errorType: error.constructor.name,
      timestamp: new Date().toISOString(),
-      suggestion: error.message.includes('socket hang up') ? 
-        'Rate limited by PDGA - try again in a few minutes.' : 
+      suggestion: error.message.includes('socket hang up') ?
+        'Rate limited by PDGA - try again in a few minutes.' :
        error.message.includes('timeout') ?
        'PDGA pages are loading slowly - try again later.' :
        'Tournament scraping failed - check server logs for details'
@@ -443,6 +441,46 @@ router.post('/api/refresh-round-history/:pdgaNumber', async (req, res) => {
  }
 });

+router.get('/api/admin/player-state/:pdgaNumber', async (req, res) => {
+  const { pdgaNumber } = req.params;
+  try {
+    const player = await getPlayerFromDB(pdgaNumber);
+    if (!player) {
+      return res.status(404).json({ error: 'Player not found', pdgaNumber: parseInt(pdgaNumber) });
+    }
+    const rounds = await getRoundHistoryFromDB(pdgaNumber);
+    const cutoff = getNextPDGAUpdateDate();
+    const twelveMonthsAgo = new Date(cutoff); twelveMonthsAgo.setFullYear(cutoff.getFullYear() - 1);
+    const twentyFourMonthsAgo = new Date(cutoff); twentyFourMonthsAgo.setFullYear(cutoff.getFullYear() - 2);
+
+    const roundsInLast12mo = rounds.filter(r => new Date(r.date) >= twelveMonthsAgo).length;
+    const roundsInLast24mo = rounds.filter(r => new Date(r.date) >= twentyFourMonthsAgo).length;
+    const dates = rounds.map(r => r.date).sort();
+
+    res.json({
+      pdgaNumber: player.pdga_number,
+      name: player.name,
+      currentRating: player.current_rating,
+      ratingChange: player.rating_change,
+      predictedRating: player.predicted_rating,
+      stdDev: player.std_dev,
+      excludedRoundsCount: player.excluded_rounds_count,
+      cutoffRating: player.cutoff_rating,
+      lastUpdated: player.last_updated,
+      lastRoundUpdate: player.last_round_update,
+      cutoffDate: cutoff.toISOString(),
+      roundCount: rounds.length,
+      roundsInLast12mo,
+      roundsInLast24mo,
+      oldestRound: dates[0] ?? null,
+      newestRound: dates[dates.length - 1] ?? null
+    });
+  } catch (err) {
+    logger.error({ err, pdgaNumber }, 'admin player-state endpoint failed');
+    res.status(500).json({ error: 'Failed to fetch player state', details: err.message });
+  }
+});
+
 router.post('/api/calculate-target-rating/:pdgaNumber', async (req, res) => {
  const { pdgaNumber } = req.params;
  const pdgaNum = parseInt(pdgaNumber, 10);
@@ -482,7 +520,7 @@ router.post('/api/calculate-target-rating/:pdgaNumber', async (req, res) => {
      competition: r.competition_name
    }));

-    const result = calculateRequiredAverage(roundRatings, target, numRounds);
+    const result = calculateRequiredAverage(roundRatings, target, numRounds, pdgaNum);

    logger.info(`Target rating calc for PDGA ${pdgaNum}: target=${target} rounds=${numRounds} -> avg=${result.requiredAverage}`);

@@ -0,0 +1,111 @@
+const logger = require('../logger');
+
+const TJING_API = 'https://api.tjing.se/graphql';
+const FETCH_TIMEOUT_MS = 8000;
+
+async function tjingFetch(query) {
+  const controller = new AbortController();
+  const timer = setTimeout(() => controller.abort(), FETCH_TIMEOUT_MS);
+
+  let response;
+  try {
+    response = await fetch(TJING_API, {
+      method: 'POST',
+      headers: { 'Content-Type': 'application/json' },
+      body: JSON.stringify({ query }),
+      signal: controller.signal
+    });
+  } catch (err) {
+    if (err.name === 'AbortError') {
+      return { error: 'Tjing API request timed out' };
+    }
+    return { error: `Network error: ${err.message}` };
+  } finally {
+    clearTimeout(timer);
+  }
+
+  if (!response.ok) {
+    return { error: `Tjing API returned ${response.status}` };
+  }
+
+  let data;
+  try {
+    data = await response.json();
+  } catch (err) {
+    return { error: 'Invalid JSON from Tjing API' };
+  }
+
+  if (data.errors && data.errors.length > 0) {
+    return { error: data.errors[0].message };
+  }
+
+  return { data: data.data };
+}
+
+async function searchTjingCourses(searchTerm) {
+  const query = `{
+    courses(first: 10, filter: { search: "${searchTerm.replace(/"/g, '\\"')}" }) {
+      id
+      name
+      address
+      type
+    }
+  }`;
+
+  const result = await tjingFetch(query);
+  if (result.error) return result;
+
+  return { data: result.data.courses || [] };
+}
+
+async function getTjingCourse(courseId) {
+  const query = `{
+    course(courseId: "${courseId.replace(/"/g, '\\"')}") {
+      id
+      name
+      address
+      layouts {
+        id
+        name
+        published
+        latestVersion {
+          holes {
+            number
+            par
+          }
+        }
+      }
+    }
+  }`;
+
+  const result = await tjingFetch(query);
+  if (result.error) return result;
+
+  const course = result.data.course;
+  if (!course) {
+    return { error: 'Course not found' };
+  }
+
+  // Calculate total par per layout from holes
+  const layouts = (course.layouts || [])
+    .filter(l => l.published && l.latestVersion && l.latestVersion.holes.length > 0)
+    .map(l => ({
+      name: l.name,
+      par: l.latestVersion.holes.reduce((sum, h) => sum + h.par, 0),
+      holes: l.latestVersion.holes.length
+    }));
+
+  return {
+    data: {
+      name: course.name,
+      address: course.address,
+      tjingId: course.id,
+      layouts
+    }
+  };
+}
+
+module.exports = {
+  searchTjingCourses,
+  getTjingCourse
+};
@@ -36,18 +36,32 @@ async function getPlayerDataFromDB(pdgaNumber, { includeMonthlyHistory = true }
  try {
    const cachedPlayer = await getPlayerFromDB(pdgaNumber);
    if (cachedPlayer) {
-      logger.debug(`Loading PDGA ${pdgaNumber} from DB (source of truth)`);
+      logger.debug({ pdgaNumber }, 'Loading player from DB (source of truth)');

      let predictedRating = cachedPlayer.predicted_rating;
      let stdDev = cachedPlayer.std_dev;
+      let excludedRoundsCount = cachedPlayer.excluded_rounds_count;
+      let cutoffRating = cachedPlayer.cutoff_rating;
+      let recomputeAttempted = false;
      if (!predictedRating || predictedRating === 0) {
+        recomputeAttempted = true;
+        logger.debug({ pdgaNumber, dbValue: cachedPlayer.predicted_rating }, 'lazy recompute triggered for predicted_rating');
        predictedRating = await getPredictedRatingFromDB(pdgaNumber);
        const updatedPlayer = await getPlayerFromDB(pdgaNumber);
        stdDev = updatedPlayer?.std_dev;
+        excludedRoundsCount = updatedPlayer?.excluded_rounds_count;
+        cutoffRating = updatedPlayer?.cutoff_rating;
+        if (!predictedRating || predictedRating === 0) {
+          logger.info({ pdgaNumber, recomputedValue: predictedRating }, 'lazy recompute did not yield a positive predicted rating');
+        }
      }

      const rating = cachedPlayer.current_rating;
      const rawRatingChange = cachedPlayer.rating_change;
+      // Only warn about ≤0 if it wasn't already explained by a failed recompute (which has its own log)
+      if (!recomputeAttempted && predictedRating != null && predictedRating <= 0) {
+        logger.warn({ pdgaNumber, dbValue: predictedRating }, 'predicted rating present but <= 0 in DB without recompute — rendered as empty');
+      }
      const resolvedPredicted = predictedRating > 0 ? predictedRating : null;
      const resolvedStdDev = stdDev > 0 ? stdDev : null;

@@ -65,6 +79,8 @@ async function getPlayerDataFromDB(pdgaNumber, { includeMonthlyHistory = true }
        ratingChange,
        predictedRating: resolvedPredicted,
        stdDev: resolvedStdDev,
+        excludedRoundsCount: (excludedRoundsCount != null && excludedRoundsCount >= 0) ? excludedRoundsCount : null,
+        cutoffRating: (cutoffRating != null && cutoffRating > 0) ? cutoffRating : null,
        lastMonthRating,
        // gap between next predicted update and current rating (null when either is missing)
        deltaPredicted: (resolvedPredicted != null && rating != null) ? resolvedPredicted - rating : null,
@@ -134,22 +150,28 @@ async function scrapePDGARating(pdgaNumber, retries = 3) {
 async function getPredictedRatingFromDB(pdgaNumber) {
  try {
    const roundHistory = await getRoundHistoryFromDB(pdgaNumber);
-    if (roundHistory.length > 0) {
-      logger.debug(`Using ${roundHistory.length} cached rounds for PDGA ${pdgaNumber} prediction`);
-
-      const roundRatings = roundHistory.map(round => ({
-        rating: round.rating,
-        date: new Date(round.date),
-        competition: round.competition_name || 'Unknown'
-      }));
-
-      const result = calculatePredictedRating(roundRatings);
-
-      await savePredictedRatingToDB(pdgaNumber, result.rating, result.stdDev);
-
-      return result.rating;
+    if (roundHistory.length === 0) {
+      logger.info({ pdgaNumber, reason: 'no_round_history_in_db' }, 'predicted recompute returning 0 — no round history available');
+      return 0;
    }
-    return 0;
+
+    logger.debug({ pdgaNumber, cachedRounds: roundHistory.length }, 'Using cached rounds for prediction');
+
+    const roundRatings = roundHistory.map(round => ({
+      rating: round.rating,
+      date: new Date(round.date),
+      competition: round.competition_name || 'Unknown'
+    }));
+
+    const result = calculatePredictedRating(roundRatings, { pdgaNumber, callsite: 'getPredictedRatingFromDB' });
+
+    if (result.rating === 0) {
+      logger.warn({ pdgaNumber, roundsInDb: roundHistory.length }, 'predicted recompute returned 0 despite having rounds in DB');
+    }
+
+    await savePredictedRatingToDB(pdgaNumber, result.rating, result.stdDev, result.excludedRoundsCount, result.cutoffRating, 'lazy-recompute');
+
+    return result.rating;
  } catch (err) {
    logger.error(`Error getting predicted rating from DB for ${pdgaNumber}:`, err.message);
    return 0;
@@ -229,6 +251,8 @@ async function getAllRatingsFromDB(progressCallback = null) {
          ratingChange: errorRatingChange,
          predictedRating: null,
          stdDev: null,
+          excludedRoundsCount: null,
+          cutoffRating: null,
          lastMonthRating: (errorRating != null && errorRatingChange != null) ? errorRating - errorRatingChange : null,
          deltaPredicted: null,
          monthlyHistory: [],
@@ -1,3 +1,5 @@
+const logger = require('../logger');
+
 function parseDate(dateStr) {
  const multiDayMatch = dateStr.match(/^(\d{1,2})(-([A-Za-z]{3}))?(\s+to\s+)(\d{1,2})-([A-Za-z]{3})-(\d{4})$/);
  if (multiDayMatch) {
@@ -79,15 +81,19 @@ function calculateStandardDeviation(ratings) {
  return Math.sqrt(variance);
 }

-function calculatePredictedRating(roundRatings) {
+function calculatePredictedRating(roundRatings, context = {}) {
+  const pdgaNumber = context.pdgaNumber ?? null;
+  const callsite = context.callsite ?? 'unknown';
  const debugLog = [];
  debugLog.push('=== PDGA RATING CALCULATION (Following Official Rules) ===');

  if (!roundRatings || roundRatings.length === 0) {
    debugLog.push('❌ No rounds provided for prediction');
-    return { rating: 0, debugLog };
+    logger.warn({ pdgaNumber, callsite, reason: 'no_rounds' }, 'predicted rating computed as 0');
+    return { rating: 0, debugLog, excludedRoundsCount: null, cutoffRating: null };
  }

+  logger.info({ pdgaNumber, callsite, inputRounds: roundRatings.length }, 'predicted rating calculation started');
  debugLog.push(`📊 Starting with ${roundRatings.length} total rounds`);

  const nextUpdateDate = getNextPDGAUpdateDate();
@@ -100,7 +106,8 @@ function calculatePredictedRating(roundRatings) {

  if (allSortedRounds.length === 0) {
    debugLog.push('❌ No valid rounds after filtering for update date');
-    return { rating: 0, debugLog };
+    logger.warn({ pdgaNumber, callsite, reason: 'no_rounds_after_date_filter', inputRounds: roundRatings.length, nextUpdateDate: nextUpdateDate.toISOString() }, 'predicted rating computed as 0');
+    return { rating: 0, debugLog, excludedRoundsCount: null, cutoffRating: null };
  }

  debugLog.push(`📊 After update date filter: ${allSortedRounds.length} rounds`);
@@ -117,8 +124,9 @@ function calculatePredictedRating(roundRatings) {
  debugLog.push('🗓️  12-MONTH FILTERING:');
  debugLog.push(`✅ Rounds in last 12 months: ${eligibleRounds.length}`);

+  let twentyFourMonthsBeforeUpdate = null;
  if (eligibleRounds.length < 8) {
-    const twentyFourMonthsBeforeUpdate = new Date(nextUpdateDate);
+    twentyFourMonthsBeforeUpdate = new Date(nextUpdateDate);
    twentyFourMonthsBeforeUpdate.setFullYear(twentyFourMonthsBeforeUpdate.getFullYear() - 2);

    eligibleRounds = allSortedRounds.filter(r => r.date >= twentyFourMonthsBeforeUpdate);
@@ -127,7 +135,8 @@ function calculatePredictedRating(roundRatings) {

  if (eligibleRounds.length === 0) {
    debugLog.push('❌ No eligible rounds found');
-    return { rating: 0, debugLog };
+    logger.warn({ pdgaNumber, callsite, reason: 'no_eligible_rounds', afterDateFilter: allSortedRounds.length, twelveMonthCutoff: twelveMonthsBeforeUpdate.toISOString(), twentyFourMonthCutoff: twentyFourMonthsBeforeUpdate ? twentyFourMonthsBeforeUpdate.toISOString() : null }, 'predicted rating computed as 0');
+    return { rating: 0, debugLog, excludedRoundsCount: null, cutoffRating: null };
  }

  debugLog.push(`📈 ELIGIBLE ROUNDS: ${eligibleRounds.length}`);
@@ -137,6 +146,8 @@ function calculatePredictedRating(roundRatings) {

  let workingRounds = [...eligibleRounds];
  let workingRatings = workingRounds.map(r => r.rating);
+  let excludedRoundsCount = 0;
+  let cutoffRating = null;

  if (workingRatings.length >= 7) {
    debugLog.push('🔍 OUTLIER EXCLUSION (≥7 rounds available):');
@@ -160,6 +171,9 @@ function calculatePredictedRating(roundRatings) {
    const stdDevOutliers = workingRatings.filter(rating => rating < stdDevCutoff);
    const hundredPointOutliers = workingRatings.filter(rating => rating < hundredPointCutoff && rating >= stdDevCutoff);

+    excludedRoundsCount = stdDevOutliers.length + hundredPointOutliers.length;
+    cutoffRating = Math.round(Math.max(stdDevCutoff, hundredPointCutoff));
+
    if (stdDevOutliers.length > 0) {
      debugLog.push(`   ❌ 2.5σ outliers removed: ${stdDevOutliers.length} rounds`);
      stdDevOutliers.forEach(rating => {
@@ -188,6 +202,8 @@ function calculatePredictedRating(roundRatings) {
      debugLog.push(`   ✅ Using ${filteredRatings.length} rounds after outlier removal`);
    } else {
      debugLog.push(`   ⚠️  Too few rounds after outlier removal (${filteredRatings.length}), keeping all rounds`);
+      excludedRoundsCount = 0;
+      cutoffRating = null;
    }
  } else {
    debugLog.push(`⏭️  OUTLIER EXCLUSION SKIPPED (only ${workingRatings.length} rounds, need ≥7)`);
@@ -228,7 +244,20 @@ function calculatePredictedRating(roundRatings) {
  debugLog.push(`   Final Rating: ${finalRating}`);
  debugLog.push('=== END PDGA CALCULATION ===');

-  return { rating: finalRating, stdDev: Math.round(stdDev), debugLog };
+  logger.info({
+    pdgaNumber,
+    callsite,
+    finalRating,
+    inputRounds: roundRatings.length,
+    afterDateFilter: allSortedRounds.length,
+    eligibleRounds: eligibleRounds.length,
+    outlierExclusionApplied: workingRatings.length >= 7,
+    doubleWeightingApplied: workingRatings.length >= 9,
+    excludedRoundsCount,
+    cutoffRating
+  }, 'predicted rating calculated');
+
+  return { rating: finalRating, stdDev: Math.round(stdDev), debugLog, excludedRoundsCount, cutoffRating };
 }

 module.exports = { parseDate, getNextPDGAUpdateDate, calculatePredictedRating, calculateStandardDeviation };
@@ -1,14 +1,14 @@
 const { calculatePredictedRating, getNextPDGAUpdateDate } = require('./rating-calculator');
 const logger = require('../logger');

-function calculateRequiredAverage(roundRatings, targetRating, numRounds) {
+function calculateRequiredAverage(roundRatings, targetRating, numRounds, pdgaNumber) {
  if (!Array.isArray(roundRatings) || roundRatings.length === 0) {
    const err = new Error('No round history');
    err.code = 'NO_ROUNDS';
    throw err;
  }

-  const currentPredicted = calculatePredictedRating(roundRatings).rating;
+  const currentPredicted = calculatePredictedRating(roundRatings, { pdgaNumber, callsite: 'target-rating-calculator' }).rating;
  const nextUpdate = getNextPDGAUpdateDate();
  const syntheticDate = new Date(nextUpdate.getTime() - 24 * 60 * 60 * 1000);

@@ -1,43 +1,32 @@
 <% var body = `
-        <div class="card-section">
-            <h3>Find Courses</h3>
-            <div class="card-section-form">
-                <input
-                    type="text"
-                    class="input"
-                    id="course-search"
-                    name="q"
-                    placeholder="Search courses by name or city..."
-                    hx-get="/partials/course-table"
-                    hx-trigger="input changed delay:300ms, search"
-                    hx-target="#courses-table"
-                    style="width: 340px;"
-                />
-                <button class="btn" onclick="scrapeCourses()" id="scrape-courses-btn">
-                    <i class="fas fa-sync-alt"></i> Scrape Courses
-                </button>
-            </div>
+<main class="page-courses">
+  <section class="action-card">
+    <div class="action-card-tabs" role="tablist">
+      <button class="action-tab is-active" role="tab" aria-selected="true" data-tab="find" id="tab-find">Find courses</button>
+      <button class="action-tab" role="tab" aria-selected="false" data-tab="tjing" id="tab-tjing">Import from Tjing</button>
+    </div>
+    <div class="action-card-body">
+      <div class="action-pane is-active" id="tab-pane-find" role="tabpanel" aria-labelledby="tab-find">
+        <input type="text" id="course-filter-input" placeholder="Find a course…" autocomplete="off">
+        <p class="action-hint">Filters the list below as you type.</p>
+      </div>
+      <div class="action-pane" id="tab-pane-tjing" role="tabpanel" aria-labelledby="tab-tjing" hidden>
+        <div class="tjing-search-row">
+          <input type="text" id="tjing-search-input" placeholder="Search Tjing courses…" autocomplete="off">
+          <button id="tjing-search-btn" class="btn-primary" onclick="searchTjing()">Search Tjing</button>
        </div>
+        <p class="action-hint">Find and import Swedish courses from tjing.se.</p>
+        <div id="tjing-results"></div>
+      </div>
+    </div>
+  </section>

-        <!-- Mobile tab pill (visible only on mobile via CSS) -->
-        <div class="m-tab-pill">
-            <button class="m-tab-pill__btn m-tab-pill__btn--active" type="button">Find courses</button>
-            <button class="m-tab-pill__btn m-tab-pill__btn--disabled" type="button" disabled>Import from Tjing</button>
-        </div>
+  <div class="results-bar">
+    <span class="results-count">Showing <strong id="visible-count">0</strong> of <strong id="total-count">0</strong> courses</span>
+  </div>

-        <!-- Mobile search input (hidden on desktop, shown on mobile via CSS) -->
-        <input
-            type="text"
-            class="m-search-input"
-            id="course-search-mobile"
-            name="q"
-            placeholder="Search courses by name or city..."
-            hx-get="/partials/course-table"
-            hx-trigger="input changed delay:300ms, search"
-            hx-target="#courses-table"
-        />
-
-        <div id="courses-table" hx-get="/partials/course-table" hx-trigger="load"></div>
+  <div id="course-table-region" hx-get="/partials/course-table" hx-trigger="load, refresh from:body" hx-swap="innerHTML"></div>
+</main>
 `; %>

 <%- include('../partials/layout', {
@@ -46,4 +35,4 @@
    cssFiles: ['courses.css'],
    jsFiles: ['courses.js'],
    body: body
-}) %>
+}) %>
@@ -95,15 +95,6 @@
 `; %>

 <% var modals = `
-    <!-- Debug Modal -->
-    <div id="debug-modal" class="debug-modal" onclick="closeDebugModal(event)">
-        <div class="debug-content" onclick="event.stopPropagation()">
-            <button class="debug-close" onclick="closeDebugModal()">&times;</button>
-            <div class="debug-header" id="debug-header">Prediction Calculation Details</div>
-            <div class="debug-log" id="debug-log">Loading...</div>
-        </div>
-    </div>
-
    <!-- Add Player Confirmation Modal -->
    <div id="add-player-modal" class="modal" onclick="closeAddPlayerModal(event)">
        <div class="modal-content" onclick="event.stopPropagation()">
@@ -1,71 +1,67 @@
-<% if (layouts.length === 0) { %>
-    <div class="no-layouts">No layouts found. Click the refresh icon to scrape layouts.</div>
+<% if (!layouts || layouts.length === 0) { %>
+  <div class="no-layouts">No layouts found. Click the refresh button to scrape layouts.</div>
 <% } else {
-    var oneYearAgo = new Date();
-    oneYearAgo.setDate(oneYearAgo.getDate() - 365);
-
-    var activeLayouts = [];
-    var inactiveLayouts = [];
-
-    layouts.forEach(function(layout) {
-        if (layout.last_played) {
-            var lastPlayedDate = new Date(layout.last_played);
-            if (lastPlayedDate >= oneYearAgo) {
-                activeLayouts.push(layout);
-            } else {
-                inactiveLayouts.push(layout);
-            }
-        } else {
-            inactiveLayouts.push(layout);
-        }
-    });
+  var oneYearAgo = new Date();
+  oneYearAgo.setDate(oneYearAgo.getDate() - 365);
+  var activeLayouts = [];
+  var inactiveLayouts = [];
+  layouts.forEach(function(l) {
+    if (l.last_played && new Date(l.last_played) >= oneYearAgo) {
+      activeLayouts.push(l);
+    } else {
+      inactiveLayouts.push(l);
+    }
+  });
+  var RATING_TIER_HIGH = 970;
+  var RATING_TIER_MID  = 940;
+  function ratingTier(r) {
+    if (r == null) return null;
+    if (r >= RATING_TIER_HIGH) return 'green';
+    if (r >= RATING_TIER_MID)  return 'amber';
+    return 'orange';
+  }
 %>
-    <h4>Layouts:</h4>
-
-    <% if (activeLayouts.length > 0) { %>
-        <% activeLayouts.forEach(function(layout) {
-            var ratingDisplay = layout.mean_rating ?
-                '<span style="color: var(--green); font-weight: 700; margin-left: 10px;">Rating: ' + layout.mean_rating + '</span>' : '';
-            var dateDisplay = layout.last_played ?
-                '<span style="color: var(--text-muted); font-size: 12px; margin-left: 10px;">Last played: ' + new Date(layout.last_played).toLocaleDateString('en-US', { year: 'numeric', month: 'short', day: 'numeric' }) + '</span>' : '';
-        %>
-            <div class="layout-item">
-                <div>
-                    <span class="layout-name"><%= layout.name %></span>
-                    <%- dateDisplay %>
-                </div>
-                <span class="layout-par">Par <%= layout.par %><%- ratingDisplay %></span>
-            </div>
-        <% }); %>
-    <% } %>
-
-    <% if (inactiveLayouts.length > 0) { %>
-        <div class="inactive-layouts-accordion">
-            <div class="accordion-header" onclick="toggleAccordion('accordion-<%= courseId %>')">
-                <span class="accordion-header-text">Inactive Layouts (<%= inactiveLayouts.length %>) - Not played in last year</span>
-                <span class="accordion-icon" id="accordion-<%= courseId %>-icon">&#9660;</span>
-            </div>
-            <div class="accordion-content" id="accordion-<%= courseId %>">
-                <% inactiveLayouts.forEach(function(layout) {
-                    var ratingDisplay = layout.mean_rating ?
-                        '<span style="color: var(--green); font-weight: 700; margin-left: 10px;">Rating: ' + layout.mean_rating + '</span>' : '';
-                    var dateDisplay = layout.last_played ?
-                        '<span style="color: var(--text-muted); font-size: 12px; margin-left: 10px;">Last played: ' + new Date(layout.last_played).toLocaleDateString('en-US', { year: 'numeric', month: 'short', day: 'numeric' }) + '</span>' :
-                        '<span style="color: var(--red); font-size: 12px; margin-left: 10px;">Never played</span>';
-                %>
-                    <div class="layout-item inactive">
-                        <div>
-                            <span class="layout-name"><%= layout.name %></span>
-                            <%- dateDisplay %>
-                        </div>
-                        <span class="layout-par">Par <%= layout.par %><%- ratingDisplay %></span>
-                    </div>
-                <% }); %>
-            </div>
-        </div>
-    <% } %>
-
-    <% if (activeLayouts.length === 0 && inactiveLayouts.length === 0) { %>
-        <div class="no-layouts">No layouts found. Click the refresh icon to scrape layouts.</div>
-    <% } %>
-<% } %>
+<div class="layouts-header">
+  <span class="layouts-kicker">LAYOUTS</span>
+  <span class="layouts-count"><%= activeLayouts.length %> active &middot; <%= inactiveLayouts.length %> inactive</span>
+</div>
+<ul class="layout-list">
+  <% activeLayouts.forEach(function(l) { var tier = ratingTier(l.mean_rating); %>
+    <li class="layout-card layout-card--active">
+      <div class="layout-info">
+        <span class="layout-name"><%= l.name %></span>
+        <span class="layout-last-played">Last played: <%= l.last_played %></span>
+      </div>
+      <div class="layout-chips">
+        <span class="chip chip-par">Par <%= l.par %></span>
+        <% if (tier) { %><span class="chip chip-rating chip-rating--<%= tier %>">Rating: <%= Math.round(l.mean_rating) %></span><% } %>
+      </div>
+    </li>
+  <% }); %>
+</ul>
+<% if (inactiveLayouts.length > 0) { %>
+  <div class="inactive-layouts">
+    <button class="inactive-toggle" type="button" onclick="toggleInactiveLayouts(this)" aria-expanded="false">
+      <span>Inactive layouts (<%= inactiveLayouts.length %>) — Not played in last year</span>
+      <i class="icon-chev fas fa-chevron-down"></i>
+    </button>
+    <ul class="layout-list inactive-layouts-body" hidden>
+      <% inactiveLayouts.forEach(function(l) { %>
+        <li class="layout-card layout-card--inactive">
+          <div class="layout-info">
+            <span class="layout-name"><%= l.name %></span>
+            <% if (l.last_played) { %>
+              <span class="layout-last-played">Last played: <%= l.last_played %></span>
+            <% } else { %>
+              <span class="layout-never-played">Never played</span>
+            <% } %>
+          </div>
+          <div class="layout-chips">
+            <span class="chip chip-par">Par <%= l.par %></span>
+          </div>
+        </li>
+      <% }); %>
+    </ul>
+  </div>
+<% } %>
+<% } %>
@@ -1,47 +1,49 @@
-<div id="search-results-info" class="search-results-info">
-    <% if (typeof query !== 'undefined' && query) { %>
-        Showing <%= courses.length %> of <%= total %> courses
-    <% } else { %>
-        Showing all <%= courses.length %> courses
-    <% } %>
-</div>
-
-<% if (courses.length === 0) { %>
-    <p>No courses found. Click "Scrape Courses" to load Swedish courses from PDGA.</p>
+<% if (!courses || courses.length === 0) { %>
+  <p style="text-align: center; color: var(--ink-3); padding: 40px 0;">No courses found. Use "Import from Tjing" or scrape courses from PDGA.</p>
 <% } else { %>
-    <table>
-        <thead>
-            <tr>
-                <th>Course Name</th>
-                <th class="mobile-hide">City</th>
-                <th class="mobile-hide">Last Updated</th>
-                <th>Actions</th>
-            </tr>
-        </thead>
-        <tbody>
-            <% courses.forEach(function(course) {
-                var lastUpdated = new Date(course.last_updated).toLocaleDateString('en-US', { year: 'numeric', month: 'short', day: 'numeric' });
-            %>
-            <tr id="row-<%= course.id %>" class="expandable-row" onclick="toggleCourseLayouts(<%= course.id %>)">
-                <td>
-                    <a href="<%= course.link %>" target="_blank" onclick="event.stopPropagation()"><%= course.name %></a>
-                    <div class="mobile-only" style="font-size: 11px; color: #999; margin-top: 2px;"><%= course.city %></div>
-                </td>
-                <td class="mobile-hide"><%= course.city %></td>
-                <td class="mobile-hide"><%= lastUpdated %></td>
-                <td>
-                    <i class="fas fa-sync-alt refresh-icon" onclick="scrapeLayouts(<%= course.id %>, '<%= course.name.replace(/'/g, "\\'") %>'); event.stopPropagation();" title="Scrape layouts for this course"></i>
-                </td>
-            </tr>
-            <tr id="layouts-<%= course.id %>" class="expanded-content">
-                <td colspan="4">
-                    <div class="layouts-container" id="layouts-container-<%= course.id %>">
-                        <div class="no-layouts">Click to load layouts...</div>
-                    </div>
-                </td>
-            </tr>
-            <% }); %>
-        </tbody>
-    </table>
-    <%- include('course-cards', { courses: courses, query: locals.query, total: locals.total }) %>
+<div class="course-grid" data-total-count="<%= courses.length %>">
+  <div class="course-row course-row--header" role="row">
+    <div class="course-header-cell">Course</div>
+    <div class="course-header-cell">City</div>
+    <div class="course-header-cell">Last updated</div>
+    <div class="course-header-cell"></div>
+  </div>
+  <% courses.forEach(function(course) {
+      var layoutCount = course.layoutCount || 0;
+      var activeLayoutCount = course.activeLayoutCount || 0;
+  %>
+    <div class="course-row expandable-row" data-course-id="<%= course.id %>" data-course-name="<%= (course.name || '').toLowerCase() %>" data-course-city="<%= (course.city || '').toLowerCase() %>" onclick="toggleCourseLayouts(<%= course.id %>)">
+      <div class="course-cell">
+        <span class="course-name"><%= course.name %></span>
+        <span class="course-meta">
+          <% if (layoutCount > 0) { %>
+            <% if (activeLayoutCount !== layoutCount) { %>
+              <%= layoutCount %> layouts &middot; <%= activeLayoutCount %> active
+            <% } else { %>
+              <%= layoutCount %> layouts
+            <% } %>
+          <% } else { %>
+            No layouts
+          <% } %>
+        </span>
+      </div>
+      <div class="course-city"><%= course.city || '—' %></div>
+      <div class="course-updated"><%= course.last_updated ? new Date(course.last_updated).toISOString().slice(0,10) : '—' %></div>
+      <div class="course-actions">
+        <button class="icon-btn refresh-icon" onclick="event.stopPropagation(); scrapeLayouts(<%= course.id %>, this)" title="Refresh layouts" aria-label="Refresh layouts">
+          <i class="fas fa-sync-alt"></i>
+        </button>
+        <button class="icon-btn icon-chev" onclick="event.stopPropagation(); toggleCourseLayouts(<%= course.id %>)" title="Expand row" aria-label="Expand">
+          <i class="fas fa-chevron-down"></i>
+        </button>
+      </div>
+    </div>
+    <div class="expanded-content" id="course-layouts-<%= course.id %>">
+      <div class="expanded-cell">
+        <div class="loading">Loading layouts…</div>
+      </div>
+    </div>
+  <% }); %>
+</div>
+<%- include('course-cards', { courses: courses, total: courses.length }) %>
 <% } %>
@@ -26,8 +26,29 @@ const chartPdgaNumber = hasPlayer ? player.pdgaNumber : pdgaNumber;
        <dt>Gap to predicted</dt>
        <dd><%- include('delta-pill', { value: player.deltaPredicted, extraClass: 'delta-predicted-pill' }) %></dd>
      </div>
+      <% if (player.stdDev != null && player.rating) { %>
+      <div>
+        <dt>Round spread</dt>
+        <dd>±<%= player.stdDev %></dd>
+      </div>
+      <div>
+        <dt>Rating range</dt>
+        <dd><%= player.rating - player.stdDev %>–<%= player.rating + player.stdDev %></dd>
+      </div>
+      <% } %>
+      <% if (player.excludedRoundsCount != null && player.rating) { %>
+      <div>
+        <dt>Excluded rounds</dt>
+        <dd><%= player.excludedRoundsCount %></dd>
+      </div>
+      <% } %>
+      <% if (player.cutoffRating != null && player.rating) { %>
+      <div>
+        <dt>Cutoff rating</dt>
+        <dd><%= player.cutoffRating %></dd>
+      </div>
+      <% } %>
    </dl>
-    <button class="link-btn" onclick="showDebugInfo(<%= player.pdgaNumber %>)" style="margin-top: 4px;">View calculation details →</button>
  </div>
  <% } %>

@@ -114,6 +114,28 @@ function renderSparkline(values, opts) {
                    <dt>Gap to predicted</dt>
                    <dd><span class="delta-pill <%= predCls %> delta-predicted-pill"><span class="delta-glyph"><%= predGlyph %></span><span class="delta-num"><%= predNum %></span></span></dd>
                </div>
+                <% if (player.stdDev != null && player.rating) { %>
+                <div>
+                    <dt>Round spread</dt>
+                    <dd>±<%= player.stdDev %></dd>
+                </div>
+                <div>
+                    <dt>Rating range</dt>
+                    <dd><%= player.rating - player.stdDev %>–<%= player.rating + player.stdDev %></dd>
+                </div>
+                <% } %>
+                <% if (player.excludedRoundsCount != null && player.rating) { %>
+                <div>
+                    <dt>Excluded rounds</dt>
+                    <dd><%= player.excludedRoundsCount %></dd>
+                </div>
+                <% } %>
+                <% if (player.cutoffRating != null && player.rating) { %>
+                <div>
+                    <dt>Cutoff rating</dt>
+                    <dd><%= player.cutoffRating %></dd>
+                </div>
+                <% } %>
            </dl>
        </div>
    </div>
@@ -67,7 +67,6 @@ function renderSparkline(values, opts) {
                    <% } else { %>
                    <span class="rating-pending">Click to load</span>
                    <% } %>
-                    <div class="std-dev-tooltip" id="tooltip-rating-<%= player.pdgaNumber %>"></div>
                </td>
                <td class="col-predicted" id="predicted-<%= player.pdgaNumber %>">
                    <% if (player.predictedRating) { %>
@@ -78,7 +77,6 @@ function renderSparkline(values, opts) {
                    <% } else { %>
                    <span class="rating-pending">—</span>
                    <% } %>
-                    <div class="std-dev-tooltip" id="tooltip-stddev-<%= player.pdgaNumber %>"></div>
                </td>
                <td class="col-actions cell-actions" onclick="event.stopPropagation()">
                    <button class="icon-btn refresh-icon" onclick="refreshPlayerData(<%= player.pdgaNumber %>)" title="Refresh rating + prediction" aria-label="Refresh rating and prediction">
Author	SHA1	Message	Date
Samuel Enocsson	31d80273b8	chore: address review feedback on predicted rating logging (#11 ) - align diagnostic endpoint date windows with getNextPDGAUpdateDate - pass pdgaNumber to initial target-rating calculation for log attribution - avoid double log when lazy-recompute returns 0 - log full error object in admin endpoint catch	2026-05-25 22:02:42 +02:00
Samuel Enocsson	83ba20d428	chore: add diagnostic logging for predicted rating lifecycle (#11 ) Add structured Pino logging across four hot-spots to make the predicted rating data-loss visible in production logs: - rating-calculator.js: log on calculation start, each 0-return early exit (no_rounds / no_rounds_after_date_filter / no_eligible_rounds), and on successful calculation with full algorithm branch metadata. Accepts optional context={pdgaNumber,callsite} to correlate logs. - models/player.js: savePlayerToDB now logs before INSERT OR REPLACE, explicitly naming the derived columns that will be reset to NULL. savePredictedRatingToDB reads the old value first and emits a warn when a positive predicted_rating is overwritten with 0. - services/player-service.js: structured debug/info/warn logs in getPlayerDataFromDB (lazy recompute trigger, failed recompute) and getPredictedRatingFromDB (no round history, 0 despite having rounds). - routes/players.js: refresh-round-history route now logs each step (official_history_scrape, rounds_scrape_save, predicted_calc) with success/count fields; outer catch uses structured error log. refresh-all and refresh-player log their INSERT OR REPLACE trigger. Also adds GET /api/admin/player-state/:pdgaNumber — a read-only diagnostic endpoint returning raw DB columns plus round count and date-window breakdowns. Closes #11	2026-05-25 21:55:53 +02:00
Release Bot	a90f2d0e86	1.4.0 Release / release (push) Successful in 5s Details Build and deploy / build-and-push (push) Successful in 21s Details Build and deploy / deploy (push) Successful in 7s Details	2026-05-25 09:18:47 +00:00
shcizo	f8233960d2	Merge pull request 'feat: show excluded rounds count and cutoff rating in player history (#21 )' (#23 ) from feat/show-excluded-rounds-count-21 into main Release / release (push) Successful in 7s Details	2026-05-25 11:18:35 +02:00
Samuel Enocsson	98a6c6be2e	feat: show cutoff rating threshold in player history accordion (#21 )	2026-05-25 11:12:01 +02:00
Samuel Enocsson	9138299ae0	Merge remote-tracking branch 'origin/main' into feat/show-excluded-rounds-count-21	2026-05-25 11:01:53 +02:00
Release Bot	3275241aa7	1.3.0 Release / release (push) Successful in 5s Details Build and deploy / build-and-push (push) Successful in 25s Details Build and deploy / deploy (push) Successful in 8s Details	2026-05-25 08:52:17 +00:00
shcizo	6faddc6232	Merge pull request 'feat: Courses-redesign + Tjing-import (#8 )' (#22 ) from feat/courses-redesign-tjing-import-8 into main Release / release (push) Successful in 7s Details	2026-05-25 10:52:06 +02:00
Samuel Enocsson	cad14def56	style: align course row typography with Players (#8 )	2026-05-25 10:36:43 +02:00
Samuel Enocsson	75b2360e96	feat: add table header row to Courses matching Players style (#8 )	2026-05-25 10:33:47 +02:00
Samuel Enocsson	2035ae0efc	fix: use FontAwesome icons matching Players page (#8 )	2026-05-25 10:29:34 +02:00
Samuel Enocsson	88396c9220	fix: remove EJS comment inside template literal causing parse error (#8 )	2026-05-25 10:25:59 +02:00
Samuel Enocsson	9cb78c9c98	fix: address code-review findings from pass 1 + 2 (#8 ) - Fix saveCourseToDB returning 0 on conflict by falling back to SELECT - Fix inactive layouts showing 'Never played' when last_played exists - Add .icon-btn.spinning to courses.css for refresh button feedback - Remove duplicate .btn-primary from courses.css (use shared.css version) - Tokenize rating tier colors into --rating-tier-{high,mid,low} CSS vars - Convert var to const/let throughout courses.js - Fix logger.error calls to use {err} object form (pino convention) - Extract RATING_TIER_HIGH/MID constants in course-layouts.ejs scriptlet - Remove dead href='#' View all link from courses.ejs (deferred) - Pass total prop explicitly from course-table.ejs to course-cards.ejs - Remove dead #search-results-info selector from mobile.css - Remove redundant .replace(/"/g, '"') from data attributes in course-table.ejs	2026-05-25 09:54:15 +02:00
Samuel Enocsson	f2e30c62aa	fix: zero excluded count in fallback, drop debug-icon orphan, align ejs guard (#21 )	2026-05-25 09:44:46 +02:00
Samuel Enocsson	4bbf6d9728	feat: redesign Courses page with tabs + restore Tjing import (#8 ) - Restore src/scrapers/tjing.js with AbortController timeout (8s), error-object returns, and verbatim GraphQL queries - Add getOrCreateLayout() to src/models/course.js - New /api/tjing/search and /api/tjing/import/:tjingId routes; course-table route now includes layoutCount/activeLayoutCount via LEFT JOIN aggregation - Rewrite courses.ejs: action-card with Find/Import tabs, results bar, HTMX course-table-region with body:refresh trigger - Rewrite course-table.ejs: CSS-grid div structure replacing <table>, lazy-load expanded layouts via JS htmx.ajax - Rewrite course-layouts.ejs: layout-card chips with rating tier colouring, collapsible inactive layouts section - Rewrite courses.js: tab switching, live client-side filter, count display, Tjing search/import using DOM API (no innerHTML with untrusted data) - Rewrite courses.css: full new design system using project tokens	2026-05-25 09:39:44 +02:00
Samuel Enocsson	0beeb98002	feat: show excluded rounds count in player history accordion (#21 )	2026-05-25 09:34:42 +02:00
Release Bot	f4c5e963d2	1.2.11 Release / release (push) Successful in 5s Details Build and deploy / build-and-push (push) Successful in 21s Details Build and deploy / deploy (push) Successful in 11s Details	2026-05-25 06:04:26 +00:00
shcizo	27d1bef8dd	Merge pull request 'fix: move std-dev info to accordion, remove broken tooltip (#19 )' (#20 ) from fix/std-dev-tooltip-positioning-and-discoverability-19 into main Release / release (push) Successful in 7s Details	2026-05-25 08:04:14 +02:00
Samuel Enocsson	088e283dcf	refactor: split round spread and rating range into separate accordion rows (#19 )	2026-05-25 08:02:05 +02:00
Samuel Enocsson	5791d8e34f	refactor: move std-dev info to accordion, remove tooltip (#19 ) - Add "Round spread" row (±stdDev, range lo–hi) to desktop accordion (player-history.ejs) and mobile card expanded section (ratings-cards.ejs) - Remove .std-dev-tooltip div and .std-dev-inline span from table partial - Remove stdDevTooltipText, updateStdDevInline, initRatingsTooltips helpers and all call sites from players.js - Remove .std-dev-tooltip and .std-dev-inline CSS rules; drop cursor:help from .rating-value	2026-05-25 07:54:46 +02:00
Samuel Enocsson	1ff768e2fa	fix: address std-dev inline span refresh + style fixes (#19 ) - A: create inline span when missing in refreshRoundHistory (was silently dropped) - B: updateStdDevInline also called from refreshHistoryThenCalculate - C: extract stdDevTooltipText + updateStdDevInline helpers; replace 3 call sites - D: remove margin-left: 4px and bump font-size to 12px on .std-dev-inline - E: guard against stdDev === 0 in EJS (truthy → != null)	2026-05-23 06:45:39 +02:00
Samuel Enocsson	c3fb850de3	fix: reposition std-dev tooltip and surface ±-spread inline (#19 )	2026-05-23 06:40:08 +02:00
Release Bot	c69efa469e	1.2.10 Release / release (push) Successful in 5s Details Build and deploy / build-and-push (push) Successful in 21s Details Build and deploy / deploy (push) Successful in 8s Details	2026-05-22 20:11:32 +00:00
Samuel Enocsson	e21e6f2ef0	fix: couple mobile add-bar offset to container padding via CSS var Release / release (push) Successful in 7s Details Previously .mobile-add-bar { margin-bottom: -80px } was hardcoded to match .container { padding-bottom: 80px }. Now both reference --m-container-pad-bottom so they can't drift out of sync.	2026-05-22 22:11:20 +02:00
Samuel Enocsson	141dc90db7	ci: add automated release workflow with conventional commits Release / release (push) Successful in 5s Details - New .gitea/workflows/release.yml triggers on push to main - Scans commits since last v* tag and bumps strict semver - feat: -> minor, fix: -> patch, !: or BREAKING CHANGE -> major - Updates package.json + package-lock.json, commits as <version>, tags v<version>, pushes both - Tag push triggers existing deploy.yml (PACKAGES_TOKEN is a PAT so cross-workflow triggers work) - Skips silently when no releaseable commits are found - Updates CLAUDE.md release section to reflect automation	2026-05-22 22:04:07 +02:00