PR_26169_001-memberships-page

PR_26169_002-membership-data-model
PR_26169_003-user-membership-assignment
PR_26169_004-membership-limit-helpers
PR_26169_005-membership-enforcement
PR_26169_006-teams-foundation
PR_26169_007-marketplace-membership-rules
PR_26169_008-legal-landing-page
PR_26169_009-terms-of-service
PR_26169_010-privacy-policy
PR_26169_011-cookie-policy
PR_26169_012-community-guidelines
PR_26169_013-copyright-policy
PR_26169_014-dmca-policy
PR_26169_015-memberships-legal-qa

Author: ToolboxAid

assets/theme-v2/js/gamefoundry-partials.js

@@ -52,6 +52,7 @@
         "game-retro": "games/retro/index.html",
         "game-strategy": "games/strategy/index.html",
         marketplace: "marketplace/index.html",
+        memberships: "memberships/index.html",
         learn: "learn/index.html",
         community: "community/index.html",
         translations: "toolbox/languages/index.html",
@@ -114,7 +115,7 @@
 
     const rootSegments = new Set([
         "account", "company", "community", "legal",
-        "admin", "docs", "games", "learn", "marketplace", "owner", "toolbox"
+        "admin", "docs", "games", "learn", "marketplace", "memberships", "owner", "toolbox"
     ]);
 
     const currentScript = document.currentScript || document.querySelector("script[src*='gamefoundry-partials.js']");

assets/theme-v2/partials/footer.html

@@ -8,6 +8,7 @@ <h2 id="footer-product">Product</h2>
           <a data-route="games" href="games/index.html">Games</a>
           <a data-route="learn" href="learn/index.html">Learn</a>
           <a data-route="marketplace" href="marketplace/index.html">Marketplace</a>
+          <a data-route="memberships" href="memberships/index.html">Memberships</a>
           <a data-route="toolbox" href="toolbox/index.html">Toolbox</a>
         </div>
       </section>

assets/theme-v2/partials/header-nav.html

@@ -95,7 +95,13 @@
           </div>
         </div>
       </div>
-      <a data-nav-link data-route="marketplace" href="marketplace/index.html">Marketplace</a>
+      <div class="nav-item">
+        <a data-nav-link data-route="marketplace" href="marketplace/index.html">Marketplace &#9662;</a>
+        <div class="sub-menu">
+          <a data-nav-link data-route="marketplace" href="marketplace/index.html">Marketplace Home</a>
+          <a data-nav-link data-route="memberships" href="memberships/index.html">Memberships</a>
+        </div>
+      </div>
       <a data-nav-link data-route="learn" href="learn/index.html">Learn</a>
       <div class="nav-item">
         <a data-nav-link data-route="account" href="account/index.html">Account &#9662;</a>

docs_build/dev/codex_commands.md

@@ -2643,3 +2643,39 @@ Required reports:
 
 Packaging:
 - `tmp/PR_26168_246-r2-backed-postgres-backups_delta.zip`
