Skip to content

refactor(api): type DataSourceApiKeyAuthBinding.to_dict with TypedDict#35001

Merged
asukaminato0721 merged 2 commits intolanggenius:mainfrom
YB0y:refactor/typed-dict-data-source-api-key-to-dict
Apr 13, 2026
Merged

refactor(api): type DataSourceApiKeyAuthBinding.to_dict with TypedDict#35001
asukaminato0721 merged 2 commits intolanggenius:mainfrom
YB0y:refactor/typed-dict-data-source-api-key-to-dict

Conversation

@YB0y
Copy link
Copy Markdown
Contributor

@YB0y YB0y commented Apr 12, 2026

Part of #32863 (api/models/)

Summary

  • Define DataSourceApiKeyAuthBindingDict TypedDict for DataSourceApiKeyAuthBinding.to_dict() return type
  • Add missing return type annotation to to_dict

Why this change

DataSourceApiKeyAuthBinding.to_dict() had no return type annotation, hiding its fixed 8-key structure. A TypedDict makes the shape and value types explicit for callers.

Changes

  • api/models/source.py: Define DataSourceApiKeyAuthBindingDict, annotate DataSourceApiKeyAuthBinding.to_dict

@YB0y YB0y requested a review from QuantumGhost as a code owner April 12, 2026 14:59
@dosubot dosubot Bot added size:S This PR changes 10-29 lines, ignoring generated files. refactor labels Apr 12, 2026
@github-actions
Copy link
Copy Markdown
Contributor

Pyrefly Diff

No changes detected.

@github-actions
Copy link
Copy Markdown
Contributor

Pyrefly Diff

No changes detected.

@dosubot dosubot Bot added the lgtm This PR has been approved by a maintainer label Apr 13, 2026
@asukaminato0721 asukaminato0721 added this pull request to the merge queue Apr 13, 2026
Merged via the queue into langgenius:main with commit 095962f Apr 13, 2026
28 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

lgtm This PR has been approved by a maintainer refactor size:S This PR changes 10-29 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants