Skip to content

Commit 8bb50a3

Browse files
qbytesdq-jpgqbytesdq-jpg
committed
Fix platform validation workflow path
1 parent 119fa88 commit 8bb50a3

8 files changed

Lines changed: 97 additions & 3 deletions

‎.github/workflows/platform-validation.yml‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -20,4 +20,4 @@ jobs:
2020
node-version: "20"
2121

2222
- name: Run Platform Validation Suite
23-
run: node ./scripts/run-platform-validation-suite.mjs
23+
run: node ./dev/scripts/run-platform-validation-suite.mjs

‎dev/docs_build/dev/reports/PR_26179_OWNER_006-move-bootstrap-scripts.md‎

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -14,6 +14,7 @@ Move local/test bootstrap scripts and dev-only runner scripts/config into the de
1414
- Moved tracked root `scripts/` files to `dev/scripts/`.
1515
- Moved root Playwright configs to `dev/config/`.
1616
- Updated `package.json` scripts to call `dev/scripts/`.
17+
- Updated GitHub Actions platform validation to call `node ./dev/scripts/run-platform-validation-suite.mjs`.
1718
- Updated moved script repo-root calculations, script self-references, PowerShell helper paths, and tests that import script modules.
1819
- Kept application runtime/business logic out of `dev/`.
1920