+
+
+## PR_26168_247-memberships-page
+
+Changes:
+- Read `docs_build/dev/PROJECT_INSTRUCTIONS.md`.
+- Verified the current branch is `main`.
+- Added the public `memberships/index.html` page with Free, Creator, and Studio tiers.
+- Wired Memberships into the shared Marketplace submenu and Product footer.
+- Registered the Memberships route/root segment for shared Theme V2 partial rewriting.
+- Kept the primary top-level header IA exception unchanged.
+- Kept the slice public/static only with no backend enforcement, database changes, teams, legal pages, marketplace enforcement, or billing integration.
+
+Validation:
+- `node --check assets/theme-v2/js/gamefoundry-partials.js`
+- `node --check tests/playwright/tools/PublicMembershipsPage.spec.mjs`
+- `node --check tests/playwright/tools/RootToolsFutureState.spec.mjs`
+- Static HTML inline script/style/event-handler guard.
+- Static Memberships route/nav/footer/tier validation.
+- `node scripts/audit-playwright-test-locations.mjs --target tests/playwright/tools/PublicMembershipsPage.spec.mjs --target tests/playwright/tools/RootToolsFutureState.spec.mjs --helper tests/helpers/playwrightRepoServer.mjs --lanes tools`
+- `node node_modules/@playwright/test/cli.js test tests/playwright/tools/PublicMembershipsPage.spec.mjs --project=playwright --workers=1 --reporter=line`
+- Attempted targeted RootToolsFutureState header validation; blocked by existing local API/toolbox fixture 500/empty-card failures unrelated to the Memberships page.
+- `git diff --check -- touched files`
+- Full samples smoke skipped because samples and `start_of_day` folders were not touched.
+
+Required reports:
+- `docs_build/dev/reports/codex_review.diff`
+- `docs_build/dev/reports/codex_changed_files.txt`
+- `docs_build/dev/reports/PR_26168_247-memberships-page.md`
+- `docs_build/dev/reports/playwright_structure_audit.md`
+- `docs_build/dev/reports/playwright_discovery_ownership_report.md`
+- `docs_build/dev/reports/playwright_discovery_scope_report.md`
+- `docs_build/dev/reports/filesystem_scan_reduction_report.md`
+
+Packaging:
+- `tmp/PR_26168_247-memberships-page_delta.zip`

docs_build/dev/commit_comment.txt

@@ -1 +1,2 @@
-R2 backed Postgres backups - PR_26168_246-r2-backed-postgres-backups
+PR_26168_247-memberships-page
+Add public Memberships page with Free, Creator, and Studio tiers.

docs_build/dev/reports/PR_26168_247-memberships-page.md

