Skip to content

Commit c657aed

Browse files
committed
Add legal governance leftover retention
1 parent 0c0f2eb commit c657aed

11 files changed

Lines changed: 339 additions & 25 deletions

IMPLEMENTATION.md

Lines changed: 127 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,127 @@
1+
# Legal Foundation Implementation Instructions
2+
3+
## PR
4+
5+
Use:
6+
7+
`PR_26175_OWNER_050-legal-foundation`
8+
9+
## Goal
10+
11+
Publish the initial Game Foundry Studio legal foundation.
12+
13+
## Required Pages
14+
15+
Create or update:
16+
17+
```text
18+
/legal/index.html
19+
/legal/terms-of-service.html
20+
/legal/privacy-policy.html
21+
/legal/cookie-policy.html
22+
/legal/community-guidelines.html
23+
/legal/copyright-policy.html
24+
/legal/dmca-policy.html
25+
/legal/legal-nav.js
26+
```
27+
28+
## Source Drafts
29+
30+
Use the markdown files in `/legal/` from this package as the source content for the HTML pages.
31+
32+
## Single Source of Truth Legal Menu
33+
34+
The left-side legal menu on all legal pages must be rendered from one shared source of truth.
35+
36+
Do not duplicate legal side-menu links in each HTML file.
37+
38+
Suggested file:
39+
40+
```text
41+
/legal/legal-nav.js
42+
```
43+
44+
Suggested SSoT:
45+
46+
```js
47+
const LEGAL_NAV_ITEMS = [
48+
{ href: "/legal/index.html", label: "Legal Overview" },
49+
{ href: "/legal/terms-of-service.html", label: "Terms of Service" },
50+
{ href: "/legal/privacy-policy.html", label: "Privacy Policy" },
51+
{ href: "/legal/cookie-policy.html", label: "Cookie Policy" },
52+
{ href: "/legal/community-guidelines.html", label: "Community Guidelines" },
53+
{ href: "/legal/copyright-policy.html", label: "Copyright Policy" },
54+
{ href: "/legal/dmca-policy.html", label: "DMCA Policy" }
55+
];
56+
```
57+
58+
Each legal page should include only a mount point:
59+
60+
```html
61+
<nav class="legal-side-nav" aria-label="Legal documents" data-legal-nav></nav>
62+
```
63+
64+
Required JavaScript behavior:
65+
66+
- Render the legal navigation links from `LEGAL_NAV_ITEMS`.
67+
- Detect the current page from `window.location.pathname`.
68+
- Normalize `/legal/` to `/legal/index.html`.
69+
- Apply an active/current class to the matching legal page.
70+
- Set `aria-current="page"` on the active link.
71+
- Preserve keyboard and tab accessibility.
72+
- Do not require inline event handlers.
73+
74+
Suggested current-page logic:
75+
76+
```js
77+
function normalizeLegalPath(pathname) {
78+
if (pathname.endsWith("/legal/")) return "/legal/index.html";
79+
if (pathname === "/legal") return "/legal/index.html";
80+
return pathname;
81+
}
82+
```
83+
84+
## Layout Requirements
85+
86+
- Use the existing Game Foundry Studio site shell/template where appropriate.
87+
- Legal pages should use a two-column layout on desktop:
88+
- Left: shared legal side navigation
89+
- Right: legal document content
90+
- On small screens, the legal navigation may stack above the content.
91+
- Keep the legal pages readable and accessible.
92+
- Do not introduce unrelated UI changes.
93+
94+
## Footer Links
95+
96+
Add footer links to:
97+
98+
- Terms
99+
- Privacy
100+
- Cookies
101+
- Community Guidelines
102+
- Copyright
103+
- DMCA
104+
105+
Footer links may point to the corresponding `/legal/*.html` pages.
106+
107+
## Validation Requirements
108+
109+
Run the repository's relevant validation before opening the PR.
110+
111+
At minimum:
112+
113+
- HTML validation or existing static-page checks, if present
114+
- JS syntax check for `/legal/legal-nav.js`
115+
- Existing legal/nav/footer tests, if present
116+
- Any repository-required governance validation
117+
118+
## Acceptance Criteria
119+
120+
- All six legal documents are published as HTML pages.
121+
- `/legal/index.html` links to all legal documents.
122+
- Footer includes legal links.
123+
- Legal side navigation is generated from one SSoT.
124+
- Current page is visually selected.
125+
- Current page has `aria-current="page"`.
126+
- No page has a hand-maintained duplicate left legal menu.
127+
- No unrelated code, route, database, API, or storage changes.
Lines changed: 40 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,40 @@
1+
# PR_26175_OWNER_055-legal-governance-leftover-retention
2+
3+
## Purpose
4+
Retain the legal/governance leftover artifacts that remained after `PR_26175_OWNER_054-legal-corrected-package` was merged and `main` was synchronized.
5+
6+
## Source Leftovers
7+
The previous `MAIN_VERIFY` identified three untracked leftover paths:
8+
9+
| Path | Classification | Retention decision |
10+
| --- | --- | --- |
11+
| `IMPLEMENTATION.md` | Legal package implementation/source notes | Retained in this follow-up PR. |
12+
| `docs_build/legal/LEGAL_CHANGELOG.md` | Legal foundation package governance notes | Retained in this follow-up PR. |
13+
| `docs_build/dev/reports/PR_26175_OWNER_current-open-pr-status.md` | Generated governance status report | Retained as a snapshot report in this follow-up PR. |
14+
15+
## Implementation
16+
- Added the retained package implementation instructions at `IMPLEMENTATION.md`.
17+
- Added the retained legal changelog at `docs_build/legal/LEGAL_CHANGELOG.md`.
18+
- Updated and retained `docs_build/dev/reports/PR_26175_OWNER_current-open-pr-status.md` as a current governance snapshot taken after OWNER_054 was merged and before OWNER_055 was opened.
19+
- Added OWNER_055 plan, validation, checklist, manual validation, changed-files, review-diff, and delta ZIP outputs.
20+
21+
## Validation Commands
22+
- PASS: `git status --short --branch`
23+
- PASS: `git rev-list --left-right --count origin/main...HEAD`
24+
- PASS: `git diff --name-only`
25+
- PASS: `git diff --check`
26+
- PASS: targeted retained-file existence checks.
27+
- PASS: targeted changed-file scope check confirmed documentation/governance/report files only.
28+
29+
## Guardrails
30+
- No product files were modified.
31+
- No runtime code was modified.
32+
- No branches were deleted.
33+
- `stash@{0}` was left untouched.
34+
- No leftover file was moved to `tmp/`.
35+
- No leftover file was deleted.
36+
37+
## Delta ZIP
38+
Repo-structured delta ZIP:
39+
40+
`tmp/PR_26175_OWNER_055-legal-governance-leftover-retention_delta.zip`
Lines changed: 19 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,19 @@
1+
# PR_26175_OWNER_055-legal-governance-leftover-retention Branch Validation
2+
3+
## Result
4+
PASS
5+
6+
## Branch
7+
- Working branch: `codex/pr-26175-owner-055-legal-governance-leftover-retention`
8+
- Base branch: `main`
9+
- Base commit at branch creation: `0c0f2ebc00b40409ea6ebc013a59729747926daf`
10+
11+
## Checks
12+
| Check | Result |
13+
| --- | --- |
14+
| Branch created from current `main` | PASS |
15+
| `origin/main` fetched before branch work | PASS |
16+
| Local branch contains one documentation/governance purpose | PASS |
17+
| Product files unchanged | PASS |
18+
| Stash left untouched | PASS |
19+
| No branch deletion performed | PASS |
Lines changed: 12 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,12 @@
1+
# PR_26175_OWNER_055-legal-governance-leftover-retention Manual Validation Notes
2+
3+
## Manual Review
4+
- Confirmed `IMPLEMENTATION.md` contains legal foundation implementation instructions/source notes rather than unrelated temporary content.
5+
- Confirmed `docs_build/legal/LEGAL_CHANGELOG.md` belongs to the legal foundation package governance notes.
6+
- Confirmed `docs_build/dev/reports/PR_26175_OWNER_current-open-pr-status.md` is a generated governance report snapshot.
7+
- Confirmed this PR does not intentionally alter product files, runtime code, UI, legal page rendering, branches, or stashes.
8+
9+
## Notes
10+
- `stash@{0}` remains untouched.
11+
- Retained files were committed to preserve legal/governance provenance after OWNER_054.
12+
- No owner-approved move-to-`tmp/` or deletion action was performed.
Lines changed: 14 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,14 @@
1+
# PR_26175_OWNER_055-legal-governance-leftover-retention Requirement Checklist
2+
3+
| Requirement | Status | Notes |
4+
| --- | --- | --- |
5+
| PLAN_PR created | PASS | Added `docs_build/pr/PLAN_PR_26175_OWNER_055-legal-governance-leftover-retention.md`. |
6+
| BUILD_PR completed | PASS | Added this documentation/governance retention report bundle. |
7+
| APPLY_PR ready | PASS | Changes are scoped for a normal PR. |
8+
| Retain `IMPLEMENTATION.md` only if it contains legal package source notes | PASS | File contains legal foundation implementation instructions/source notes. |
9+
| Retain `docs_build/legal/LEGAL_CHANGELOG.md` only if it belongs to the legal package | PASS | File documents legal foundation package version 1.0. |
10+
| Retain current open PR status report only if creating report-retention PR | PASS | This PR is the report-retention/legal-governance follow-up. |
11+
| Do not modify product files | PASS | Only documentation/governance/report paths are changed. |
12+
| Do not alter `stash@{0}` | PASS | Stash was not touched. |
13+
| Do not delete branches | PASS | No branch deletion was performed. |
14+
| Produce repo-structured ZIP under `tmp/` | PASS | `tmp/PR_26175_OWNER_055-legal-governance-leftover-retention_delta.zip`. |
Lines changed: 19 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,19 @@
1+
# PR_26175_OWNER_055-legal-governance-leftover-retention Validation Lane
2+
3+
## Lane
4+
Documentation/governance retention lane.
5+
6+
## Validation Performed
7+
| Validation | Result |
8+
| --- | --- |
9+
| `git status --short --branch` | PASS |
10+
| `git rev-list --left-right --count origin/main...HEAD` | PASS |
11+
| `git diff --name-only` | PASS |
12+
| `git diff --check` | PASS |
13+
| Required retained files exist | PASS |
14+
| Required OWNER_055 reports exist | PASS |
15+
| Changed-file scope excludes product/runtime files | PASS |
16+
| Delta ZIP exists under `tmp/` | PASS |
17+
18+
## Runtime Validation
19+
Not run. This PR is documentation/governance-only and does not change runtime code, UI, legal page content, or tests.
Lines changed: 31 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,31 @@
1+
# PR_26175_OWNER_current-open-pr-status
2+
3+
## Snapshot Scope
4+
This governance snapshot was refreshed after `PR_26175_OWNER_054-legal-corrected-package` was merged and before opening `PR_26175_OWNER_055-legal-governance-leftover-retention`.
5+
6+
## MAIN_VERIFY Snapshot
7+
| Item | Result |
8+
| --- | --- |
9+
| Current branch at verification | `main` |
10+
| Current main commit | `0c0f2ebc00b40409ea6ebc013a59729747926daf` (`0c0f2ebc`) |
11+
| Local/origin sync | `0 0` |
12+
| Open PR count before OWNER_055 | `0` |
13+
| Draft PR count before OWNER_055 | `0` |
14+
| Worktree state before OWNER_055 branch | Dirty, untracked leftovers only |
15+
| Product files modified | No |
16+
| Stashes altered | No |
17+
18+
## Retained Leftovers
19+
| Path | Classification | OWNER_055 disposition |
20+
| --- | --- | --- |
21+
| `IMPLEMENTATION.md` | Legal package implementation/source notes | Retain in legal governance follow-up PR. |
22+
| `docs_build/legal/LEGAL_CHANGELOG.md` | Legal foundation package governance notes | Retain in legal governance follow-up PR. |
23+
| `docs_build/dev/reports/PR_26175_OWNER_current-open-pr-status.md` | Generated governance status report | Retain as this refreshed snapshot. |
24+
25+
## Stashes
26+
| Stash | Description | Action |
27+
| --- | --- | --- |
28+
| `stash@{0}` | `On codex/pr-26175-alfa-047-theme-v2-svg-icon-registry: MAIN_VERIFY preserve pre-existing uncommitted OWNER_050 files before main sync` | Retained; not altered |
29+
30+
## Recommendation
31+
Proceed with `PR_26175_OWNER_055-legal-governance-leftover-retention` as a small documentation/governance PR. Do not delete leftovers, move leftovers to `tmp/`, alter stashes, or delete branches unless separately approved by the owner.
Lines changed: 9 additions & 25 deletions
Original file line numberDiff line numberDiff line change
@@ -1,27 +1,11 @@
1-
assets/theme-v2/js/gamefoundry-partials.js
2-
assets/theme-v2/js/legal-document-page.js
3-
assets/theme-v2/partials/footer.html
4-
docs_build/dev/reports/PR_26175_OWNER_054-legal-corrected-package.md
1+
IMPLEMENTATION.md
2+
docs_build/dev/reports/PR_26175_OWNER_055-legal-governance-leftover-retention.md
3+
docs_build/dev/reports/PR_26175_OWNER_055-legal-governance-leftover-retention_branch-validation.md
4+
docs_build/dev/reports/PR_26175_OWNER_055-legal-governance-leftover-retention_manual-validation-notes.md
5+
docs_build/dev/reports/PR_26175_OWNER_055-legal-governance-leftover-retention_requirement-checklist.md
6+
docs_build/dev/reports/PR_26175_OWNER_055-legal-governance-leftover-retention_validation-lane.md
7+
docs_build/dev/reports/PR_26175_OWNER_current-open-pr-status.md
58
docs_build/dev/reports/codex_changed_files.txt
69
docs_build/dev/reports/codex_review.diff
7-
docs_build/dev/reports/coverage_changed_js_guardrail.txt
8-
docs_build/dev/reports/playwright_v8_coverage_report.txt
9-
legal/community-guidelines.html
10-
legal/community-guidelines.md
11-
legal/cookie-policy.html
12-
legal/cookie-policy.md
13-
legal/cookies-policy.html
14-
legal/copyright-policy.html
15-
legal/copyright-policy.md
16-
legal/disclaimer.html
17-
legal/dmca-policy.html
18-
legal/dmca-policy.md
19-
legal/index.html
20-
legal/index.md
21-
legal/legal-nav.js
22-
legal/privacy-policy.html
23-
legal/privacy-policy.md
24-
legal/terms.html
25-
legal/terms-of-service.html
26-
legal/terms-of-service.md
27-
tests/playwright/tools/RemainingLegalPages.spec.mjs
10+
docs_build/legal/LEGAL_CHANGELOG.md
11+
docs_build/pr/PLAN_PR_26175_OWNER_055-legal-governance-leftover-retention.md
-190 KB
Binary file not shown.
Lines changed: 24 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,24 @@
1+
# Legal Changelog
2+
3+
## Version 1.0 — June 24, 2026
4+
5+
Initial legal foundation package for Game Foundry Studio LLC.
6+
7+
Included:
8+
9+
- Legal Overview
10+
- Terms of Service
11+
- Privacy Policy
12+
- Cookie Policy
13+
- Community Guidelines
14+
- Copyright Policy
15+
- DMCA Policy
16+
17+
Implementation requirements added:
18+
19+
- Public `/legal/` page structure
20+
- Footer legal links
21+
- Shared left-side legal navigation
22+
- `/legal/legal-nav.js` as the single source of truth
23+
- JavaScript current-page selection using `window.location.pathname`
24+
- `aria-current="page"` on the selected legal page

0 commit comments

Comments
 (0)