Skip to content

fix(live): extend Robinhood OAuth init timeout#263

Merged
warren618 merged 1 commit into
mainfrom
codex/issue-259-robinhood-oauth-timeout
Jun 18, 2026
Merged

fix(live): extend Robinhood OAuth init timeout#263
warren618 merged 1 commit into
mainfrom
codex/issue-259-robinhood-oauth-timeout

Conversation

@warren618

@warren618 warren618 commented Jun 18, 2026

Copy link
Copy Markdown
Collaborator

Summary

  • add an explicit init_timeout field for MCP server config and Robinhood's seed config
  • keep normal remote tool calls on tool_timeout while allowing OAuth/initialize to wait up to 300s
  • make connector authorize lift older Robinhood configs to the 300s init window without widening tool calls

Fixes #259

Tests

  • python -m pytest tests/test_cli_live.py tests/test_mcp_oauth_schema.py -q
@warren618 warren618 merged commit da11efd into main Jun 18, 2026
1 check passed
@warren618 warren618 deleted the codex/issue-259-robinhood-oauth-timeout branch June 18, 2026 13:52
@warren618 warren618 mentioned this pull request Jun 18, 2026
Merged
2 tasks
warren618 added a commit that referenced this pull request Jun 18, 2026
@warren618
chore: finish proposal_id hardening + document MCP initTimeout (#264)
5e328d6 
Completes work the 2026-06-18 self-merged batch left half-applied:
- Tighten the mandate proposal_id relay/preview parsers in api_server.py
  and cli/_legacy.py to the strict mp_[0-9a-f]{32} format already enforced
  by commit.py (#256), so the relay and the commit gate agree on format.
- Document the MCP initTimeout config key (shipped in #263) in README and
  wire its base-default passthrough in config/loader.py.
- Add the registered pytest.mark.unit marker to test_mandate_commit_security.py.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

1 participant

@warren618