@@ -0,0 +1,90 @@
+# PR_26168_247-memberships-page
+
+## Branch Validation
+
+PASS - Current branch is `main`.
+
+Note: `test.sql` was already deleted in the working tree before this PR work and was left untouched/excluded from the PR package.
+
+## Requirement Checklist
+
+| Requirement | Status | Notes |
+| --- | --- | --- |
+| Follow `docs_build/dev/PROJECT_INSTRUCTIONS.md` | PASS | Read before implementation; kept top-level header IA exception intact. |
+| Create only `PR_26168_247-memberships-page` | PASS | Scope limited to public Memberships page, shared navigation wiring, focused tests, and required reports/package. |
+| Add `/memberships/index.html` | PASS | New public Theme V2 page added with Free, Creator, and Studio tier content. |
+| Wire Memberships into shared nav | PASS | Added under shared Marketplace submenu in production header partial and local repo-server header mirror. |
+| Wire Memberships into shared footer | PASS | Added to Product footer links. |
+| Wire Memberships into mobile nav | PASS | Shared header submenu is the mobile/narrow navigation surface; Playwright validates the Memberships link at 390px width. |
+| Use existing GFS shared layout and CSS | PASS | Page uses Theme V2 shared partials/classes and existing CSS only. |
+| No backend enforcement | PASS | No API/service/runtime enforcement code added. |
+| No database changes | PASS | No schema, migration, seed, or database files touched. |
+| No legal pages | PASS | No legal pages added or modified. Existing shared footer legal links were not changed. |
+| No teams | PASS | No team model, team UI, or team copy added. |
+| No marketplace enforcement | PASS | Navigation placement only; no marketplace access logic or enforcement added. |
+| No billing integration | PASS | No billing, checkout, payment, or provider code/copy added. |
+| Commit message prepared | PASS | `docs_build/dev/commit_comment.txt` contains the requested title/body. |
+
+## Intentional Order Notes
+
+PASS - Primary top-level header navigation remains the documented product IA exception:
+`Games`, `Toolbox`, `Marketplace`, `Learn`, followed by session-dependent account/admin items.
+
+PASS - Memberships is placed inside the Marketplace submenu so the top-level header order remains unchanged.
+
+PASS - New browseable lists remain alphabetized:
+- Marketplace submenu: `Marketplace Home`, `Memberships`.
+- Product footer: `Games`, `Learn`, `Marketplace`, `Memberships`, `Toolbox`.
+
+## Validation Lane Report
+
+| Lane | Status | Command / Notes |
+| --- | --- | --- |
+| JS syntax | PASS | `node --check assets/theme-v2/js/gamefoundry-partials.js` |
+| Playwright spec syntax | PASS | `node --check tests/playwright/tools/PublicMembershipsPage.spec.mjs` |
+| Existing header contract spec syntax | PASS | `node --check tests/playwright/tools/RootToolsFutureState.spec.mjs` |
+| HTML inline guard | PASS | `rg --pcre2 -n "<script(?![^>]+src=)\|<style\|\\s(onclick\|onchange\|oninput\|onsubmit\|onkeydown\|onkeyup\|onload)=" ...` returned no matches. |
+| Out-of-scope term guard | PASS | Targeted search for billing/checkout/payment/stripe/team/entitlement/enforcement/database/migration terms returned no matches in touched implementation files. |
+| Static membership wiring | PASS | Node validation confirmed production header, local header mirror, footer, route map/root segment, and Free/Creator/Studio tier labels. |
+| Playwright ownership audit | PASS | Scoped audit passed for `PublicMembershipsPage.spec.mjs`, `RootToolsFutureState.spec.mjs`, and `playwrightRepoServer.mjs`. |
+| Targeted Memberships Playwright | PASS | `node node_modules/@playwright/test/cli.js test tests/playwright/tools/PublicMembershipsPage.spec.mjs --project=playwright --workers=1 --reporter=line` - 2 passed. |
+| Targeted shared-header RootTools attempt | FAIL (pre-existing fixture) | Attempted RootTools header validation failed on unrelated local API/toolbox fixture state: empty toolbox card list and 500s from `/api/platform-settings/banner`, `/api/toolbox/game-workspace/repositories`, and `/api/toolbox/registry/snapshot`. Memberships coverage was validated by the focused public-page spec and static production partial check. |
+| Diff whitespace | PASS | `git diff --check -- PR-scoped tracked files` passed; generated `codex_review.diff` is excluded because diff context lines trigger false trailing-whitespace positives. |
+| Full samples smoke | SKIP | Samples and `start_of_day` were not touched; repo instructions say not to run broad samples smoke for this slice. |
+
+## Manual Validation Notes
+
+1. Open `/memberships/index.html`.
+2. Confirm the page uses the shared header and footer.
+3. Confirm the H1 reads `Choose the membership shape for your next game.`
+4. Confirm Free, Creator, and Studio tier cards are visible.
+5. Open the Marketplace submenu and confirm Memberships is listed.
+6. Resize to a narrow mobile width and confirm the shared navigation still exposes Memberships.
+7. Confirm Product footer includes Memberships.
+8. Confirm there is no billing, backend enforcement, database, teams, legal page, or marketplace enforcement behavior.
+
+Expected outcome: Memberships is a public informational page only, with shared Theme V2 chrome and no backend or data-model side effects.
+
+## Changed Files
+
+- `assets/theme-v2/js/gamefoundry-partials.js`
+- `assets/theme-v2/partials/footer.html`
+- `assets/theme-v2/partials/header-nav.html`
+- `docs_build/dev/codex_commands.md`
+- `docs_build/dev/commit_comment.txt`
+- `docs_build/dev/reports/PR_26168_247-memberships-page.md`
+- `docs_build/dev/reports/filesystem_scan_reduction_report.md`
+- `docs_build/dev/reports/playwright_discovery_ownership_report.md`
+- `docs_build/dev/reports/playwright_discovery_scope_report.md`
+- `docs_build/dev/reports/playwright_structure_audit.md`
+- `memberships/index.html`
+- `src/dev-runtime/admin/header-nav.local.html`
+- `tests/playwright/tools/PublicMembershipsPage.spec.mjs`
+- `tests/playwright/tools/RootToolsFutureState.spec.mjs`
+
+## Package
+
+Generated artifacts:
+- `docs_build/dev/reports/codex_review.diff`
+- `docs_build/dev/reports/codex_changed_files.txt`
+- `tmp/PR_26168_247-memberships-page_delta.zip`

