Skip to content

Warn on SDK config load failures#617

Open
d42me wants to merge 1 commit into
mainfrom
fix/config-load-warnings
Open

Warn on SDK config load failures#617
d42me wants to merge 1 commit into
mainfrom
fix/config-load-warnings

Conversation

@d42me
Copy link
Copy Markdown
Contributor

@d42me d42me commented May 8, 2026

Adds warning logs when SDK config files cannot be parsed/read across MCP, evals, sandboxes, and tunnel packages, with a caplog regression test.


Note

Low Risk
Low risk: only adds warning logs and slightly broadens exception handling when reading local config files; no behavior change beyond improved visibility and consistent fallback.

Overview
Adds warning logs across the various SDK Config loaders (evals, MCP server, sandboxes, tunnel) when ~/.prime/config.json cannot be read or parsed, and standardizes the failure path to fall back to an empty config (catching OSError instead of IOError).

Introduces a regression test (test_sdk_config_load_warnings.py) that parametrically verifies each package emits the warning and returns {} when the config file contains invalid JSON.

Reviewed by Cursor Bugbot for commit de7a284. Bugbot is set up for automated code reviews on this repo. Configure here.

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