fix(plugin-engine): refresh shell env values

[robinebers]

Summary

• Re-read shell-derived env vars instead of caching them for the lifetime of the app.
• Keeps process env precedence intact while allowing Spotlight-launched OpenUsage to pick up updated shell config values without restart.
• Addresses Minimax "Request failed" unless started from the terminal #305.

Test plan

• cargo test resolve_env_value_rechecks_shell_when_process_env_missing
• cargo test env_api

Made with Cursor

---

Note

^{Cursor Bugbot is generating a summary for commit b53dc4f. Configure here.}

---

Summary by cubic

Re-read shell-derived env vars on each lookup (no caching) while keeping process env precedence. This lets Spotlight-launched OpenUsage pick up updated shell config without restart and fixes #305.

• Bug Fixes
  • Removed terminal env cache; if a var isn’t in the process env, we now re-read from interactive shells every time.
  • Introduced resolve_env_value_with_readers for clearer logic and testability, plus a test to verify repeated shell reads.

^{Written for commit b53dc4f. Summary will update on new commits. Review in cubic}

[cubic-dev-ai]

No issues found across 1 file

_{Re-trigger cubic}