Skip to content

Commit 119fa88

Browse files
ToolboxAidToolboxAid
committed
Move bootstrap scripts into dev workspace
1 parent c36a053 commit 119fa88

92 files changed

Lines changed: 24003 additions & 19847 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.

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: 58 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,58 @@
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 moved script repo-root calculations, script self-references, PowerShell helper paths, and tests that import script modules.
18+
- Kept application runtime/business logic out of `dev/`.
19+
20+
## Scope Confirmation
21+
22+
Documentation/dev-runner/config only. No product UI, API, database, `src/`, `docs/`, `games/`, `toolbox/`, `account/`, `admin/`, `legal/`, or asset runtime behavior was moved.
23+
24+
## Requirement Checklist
25+
26+
| Status | Requirement | Evidence |
27+
| --- | --- | --- |
28+
| 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/. |
29+
| 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/. |
30+
| 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. |
31+
| PASS | No Creator-writeable repo folder introduced. | This PR moves dev tooling only and adds no Creator data write path. |
32+
| PASS | No runtime/business logic scope expansion. | Changes are limited to dev scripts/config, package script paths, validation tests/docs, and generated reports. |
33+
| PASS | No broad unrelated cleanup. | No product/runtime implementation files were changed. |
34+
35+
## Validation
36+
37+
| Status | Lane | Evidence |
38+
| --- | --- | --- |
39+
| PASS | Current branch | PR_26179_OWNER_006-move-bootstrap-scripts |
40+
| PASS | Base branch for stack | PR_26179_OWNER_005-move-tests-to-dev |
41+
| PASS | node --check changed JS/CJS/MJS files | 32 changed JS/CJS/MJS files checked successfully. |
42+
| PASS | Local API bootstrap import | Imported dev/scripts/start-local-api-server.mjs and verified formatStartupLogLines export. |
43+
| 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 |
44+
| PASS | Service API lane through moved runner | npm run test:service:api |
45+
| PASS | Playwright structure audit through moved runner | npm run test:audit:locations |
46+
| PASS | Playwright moved config discovery | npx playwright test --config=dev/config/playwright.config.cjs --list |
47+
| PASS | Canonical structure validation | npm run validate:canonical-structure |
48+
| PASS | Whitespace validation | git diff --check -- . |
49+
50+
## Manual Validation Notes
51+
52+
- No UI changes were made.
53+
- 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.
54+
- `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.
55+
56+
## Changed Files
57+
58+
`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. |
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: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,6 @@
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.

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

Lines changed: 10 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,10 @@
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 | No Creator-writeable repo folder introduced. | This PR moves dev tooling only and adds no Creator data write path. |
9+
| PASS | No runtime/business logic scope expansion. | Changes are limited to dev scripts/config, package script paths, validation tests/docs, and generated reports. |
10+
| 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: 14 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,14 @@
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 | Whitespace validation | git diff --check -- . |

0 commit comments

Comments
 (0)