Skip to content

Commit 1f5e4dc

Browse files
ToolboxAidToolboxAid
committed
Lock root tools index parity with approved GameFoundryStudio source - PR_26152_048-root-tools-index-parity-lock & Recover root Admin page content parity from pre-V2 GameFoundryStudio sources - PR_26152_049-root-admin-original-parity-recovery
1 parent 549dce2 commit 1f5e4dc

2 files changed

Lines changed: 155 additions & 0 deletions

File tree

‎docs/dev/reports/root_admin_original_parity_recovery_validation.md‎

Lines changed: 82 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,82 @@
1+
# PR_26152_049 Root Admin Original Parity Recovery Validation
2+
3+
## Scope
4+
5+
Validated only the listed root Admin pages and supporting root Admin path assumptions.
6+
7+
No Admin page content was changed because original pre-Theme-V2 source content could not be retrieved.
8+
9+
## Source Recovery Attempt
10+
11+
Target source family:
12+
- `GameFoundryStudio/admin/site-settings.html`
13+
- `GameFoundryStudio/admin/branding.html`
14+
- `GameFoundryStudio/admin/themes.html`
15+
- `GameFoundryStudio/admin/design-system.html`
16+
- `GameFoundryStudio/admin/controls.html`
17+
- `GameFoundryStudio/admin/grouping-colors.html`
18+
- `GameFoundryStudio/admin/ratings.html`
19+
- `GameFoundryStudio/admin/users.html`
20+
- `GameFoundryStudio/admin/roles.html`
21+
- `GameFoundryStudio/admin/moderation.html`
22+
- `GameFoundryStudio/admin/analytics.html`
23+
24+
Result: Blocked.
25+
26+
Details:
27+
- Original `GameFoundryStudio/admin/**` source pages are not present as live working-tree files.
28+
- Scoped repository searches found current root Admin pages and reports, not preserved original Admin bodies.
29+
- Git history/deleted-file commands for the source family repeatedly failed with the Windows sandbox `spawn setup refresh` error.
30+
- Original-content parity could not be validated or restored without inventing content.
31+
32+
## Root Admin Pages Checked
33+
34+
- `admin/site-settings.html`
35+
- `admin/branding.html`
36+
- `admin/themes.html`
37+
- `admin/design-system.html`
38+
- `admin/controls.html`
39+
- `admin/grouping-colors.html`
40+
- `admin/ratings.html`
41+
- `admin/users.html`
42+
- `admin/roles.html`
43+
- `admin/moderation.html`
44+
- `admin/analytics.html`
45+
46+
## Inline CSS / Script / Event Handler Scan
47+
48+
Command:
49+
```powershell
50+
rg -n --pcre2 '<script(?![^>]*\bsrc=)|<style|\son[a-z]+\s*=' admin/site-settings.html admin/branding.html admin/themes.html admin/design-system.html admin/controls.html admin/grouping-colors.html admin/ratings.html admin/users.html admin/roles.html admin/moderation.html admin/analytics.html
51+
```
52+
53+
Result: Passed. No matches found.
54+
55+
## Root Path Checks
56+
57+
Verified root Admin pages retain:
58+
- `assets/css/theme/v2/theme.css`
59+
- `assets/js/gamefoundry-partials.js`
60+
- `data-partial="header-nav"`
61+
- `data-partial="footer"`
62+
63+
Result: Passed.
64+
65+
## CSS Guard
66+
67+
Command:
68+
```powershell
69+
git status --short -- GameFoundryStudio/assets/css/theme/v2 GameFoundryStudio/assets/css/styles.css
70+
```
71+
72+
Result: Passed. No scoped CSS changes.
73+
74+
## Non-Admin Scope Guard
75+
76+
No root Admin page files were edited and no non-Admin pages were edited for this PR.
77+
78+
## Skipped
79+
80+
- Repo-wide tests were not run by request.
81+
- Tests outside affected root/Admin/GameFoundryStudio paths were not run by request.
82+
- Direct original-content parity validation was blocked by unavailable source content.

‎docs/dev/reports/root_tools_index_parity_lock_validation.md‎

Lines changed: 73 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,73 @@
1+
# PR_26152_048 Root Tools Index Parity Lock Validation
2+
3+
## Scope
4+
5+
Validated only:
6+
- `/tools/index.html`
7+
- `GameFoundryStudio/assets/js/tools-page-accordions.js`
8+
- Reports
9+
10+
No additional pages were migrated.
11+
No CSS files were changed.
12+
13+
## Project Instructions
14+
15+
Read:
16+
- `docs/dev/PROJECT_INSTRUCTIONS.md`
17+
18+
## Static Inspection
19+
20+
Inspected current root Tools index:
21+
- `tools/index.html`
22+
23+
Inspected supporting renderer:
24+
- `GameFoundryStudio/assets/js/tools-page-accordions.js`
25+
26+
Findings:
27+
- `/tools/index.html` loads `assets/css/theme/v2/theme.css`.
28+
- `/tools/index.html` uses `[data-tools-accordion-list]` as the render target.
29+
- `tools-page-accordions.js` renders Theme V2-oriented `.control-card` tiles inside `.card-grid`.
30+
- `tools-page-accordions.js` still defines tool images, badges, descriptions, grouping names, grouping colors, outlines, sorting, grouping, and links.
31+
32+
## Parity Blocker
33+
34+
Root Tools index visual parity with the older GameFoundryStudio Tools index requires changing generated classes/layout and/or stylesheet references.
35+
36+
The PR instruction says:
37+
38+
> If parity requires CSS/class/id changes, stop and document the issue instead.
39+
40+
Result: Blocked. No root Tools index code changes were made.
41+
42+
## Inline CSS / Script / Event Handler Scan
43+
44+
Command:
45+
```powershell
46+
rg -n --pcre2 '<script(?![^>]*\bsrc=)|<style|\son[a-z]+\s*=' tools/index.html
47+
```
48+
49+
Result: Passed. No matches found.
50+
51+
## JavaScript Syntax Check
52+
53+
Command:
54+
```powershell
55+
node --check GameFoundryStudio/assets/js/tools-page-accordions.js
56+
```
57+
58+
Result: Passed.
59+
60+
## CSS Guard
61+
62+
Command:
63+
```powershell
64+
git status --short -- GameFoundryStudio/assets/css/theme/v2 GameFoundryStudio/assets/css/styles.css
65+
```
66+
67+
Result: Passed. No scoped CSS changes.
68+
69+
## Skipped
70+
71+
- Repo-wide tests were not run by request.
72+
- Tests outside the root Tools index surface were not run by request.
73+
- No Playwright run was performed because no index code was changed.

0 commit comments

Comments
 (0)