Skip to content

refactor: replace bare dict with dict[str, Any] in datasource_entities#35062

Merged
asukaminato0721 merged 1 commit intolanggenius:mainfrom
wdeveloper16:refactor/typed-dict-datasource-entities
Apr 13, 2026
Merged

refactor: replace bare dict with dict[str, Any] in datasource_entities#35062
asukaminato0721 merged 1 commit intolanggenius:mainfrom
wdeveloper16:refactor/typed-dict-datasource-entities

Conversation

@wdeveloper16
Copy link
Copy Markdown
Contributor

Summary

  • Tighten six bare dict / dict | None Pydantic field annotations in core/datasource/entities/datasource_entities.py:
    • DatasourceEntity.output_schema
    • DatasourceInvokeMeta.tool_config
    • OnlineDocumentPage.page_icon
    • GetWebsiteCrawlRequest.crawl_parameters
    • OnlineDriveFileBucket.next_page_parameters
    • OnlineDriveBrowseFilesRequest.next_page_parameters
  • All are dynamic, provider-specific payloads; dict[str, Any] is the correct shape. Any is already imported in the module.

No behavior change — types only.

Part of #22651.

Test plan

  • make lint passes
  • make type-check-core passes

@dosubot dosubot Bot added the size:S This PR changes 10-29 lines, ignoring generated files. label Apr 13, 2026
@github-actions
Copy link
Copy Markdown
Contributor

Pyrefly Diff

No changes detected.

@asukaminato0721 asukaminato0721 added this pull request to the merge queue Apr 13, 2026
@dosubot dosubot Bot added the lgtm This PR has been approved by a maintainer label Apr 13, 2026
Merged via the queue into langgenius:main with commit e243e8d 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 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