docs_build/dev/reports/codex_changed_files.txt

@@ -1,48 +1,68 @@
-# git status --short
-M .env.example
- M docs_build/database/README.md
- M docs_build/database/backup-restore-lane.md
- M docs_build/database/runbook.md
+# Codex Changed Files
+
+## git status --short (PR-scoped)
+
+M assets/theme-v2/js/gamefoundry-partials.js
+ M assets/theme-v2/partials/footer.html
+ M assets/theme-v2/partials/header-nav.html
+ M docs_build/dev/codex_commands.md
+ M docs_build/dev/commit_comment.txt
+ M docs_build/dev/reports/codex_changed_files.txt
+ M docs_build/dev/reports/codex_review.diff
+ M docs_build/dev/reports/filesystem_scan_reduction_report.md
+ M docs_build/dev/reports/playwright_discovery_ownership_report.md
+ M docs_build/dev/reports/playwright_discovery_scope_report.md
+ M docs_build/dev/reports/playwright_structure_audit.md
+ M src/dev-runtime/admin/header-nav.local.html
+ M tests/playwright/tools/RootToolsFutureState.spec.mjs
+?? docs_build/dev/reports/PR_26168_247-memberships-page.md
+?? memberships/index.html
+?? tests/playwright/tools/PublicMembershipsPage.spec.mjs
+
+## git diff --stat (PR-scoped)
+
+assets/theme-v2/js/gamefoundry-partials.js         |  3 +-
+ assets/theme-v2/partials/footer.html               |  1 +
+ assets/theme-v2/partials/header-nav.html           |  8 ++++-
+ docs_build/dev/codex_commands.md                   | 36 ++++++++++++++++++++++
+ docs_build/dev/commit_comment.txt                  |  3 +-
+ .../reports/filesystem_scan_reduction_report.md    |  2 +-
+ .../playwright_discovery_ownership_report.md       |  6 ++--
+ .../reports/playwright_discovery_scope_report.md   |  6 ++--
+ .../dev/reports/playwright_structure_audit.md      |  8 ++---
+ src/dev-runtime/admin/header-nav.local.html        |  8 ++++-
+ .../playwright/tools/RootToolsFutureState.spec.mjs |  4 +--
+ 11 files changed, 65 insertions(+), 20 deletions(-)
+ .../dev/reports/PR_26168_247-memberships-page.md   | 90 ++++++++++++++++++++++
+ 1 file changed, 90 insertions(+)
+ /dev/null => memberships/index.html | 122 ++++++++++++++++++++++++++++++++++++
+ 1 file changed, 122 insertions(+)
+ .../tools/PublicMembershipsPage.spec.mjs           | 110 +++++++++++++++++++++
+ 1 file changed, 110 insertions(+)
+ docs_build/dev/reports/codex_changed_files.txt |  108 +-
+ docs_build/dev/reports/codex_review.diff       | 1918 +++++++-----------------
+ 2 files changed, 603 insertions(+), 1423 deletions(-)
+
+## Full working tree note
+
+- Unrelated pre-existing working tree change remains: `D test.sql` (excluded from PR diff and ZIP).
+
+## Full git status --short
+
+M assets/theme-v2/js/gamefoundry-partials.js
+ M assets/theme-v2/partials/footer.html
+ M assets/theme-v2/partials/header-nav.html
  M docs_build/dev/codex_commands.md
  M docs_build/dev/commit_comment.txt
  M docs_build/dev/reports/codex_changed_files.txt
  M docs_build/dev/reports/codex_review.diff
