Merged
Keep Admin maintenance target and intent synchronized #147
urandonAdd on 28 days ago
Keep Admin maintenance target and intent synchronized #147
Merged
Merged fromcodex/issue-150-maintenance-targetintomain
Pull Request has been successfully merged by urandon.
(Thank you for urandon's contribution.)28 days ago Associated issue: [Feedback/bug][setup] Admin Maintenance renders selected target but posts an empty repo_id after Caches navigation,[Feedback/bug][cache] Concurrent local Go gate processes can collide on the fixture service fallback writer lock
28 days ago Associated issue: [Feedback/bug][setup] Admin Maintenance renders selected target but posts an empty repo_id after Caches navigation,[Feedback/bug][cache] Concurrent local Go gate processes can collide on the fixture service fallback writer lock
28 days ago Pushed 3 commits:13a7b5d8-2 commits from branch
urandon/gitcode-mcp:main,8a51599c-Merge remote-tracking branch 'origin/main' into codex/issue-150-maintenance-target28 days ago Pushed 3 commits:13a7b5d8-2 commits from branch
urandon/gitcode-mcp:main,8a51599c-Merge remote-tracking branch 'origin/main' into codex/issue-150-maintenance-target28 days ago Pushed 6 commits:4e588123-5 commits from branch
urandon/gitcode-mcp:main,ebf21198-Merge branch 'main' into codex/issue-150-maintenance-target28 days ago Pushed 6 commits:4e588123-5 commits from branch
urandon/gitcode-mcp:main,ebf21198-Merge branch 'main' into codex/issue-150-maintenance-target28 days ago Merged pull request, merge node SHA: e8250ee6de503201594da60407828ba1b5883895
Closes #150
Scope/design
Synchronize selected opaque cache/repository identity with submitted maintenance intent across Caches navigation, URL/registration restoration, reload, history, refresh and explicit selection. Preserve unsaved same-target policy drafts. Target changes invalidate old plans/receipts; intent generations prevent late plan/error responses from resurfacing on another target or edited policy. Keep unenrolled targets URL-addressable. Embedded assets rebuilt deterministically. No backend/CLI/MCP contract or browser authority change.
Verification
Exact candidate: 4b385840f739f56966fba905bd447f39acf5a7ff
Base: 57b2359175b18c0ca08527046c05adf072b06682
Fail-first regressions demonstrated empty IDs after deep-link navigation and old repository/profile after list navigation. 11 new semantic regressions pass, covering multi-repository/cache, unenrolled target, reload/history, same-target drafts, target/plan invalidation and late success/failure.
Passed full semantic browser suite: 50 passed, 1 opt-in live visual smoke skipped (no local QA launch URL/output directory). Artifact guard passed; no raster/trace/video output. Frontend types/unit, build/repeat asset validation/licenses, go test -count=1 ./..., go vet ./..., docs and whitespace passed.
Race: first overlapping regular-Go/race process run had two fixture service writer-contention failures. Complete isolated service race and then complete five-package Admin race command passed without disabling instrumentation or internal test parallelism. First-failure evidence and independent test-isolation follow-up #166 are recorded on #150/#165; not bundled in this UI patch.
Risk/recovery/UX
Existing Admin selection/control cohort repair, no new action or resource. Main risk is stale intent/plan crossing a target or wiping drafts; semantic JSON/DOM tests cover both. Recovery is a normal revert; no cache/schema/config mutation. Dependencies unchanged; existing npm audit warnings are not claimed fixed.
Independent exact-SHA review and required CI remain merge gates.