Skip to content

Fix default templates#1228

Open
Arshardh wants to merge 1 commit intowso2:mainfrom
Arshardh:spec-mods
Open

Fix default templates#1228
Arshardh wants to merge 1 commit intowso2:mainfrom
Arshardh:spec-mods

Conversation

@Arshardh
Copy link
Contributor

@Arshardh Arshardh commented Feb 18, 2026

Purpose

  • Fix openAI endpoint according to how the SDK is developed.
  • Modify the azure specification to use the latest with responses API.

Summary by CodeRabbit

Release Notes

  • Bug Fixes
    • Azure OpenAI provider updated to reference the provider's v2 connector assets and to align token field names with current usage reporting (prompt → input, completion → output).
    • OpenAI provider API base path corrected to the proper v1 endpoint for reliable integration.

@coderabbitai
Copy link
Contributor

coderabbitai bot commented Feb 18, 2026

No actionable comments were generated in the recent review. 🎉


Walkthrough

Updated two LLM provider templates: Azure OpenAI template had metadata URLs moved to v2 and token identifier mappings changed; OpenAI template endpoint adjusted to include the /v1 path.

Changes

Cohort / File(s) Summary
Azure OpenAI template
platform-api/src/resources/default-llm-provider-templates/azureopenai-template.yaml
spec.metadata.logoUrl and spec.metadata.openapiSpecUrl updated to the v2 raw GitHub paths; spec.promptTokens.identifier changed from $.usage.prompt_tokens$.usage.input_tokens; spec.completionTokens.identifier changed from $.usage.completion_tokens$.usage.output_tokens.
OpenAI template
platform-api/src/resources/default-llm-provider-templates/openai-template.yaml
Updated provider endpointUrl from https://api.openai.comhttps://api.openai.com/v1. No other fields changed.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Poem

🐰 I nibbled through YAML, neat and spry,

URLs hopped to v2, tokens learnt to fly,
A tiny path to /v1 in view,
Small crunchy changes — fresh and new! 🥕

🚥 Pre-merge checks | ✅ 1 | ❌ 2

❌ Failed checks (1 warning, 1 inconclusive)

Check name Status Explanation Resolution
Description check ⚠️ Warning The pull request description is incomplete, missing most required sections from the template including Goals, Approach, User stories, Documentation, Automation tests, Security checks, Samples, Related PRs, and Test environment. Complete the PR description by filling in all required template sections, particularly Goals, Approach, test coverage details, security verification, and test environment information.
Title check ❓ Inconclusive The title 'Fix default templates' is vague and generic, using non-descriptive language that doesn't convey specific information about the changes. Provide a more specific title such as 'Update Azure OpenAI and OpenAI provider templates' to clearly indicate which templates are being fixed and why.
✅ Passed checks (1 passed)
Check name Status Explanation
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.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment

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

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

Copy link
Contributor

@coderabbitai coderabbitai bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Actionable comments posted: 1

🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.

Inline comments:
In
`@platform-api/src/resources/default-llm-provider-templates/azureopenai-template.yaml`:
- Around line 29-30: The logoUrl value is inconsistent with openapiSpecUrl;
update the logoUrl in azureopenai-template.yaml to reference the icon in the
azure.openai.v2 directory so it matches openapiSpecUrl—replace the current
logoUrl that points to openapi/azure.openai/icon.png with the URL under
openapi/azure.openai.v2/icon.png (ensure the logoUrl key is updated to the new
raw.githubusercontent URL).

Copy link
Contributor

@coderabbitai coderabbitai bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.

Duplicate comments:
In
`@platform-api/src/resources/default-llm-provider-templates/azureopenai-template.yaml`:
- Around line 29-30: Update the logoUrl to point to the matching azure.openai.v2
directory so it aligns with openapiSpecUrl; change the logoUrl value that
currently references .../openapi/azure.openai/icon.png to the corresponding path
under .../openapi/azure.openai.v2/icon.png so both logoUrl and openapiSpecUrl
use the same azure.openai.v2 directory.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants

Comments