Skip to content

fix(config): fail open to default local endpoints#3

Merged
vicary merged 2 commits into
mainfrom
config-default-fallback
Mar 27, 2026
Merged

fix(config): fail open to default local endpoints#3
vicary merged 2 commits into
mainfrom
config-default-fallback

Conversation

@vicary

@vicary vicary commented Mar 27, 2026

Copy link
Copy Markdown
Owner

Summary

  • fall back to default localhost Redis and Graphiti endpoints when config discovery fails or no usable config is found
  • surface malformed discovered or legacy config sources through OpenCode warnings instead of throwing during startup
  • infer the missing sibling endpoint from the configured host and add regression coverage for warning delivery and config resolution

Test Plan

  • deno test src/config.test.ts src/services/opencode-warning.test.ts
  • deno task check
  • deno task lint
  • deno fmt --check

vicary added 2 commits March 27, 2026 14:57
Avoid startup failures when config discovery or validation fails by falling back to default Redis and Graphiti endpoints, surfacing recoverable file issues as warnings, and inferring a sibling endpoint from the configured host.

This comment was marked as resolved.

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Copilot reviewed 7 out of 7 changed files in this pull request and generated no new comments.


💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@vicary vicary merged commit 9c4746d into main Mar 27, 2026
6 checks passed
@vicary vicary deleted the config-default-fallback branch March 27, 2026 09:27
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.

2 participants