Skip to content

Commit 3594ef8

Browse files
committed
ALFA 009 add owner browser launch
1 parent 149c383 commit 3594ef8

8 files changed

Lines changed: 555 additions & 904 deletions

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

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -12,6 +12,7 @@ Expected diagnostics:
1212
- `Role: owner`
1313
- `Web URL: http://127.0.0.1:5510`
1414
- `API URL: http://127.0.0.1:5511/api`
15+
- `Browser launch: opened http://127.0.0.1:5510/index.html`
1516

1617
## Primary Alfa Codex Command
1718

@@ -25,6 +26,7 @@ Expected diagnostics:
2526
- `Role: codex`
2627
- `Web URL: http://127.0.0.1:5512`
2728
- `API URL: http://127.0.0.1:5513/api`
29+
- `Browser launch: suppressed for codex role`
2830

2931
## Legacy Command
3032

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

Lines changed: 10 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -28,6 +28,8 @@ Per OWNER instruction, those uncommitted changes were discarded with `git reset
2828
- Added `dev/scripts/team-port-config.mjs` for full team name and role-aware port resolution.
2929
- Added `dev/scripts/start-dev.mjs` as the bootstrap orchestrator.
3030
- Added focused node tests for team resolution, role offsets, invalid team/role validation, script wiring, `.env` loading behavior, and bootstrap diagnostics.
31+
- Added automatic browser launch for owner-role `dev:bootstrap` after the API and web servers are both ready.
32+
- Suppressed automatic browser launch for the `codex` role.
3133

3234
## Supported Teams
3335

@@ -87,6 +89,14 @@ Codex role example:
8789
npm run dev:bootstrap -- --team alfa --role codex
8890
```
8991

92+
## Browser Launch Behavior
93+
94+
- `owner` role launches the browser automatically to the selected team's `index.html`.
95+
- `codex` role suppresses browser launch.
96+
- Browser launch happens only after both API and web servers are ready.
97+
- The Alfa owner launch target is `http://127.0.0.1:5510/index.html`.
98+
- The Alfa codex launch target is suppressed while using web `5512` and API `5513`.
99+
90100
## Files Changed
91101

92102
- `package.json` - updated

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

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -21,5 +21,8 @@
2121
| Start Local API | PASS | `dev:bootstrap` starts the existing Local API server on resolved API port. |
2222
| Start web server when requested | PASS | `dev:bootstrap` starts web plus API; `dev:web` mode is wired. |
2323
| Print startup diagnostics | PASS | Diagnostics include mode, team, role, web URL, API URL, environment source, supported teams, and supported roles. |
24+
| Owner role launches browser automatically | PASS | Live startup validation opened `http://127.0.0.1:5510/index.html` after API and web were ready. |
25+
| Codex role suppresses browser launch | PASS | Live startup validation reported `Browser launch: suppressed for codex role`. |
26+
| Wait for both servers before browser launch | PASS | Targeted unit test asserts API ready, web ready, then browser launch order. |
2427
| Preserve legacy behavior | PASS | `dev:local-api` command unchanged and startup validation passed. |
2528
| 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

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -8,15 +8,15 @@
88
| `node --check dev/scripts/start-dev.mjs` | PASS | Syntax valid. |
99
| `node --check dev/scripts/start-local-api-server.mjs` | PASS | Legacy startup entrypoint remains valid. |
1010
| `node --check dev/tests/dev-runtime/TeamAwareBootstrap.test.mjs` | PASS | Test syntax valid. |
11-
| `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; 10 subtests passed. |
11+
| `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. |
1212
| Unsupported team token scan in new bootstrap files | PASS | No unsupported old team names or single-letter public team commands found. |
1313
| `npm run validate:canonical-structure` | PASS | Canonical repository structure guardrail passed. |
14-
| `npm run dev:bootstrap -- --team alfa` | PASS | Started with team `alfa`, role `owner`, web `5510`, API `5511`, then process tree was stopped. |
15-
| `npm run dev:bootstrap -- --team alfa --role codex` | PASS | Started with team `alfa`, role `codex`, web `5512`, API `5513`, then process tree was stopped. |
14+
| `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. |
15+
| `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. |
1616
| `npm run dev:bootstrap -- --team omega` | PASS | Failed clearly with supported team list. |
1717
| `npm run dev:bootstrap -- --team alfa --role reviewer` | PASS | Failed clearly with supported role list. |
1818
| `npm run dev:local-api` | PASS | Legacy command started and printed legacy startup diagnostics, then process tree was stopped. |
1919

2020
## Notes
2121

22-
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.
22+
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.
Lines changed: 14 additions & 19 deletions
Original file line numberDiff line numberDiff line change
@@ -1,24 +1,19 @@
11
# git status --short
2-
M package.json
3-
?? dev/reports/PR_26171_ALFA_009-team-aware-bootstrap_branch-validation.md
4-
?? dev/reports/PR_26171_ALFA_009-team-aware-bootstrap_manual-validation-notes.md
5-
?? dev/reports/PR_26171_ALFA_009-team-aware-bootstrap_report.md
6-
?? dev/reports/PR_26171_ALFA_009-team-aware-bootstrap_requirement-checklist.md
7-
?? dev/reports/PR_26171_ALFA_009-team-aware-bootstrap_validation-report.md
8-
?? dev/scripts/start-dev.mjs
9-
?? dev/scripts/team-port-config.mjs
10-
?? dev/tests/dev-runtime/TeamAwareBootstrap.test.mjs
2+
M dev/reports/PR_26171_ALFA_009-team-aware-bootstrap_manual-validation-notes.md
3+
M dev/reports/PR_26171_ALFA_009-team-aware-bootstrap_report.md
4+
M dev/reports/PR_26171_ALFA_009-team-aware-bootstrap_requirement-checklist.md
5+
M dev/reports/PR_26171_ALFA_009-team-aware-bootstrap_validation-report.md
6+
M dev/scripts/start-dev.mjs
7+
M dev/tests/dev-runtime/TeamAwareBootstrap.test.mjs
118

129
# git ls-files --others --exclude-standard
13-
dev/reports/PR_26171_ALFA_009-team-aware-bootstrap_branch-validation.md
14-
dev/reports/PR_26171_ALFA_009-team-aware-bootstrap_manual-validation-notes.md
15-
dev/reports/PR_26171_ALFA_009-team-aware-bootstrap_report.md
16-
dev/reports/PR_26171_ALFA_009-team-aware-bootstrap_requirement-checklist.md
17-
dev/reports/PR_26171_ALFA_009-team-aware-bootstrap_validation-report.md
18-
dev/scripts/start-dev.mjs
19-
dev/scripts/team-port-config.mjs
20-
dev/tests/dev-runtime/TeamAwareBootstrap.test.mjs
10+
(no output)
2111

2212
# git diff --stat
23-
package.json | 3 +++
24-
1 file changed, 3 insertions(+)
13+
...team-aware-bootstrap_manual-validation-notes.md | 2 +
14+
...R_26171_ALFA_009-team-aware-bootstrap_report.md | 10 ++
15+
...9-team-aware-bootstrap_requirement-checklist.md | 3 +
16+
...A_009-team-aware-bootstrap_validation-report.md | 8 +-
17+
dev/scripts/start-dev.mjs | 76 +++++++++++--
18+
dev/tests/dev-runtime/TeamAwareBootstrap.test.mjs | 119 +++++++++++++++++++++
19+
6 files changed, 208 insertions(+), 10 deletions(-)

0 commit comments

Comments
 (0)