Skip to content

fix(aidp): support legacy and new model APIs - #3938

Open
cj2026-bit wants to merge 1 commit into
developfrom
cj-fix-aidp-model-api-compatibility
Open

cj2026-bit wants to merge 1 commit into
developfrom
cj-fix-aidp-model-api-compatibility

Conversation

@cj2026-bit

Copy link
Copy Markdown
Collaborator

Summary

  • Support both AIDP model-list endpoint contracts: legacy Service and new Query.
  • Probe both endpoints once on cold start, prefer Service when both are non-404, and cache SERVICE/QUERY in process memory.
  • Translate version-specific query parameters and normalize wrapped-object and direct-array responses while preserving the backend API contract.
  • Update the local AIDP mock server and add compatibility, retry, cache, response, and error tests.

Validation

  • pytest test/ext_components/aidp -q: 510 passed
  • Local mock verified both endpoint modes and backend model filtering.
  • Full project test entrypoint generated coverage (88% overall) but had unrelated baseline/environment failures outside the changed files.

Issue association: none.

Add endpoint probing and process-local SERVICE/QUERY caching for AIDP model listing. Translate the version-specific query parameters and normalize the legacy wrapped and new direct-array responses while preserving the backend contract.

Update the local AIDP mock server and cover endpoint selection, retry behavior, cache reuse, response parsing, and error handling.

Co-authored-by: Codex <noreply@openai.com>

Generated-by: gpt-5-codex
@codecov

codecov Bot commented Sep 16, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 91.54930% with 6 lines in your changes missing coverage. Please review.

Files with missing lines Patch % Lines
...ckend/ext_components/aidp/services/aidp_service.py 91.54% 5 Missing and 1 partial ⚠️

📢 Thoughts on this report? Let us know!

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.

1 participant