Skip to content

fix: scope env substitution to selected server#208

Open
haosenwang1018 wants to merge 1 commit intophilschmid:mainfrom
haosenwang1018:fix/scoped-env-warnings
Open

fix: scope env substitution to selected server#208
haosenwang1018 wants to merge 1 commit intophilschmid:mainfrom
haosenwang1018:fix/scoped-env-warnings

Conversation

@haosenwang1018
Copy link

Summary

  • stop substituting environment variables for every server during config load
  • only resolve placeholders when getServerConfig() returns the selected server
  • add regression coverage for unrelated missing env vars and the new lazy substitution flow

Testing

  • bun test tests/config.test.ts
  • bun run typecheck

Closes #20.

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.

Environment variable warnings shown for all servers, not just the one being queried

1 participant