ALFA 009 harden bootstrap role parsing

Author: ToolboxAid

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

@@ -26,7 +26,7 @@ Expected diagnostics:
 - `Role: codex`
 - `Web URL: http://127.0.0.1:5512`
 - `API URL: http://127.0.0.1:5513/api`
-- `Browser launch: suppressed for codex role`
+- `Browser launch: skipped for codex role`
 
 ## Legacy Command
 

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

@@ -29,7 +29,7 @@ Per OWNER instruction, those uncommitted changes were discarded with `git reset
 - Added `dev/scripts/start-dev.mjs` as the bootstrap orchestrator.
 - Added focused node tests for team resolution, role offsets, invalid team/role validation, script wiring, `.env` loading behavior, and bootstrap diagnostics.
 - 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.
+- Skipped 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.
 
@@ -94,10 +94,10 @@ npm run dev:bootstrap -- --team alfa --role codex
 ## Browser Launch Behavior
 
 - `owner` role launches the browser automatically to the selected team's `index.html`.
-- `codex` role suppresses browser launch.
+- `codex` role skips browser launch.
 - Browser launch happens only after both API and web servers are ready.
 - 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 Alfa codex browser launch is skipped 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` 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.

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

@@ -18,14 +18,15 @@
 | 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. |
+| Support positional team/role forwarding | PASS | `npm run dev:bootstrap -- alfa codex` resolves Alfa codex ports and skips browser launch. |
 | 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. |
 | Start web server when requested | PASS | `dev:bootstrap` starts web plus API; `dev:web` mode is wired. |
 | Print startup diagnostics | PASS | Diagnostics include mode, team, role, web URL, API URL, environment source, supported teams, and supported roles. |
 | Owner role launches browser automatically | PASS | Live startup validation opened `http://127.0.0.1:5510/index.html` after API and web were ready. |
-| Codex role suppresses browser launch | PASS | Live startup validation reported `Browser launch: suppressed for codex role`. |
+| Codex role skips browser launch | PASS | Live startup validation reported `Browser launch: skipped for codex role`. |
 | Wait for both servers before browser launch | PASS | Targeted unit test asserts API ready, web ready, then browser launch order. |
 | Preserve legacy behavior | PASS | `dev:local-api` command unchanged and startup validation passed. |
 | No unrelated cleanup | PASS | Scope limited to package scripts, new dev scripts, targeted tests, and reports. |

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

@@ -12,7 +12,8 @@
 | 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 alfa --role codex` | PASS | Started with team `alfa`, role `codex`, web `5512`, API `5513`, skipped browser launch, then process tree was stopped. |
+| `npm run dev:bootstrap -- alfa codex` | PASS | Positional team/role forwarding selected team `alfa`, role `codex`, web `5512`, API `5513`, and skipped browser launch. |
 | `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`. |

dev/reports/codex_changed_files.txt

@@ -4,16 +4,18 @@ 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
-...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(-)
+...team-aware-bootstrap_manual-validation-notes.md |  2 +-
+ ...R_26171_ALFA_009-team-aware-bootstrap_report.md |  6 +-
+ ...9-team-aware-bootstrap_requirement-checklist.md |  3 +-
+ ...A_009-team-aware-bootstrap_validation-report.md |  3 +-
+ dev/scripts/start-dev.mjs                          |  2 +-
+ dev/scripts/team-port-config.mjs                   | 31 ++++++++--
+ dev/tests/dev-runtime/TeamAwareBootstrap.test.mjs  | 71 ++++++++++++++++------
+ 7 files changed, 87 insertions(+), 31 deletions(-)