Skip to content

fix: open Project chats on stock Hermes gateways - #102

Open
rusty4444 wants to merge 1 commit into
mainfrom
fix/project-chat-stock-gateway
Open

rusty4444 wants to merge 1 commit into
mainfrom
fix/project-chat-stock-gateway

Conversation

@rusty4444

Copy link
Copy Markdown
Owner

Summary

Fixes #100 with the stock Hermes gateway contract.

Project chats currently require projects.assign_session, but stock Hermes does not ship that sibling RPC. The app therefore blocks the chat before it can open. The existing PR #101 identifies the right cwd-based fallback, but its session.create request still includes the unsupported session_id field and also contains an unrelated feature commit.

Changes

  • treat only a missing projects.assign_session sibling as a non-fatal compatibility fallback
  • carry the selected Project's primary working directory into the chat
  • create the new gateway session with cwd and no client-supplied session_id, matching stock Hermes SessionCreateParams
  • retain the runtime session ID returned by the gateway in the existing mobile-to-gateway mapping
  • keep transport and genuine Project assignment failures blocking and retryable
  • add widget, draft-model, and WebSocket request-shape regression coverage

A future gateway that implements projects.assign_session still uses the explicit assignment path. The cwd fallback is only used when that method is absent.

Verification

  • flutter analyze --no-pub: no issues
  • flutter test --no-pub: 949 passed
  • git diff --check

This is a clean, single-scope alternative to #101. The original cwd fallback was proposed by @Thaeland and is credited in the commit.

Treat the missing projects.assign_session sibling as a supported fallback, create the gateway session in the Project cwd using the real stock session.create request shape, and retain the returned runtime mapping. Add widget and wire-contract regressions.

Co-authored-by: Thaeland <Jon.Beets@paceremail.com>
@rusty4444
rusty4444 force-pushed the fix/project-chat-stock-gateway branch from 2be221e to 8b0d4c1 Compare September 15, 2026 21:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Project chats permanently blocked: projects.assign_session does not exist on stock Hermes gateways

1 participant