ALFA 009 fix bootstrap env port overrides

Author: ToolboxAid

dev/reports/PR_26171_ALFA_009-team-aware-bootstrap_manual-validation-notes.md

@@ -12,7 +12,7 @@ Expected diagnostics:
 - `Role: owner`
 - `Web URL: http://127.0.0.1:5510`
 - `API URL: http://127.0.0.1:5511/api`
-- `Browser launch: opened http://127.0.0.1:5510/index.html`
+- `Browser launch: http://127.0.0.1:5510/index.html`
 
 ## Primary Alfa Codex Command
 
@@ -60,6 +60,8 @@ For Charlie, both forms should show:
 - `API URL: http://127.0.0.1:5531/api`
 - `Browser launch: http://127.0.0.1:5530/index.html`
 
+If `.env` or the current shell already has `GAMEFOUNDRY_SITE_URL` or `GAMEFOUNDRY_API_URL`, bootstrap should still show the resolved team/role ports in diagnostics and launch the browser from the computed `GAMEFOUNDRY_SITE_URL`.
+
 ## Failure Checks
 
 ```powershell

dev/reports/PR_26171_ALFA_009-team-aware-bootstrap_report.md

@@ -31,6 +31,7 @@ Per OWNER instruction, those uncommitted changes were discarded with `git reset
 - Added automatic browser launch for owner-role `dev:bootstrap` after the API and web servers are both ready.
 - Suppressed automatic browser launch for the `codex` role.
 - Fixed npm argument forwarding/parsing so both `--team charlie` and positional `charlie` select the same team after the package script's `--mode bootstrap` argument.
+- Treats `.env` `GAMEFOUNDRY_SITE_URL` and `GAMEFOUNDRY_API_URL` as legacy/default local values for bootstrap mode, then overwrites runtime URLs from the resolved team/role ports.
 
 ## Supported Teams
 
@@ -98,7 +99,8 @@ npm run dev:bootstrap -- --team alfa --role codex
 - The Alfa owner launch target is `http://127.0.0.1:5510/index.html`.
 - The Alfa codex launch target is suppressed while using web `5512` and API `5513`.
 - The Charlie launch target is `http://127.0.0.1:5530/index.html` for both `npm run dev:bootstrap -- --team charlie` and `npm run dev:bootstrap -- charlie`.
-- Computed team/role ports override any `.env` port values loaded before bootstrap startup.
+- Computed team/role ports override any `.env` or inherited process `GAMEFOUNDRY_SITE_URL` / `GAMEFOUNDRY_API_URL` values loaded before bootstrap startup.
+- Bootstrap runtime sets `GAMEFOUNDRY_SITE_URL=http://127.0.0.1:<webPort>` and `GAMEFOUNDRY_API_URL=http://127.0.0.1:<apiPort>/api` after team/role resolution.
 
 ## Files Changed
 

dev/reports/PR_26171_ALFA_009-team-aware-bootstrap_requirement-checklist.md

@@ -18,7 +18,7 @@
 | Validate unknown role names | PASS | `reviewer` fails clearly with supported role list. |
 | Support `--team` argument forwarding | PASS | `npm run dev:bootstrap -- --team charlie` resolves Charlie ports. |
 | Support positional team forwarding | PASS | `npm run dev:bootstrap -- charlie` resolves Charlie ports. |
-| Prevent `.env` port override | PASS | Unit test confirms computed team/role ports overwrite loaded API/site port values. |
+| Prevent `.env` and process URL port override | PASS | Unit and live startup tests confirm computed team/role ports overwrite loaded or inherited API/site URL port values. |
 | Load environment | PASS | Orchestrator loads `.env` values without overriding existing environment values. |
 | Resolve team ports | PASS | Unit tests cover all teams and both roles. |
 | Start Local API | PASS | `dev:bootstrap` starts the existing Local API server on resolved API port. |

dev/reports/PR_26171_ALFA_009-team-aware-bootstrap_validation-report.md

@@ -9,11 +9,11 @@
 | `node --check dev/scripts/start-local-api-server.mjs` | PASS | Legacy startup entrypoint remains valid. |
 | `node --check dev/tests/dev-runtime/TeamAwareBootstrap.test.mjs` | PASS | Test syntax valid. |
 | `node ./dev/scripts/run-node-test-files.mjs dev/tests/dev-runtime/TeamAwareBootstrap.test.mjs dev/tests/dev-runtime/LocalApiStartupLogging.test.mjs` | PASS | 2 targeted files passed; 14 subtests passed. |
