Skip to content

[FIX] Azure AI schema filename mismatch#1846

Merged
chandrasekharan-zipstack merged 2 commits intomainfrom
fix/azure-ai-schema-filename
Mar 11, 2026
Merged

[FIX] Azure AI schema filename mismatch#1846
chandrasekharan-zipstack merged 2 commits intomainfrom
fix/azure-ai-schema-filename

Conversation

@chandrasekharan-zipstack
Copy link
Contributor

What

Rename Azure AI Foundry adapter schema file from azure_ai_foundry.json to azure_ai.json to match the get_provider() return value.

Why

The base class get_json_schema() constructs schema paths using get_provider(). The Azure AI adapter returns 'azure_ai' from get_provider(), but the schema file was named azure_ai_foundry.json. This mismatch caused a FileNotFoundError when loading the adapter, which crashed the entire /supported_adapters/ endpoint (returning 500 error) and broke adapter icon loading across all settings pages.

How

Renamed the schema file to match the provider name convention used by all other adapters.

Can this PR break any existing features. If yes, please list possible items. If no, please explain why.

No. This fixes a breaking issue. The schema file was inaccessible with its previous name, so this restores intended functionality without changing any API contracts or behavior.

Database Migrations

  • None

Env Config

  • None

Relevant Docs

Related Issues or PRs

Dependencies Versions

  • None

Notes on Testing

Verify on staging:

  1. /supported_adapters/ endpoint returns 200 (not 500)
  2. Adapter icons load correctly (no ⚠️ fallback icons)
  3. LLM/Embedding/VectorDB/TextExtractor settings pages render correctly

Screenshots

N/A

Schema file was named azure_ai_foundry.json but get_provider() returns
'azure_ai'. The base class get_json_schema() uses get_provider() to
construct the schema path, so it looked for azure_ai.json (non-existent),
causing FileNotFoundError that crashed /supported_adapters/ endpoint
with 500 error and broke all adapter icon loading.

Rename to azure_ai.json to match get_provider() and fix adapter list
rendering on settings pages.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@coderabbitai
Copy link
Contributor

coderabbitai bot commented Mar 11, 2026

Important

Review skipped

Review was skipped as selected files did not have any reviewable changes.

💤 Files selected but had no reviewable changes (1)
  • unstract/sdk1/src/unstract/sdk1/adapters/llm1/static/azure_ai.json
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: 95234afb-1d94-49d1-bf98-12658625d7be

📥 Commits

Reviewing files that changed from the base of the PR and between 16bc418 and f0ed10d.

📒 Files selected for processing (1)
  • unstract/sdk1/src/unstract/sdk1/adapters/llm1/static/azure_ai.json

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch fix/azure-ai-schema-filename

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.

@github-actions
Copy link
Contributor

Test Results

Summary
  • Runner Tests: 11 passed, 0 failed (11 total)

Runner Tests - Full Report
filepath function $$\textcolor{#23d18b}{\tt{passed}}$$ SUBTOTAL
$$\textcolor{#23d18b}{\tt{runner/src/unstract/runner/clients/test\_docker.py}}$$ $$\textcolor{#23d18b}{\tt{test\_logs}}$$ $$\textcolor{#23d18b}{\tt{1}}$$ $$\textcolor{#23d18b}{\tt{1}}$$
$$\textcolor{#23d18b}{\tt{runner/src/unstract/runner/clients/test\_docker.py}}$$ $$\textcolor{#23d18b}{\tt{test\_cleanup}}$$ $$\textcolor{#23d18b}{\tt{1}}$$ $$\textcolor{#23d18b}{\tt{1}}$$
$$\textcolor{#23d18b}{\tt{runner/src/unstract/runner/clients/test\_docker.py}}$$ $$\textcolor{#23d18b}{\tt{test\_cleanup\_skip}}$$ $$\textcolor{#23d18b}{\tt{1}}$$ $$\textcolor{#23d18b}{\tt{1}}$$
$$\textcolor{#23d18b}{\tt{runner/src/unstract/runner/clients/test\_docker.py}}$$ $$\textcolor{#23d18b}{\tt{test\_client\_init}}$$ $$\textcolor{#23d18b}{\tt{1}}$$ $$\textcolor{#23d18b}{\tt{1}}$$
$$\textcolor{#23d18b}{\tt{runner/src/unstract/runner/clients/test\_docker.py}}$$ $$\textcolor{#23d18b}{\tt{test\_get\_image\_exists}}$$ $$\textcolor{#23d18b}{\tt{1}}$$ $$\textcolor{#23d18b}{\tt{1}}$$
$$\textcolor{#23d18b}{\tt{runner/src/unstract/runner/clients/test\_docker.py}}$$ $$\textcolor{#23d18b}{\tt{test\_get\_image}}$$ $$\textcolor{#23d18b}{\tt{1}}$$ $$\textcolor{#23d18b}{\tt{1}}$$
$$\textcolor{#23d18b}{\tt{runner/src/unstract/runner/clients/test\_docker.py}}$$ $$\textcolor{#23d18b}{\tt{test\_get\_container\_run\_config}}$$ $$\textcolor{#23d18b}{\tt{1}}$$ $$\textcolor{#23d18b}{\tt{1}}$$
$$\textcolor{#23d18b}{\tt{runner/src/unstract/runner/clients/test\_docker.py}}$$ $$\textcolor{#23d18b}{\tt{test\_get\_container\_run\_config\_without\_mount}}$$ $$\textcolor{#23d18b}{\tt{1}}$$ $$\textcolor{#23d18b}{\tt{1}}$$
$$\textcolor{#23d18b}{\tt{runner/src/unstract/runner/clients/test\_docker.py}}$$ $$\textcolor{#23d18b}{\tt{test\_run\_container}}$$ $$\textcolor{#23d18b}{\tt{1}}$$ $$\textcolor{#23d18b}{\tt{1}}$$
$$\textcolor{#23d18b}{\tt{runner/src/unstract/runner/clients/test\_docker.py}}$$ $$\textcolor{#23d18b}{\tt{test\_get\_image\_for\_sidecar}}$$ $$\textcolor{#23d18b}{\tt{1}}$$ $$\textcolor{#23d18b}{\tt{1}}$$
$$\textcolor{#23d18b}{\tt{runner/src/unstract/runner/clients/test\_docker.py}}$$ $$\textcolor{#23d18b}{\tt{test\_sidecar\_container}}$$ $$\textcolor{#23d18b}{\tt{1}}$$ $$\textcolor{#23d18b}{\tt{1}}$$
$$\textcolor{#23d18b}{\tt{TOTAL}}$$ $$\textcolor{#23d18b}{\tt{11}}$$ $$\textcolor{#23d18b}{\tt{11}}$$

@sonarqubecloud
Copy link

@chandrasekharan-zipstack chandrasekharan-zipstack merged commit c41e05e into main Mar 11, 2026
7 checks passed
@chandrasekharan-zipstack chandrasekharan-zipstack deleted the fix/azure-ai-schema-filename branch March 11, 2026 08:22
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.

3 participants