fix(shared): Fix loadConfig import in resolveEnvOverrides.ts

[CyberT17]

Fixes #801

Summary by CodeRabbit

• Refactor

  • Simplified internal module imports to streamline code structure; no changes to runtime behavior or user experience.

• Documentation

  • Updated changelog to record the import consolidation fix.

Note: This release contains no user-facing changes or new features.

_{✏️ Tip: You can customize this high-level summary in your review settings.}

[coderabbitai]

Walkthrough

Consolidates imports in packages/shared/tools/resolveEnvOverrides.ts, importing both loadConfig and resolveEnvironmentVariableOverridesFromConfig from ../src/env.server.js. Updates CHANGELOG.md with an Unreleased Fixed entry referencing the corrected import.

Changes

Cohort / File(s)	Summary
Import consolidation packages/shared/tools/resolveEnvOverrides.ts	Replaced separate imports (loadConfig from ../src/utils.js and resolveEnvironmentVariableOverridesFromConfig from ../src/env.server.js) with a single named import from ../src/env.server.js.
Changelog update CHANGELOG.md	Added Unreleased → Fixed entry noting the corrected loadConfig import in resolveEnvOverrides (PR #801).

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

🚥 Pre-merge checks | ✅ 5

✅ Passed checks (5 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	The title 'fix(shared): Fix loadConfig import in resolveEnvOverrides.ts' accurately describes the main change: updating the import source of loadConfig in resolveEnvOverrides.ts from the old location to the new one.
Linked Issues check	✅ Passed	The pull request directly addresses issue #801 by updating the loadConfig import in resolveEnvOverrides.ts from ../src/utils.js to ../src/env.server.js, resolving the failed deployment issue caused by the unmigrated import reference.
Out of Scope Changes check	✅ Passed	All changes are narrowly scoped to fixing the import issue described in #801: consolidating imports in resolveEnvOverrides.ts and documenting the fix in CHANGELOG.md, with no extraneous modifications.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

[coderabbitai]

Actionable comments posted: 1

🤖 Fix all issues with AI agents

In `@CHANGELOG.md`:
- Around line 10-11: Update the changelog entry "Fixed loadConfig import in
resolveEnvOverrides" to reference PR `#802` instead of `#801` by changing the
trailing PR id and link to use /pull/802 (i.e., replace [`#801`](.../pull/801)
with [`#802`](.../pull/802)) so the entry points to the correct PR.

[brendan-kellam]

thank you for the fix 🙏