bedrock provider cleanup

[narengogi]

Changes included in this PR:

• utility for fetching environment variables
• Bedrock/AWS proxy routes support

Changes not included:
Support for IRSA and IMDS based auth

[matter-code-review]

Description

Summary By MatterAI

🔄 What Changed

This pull request introduces a minor whitespace adjustment in src/types/requestBody.ts. Specifically, a blank line was removed and then re-added, resulting in no functional change to the code.

🔍 Impact of the Change

There is no functional or behavioral impact on the application. This change is purely cosmetic, aimed at code formatting or cleanup.

📁 Total Files Changed

• src/types/requestBody.ts: Whitespace adjustment (1 addition, 1 deletion).

🧪 Test Added

N/A

🔒Security Vulnerabilities

N/A

Motivation

Minor formatting cleanup to align with code style guidelines.

Type of Change

• Bug fix (non-breaking change which fixes an issue)
• New feature (non-breaking change which adds functionality)
• Breaking change (fix or feature that would cause existing functionality to not work as expected)
• Documentation update
• Refactoring (no functional changes)

How Has This Been Tested?

N/A

Screenshots (if applicable)

N/A

Checklist

• My code follows the style guidelines of this project
• I have performed a self-review of my own code
• I have commented my code, particularly in hard-to-understand areas
• I have made corresponding changes to the documentation
• My changes generate no new warnings
• I have added tests that prove my fix is effective or that my feature works
• New and existing unit tests pass locally with my changes

Related Issues

N/A

Tanka Poem ♫

Blank line removed,
Then added, a cosmic dance.
Code breathes, silent.
No new warnings, just clean space,
Matter's touch, a gentle breeze. ✨

Sequence Diagram

sequenceDiagram
    participant Developer
    participant File as src/types/requestBody.ts

    Developer->>File: Modify whitespace
    Note over File: Removed and added blank line

Loading

[matter-code-review]

Code review completed. Found several areas for improvement including missing error handling, potential security issues, and documentation gaps.

[matter-code-review]

Important

PR Review Skipped

PR review skipped as per the configuration setting. Run a manually review by commenting /matter review

💡Tips to use Matter AI

Command List

• /matter summary: Generate AI Summary for the PR
• /matter review: Generate AI Reviews for the latest commit in the PR
• /matter review-full: Generate AI Reviews for the complete PR
• /matter release-notes: Generate AI release-notes for the PR
• /matter : Chat with your PR with Matter AI Agent
• /matter remember : Generate AI memories for the PR
• /matter explain: Get an explanation of the PR
• /matter help: Show the list of available commands and documentation
• Need help? Join our Discord server: https://discord.gg/fJU5DvanU3

[matter-code-review]

Bedrock provider cleanup with proxy support and AWS endpoint domain improvements. Found critical null safety issues and missing error handling.

[matter-code-review]

Dynamic import implementation looks correct but has potential type safety and performance issues

[matter-code-review]

High complexity PR with bedrock provider cleanup and AWS endpoint domain support. Key changes include renaming azureExtraParams to azureExtraParameters, adding AWS endpoint domain support, and refactoring bedrock API configurations.

[matter-code-review]

Bedrock provider cleanup with improved AWS endpoint handling and parameter standardization

[matter-code-review]

Minor cleanup in requestBody type definition