- M docs_build/dev/reports/coverage_changed_js_guardrail.txt
- M docs_build/dev/reports/playwright_v8_coverage_report.txt
- M src/dev-runtime/database/postgres-backup-service.mjs
- M src/dev-runtime/server/local-api-router.mjs
- M src/dev-runtime/storage/r2-project-asset-storage.mjs
- M src/dev-runtime/storage/storage-config.mjs
- M tests/dev-runtime/PostgresBackupService.test.mjs
- M tests/playwright/tools/AdminPlatformToolsWireframes.spec.mjs
-?? docs_build/dev/reports/PR_26168_246-r2-backed-postgres-backups.md
-
-# git ls-files --others --exclude-standard
-docs_build/dev/reports/PR_26168_246-r2-backed-postgres-backups.md
-
-# git diff --stat
-.env.example                                       |   16 +
- docs_build/database/README.md                      |    2 +-
- docs_build/database/backup-restore-lane.md         |   80 +-
- docs_build/database/runbook.md                     |   23 +-
- docs_build/dev/codex_commands.md                   |   30 +
- docs_build/dev/commit_comment.txt                  |    2 +-
- docs_build/dev/reports/codex_changed_files.txt     |   51 +-
- docs_build/dev/reports/codex_review.diff           | 9458 +++-----------------
- .../dev/reports/coverage_changed_js_guardrail.txt  |   15 +-
- .../dev/reports/playwright_v8_coverage_report.txt  |   22 +-
- .../database/postgres-backup-service.mjs           |  141 +-
- src/dev-runtime/server/local-api-router.mjs        |    5 +-
- .../storage/r2-project-asset-storage.mjs           |   64 +-
- src/dev-runtime/storage/storage-config.mjs         |  103 +-
- tests/dev-runtime/PostgresBackupService.test.mjs   |  269 +-
- .../tools/AdminPlatformToolsWireframes.spec.mjs    |   16 +-
- 16 files changed, 1989 insertions(+), 8308 deletions(-)
-warning: in the working copy of '.env.example', LF will be replaced by CRLF the next time Git touches it
-warning: in the working copy of 'docs_build/dev/reports/codex_changed_files.txt', CRLF will be replaced by LF the next time Git touches it
-warning: in the working copy of 'src/dev-runtime/database/postgres-backup-service.mjs', LF will be replaced by CRLF the next time Git touches it
-warning: in the working copy of 'src/dev-runtime/server/local-api-router.mjs', LF will be replaced by CRLF the next time Git touches it
-warning: in the working copy of 'src/dev-runtime/storage/r2-project-asset-storage.mjs', LF will be replaced by CRLF the next time Git touches it
-warning: in the working copy of 'src/dev-runtime/storage/storage-config.mjs', LF will be replaced by CRLF the next time Git touches it
-warning: in the working copy of 'tests/dev-runtime/PostgresBackupService.test.mjs', LF will be replaced by CRLF the next time Git touches it
-warning: in the working copy of 'tests/playwright/tools/AdminPlatformToolsWireframes.spec.mjs', LF will be replaced by CRLF the next time Git touches it
+ M docs_build/dev/reports/filesystem_scan_reduction_report.md
+ M docs_build/dev/reports/playwright_discovery_ownership_report.md
+ M docs_build/dev/reports/playwright_discovery_scope_report.md
+ M docs_build/dev/reports/playwright_structure_audit.md
+ M src/dev-runtime/admin/header-nav.local.html
+ D test.sql
+ M tests/playwright/tools/RootToolsFutureState.spec.mjs
+?? docs_build/dev/reports/PR_26168_247-memberships-page.md
+?? memberships/index.html
+?? tests/playwright/tools/PublicMembershipsPage.spec.mjs