Skip to content

Commit 74b8960

Browse files
ToolboxAidToolboxAid
authored
Merge PR #241: PR_26179_OWNER_006-move-bootstrap-scripts
Owner-approved restructure stack closeout.
2 parents 3060427 + 8bb50a3 commit 74b8960

93 files changed

Lines changed: 24098 additions & 19848 deletions

File tree

Some content is hidden

Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.

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

codex_playwright_system_chrome.config.cjs renamed to dev/config/codex_playwright_system_chrome.config.cjs

File renamed without changes.

‎dev/config/playwright.config.cjs‎

Lines changed: 33 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,33 @@
1+
const path = require("node:path");
2+
3+
const repoRoot = path.resolve(__dirname, "../..");
4+
const repoPath = (...segments) => path.join(repoRoot, ...segments);
5+
6+
module.exports = {
7+
timeout: 120000,
8+
outputDir: repoPath("tmp", "test-results"),
9+
projects: [
10+
{
11+
name: "ui",
12+
testDir: repoPath("dev", "tests", "ui"),
13+
outputDir: repoPath("tmp", "test-results", "artifacts")
14+
},
15+
{
16+
name: "playwright",
17+
testDir: repoPath("dev", "tests", "playwright"),
18+
outputDir: repoPath("tmp", "test-results", "artifacts")
19+
}
20+
],
21+
reporter: [
22+
["list"],
23+
["html", { outputFolder: repoPath("tmp", "test-results", "report"), open: "always" }],
24+
["json", { outputFile: repoPath("tmp", "test-results", "playwright-results.json") }]
25+
],
26+
use: {
27+
headless: false,
28+
launchOptions: {
29+
slowMo: 5
30+
},
31+
trace: "on"
32+
}
33+
};

‎dev/docs_build/dev/ProjectInstructions/addendums/workspace_v2_playwright_gate.md‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -13,7 +13,7 @@ Use this gate for repeatable local validation of Workspace Manager V2 UI behavio
1313
- Test artifacts are written under `tmp/test-results/`.
1414
- Trace artifacts are written under `tmp/test-results/**` (for example in artifacts folders).
1515
- HTML report is written under `tmp/test-results/report`.
16-
- Reporter auto-open behavior is controlled by `playwright.config.cjs`.
16+
- Reporter auto-open behavior is controlled by `dev/config/playwright.config.cjs`.
1717

1818
## Pass/Fail Behavior
1919
- Success:

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

