Skip to content

fix: Loosen StoreListActor.picture_url to str and add new API error type - #1038

Merged
vdusek merged 1 commit into
masterfrom
ci/regenerate-models
Aug 28, 2026
Merged

fix: Loosen StoreListActor.picture_url to str and add new API error type#1038
vdusek merged 1 commit into
masterfrom
ci/regenerate-models

Conversation

@apify-service-account

@apify-service-account apify-service-account commented Aug 28, 2026

Copy link
Copy Markdown
Contributor

Regenerates the Pydantic models, TypedDicts, and literal aliases from the published OpenAPI specification (v2-2026-08-24T085852Z -> v2-2026-08-27T071624Z), picking up the spec fixes from apify/apify-docs#2859.

  • StoreListActor.picture_url is now str | None instead of AnyUrl | None. The API doesn't guarantee an RFC 3986 URI here - legacy Actors can carry an empty or malformed pictureUrl - so such Store items no longer fail validation.
  • ErrorType gains cannot-publish-actor-task, returned with 400 by POST /v2/actor-tasks and PUT /v2/actor-tasks/{actorTaskId}.
  • ActorNotice gains a NONE_TYPE_NONE = None member. The spec now lists null in the enum (the API does return notice: null); since the field is typed ActorNotice | None, None still validates to plain None, so this is cosmetic.

✍️ Drafted by Claude Code

@apify-service-account apify-service-account added the t-tooling Issues with this label are in the ownership of the tooling team. label Aug 28, 2026
@codecov

codecov Bot commented Aug 28, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 95.29%. Comparing base (79e205b) to head (54f9c68).
⚠️ Report is 1 commits behind head on master.

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #1038      +/-   ##
==========================================
+ Coverage   95.27%   95.29%   +0.01%     
==========================================
  Files          59       59              
  Lines        5501     5502       +1     
==========================================
+ Hits         5241     5243       +2     
+ Misses        260      259       -1     
Flag Coverage Δ
integration 92.42% <100.00%> (+0.16%) ⬆️
unit 87.36% <100.00%> (+<0.01%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@vdusek vdusek changed the title chore: Regenerate models from the published OpenAPI spec fix: Loosen StoreListActor.picture_url to str and add new API error type Aug 28, 2026
@vdusek
vdusek merged commit a098dd5 into master Aug 28, 2026
55 of 57 checks passed
@vdusek
vdusek deleted the ci/regenerate-models branch August 28, 2026 06:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

t-tooling Issues with this label are in the ownership of the tooling team.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants