Skip to content

Commit d581b01

Browse files
committed
Merge remote-tracking branch 'origin/main' into PR_26179_OWNER_008-update-path-governance-final
2 parents 6831a2c + 17600f9 commit d581b01

5 files changed

Lines changed: 10 additions & 1 deletion

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/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/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/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/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 -- . |

0 commit comments

Comments
 (0)