@@ -45,13 +46,16 @@ Documentation/dev-runner/config only. No product UI, API, database, `src/`, `doc
4546
| PASS | Playwright structure audit through moved runner | npm run test:audit:locations |
4647
| PASS | Playwright moved config discovery | npx playwright test --config=dev/config/playwright.config.cjs --list |
4748
| PASS | Canonical structure validation | npm run validate:canonical-structure |
49+
| PASS | GitHub Actions platform-validation path | Workflow now calls node ./dev/scripts/run-platform-validation-suite.mjs; targeted grep confirms no .github workflow still calls node ./scripts/run-platform-validation-suite.mjs. |
50+
| PASS | Platform validation suite | node ./dev/scripts/run-platform-validation-suite.mjs completed 8/8 scenarios. |
4851
| PASS | Whitespace validation | git diff --check -- . |
4952

5053
## Manual Validation Notes
5154

5255
- No UI changes were made.
5356
- Full browser Playwright execution was not run; PR_006 changed runner/config placement, so validation used structure audit and `--list` discovery with the moved config.
5457
- `npm run dev:local-api` was not left running; the moved startup module was imported successfully and the package script points to the moved file.
58+
- PR #241 CI blocker was a workflow script path issue only; no runtime behavior or file moves were changed for this fix.
5559

5660
## Changed Files
5761

‎dev/docs_build/dev/reports/PR_26179_OWNER_006-move-bootstrap-scripts_branch-validation.md‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -6,5 +6,5 @@ Generated: 2026-06-27T19:11:00.890Z
66
| --- | --- | --- |
77
| Current branch is PR branch | PASS | PR_26179_OWNER_006-move-bootstrap-scripts |
88
| Branch is stacked on previous PR branch | PASS | Base branch: PR_26179_OWNER_005-move-tests-to-dev |
9-
| Worktree contains only intended project changes before commit | PASS | Script/config move, package path updates, validation docs/tests, reports. |
9+
| Worktree contains only intended project changes before commit | PASS | Script/config move, package path updates, validation docs/tests, reports, and GitHub Actions moved-script path correction. |
1010
| Product/runtime path hard stop avoided | PASS | No product/runtime directory was moved into dev/. |

‎dev/docs_build/dev/reports/PR_26179_OWNER_006-move-bootstrap-scripts_manual-validation-notes.md‎

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -4,3 +4,5 @@
44
- No browser storage, API contract, database schema, or product behavior changed.
55
- Playwright browser execution was not run because this PR moves dev runner/config paths only; moved config discovery and structure audit passed.
66
- Local API startup was validated by importing the moved startup module and by checking `package.json` command routing.
7+
- GitHub Actions platform-validation path was corrected from `node ./scripts/run-platform-validation-suite.mjs` to `node ./dev/scripts/run-platform-validation-suite.mjs`.
8+
- Local platform validation completed 8/8 scenarios after the workflow path fix.

‎dev/docs_build/dev/reports/PR_26179_OWNER_006-move-bootstrap-scripts_requirement-checklist.md‎

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -5,6 +5,7 @@
55
| PASS | Move local/test bootstrap scripts and dev-only runner scripts/config into dev/bootstrap/, dev/scripts/, or dev/config/. | Moved tracked root scripts/ to dev/scripts/ and Playwright configs to dev/config/. |
66
| PASS | Keep application runtime/business logic out of dev/. | No src/, product UI, API contract, database, assets, docs/, games/, toolbox/, account/, admin/, or legal runtime files were moved into dev/. |
77
| PASS | Ensure npm run dev:local-api still works. | package.json now points dev:local-api at dev/scripts/start-local-api-server.mjs; syntax/import validation passed without launching a long-running server. |
8+
| PASS | GitHub Actions references moved script paths. | .github/workflows/platform-validation.yml now runs node ./dev/scripts/run-platform-validation-suite.mjs. |
89
| PASS | No Creator-writeable repo folder introduced. | This PR moves dev tooling only and adds no Creator data write path. |
910
| PASS | No runtime/business logic scope expansion. | Changes are limited to dev scripts/config, package script paths, validation tests/docs, and generated reports. |
1011
| PASS | No broad unrelated cleanup. | No product/runtime implementation files were changed. |

‎dev/docs_build/dev/reports/PR_26179_OWNER_006-move-bootstrap-scripts_validation-lane.md‎

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -11,4 +11,6 @@
1111
| PASS | Playwright structure audit through moved runner | npm run test:audit:locations |
1212
| PASS | Playwright moved config discovery | npx playwright test --config=dev/config/playwright.config.cjs --list |
1313
| PASS | Canonical structure validation | npm run validate:canonical-structure |
14+
| PASS | GitHub Actions platform-validation path | Workflow now calls node ./dev/scripts/run-platform-validation-suite.mjs; targeted grep confirms no .github workflow still calls node ./scripts/run-platform-validation-suite.mjs. |
15+
| PASS | Platform validation suite | node ./dev/scripts/run-platform-validation-suite.mjs completed 8/8 scenarios. |
1416
| PASS | Whitespace validation | git diff --check -- . |

‎dev/docs_build/dev/reports/codex_changed_files.txt‎

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,5 @@
1-
R100 codex_playwright_system_chrome.config.cjs dev/config/codex_playwright_system_chrome.config.cjs
1+
M .github/workflows/platform-validation.yml
2+
R100 codex_playwright_system_chrome.config.cjs dev/config/codex_playwright_system_chrome.config.cjs
23
A dev/config/playwright.config.cjs
34
M dev/docs_build/dev/ProjectInstructions/addendums/workspace_v2_playwright_gate.md
45
A dev/docs_build/dev/reports/PR_26179_OWNER_006-move-bootstrap-scripts.md

‎dev/docs_build/dev/reports/codex_review.diff‎

Lines changed: 84 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,3 +1,87 @@
1+
diff --git a/.github/workflows/platform-validation.yml b/.github/workflows/platform-validation.yml
2+
index e618de60b..c08529826 100644
3+
--- a/.github/workflows/platform-validation.yml
4+
+++ b/.github/workflows/platform-validation.yml
5+
@@ -20,4 +20,4 @@ jobs:
6+
node-version: "20"
7+
8+
- name: Run Platform Validation Suite
9+
- run: node ./scripts/run-platform-validation-suite.mjs
10+
+ run: node ./dev/scripts/run-platform-validation-suite.mjs
11+
diff --git a/dev/docs_build/dev/reports/PR_26179_OWNER_006-move-bootstrap-scripts.md b/dev/docs_build/dev/reports/PR_26179_OWNER_006-move-bootstrap-scripts.md
12+
index 0a07f079c..a8174a6fd 100644
13+
--- a/dev/docs_build/dev/reports/PR_26179_OWNER_006-move-bootstrap-scripts.md
14+
+++ b/dev/docs_build/dev/reports/PR_26179_OWNER_006-move-bootstrap-scripts.md
15+
@@ -14,6 +14,7 @@ Move local/test bootstrap scripts and dev-only runner scripts/config into the de
16+
- Moved tracked root `scripts/` files to `dev/scripts/`.
17+
- Moved root Playwright configs to `dev/config/`.
18+
- Updated `package.json` scripts to call `dev/scripts/`.
19+
+- Updated GitHub Actions platform validation to call `node ./dev/scripts/run-platform-validation-suite.mjs`.
20+
- Updated moved script repo-root calculations, script self-references, PowerShell helper paths, and tests that import script modules.
21+
- Kept application runtime/business logic out of `dev/`.
22+
23+
@@ -45,6 +46,8 @@ Documentation/dev-runner/config only. No product UI, API, database, `src/`, `doc
24+
| PASS | Playwright structure audit through moved runner | npm run test:audit:locations |
25+
| PASS | Playwright moved config discovery | npx playwright test --config=dev/config/playwright.config.cjs --list |
26+
| PASS | Canonical structure validation | npm run validate:canonical-structure |
27+
+| PASS | GitHub Actions platform-validation path | Workflow now calls node ./dev/scripts/run-platform-validation-suite.mjs; targeted grep confirms no .github workflow still calls node ./scripts/run-platform-validation-suite.mjs. |
28+
+| PASS | Platform validation suite | node ./dev/scripts/run-platform-validation-suite.mjs completed 8/8 scenarios. |
29+
| PASS | Whitespace validation | git diff --check -- . |
30+
31+
## Manual Validation Notes
32+
@@ -52,6 +55,7 @@ Documentation/dev-runner/config only. No product UI, API, database, `src/`, `doc
33+
- No UI changes were made.
34+
- Full browser Playwright execution was not run; PR_006 changed runner/config placement, so validation used structure audit and `--list` discovery with the moved config.
35+
- `npm run dev:local-api` was not left running; the moved startup module was imported successfully and the package script points to the moved file.
36+
+- PR #241 CI blocker was a workflow script path issue only; no runtime behavior or file moves were changed for this fix.
37+
38+
## Changed Files
39+
40+
diff --git a/dev/docs_build/dev/reports/PR_26179_OWNER_006-move-bootstrap-scripts_branch-validation.md b/dev/docs_build/dev/reports/PR_26179_OWNER_006-move-bootstrap-scripts_branch-validation.md
41+
index e30418103..d00f59e1a 100644
42+
--- a/dev/docs_build/dev/reports/PR_26179_OWNER_006-move-bootstrap-scripts_branch-validation.md
43+
+++ b/dev/docs_build/dev/reports/PR_26179_OWNER_006-move-bootstrap-scripts_branch-validation.md
44+
@@ -6,5 +6,5 @@ Generated: 2026-06-27T19:11:00.890Z
45+
| --- | --- | --- |
46+
| Current branch is PR branch | PASS | PR_26179_OWNER_006-move-bootstrap-scripts |
47+
| Branch is stacked on previous PR branch | PASS | Base branch: PR_26179_OWNER_005-move-tests-to-dev |
48+
-| Worktree contains only intended project changes before commit | PASS | Script/config move, package path updates, validation docs/tests, reports. |
49+
+| Worktree contains only intended project changes before commit | PASS | Script/config move, package path updates, validation docs/tests, reports, and GitHub Actions moved-script path correction. |
50+
| Product/runtime path hard stop avoided | PASS | No product/runtime directory was moved into dev/. |
51+
diff --git a/dev/docs_build/dev/reports/PR_26179_OWNER_006-move-bootstrap-scripts_manual-validation-notes.md b/dev/docs_build/dev/reports/PR_26179_OWNER_006-move-bootstrap-scripts_manual-validation-notes.md
52+
index acd139c6a..f82cff1cb 100644
53+
--- a/dev/docs_build/dev/reports/PR_26179_OWNER_006-move-bootstrap-scripts_manual-validation-notes.md
54+
+++ b/dev/docs_build/dev/reports/PR_26179_OWNER_006-move-bootstrap-scripts_manual-validation-notes.md
55+
@@ -4,3 +4,5 @@
56+
- No browser storage, API contract, database schema, or product behavior changed.
57+
- Playwright browser execution was not run because this PR moves dev runner/config paths only; moved config discovery and structure audit passed.
58+
- Local API startup was validated by importing the moved startup module and by checking `package.json` command routing.
59+
+- GitHub Actions platform-validation path was corrected from `node ./scripts/run-platform-validation-suite.mjs` to `node ./dev/scripts/run-platform-validation-suite.mjs`.
60+
+- Local platform validation completed 8/8 scenarios after the workflow path fix.
61+
diff --git a/dev/docs_build/dev/reports/PR_26179_OWNER_006-move-bootstrap-scripts_requirement-checklist.md b/dev/docs_build/dev/reports/PR_26179_OWNER_006-move-bootstrap-scripts_requirement-checklist.md
62+
index b69e7145f..8939747f1 100644
63+
--- a/dev/docs_build/dev/reports/PR_26179_OWNER_006-move-bootstrap-scripts_requirement-checklist.md
64+
+++ b/dev/docs_build/dev/reports/PR_26179_OWNER_006-move-bootstrap-scripts_requirement-checklist.md
65+
@@ -5,6 +5,7 @@
66+
| PASS | Move local/test bootstrap scripts and dev-only runner scripts/config into dev/bootstrap/, dev/scripts/, or dev/config/. | Moved tracked root scripts/ to dev/scripts/ and Playwright configs to dev/config/. |
67+
| PASS | Keep application runtime/business logic out of dev/. | No src/, product UI, API contract, database, assets, docs/, games/, toolbox/, account/, admin/, or legal runtime files were moved into dev/. |
68+
| PASS | Ensure npm run dev:local-api still works. | package.json now points dev:local-api at dev/scripts/start-local-api-server.mjs; syntax/import validation passed without launching a long-running server. |
69+
+| PASS | GitHub Actions references moved script paths. | .github/workflows/platform-validation.yml now runs node ./dev/scripts/run-platform-validation-suite.mjs. |
70+
| PASS | No Creator-writeable repo folder introduced. | This PR moves dev tooling only and adds no Creator data write path. |
71+
| PASS | No runtime/business logic scope expansion. | Changes are limited to dev scripts/config, package script paths, validation tests/docs, and generated reports. |
72+
| PASS | No broad unrelated cleanup. | No product/runtime implementation files were changed. |
73+
diff --git a/dev/docs_build/dev/reports/PR_26179_OWNER_006-move-bootstrap-scripts_validation-lane.md b/dev/docs_build/dev/reports/PR_26179_OWNER_006-move-bootstrap-scripts_validation-lane.md
74+
index 2592e1ef6..dd8d9f18b 100644
75+
--- a/dev/docs_build/dev/reports/PR_26179_OWNER_006-move-bootstrap-scripts_validation-lane.md
76+
+++ b/dev/docs_build/dev/reports/PR_26179_OWNER_006-move-bootstrap-scripts_validation-lane.md
77+
@@ -11,4 +11,6 @@
78+
| PASS | Playwright structure audit through moved runner | npm run test:audit:locations |
79+
| PASS | Playwright moved config discovery | npx playwright test --config=dev/config/playwright.config.cjs --list |
80+
| PASS | Canonical structure validation | npm run validate:canonical-structure |
81+
+| PASS | GitHub Actions platform-validation path | Workflow now calls node ./dev/scripts/run-platform-validation-suite.mjs; targeted grep confirms no .github workflow still calls node ./scripts/run-platform-validation-suite.mjs. |
82+
+| PASS | Platform validation suite | node ./dev/scripts/run-platform-validation-suite.mjs completed 8/8 scenarios. |
83+
| PASS | Whitespace validation | git diff --check -- . |
84+
185
diff --git a/codex_playwright_system_chrome.config.cjs b/dev/config/codex_playwright_system_chrome.config.cjs
286
similarity index 100%
387
rename from codex_playwright_system_chrome.config.cjs

0 commit comments

Comments
 (0)