ALFA 009 fix bootstrap team forwarding

Author: ToolboxAid

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

@@ -45,13 +45,21 @@ Expected result:
 npm run dev:bootstrap -- --team owner
 npm run dev:bootstrap -- --team bravo
 npm run dev:bootstrap -- --team charlie
+npm run dev:bootstrap -- charlie
 npm run dev:bootstrap -- --team delta
 npm run dev:bootstrap -- --team echo
 npm run dev:bootstrap -- --team foxtrot
 npm run dev:bootstrap -- --team golf
 npm run dev:bootstrap -- --team hotel
 ```
 
+For Charlie, both forms should show:
+
+- `Team: charlie`
+- `Web URL: http://127.0.0.1:5530`
+- `API URL: http://127.0.0.1:5531/api`
+- `Browser launch: http://127.0.0.1:5530/index.html`
+
 ## Failure Checks
 
 ```powershell

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

@@ -30,6 +30,7 @@ Per OWNER instruction, those uncommitted changes were discarded with `git reset
 - 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.
+- Fixed npm argument forwarding/parsing so both `--team charlie` and positional `charlie` select the same team after the package script's `--mode bootstrap` argument.
 
 ## Supported Teams
 
@@ -96,6 +97,8 @@ npm run dev:bootstrap -- --team alfa --role codex
 - 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 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.
 
 ## Files Changed
 

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

@@ -16,6 +16,9 @@
 | Do not use single-letter public team commands | PASS | Scan returned no matches in new bootstrap files. |
 | Validate unknown team names | PASS | `omega` fails clearly with supported team list. |
 | 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. |
 | 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

@@ -13,10 +13,13 @@
 | `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 -- 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. |
 | `npm run dev:bootstrap -- --team alfa --role reviewer` | PASS | Failed clearly with supported role list. |
 | `npm run dev:local-api` | PASS | Legacy command started and printed legacy startup diagnostics, then process tree was stopped. |
 
 ## 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. The owner-role validation opened the configured browser target after both server diagnostics were available.
+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.

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
-...team-aware-bootstrap_manual-validation-notes.md |   2 +
- ...R_26171_ALFA_009-team-aware-bootstrap_report.md |  10 ++
- ...9-team-aware-bootstrap_requirement-checklist.md |   3 +
- ...A_009-team-aware-bootstrap_validation-report.md |   8 +-
- dev/scripts/start-dev.mjs                          |  76 +++++++++++--
- dev/tests/dev-runtime/TeamAwareBootstrap.test.mjs  | 119 +++++++++++++++++++++
- 6 files changed, 208 insertions(+), 10 deletions(-)
+...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(-)