Rename injected transformer provider parameter to provider-id

[Arshardh]

No description provided.

[coderabbitai]

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

Run ID: bdc00812-216b-4323-80a4-04ae5b4f970f

📥 Commits

Reviewing files that changed from the base of the PR and between b5c2b83 and 238e5ef.

📒 Files selected for processing (3)

• gateway/examples/openai-multi-provider-proxy.yaml
• gateway/gateway-controller/pkg/utils/llm_transformer.go
• gateway/gateway-controller/pkg/utils/llm_transformer_multiprovider_test.go

🚧 Files skipped from review as they are similar to previous changes (2)

• gateway/gateway-controller/pkg/utils/llm_transformer_multiprovider_test.go
• gateway/gateway-controller/pkg/utils/llm_transformer.go

---

📝 Walkthrough

Walkthrough

The additional provider transformer now passes and validates the selected upstream using the providerId parameter instead of id. Related comments and the multiprovider test assertion were updated accordingly.

Changes

Multi-provider transformer routing

Layer / File(s)	Summary
Update providerId contract gateway/gateway-controller/pkg/utils/llm_transformer.go, gateway/gateway-controller/pkg/utils/llm_transformer_multiprovider_test.go, gateway/examples/openai-multi-provider-proxy.yaml	The transformer policy uses providerId for the selected upstream name, with documentation, example comments, and test expectations updated.

Estimated code review effort: 1 (Trivial) | ~5 minutes

Possibly related PRs

• wso2/api-platform#2486: Updates the same multi-provider translator policy contract and related YAML/tests.
• wso2/api-platform#2586: Changes the inline transformer parameter key from id to providerId.

Suggested reviewers: krishanx92, renuka-fernando, anugayan, chamilaadhi, malinthaprasan

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 warning)

Check name	Status	Explanation	Resolution
Description check	⚠️ Warning	The description is missing entirely and does not include any required template sections or details.	Add the full template sections: Purpose, Goals, Approach, User stories, Documentation, Automation tests, Security checks, Samples, Related PRs, and Test environment.

✅ Passed checks (4 passed)

Check name	Status	Explanation
Title check	✅ Passed	The title clearly matches the main change: renaming the injected transformer provider parameter.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check	✅ Passed	Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check	✅ Passed	Check skipped because no linked issues were found for this pull request.

✨ Finishing Touches

📝 Generate docstrings

• Create stacked PR
• Commit on current branch

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Commit unit tests in branch multi-provider-routing

Warning

There were issues while running some tools. Please review the errors and either fix the tool's configuration or disable the tool if it's a critical failure.

🔧 golangci-lint (2.12.2)

level=error msg="[linters_context] typechecking error: pattern ./...: directory prefix . does not contain modules listed in go.work or their selected dependencies"

---

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.}