Skip to content

Set Obsidian client keep-alive timeout#57

Open
fa1k3 wants to merge 1 commit into
cyanheads:mainfrom
fa1k3:fix/obsidian-agent-keepalive
Open

Set Obsidian client keep-alive timeout#57
fa1k3 wants to merge 1 commit into
cyanheads:mainfrom
fa1k3:fix/obsidian-agent-keepalive

Conversation

@fa1k3
Copy link
Copy Markdown

@fa1k3 fa1k3 commented May 20, 2026

Summary

  • set explicit undici Agent keep-alive timeouts for the Obsidian REST client
  • keep the client idle timeout below the Local REST API server keep-alive timeout
  • add regression coverage for the Agent option builder

Why

Short, explicit client-side keep-alive avoids stale-socket reuse races against upstream servers with short keep-alive windows.

Tests

  • bun test tests/services/obsidian-service.test.ts
  • bun run test
  • bun run build
  • bun run lint:mcp

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.

1 participant