Merge PR #217 PR_26177_CHARLIE_008-environment-management-complete

# Conflicts:
#	docs_build/dev/reports/codex_changed_files.txt
#	docs_build/dev/reports/codex_review.diff

Author: ToolboxAid

docs_build/dev/reports/PR_26177_CHARLIE_008-environment-management-complete.md

@@ -0,0 +1,52 @@
+# PR_26177_CHARLIE_008-environment-management-complete
+
+## Summary
+
+Team Charlie completed the Environment Management closeout slice.
+
+This PR keeps runtime behavior scoped to environment diagnostics and validation:
+
+- Environment Banner diagnostics now expose the environment safeguard state.
+- DEV/UAT-style labels remain visible as non-production safeguards.
+- Production labels remain hidden by default.
+- Missing local environment labels produce the existing actionable danger banner.
+- Configurable Runtime Ports are marked deprecated/superseded instead of pending/open work.
+- Environment Banner Playwright coverage now targets an existing legal page.
+
+## Changed Files
+
+- `src/dev-runtime/server/local-api-router.mjs`
+- `tests/dev-runtime/AdminHealthOperations.test.mjs`
+- `tests/dev-runtime/PublicEnvironmentConfig.test.mjs`
+- `tests/playwright/tools/AdminHealthOperationsPage.spec.mjs`
+- `tests/playwright/tools/EnvironmentBannerCoverage.spec.mjs`
+- `docs_build/dev/reports/PR_26177_CHARLIE_008-environment-management-complete.md`
+- `docs_build/dev/reports/PR_26177_CHARLIE_008-environment-management-complete_branch-validation.md`
+- `docs_build/dev/reports/PR_26177_CHARLIE_008-environment-management-complete_manual-validation-notes.md`
+- `docs_build/dev/reports/PR_26177_CHARLIE_008-environment-management-complete_requirements-checklist.md`
+- `docs_build/dev/reports/PR_26177_CHARLIE_008-environment-management-complete_validation-lane.md`
+- `docs_build/dev/reports/codex_changed_files.txt`
+- `docs_build/dev/reports/codex_review.diff`
+
+## Implementation Notes
+
+- Added `environmentSafeguard`, `environmentLabelNormalized`, `environmentBannerSource`, and `environmentBannerTone` diagnostics to `/api/public/config`.
+- Preserved the existing Environment Banner rendering contract and Theme V2 partial flow.
+- Changed System Health Local API startup diagnostics for Configurable Runtime Ports from `PENDING/deferred/cancelled` to `PASS/deprecated/superseded`.
+- Corrected Environment Banner Playwright coverage from missing `/legal/disclaimer.html` to existing `/legal/privacy-policy.html`.
+- No UI content, API data ownership, database, or storage behavior changes were introduced.
+
+## Validation
+
+- PASS: `node --check src/dev-runtime/server/local-api-router.mjs`
+- PASS: `node --check tests/dev-runtime/PublicEnvironmentConfig.test.mjs`
+- PASS: `node --check tests/dev-runtime/AdminHealthOperations.test.mjs`
+- PASS: `node --check tests/playwright/tools/EnvironmentBannerCoverage.spec.mjs`
+- PASS: `node --test tests/dev-runtime/PublicEnvironmentConfig.test.mjs tests/dev-runtime/AdminHealthOperations.test.mjs tests/api/admin-system-health/contract.test.mjs`
+- PASS: `npx playwright test tests/playwright/tools/EnvironmentBannerCoverage.spec.mjs --workers=1`
+- PASS: `npx playwright test tests/playwright/tools/AdminHealthOperationsPage.spec.mjs --workers=1`
+- PASS: `git diff --check`
+
+## ZIP
+
+- `tmp/PR_26177_CHARLIE_008-environment-management-complete_delta.zip`

docs_build/dev/reports/PR_26177_CHARLIE_008-environment-management-complete_branch-validation.md

@@ -0,0 +1,24 @@
+# PR_26177_CHARLIE_008 Branch Validation
+
+## Branch
+
+- Branch: `PR_26177_CHARLIE_008-environment-management-complete`
+- Start branch: `main`
+- Main start commit: `8cdd87bf2eb2b9c0625e80881f1d359e902fa8fc`
+
+## Checks
+
+| Check | Result | Notes |
+| --- | --- | --- |
+| Started from `main` | PASS | Branch was created after `main` was clean and synchronized. |
+| Worktree clean before branch work | PASS | Startup status check returned no changes. |
+| One PR purpose only | PASS | Environment management diagnostics/tests only. |
+| No `start_of_day` changes | PASS | Changed-file list contains no `start_of_day` paths. |
+| Environment Banner complete | PASS | DEV/UAT visible safeguards, production hidden, missing local label diagnostic covered. |
+| Configurable Runtime Ports not open work | PASS | System Health reports deprecated/superseded. |
+| No runtime data ownership regression | PASS | Browser remains a consumer of public config and System Health API contracts. |
+| Repo-structured ZIP created | PASS | `tmp/PR_26177_CHARLIE_008-environment-management-complete_delta.zip`. |
+
+## Result
+
+PASS

docs_build/dev/reports/PR_26177_CHARLIE_008-environment-management-complete_manual-validation-notes.md

