PR_26171_ALFA_009-team-aware-bootstrap#250
Merged
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
PR_26171_ALFA_009 Team-Aware Bootstrap Report
Purpose
Implement team-aware local dev bootstrap commands with role-aware port assignment while preserving the legacy local API command.
Reset Note
Before rebuilding from clean
main, the failed dirty attempt included:dev/reports/codex_changed_files.txtdev/reports/codex_review.diffdev/scripts/start-local-api-server.mjspackage.jsondev/scripts/start-dev.mjsdev/scripts/team-port-config.mjsdev/tests/dev-runtime/TeamAwareBootstrap.test.mjsPer OWNER instruction, those uncommitted changes were discarded with
git reset --hardandgit clean -fd, then this branch was recreated from cleanmain.Implementation Summary
npm run dev:bootstrap.npm run dev:api.npm run dev:web.npm run dev:local-apiunchanged as the legacy alias.dev/scripts/team-port-config.mjsfor full team name and role-aware port resolution.dev/scripts/start-dev.mjsas the bootstrap orchestrator..envloading behavior, and bootstrap diagnostics.dev:bootstrapafter the API and web servers are both ready.codexrole.--team charlieand positionalcharlieselect the same team after the package script's--mode bootstrapargument..envGAMEFOUNDRY_SITE_URLandGAMEFOUNDRY_API_URLas legacy/default local values for bootstrap mode, then overwrites runtime URLs from the resolved team/role ports.Supported Teams
owner,alfa,bravo,charlie,delta,echo,foxtrot,golf,hotelSupported Roles
owner,codexPort Mapping
ownerrole uses base ports.codexrole uses base ports plus 2.Commands Added
Legacy Command Preserved
npm run dev:local-apiExact Startup Commands
Codex role example:
Browser Launch Behavior
ownerrole launches the browser automatically to the selected team'sindex.html.codexrole skips browser launch.http://127.0.0.1:5510/index.html.5512and API5513.http://127.0.0.1:5530/index.htmlfor bothnpm run dev:bootstrap -- --team charlieandnpm run dev:bootstrap -- charlie..envor inherited processGAMEFOUNDRY_SITE_URL/GAMEFOUNDRY_API_URLvalues loaded before bootstrap startup.GAMEFOUNDRY_SITE_URL=http://127.0.0.1:<webPort>andGAMEFOUNDRY_API_URL=http://127.0.0.1:<apiPort>/apiafter team/role resolution.Files Changed
package.json- updateddev/scripts/start-dev.mjs- addeddev/scripts/team-port-config.mjs- addeddev/tests/dev-runtime/TeamAwareBootstrap.test.mjs- addedValidation Summary
All requested targeted validation passed.