-| Unsupported team token scan in new bootstrap files | PASS | No unsupported old team names or single-letter public team commands found. |
+| Unsupported team token scan in new bootstrap files | PASS | No unsupported old team names or single-letter public team commands found in package scripts, bootstrap scripts, or targeted bootstrap tests. |
 | `npm run validate:canonical-structure` | PASS | Canonical repository structure guardrail passed. |
 | `npm run dev:bootstrap -- --team alfa` | PASS | Started with team `alfa`, role `owner`, web `5510`, API `5511`, launched browser to `http://127.0.0.1:5510/index.html`, then process tree was stopped. |
 | `npm run dev:bootstrap -- --team alfa --role codex` | PASS | Started with team `alfa`, role `codex`, web `5512`, API `5513`, suppressed browser launch, then process tree was stopped. |
-| `npm run dev:bootstrap -- --team charlie` | PASS | Started with team `charlie`, role `owner`, web `5530`, API `5531`, launched browser to `http://127.0.0.1:5530/index.html`, then process tree was stopped. |
+| `npm run dev:bootstrap -- --team charlie` with inherited stale URL env values | PASS | Started with team `charlie`, role `owner`, web `5530`, API `5531`, launched browser to `http://127.0.0.1:5530/index.html`, and ignored inherited `GAMEFOUNDRY_SITE_URL` / `GAMEFOUNDRY_API_URL` port values before process cleanup. |
 | `npm run dev:bootstrap -- charlie` | PASS | Positional team forwarding selected team `charlie`, web `5530`, API `5531`, and browser target `http://127.0.0.1:5530/index.html`. |
 | `npm run dev:bootstrap -- --team bravo` | PASS | Started with team `bravo`, role `owner`, web `5520`, API `5521`, and browser target `http://127.0.0.1:5520/index.html`. |
 | `npm run dev:bootstrap -- --team omega` | PASS | Failed clearly with supported team list. |
@@ -22,4 +22,4 @@
 
 ## Notes
 
-The startup-command validations used short-lived process launches and `taskkill /T /F` cleanup after expected diagnostics appeared, so no dev servers were left running. Owner-role validations opened the configured browser target after both server diagnostics were available. Unit coverage verifies loaded `.env` port values are overwritten by computed team/role ports.
+The startup-command validations used short-lived process launches and `taskkill /T /F` cleanup after expected diagnostics appeared, so no dev servers were left running. Owner-role validations opened the configured browser target after both server diagnostics were available. Unit coverage verifies loaded `.env` and inherited process URL values are overwritten by computed team/role ports.

dev/reports/codex_changed_files.txt

@@ -4,18 +4,16 @@ M dev/reports/PR_26171_ALFA_009-team-aware-bootstrap_manual-validation-notes.md
  M dev/reports/PR_26171_ALFA_009-team-aware-bootstrap_requirement-checklist.md
  M dev/reports/PR_26171_ALFA_009-team-aware-bootstrap_validation-report.md
  M dev/scripts/start-dev.mjs
- M dev/scripts/team-port-config.mjs
  M dev/tests/dev-runtime/TeamAwareBootstrap.test.mjs
 
 # git ls-files --others --exclude-standard
 (no output)
 
 # git diff --stat
-...team-aware-bootstrap_manual-validation-notes.md |  8 ++++++
- ...R_26171_ALFA_009-team-aware-bootstrap_report.md |  3 ++
- ...9-team-aware-bootstrap_requirement-checklist.md |  3 ++
- ...A_009-team-aware-bootstrap_validation-report.md |  5 +++-
- dev/scripts/start-dev.mjs                          |  2 +-
- dev/scripts/team-port-config.mjs                   | 29 +++++++++++++++++++-
- dev/tests/dev-runtime/TeamAwareBootstrap.test.mjs  | 32 +++++++++++++++++++++-
- 7 files changed, 78 insertions(+), 4 deletions(-)
+...LFA_009-team-aware-bootstrap_manual-validation-notes.md |  4 +++-
+ .../PR_26171_ALFA_009-team-aware-bootstrap_report.md       |  4 +++-
+ ..._ALFA_009-team-aware-bootstrap_requirement-checklist.md |  2 +-
+ ...6171_ALFA_009-team-aware-bootstrap_validation-report.md |  6 +++---
+ dev/scripts/start-dev.mjs                                  |  6 +++---
+ dev/tests/dev-runtime/TeamAwareBootstrap.test.mjs          | 14 +++++++++++---
+ 6 files changed, 24 insertions(+), 12 deletions(-)