This repository was archived by the owner on Jun 8, 2026. It is now read-only.
Conversation
Add CL_LOCAL_GATEWAY_NO_AUTH=1 mode that bypasses gateway auth for local development. Guarded by !app.isPackaged so it cannot be enabled in production builds. The exchange endpoint issues a 24h session immediately without challenge verification. Implement GET /api/engineer/symphony/sessions/unread-count to count sessions with unanswered assistant replies, matching the Next.js route behavior for LocalElectron mode. FEAT-68
Code Review SummaryStatus: Approved Reviewers: Bug Hunter A, Bug Hunter B, Unified Auditor, Premise Reviewer, Gateway Core Architect Findings
MEDIUM Issues (consider)
Validation Stats
Recommendation: Approve — no blocking or high-priority issues. The medium items around no-auth mode (loopback bypass, static fallback token) are worth addressing before this pattern is widened. |
- Add optional chain on history.messages before .at() call - Check provider-specific chat history files (claude, codex) not just default - Move no-auth exchange block after loopback address check - Replace static "no-auth" token fallback with fail-fast sessionStore check
- Add optional chain on history.messages before .at() call - Check provider-specific chat history files (claude, codex) not just default - Move no-auth exchange block after loopback address check - Replace static "no-auth" token fallback with fail-fast sessionStore check - Extract version-bump-check into its own workflow file
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 subscribe to this conversation on GitHub.
Already have an account?
Sign in.
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.
Add CL_LOCAL_GATEWAY_NO_AUTH=1 mode that bypasses gateway auth for local development. Guarded by !app.isPackaged so it cannot be enabled in production builds. The exchange endpoint issues a 24h session immediately without challenge verification.
Implement GET /api/engineer/symphony/sessions/unread-count to count sessions with unanswered assistant replies, matching the Next.js route behavior for LocalElectron mode.