From fcf673494990fa4f9eea32fa63ab5dfea2dcdf13 Mon Sep 17 00:00:00 2001 From: watany <76135106+watany-dev@users.noreply.github.com> Date: Sun, 2 Mar 2025 09:39:10 +0900 Subject: [PATCH] feat(bedrock): Add APAC Support for Cross-Region Inference Profiles (#1913) * adding apac * Remove unreachable 'break' after return in case 'ap-'. Co-authored-by: ellipsis-dev[bot] <65095814+ellipsis-dev[bot]@users.noreply.github.com> * changeset * adding testing * revert * revart default model * revert package-lock * Remove unnecessary test --------- Co-authored-by: ellipsis-dev[bot] <65095814+ellipsis-dev[bot]@users.noreply.github.com> Co-authored-by: Saoud Rizwan <7799382+saoudrizwan@users.noreply.github.com> --- .changeset/twelve-terms-kiss.md | 5 +++++ src/api/providers/bedrock.ts | 4 +++- 2 files changed, 8 insertions(+), 1 deletion(-) create mode 100644 .changeset/twelve-terms-kiss.md diff --git a/.changeset/twelve-terms-kiss.md b/.changeset/twelve-terms-kiss.md new file mode 100644 index 0000000000..a6daecd2ca --- /dev/null +++ b/.changeset/twelve-terms-kiss.md @@ -0,0 +1,5 @@ +--- +"claude-dev": patch +--- + +Add APAC Support for Cross-Region Inference Profiles diff --git a/src/api/providers/bedrock.ts b/src/api/providers/bedrock.ts index 0c8908363b..1cd64354d0 100644 --- a/src/api/providers/bedrock.ts +++ b/src/api/providers/bedrock.ts @@ -131,7 +131,7 @@ export class AwsBedrockHandler implements ApiHandler { }) } - private async getModelId(): Promise { + async getModelId(): Promise { if (this.options.awsUseCrossRegionInference) { let regionPrefix = (this.options.awsRegion || "").slice(0, 3) switch (regionPrefix) { @@ -139,6 +139,8 @@ export class AwsBedrockHandler implements ApiHandler { return `us.${this.getModel().id}` case "eu-": return `eu.${this.getModel().id}` + case "ap-": + return `apac.${this.getModel().id}` default: // cross region inference is not supported in this region, falling back to default model return this.getModel().id