fix: secret refs and config injection

[aaronsteers]

No description provided.

[github-actions]

PyTest Results (Fast)

0 tests  ±0   0 ✅ ±0   0s ⏱️ ±0s
0 suites ±0   0 💤 ±0 
0 files   ±0   0 ❌ ±0 

Results for commit 990e318. ± Comparison against base commit b4ae513.

♻️ This comment has been updated with latest results.

[github-actions]

PyTest Results (Full)

0 tests  ±0   0 ✅ ±0   0s ⏱️ ±0s
0 suites ±0   0 💤 ±0 
0 files   ±0   0 ❌ ±0 

Results for commit 990e318. ± Comparison against base commit b4ae513.

♻️ This comment has been updated with latest results.

[Copilot]

Pull Request Overview

This PR refactors the secrets handling module by making the load_secrets function private and enhancing its functionality. The changes improve the module's internal structure while adding better support for nested configurations and error handling.

• Renames load_secrets to _load_secrets to make it a private function
• Adds support for nested dictionary structures using dot notation in secrets
• Improves error handling with proper FileNotFoundError exceptions
• Updates all test files and imports to use the new private function

Reviewed Changes

Copilot reviewed 7 out of 8 changed files in this pull request and generated 1 comment.

Show a summary per file

File	Description
connector_builder_mcp/_secrets.py	Core refactoring of secrets loading with nested dict support and improved error handling
tests/test_secrets.py	Updates tests to use private function and adds proper fixtures for testing
tests/test_integration.py	Updates import to use the new private function name
connector_builder_mcp/manifest_scaffold.py	Minor code style improvements with import organization
pyproject.toml	Adds metadata configuration for direct references
poe_tasks.toml	Adds sandbox directory configuration
README.md	Updates configuration examples and removes upgrade flags

---

_{Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.}