Skip to content

PR_26180_OWNER_007-www-route-root-compatibility#286

Draft
ToolboxAid wants to merge 1 commit into
PR_26180_OWNER_006-www-migration-mapfrom
PR_26180_OWNER_007-www-route-root-compatibility
Draft

PR_26180_OWNER_007-www-route-root-compatibility#286
ToolboxAid wants to merge 1 commit into
PR_26180_OWNER_006-www-migration-mapfrom
PR_26180_OWNER_007-www-route-root-compatibility

Conversation

@ToolboxAid

Copy link
Copy Markdown
Owner

Summary

Prepares runtime/test route-root compatibility for the future www/ migration without moving browser-served files or changing public URLs.

Scope

  • Adds shared local static route resolution in src/dev-runtime/server/static-web-root.mjs.
  • Updates local static serving and Playwright repo helpers to support a configurable web root.
  • Keeps default behavior as current repository-root serving.
  • Documents the future toggle: GAMEFOUNDRY_LOCAL_WEB_ROOT=www.
  • Adds targeted Node and Playwright route-root compatibility coverage.

Validation

  • node --check on changed JS/test files: PASS
  • node ./dev/scripts/run-node-test-files.mjs dev/tests/dev-runtime/StaticWebRootCompatibility.test.mjs: PASS
  • npx playwright test dev/tests/playwright/tools/StaticWebRootCompatibility.spec.mjs --reporter=line: PASS, 2/2
  • git diff --check: PASS
  • npm run validate:canonical-structure: PASS

Reports

  • dev/reports/codex_review.diff
  • dev/reports/codex_changed_files.txt
  • dev/reports/PR_26180_OWNER_007-www-route-root-compatibility_report.md
  • dev/reports/PR_26180_OWNER_007-www-route-root-compatibility_branch-validation.md
  • dev/reports/PR_26180_OWNER_007-www-route-root-compatibility_requirement-checklist.md
  • dev/reports/PR_26180_OWNER_007-www-route-root-compatibility_validation-report.md
  • dev/reports/PR_26180_OWNER_007-www-route-root-compatibility_manual-validation-notes.md

Artifact

  • dev/workspace/zips/PR_26180_OWNER_007-www-route-root-compatibility_delta.zip

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants