Skip to content

Commit c0755bb

Browse files
ToolboxAidToolboxAid
committed
Review remote branches for missing work - PR_26159_045-remote-branch-review
1 parent 638dd1d commit c0755bb

1 file changed

Lines changed: 371 additions & 0 deletions

File tree

‎docs_build/dev/reports/remote_branch_review.md‎

Lines changed: 371 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,371 @@
1+
# Remote Branch Review
2+
3+
PR: PR_26159_045-remote-branch-review
4+
Generated: 2026-06-08
5+
Runtime behavior changed: No
6+
Playwright impacted: No
7+
8+
## Branch Guard
9+
10+
- Current branch: `main`
11+
- Expected branch: `main`
12+
- Branch validation: PASS
13+
- Local branches found:
14+
15+
```text
16+
* main
17+
```
18+
19+
## Method
20+
21+
- Remote refs were refreshed with `git fetch origin` without pruning or deleting branches.
22+
- Behind/ahead counts use `git rev-list --left-right --count main...origin/<branch>`.
23+
- Commits not in main use `git log --oneline --decorate main..origin/<branch>`.
24+
- Changed files and stats use `git diff main...origin/<branch>`.
25+
- No branches were deleted, merged, or cherry-picked.
26+
27+
## Remote Branches Found
28+
29+
```text
30+
origin/PR_26151_004-gamefoundry-site-structure-ssot
31+
origin/PR_26159_035-colors-picker-layout-tags
32+
origin/PR_26159_036-colors-picker-preview-layout
33+
origin/recover/70f1301b
34+
```
35+
36+
## Executive Summary
37+
38+
| Branch | Behind | Ahead | Unique files | Already merged/superseded | Unique work may be missing | Recommended action |
39+
| --- | ---: | ---: | ---: | --- | --- | --- |
40+
| `PR_26159_036-colors-picker-preview-layout` | 7 | 0 | 0 | Yes | No | delete |
41+
| `PR_26159_035-colors-picker-layout-tags` | 8 | 0 | 0 | Yes | No | delete |
42+
| `PR_26151_004-gamefoundry-site-structure-ssot` | 337 | 39 | 38 | Partially | Possibly | convert to new small PR |
43+
| `recover/70f1301b` | 1605 | 0 | 0 | Yes | No | delete |
44+
45+
## PR_26159_036-colors-picker-preview-layout
46+
47+
Remote ref: `origin/PR_26159_036-colors-picker-preview-layout`
48+
49+
- Behind count: 7
50+
- Ahead count: 0
51+
- Commits not in main: 0
52+
- Changed files not in main: 0
53+
54+
### Plain-English Summary
55+
56+
Historical Colors picker-preview layout branch. Against current main it has no unique commits, no unique files, and no diff; the picker preview layout work has already been incorporated into or superseded by later Colors PRs on main.
57+
58+
### Already Merged or Superseded?
59+
60+
Yes. Ahead count is 0, the branch is an ancestor/older state relative to main, and there are no changed files not in main.
61+
62+
### Unique Work That May Be Missing from main
63+
64+
No unique missing work detected from this branch.
65+
66+
### Recommended Action
67+
68+
delete
69+
70+
### File Category Classification
71+
72+
No unique changed files.
73+
74+
### Commits Not in main
75+
76+
```text
77+
(none)
78+
```
79+
80+
### Diff Stat
81+
82+
```text
83+
(no diff)
84+
```
85+
86+
### Changed Files Not in main
87+
88+
```text
89+
(none)
90+
```
91+
92+
## PR_26159_035-colors-picker-layout-tags
93+
94+
Remote ref: `origin/PR_26159_035-colors-picker-layout-tags`
95+
96+
- Behind count: 8
97+
- Ahead count: 0
98+
- Commits not in main: 0
99+
- Changed files not in main: 0
100+
101+
### Plain-English Summary
102+
103+
Historical Colors picker layout/tags branch. Against current main it has no unique commits, no unique files, and no diff; the useful tag/layout work appears already incorporated into or superseded by later Colors PRs on main.
104+
105+
### Already Merged or Superseded?
106+
107+
Yes. Ahead count is 0, the branch is an ancestor/older state relative to main, and there are no changed files not in main.
108+
109+
### Unique Work That May Be Missing from main
110+
111+
No unique missing work detected from this branch.
112+
113+
### Recommended Action
114+
115+
delete
116+
117+
### File Category Classification
118+
119+
No unique changed files.
120+
121+
### Commits Not in main
122+
123+
```text
124+
(none)
125+
```
126+
127+
### Diff Stat
128+
129+
```text
130+
(no diff)
131+
```
132+
133+
### Changed Files Not in main
134+
135+
```text
136+
(none)
137+
```
138+
139+
## PR_26151_004-gamefoundry-site-structure-ssot
140+
141+
Remote ref: `origin/PR_26151_004-gamefoundry-site-structure-ssot`
142+
143+
- Behind count: 337
144+
- Ahead count: 39
145+
- Commits not in main: 39
146+
- Changed files not in main: 38
147+
148+
### Plain-English Summary
149+
150+
Older GameFoundryStudio site-structure branch. It adds a standalone GameFoundryStudio page tree, split CSS files, partial navigation changes, organized account/tool/group pages, and PR validation artifacts. It appears to predate the current Theme V2/root page structure and the newer local API/dev-runtime work on main.
151+
152+
### Already Merged or Superseded?
153+
154+
Partially. The branch has 39 unique commits, so it is not merged. However it is 337 commits behind main and targets older GameFoundryStudio paths/styles that appear structurally superseded by current root/assets/theme-v2 organization.
155+
156+
### Unique Work That May Be Missing from main
157+
158+
Possibly. IA/content ideas such as organized Tool Builder/Creator/Publisher pages and grouped tool pages may be useful, but they should not be merged wholesale because the branch uses older path and CSS ownership patterns.
159+
160+
### Recommended Action
161+
162+
convert to new small PR
163+
164+
### File Category Classification
165+
166+
- config: 1
167+
- CSS: 10
168+
- docs: 5
169+
- generated artifacts: 5
170+
- runtime: 1
171+
- UI: 22
172+
173+
### Commits Not in main
174+
175+
```text
176+
0aee05215 (origin/PR_26151_004-gamefoundry-site-structure-ssot) Add commit comment for PR_26151_004
177+
25cedf280 Add codex command log for PR_26151_004
178+
ba448e9cf Add validation report for PR_26151_004
179+
c06619ff5 Add codex review diff report for PR_26151_004
180+
829f608ae Add changed files report for PR_26151_004
181+
ef47309bd Restore partial fallback for legacy pages
182+
5e21ad5cf Add organized Tool Publisher page
183+
e5a11aff4 Add organized Tool Creator page
184+
9d8bdfa57 Add organized Tool Builder page
185+
9dd7de321 Add organized Publish page
186+
07cffbcf6 Add organized Cloud page
187+
df9a69b4a Add organized Docs page
188+
3f41ebbed Add organized Community page
189+
17e6f4d05 Add organized Learn page
190+
5216f30db Add organized Marketplace page
191+
2d0f25a97 Add organized Arcade page
192+
9ce8bbb4e Add organized Configuration Admin group page
193+
e59d22ba3 Add organized Design Animation group page
194+
e284e99fc Add organized Media Community group page
195+
e1e0799f0 Add organized Assets Content group page
196+
3bfb96259 Add organized Technology System group page
197+
c532e0e80 Add organized Building Creation group page
198+
481edb60c Add organized Tools index page
199+
aeeba13b8 Add organized Controls page
200+
8157042bd Add organized Branding page
201+
152aa63de Add organized Account page
202+
e7f1524fd Use shared GameFoundryStudio shell on home page
203+
988d0505e Add GameFoundryStudio tool CSS
204+
04a254d07 Add GameFoundryStudio page-specific CSS
205+
4fd54fae4 Add steel gray meaning CSS
206+
2c86d7e13 Add purple meaning CSS
207+
dc4a269ef Add arcade cyan meaning CSS
208+
b649b6cc5 Add forge gold meaning CSS
209+
f2ddbb7e4 Add electric blue meaning CSS
210+
af52eda34 Add molten orange meaning CSS
211+
c06991e2d Add GameFoundryStudio shared base CSS
212+
da4cb2601 Split GameFoundryStudio CSS entry point
213+
dffa67dfc Make GameFoundryStudio partials path aware
214+
53cc15ec4 Update GameFoundryStudio shared nav routes
215+
```
216+
217+
### Diff Stat
218+
219+
```text
220+
GameFoundryStudio/account/branding.html | 14 +
221+
GameFoundryStudio/account/controls.html | 13 +
222+
GameFoundryStudio/account/index.html | 14 +
223+
GameFoundryStudio/arcade/index.html | 2 +
224+
GameFoundryStudio/assets/css/base.css | 18 +
225+
.../assets/css/colors/arcade-cyan.css | 5 +
226+
.../assets/css/colors/electric-blue.css | 5 +
227+
GameFoundryStudio/assets/css/colors/forge-gold.css | 5 +
228+
.../assets/css/colors/molten-orange.css | 5 +
229+
GameFoundryStudio/assets/css/colors/purple.css | 5 +
230+
GameFoundryStudio/assets/css/colors/steel-gray.css | 5 +
231+
GameFoundryStudio/assets/css/pages.css | 2 +
232+
GameFoundryStudio/assets/css/styles.css | 1087 +-------------------
233+
GameFoundryStudio/assets/css/tools.css | 5 +
234+
.../assets/js/gamefoundry-partials.js | 97 +-
235+
GameFoundryStudio/assets/partials/header-nav.html | 42 +-
236+
GameFoundryStudio/cloud/index.html | 2 +
237+
GameFoundryStudio/community/index.html | 2 +
238+
GameFoundryStudio/docs/index.html | 2 +
239+
GameFoundryStudio/index.html | 57 +-
240+
GameFoundryStudio/learn/index.html | 2 +
241+
GameFoundryStudio/marketplace/index.html | 2 +
242+
GameFoundryStudio/publish/index.html | 2 +
243+
GameFoundryStudio/tools/groups/assets-content.html | 2 +
244+
.../tools/groups/building-creation.html | 2 +
245+
.../tools/groups/configuration-admin.html | 2 +
246+
.../tools/groups/design-animation.html | 2 +
247+
.../tools/groups/media-community.html | 2 +
248+
.../tools/groups/technology-system.html | 2 +
249+
GameFoundryStudio/tools/index.html | 13 +
250+
GameFoundryStudio/tools/tool-builder.html | 2 +
251+
GameFoundryStudio/tools/tool-creator.html | 2 +
252+
GameFoundryStudio/tools/tool-publisher.html | 2 +
253+
docs/dev/codex_commands.md | 39 +
254+
docs/dev/commit_comment.txt | 15 +
255+
...4-gamefoundry-site-structure-ssot-validation.md | 65 ++
256+
docs/dev/reports/codex_changed_files.txt | 33 +
257+
docs/dev/reports/codex_review.diff | 15 +
258+
38 files changed, 434 insertions(+), 1157 deletions(-)
259+
```
260+
261+
### Changed Files Not in main
262+
263+
```text
264+
A GameFoundryStudio/account/branding.html
265+
A GameFoundryStudio/account/controls.html
266+
A GameFoundryStudio/account/index.html
267+
A GameFoundryStudio/arcade/index.html
268+
A GameFoundryStudio/assets/css/base.css
269+
A GameFoundryStudio/assets/css/colors/arcade-cyan.css
270+
A GameFoundryStudio/assets/css/colors/electric-blue.css
271+
A GameFoundryStudio/assets/css/colors/forge-gold.css
272+
A GameFoundryStudio/assets/css/colors/molten-orange.css
273+
A GameFoundryStudio/assets/css/colors/purple.css
274+
A GameFoundryStudio/assets/css/colors/steel-gray.css
275+
A GameFoundryStudio/assets/css/pages.css
276+
M GameFoundryStudio/assets/css/styles.css
277+
A GameFoundryStudio/assets/css/tools.css
278+
M GameFoundryStudio/assets/js/gamefoundry-partials.js
279+
M GameFoundryStudio/assets/partials/header-nav.html
280+
A GameFoundryStudio/cloud/index.html
281+
A GameFoundryStudio/community/index.html
282+
A GameFoundryStudio/docs/index.html
283+
M GameFoundryStudio/index.html
284+
A GameFoundryStudio/learn/index.html
285+
A GameFoundryStudio/marketplace/index.html
286+
A GameFoundryStudio/publish/index.html
287+
A GameFoundryStudio/tools/groups/assets-content.html
288+
A GameFoundryStudio/tools/groups/building-creation.html
289+
A GameFoundryStudio/tools/groups/configuration-admin.html
290+
A GameFoundryStudio/tools/groups/design-animation.html
291+
A GameFoundryStudio/tools/groups/media-community.html
292+
A GameFoundryStudio/tools/groups/technology-system.html
293+
A GameFoundryStudio/tools/index.html
294+
A GameFoundryStudio/tools/tool-builder.html
295+
A GameFoundryStudio/tools/tool-creator.html
296+
A GameFoundryStudio/tools/tool-publisher.html
297+
A docs/dev/codex_commands.md
298+
A docs/dev/commit_comment.txt
299+
A docs/dev/reports/PR_26151_004-gamefoundry-site-structure-ssot-validation.md
300+
A docs/dev/reports/codex_changed_files.txt
301+
A docs/dev/reports/codex_review.diff
302+
```
303+
304+
## recover/70f1301b
305+
306+
Remote ref: `origin/recover/70f1301b`
307+
308+
- Behind count: 1605
309+
- Ahead count: 0
310+
- Commits not in main: 0
311+
- Changed files not in main: 0
312+
313+
### Plain-English Summary
314+
315+
Recovery branch with no commits ahead of main. It contains no unique changed files or diff against main and appears to be a stale recovery pointer.
316+
317+
### Already Merged or Superseded?
318+
319+
Yes. Ahead count is 0 and there are no commits or files not in main.
320+
321+
### Unique Work That May Be Missing from main
322+
323+
No unique missing work detected from this branch.
324+
325+
### Recommended Action
326+
327+
delete
328+
329+
### File Category Classification
330+
331+
No unique changed files.
332+
333+
### Commits Not in main
334+
335+
```text
336+
(none)
337+
```
338+
339+
### Diff Stat
340+
341+
```text
342+
(no diff)
343+
```
344+
345+
### Changed Files Not in main
346+
347+
```text
348+
(none)
349+
```
350+
351+
## Special Attention Notes
352+
353+
- `PR_26151_004-gamefoundry-site-structure-ssot`: 39 commits ahead and 337 behind. It contains potentially useful site IA/page organization ideas, but should be converted into new small PRs rather than merged wholesale.
354+
- `recover/70f1301b`: 0 ahead and no unique files. It appears safe to delete later, but this PR intentionally does not delete branches.
355+
- Colors branches: both are 0 ahead with no unique files, so useful picker behavior appears already merged or superseded by current main.
356+
357+
## Validation Checklist
358+
359+
| Requirement | Status | Evidence |
360+
| --- | --- | --- |
361+
| Report includes all four requested branches. | PASS | Four branch sections are present. |
362+
| Report includes recommended action for each branch. | PASS | Each branch section has `Recommended Action`. |
363+
| Behind/ahead counts included. | PASS | Executive summary and each branch section include counts. |
364+
| Commits not in main included. | PASS | Each branch section includes `git log main..origin/<branch>` output or `(none)`. |
365+
| Changed files not in main included. | PASS | Each branch section includes `git diff --name-status main...origin/<branch>` output or `(none)`. |
366+
| Diff stat included. | PASS | Each branch section includes `git diff --stat main...origin/<branch>` output or `(no diff)`. |
367+
| No branches deleted. | PASS | No delete command was run. |
368+
| No branches merged. | PASS | No merge command was run. |
369+
| No branches cherry-picked. | PASS | No cherry-pick command was run. |
370+
| Playwright impacted: No. | PASS | Report-only branch review; no runtime/UI behavior changed. |
371+
| Full samples validation skipped. | PASS | Report-only branch review; no samples or runtime behavior changed. |

0 commit comments

Comments
 (0)