Lines changed: 62 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,62 @@
1+
# PR_26179_OWNER_006-move-bootstrap-scripts
2+
3+
Generated: 2026-06-27T19:11:00.890Z
4+
Branch: PR_26179_OWNER_006-move-bootstrap-scripts
5+
Base: PR_26179_OWNER_005-move-tests-to-dev
6+
HEAD before report commit: c36a05304b3593ca2a0b6d8cbb99fefc2bb92922
7+
8+
## Purpose
9+
10+
Move local/test bootstrap scripts and dev-only runner scripts/config into the dev workspace while preserving existing command surfaces.
11+
12+
## Summary
13+
14+
- Moved tracked root `scripts/` files to `dev/scripts/`.
15+
- Moved root Playwright configs to `dev/config/`.
16+
- Updated `package.json` scripts to call `dev/scripts/`.
17+
- Updated GitHub Actions platform validation to call `node ./dev/scripts/run-platform-validation-suite.mjs`.
18+
- Updated moved script repo-root calculations, script self-references, PowerShell helper paths, and tests that import script modules.
19+
- Kept application runtime/business logic out of `dev/`.
20+
21+
## Scope Confirmation
22+
23+
Documentation/dev-runner/config only. No product UI, API, database, `src/`, `docs/`, `games/`, `toolbox/`, `account/`, `admin/`, `legal/`, or asset runtime behavior was moved.
24+
25+
## Requirement Checklist
26+
27+
| Status | Requirement | Evidence |
28+
| --- | --- | --- |
29+
| 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/. |
30+
| 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/. |
31+
| 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. |
32+
| PASS | No Creator-writeable repo folder introduced. | This PR moves dev tooling only and adds no Creator data write path. |
33+
| PASS | No runtime/business logic scope expansion. | Changes are limited to dev scripts/config, package script paths, validation tests/docs, and generated reports. |
34+
| PASS | No broad unrelated cleanup. | No product/runtime implementation files were changed. |
35+
36+
## Validation
37+
38+
| Status | Lane | Evidence |
39+
| --- | --- | --- |
40+
| PASS | Current branch | PR_26179_OWNER_006-move-bootstrap-scripts |
41+
| PASS | Base branch for stack | PR_26179_OWNER_005-move-tests-to-dev |
42+
| PASS | node --check changed JS/CJS/MJS files | 32 changed JS/CJS/MJS files checked successfully. |
43+
| PASS | Local API bootstrap import | Imported dev/scripts/start-local-api-server.mjs and verified formatStartupLogLines export. |
44+
| PASS | Targeted moved-script tests | node ./dev/scripts/run-node-test-files.mjs dev/tests/dev-runtime/LocalApiStartupLogging.test.mjs dev/tests/tools/AssetOwnershipStrategyCloseout.test.mjs dev/tests/regression/CanonicalRepositoryStructureGuardrail.test.mjs |
45+
| PASS | Service API lane through moved runner | npm run test:service:api |
46+
| PASS | Playwright structure audit through moved runner | npm run test:audit:locations |
47+
| PASS | Playwright moved config discovery | npx playwright test --config=dev/config/playwright.config.cjs --list |
48+
| 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. |
51+
| PASS | Whitespace validation | git diff --check -- . |
52+
53+
## Manual Validation Notes
54+
55+
- No UI changes were made.
56+
- 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.
57+
- `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.
59+
60+
## Changed Files
61+
62+
`dev/docs_build/dev/reports/codex_changed_files.txt` contains the full name-status list.

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

Lines changed: 10 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,10 @@
1+
# Branch Validation - PR_26179_OWNER_006-move-bootstrap-scripts
2+
3+
Generated: 2026-06-27T19:11:00.890Z
4+
5+
| Check | Result | Evidence |
6+
| --- | --- | --- |
7+
| Current branch is PR branch | PASS | PR_26179_OWNER_006-move-bootstrap-scripts |
8+
| 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, and GitHub Actions moved-script path correction. |
10+
| 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: 8 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,8 @@
1+
# Manual Validation Notes - PR_26179_OWNER_006-move-bootstrap-scripts
2+
3+
- No runtime UI was changed.
4+
- No browser storage, API contract, database schema, or product behavior changed.
5+
- Playwright browser execution was not run because this PR moves dev runner/config paths only; moved config discovery and structure audit passed.
6+
- 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: 11 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,11 @@
1+
# Requirement Checklist - PR_26179_OWNER_006-move-bootstrap-scripts
2+
3+
| Status | Requirement | Evidence |
4+
| --- | --- | --- |
5+
| 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/. |
6+
| 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/. |
7+
| 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. |
9+
| PASS | No Creator-writeable repo folder introduced. | This PR moves dev tooling only and adds no Creator data write path. |
10+
| PASS | No runtime/business logic scope expansion. | Changes are limited to dev scripts/config, package script paths, validation tests/docs, and generated reports. |
11+
| 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: 16 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,16 @@
1+
# Validation Lane - PR_26179_OWNER_006-move-bootstrap-scripts
2+
3+
| Status | Validation | Notes |
4+
| --- | --- | --- |
5+
| PASS | Current branch | PR_26179_OWNER_006-move-bootstrap-scripts |
6+
| PASS | Base branch for stack | PR_26179_OWNER_005-move-tests-to-dev |
7+
| PASS | node --check changed JS/CJS/MJS files | 32 changed JS/CJS/MJS files checked successfully. |
8+
| PASS | Local API bootstrap import | Imported dev/scripts/start-local-api-server.mjs and verified formatStartupLogLines export. |
9+
| PASS | Targeted moved-script tests | node ./dev/scripts/run-node-test-files.mjs dev/tests/dev-runtime/LocalApiStartupLogging.test.mjs dev/tests/tools/AssetOwnershipStrategyCloseout.test.mjs dev/tests/regression/CanonicalRepositoryStructureGuardrail.test.mjs |
10+
| PASS | Service API lane through moved runner | npm run test:service:api |
11+
| PASS | Playwright structure audit through moved runner | npm run test:audit:locations |
12+
| PASS | Playwright moved config discovery | npx playwright test --config=dev/config/playwright.config.cjs --list |
13+
| 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. |
16+
| PASS | Whitespace validation | git diff --check -- . |

0 commit comments

Comments
 (0)