@@ -0,0 +1,20 @@
+# PR_26177_CHARLIE_008 Manual Validation Notes
+
+## Manual Review
+
+- Confirmed changed files are limited to environment diagnostics, targeted tests, and required reports.
+- Confirmed no `start_of_day` paths were modified.
+- Confirmed no inline styles, style blocks, script blocks, page-local CSS, or inline event handlers were introduced.
+- Confirmed no browser-owned authoritative product data was added.
+- Confirmed no secrets are exposed in public config diagnostics.
+
+## Notes
+
+- Environment Banner coverage previously referenced missing `/legal/disclaimer.html`; the targeted legal coverage route now uses existing `/legal/privacy-policy.html`.
+- DEV/custom labels and UAT labels render visible non-production banners.
+- Production labels are normalized to `PROD` diagnostics and hidden by default.
+- Configurable Runtime Ports are closed as deprecated/superseded and are not tracked as active work.
+
+## Result
+
+PASS

docs_build/dev/reports/PR_26177_CHARLIE_008-environment-management-complete_requirements-checklist.md

@@ -0,0 +1,19 @@
+# PR_26177_CHARLIE_008 Requirements Checklist
+
+| Requirement | Result | Evidence |
+| --- | --- | --- |
+| Complete Environment Management from 90% to 100% | PASS | Environment safeguards and runtime-port closeout are explicit. |
+| Environment Banner complete | PASS | Public config diagnostics and Playwright banner coverage passed. |
+| Update reports/checklists accordingly | PASS | This report set contains branch validation, checklist, validation lane, manual notes, and shared diff/list artifacts. |
+| Mark Configurable Runtime Ports deprecated/superseded, not open work | PASS | Local API startup diagnostics now return `deprecated/superseded` with PASS. |
+| DEV/UAT/PROD environment safeguards | PASS | DEV/custom and UAT labels render visible safeguards; Production hides by default. |
+| Environment-aware diagnostics | PASS | `/api/public/config` reports normalized label and safeguard state. |
+| Final Charlie completion checklist | PASS | This checklist records the final Environment Management completion items. |
+| Targeted tests | PASS | Public config, Admin System Health, Environment Banner, and Admin System Health Playwright tests passed. |
+| No MEM DB/local-mem/fake-login/browser SSoT | PASS | No product-data ownership changes. |
+| No SQLite direction | PASS | No SQLite additions or terminology. |
+| Theme V2 rules | PASS | Existing Theme V2 partial flow preserved; no inline styles/scripts added. |
+
+## Result
+
+PASS

docs_build/dev/reports/PR_26177_CHARLIE_008-environment-management-complete_validation-lane.md

@@ -0,0 +1,24 @@
+# PR_26177_CHARLIE_008 Validation Lane
+
+## Commands
+
+| Command | Result |
+| --- | --- |
+| `node --check src/dev-runtime/server/local-api-router.mjs` | PASS |
+| `node --check tests/dev-runtime/PublicEnvironmentConfig.test.mjs` | PASS |
+| `node --check tests/dev-runtime/AdminHealthOperations.test.mjs` | PASS |
+| `node --check tests/playwright/tools/EnvironmentBannerCoverage.spec.mjs` | PASS |
+| `node --test tests/dev-runtime/PublicEnvironmentConfig.test.mjs tests/dev-runtime/AdminHealthOperations.test.mjs tests/api/admin-system-health/contract.test.mjs` | PASS, 10 tests |
+| `npx playwright test tests/playwright/tools/EnvironmentBannerCoverage.spec.mjs --workers=1` | PASS, 3 tests |
+| `npx playwright test tests/playwright/tools/AdminHealthOperationsPage.spec.mjs --workers=1` | PASS, 3 tests |
+| `git diff --check` | PASS |
+
+## Playwright
+
+Impacted: Yes, Environment Banner diagnostics and System Health startup diagnostics changed.
+
+Result: PASS.
+
+## Full Samples Smoke
+
+Not run. Not required for this targeted Environment Management PR.

docs_build/dev/reports/codex_changed_files.txt

@@ -1,13 +1,12 @@
 docs_build/dev/reports/codex_changed_files.txt
 docs_build/dev/reports/codex_review.diff
-docs_build/dev/reports/PR_26177_CHARLIE_007-runtime-configuration-complete.md
-docs_build/dev/reports/PR_26177_CHARLIE_007-runtime-configuration-complete_branch-validation.md
-docs_build/dev/reports/PR_26177_CHARLIE_007-runtime-configuration-complete_manual-validation-notes.md
-docs_build/dev/reports/PR_26177_CHARLIE_007-runtime-configuration-complete_requirements-checklist.md
-docs_build/dev/reports/PR_26177_CHARLIE_007-runtime-configuration-complete_validation-lane.md
-scripts/start-local-api-server.mjs
+docs_build/dev/reports/PR_26177_CHARLIE_008-environment-management-complete.md
+docs_build/dev/reports/PR_26177_CHARLIE_008-environment-management-complete_branch-validation.md
+docs_build/dev/reports/PR_26177_CHARLIE_008-environment-management-complete_manual-validation-notes.md
+docs_build/dev/reports/PR_26177_CHARLIE_008-environment-management-complete_requirements-checklist.md
+docs_build/dev/reports/PR_26177_CHARLIE_008-environment-management-complete_validation-lane.md
 src/dev-runtime/server/local-api-router.mjs
-src/dev-runtime/storage/storage-config.mjs
 tests/dev-runtime/AdminHealthOperations.test.mjs
-tests/dev-runtime/LocalApiStartupLogging.test.mjs
-tests/dev-runtime/StorageConfig.test.mjs
+tests/dev-runtime/PublicEnvironmentConfig.test.mjs
+tests/playwright/tools/AdminHealthOperationsPage.spec.mjs
+tests/playwright/tools/EnvironmentBannerCoverage.spec.mjs