Skip to content

fix(actors info): follow the Actor's default build for --input and --readme - #1473

Open
l2ysho wants to merge 6 commits into
masterfrom
fix/actors-info-default-build
Open

l2ysho wants to merge 6 commits into
masterfrom
fix/actors-info-default-build

Conversation

@l2ysho

@l2ysho l2ysho commented Oct 5, 2026 •

Copy link
Copy Markdown
Contributor

Note

TL;DR — fix failing e2e

Closes #1454

Changes

  • src/commands/actors/info.ts: for --input and --readme, resolve the build with client.actor(id).defaultBuild(). Fall back to taggedBuilds.latest if that call fails. This follows the approach agreed on the issue, and it also handles defaultRunOptions.build set to a build number.
  • test/local/commands/actors-info.test.ts: 3 tests with a mocked client: no latest tag, default build preferred over latest, fallback to latest. The first two fail without the fix.

Evidence

  • apify/rag-web-browser now has only beta-3 and version-1 tags. defaultRunOptions.build is version-1.
  • The E2E test actors/info.test.ts > shows input schema with --input flag fails on the last 3 master nightlies (10-03 to 10-05) with SyntaxError: Unexpected token 'E', because the CLI printed the error text instead of JSON: failing job 10-05.
  • defaultBuild() returned the right input schema for apify/rag-web-browser (15 properties), apify/web-scraper (39) and apify/hello-world (1), without a token.

Not verified

  • I did not run the E2E test locally (no TEST_USER_TOKEN). The next E2E nightly will show it.

🤖 Generated with Claude Code

…readme

Closes #1454

Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
@l2ysho l2ysho added the t-builders Issues owned by the Builders team. label Oct 5, 2026
…uild

Only a 404 means the Actor has no default build. Any other error now fails the
command instead of silently printing the `latest` schema with exit 0.

Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
@l2ysho l2ysho self-assigned this Oct 8, 2026
@l2ysho
l2ysho marked this pull request as ready for review October 8, 2026 08:39
@l2ysho
l2ysho requested a review from DaveHanns as a code owner October 8, 2026 08:39
@apify-service-account apify-service-account added the tested Temporary label used only programatically for some analytics. label Oct 8, 2026

This branch has not been deployed

No deployments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

t-builders Issues owned by the Builders team. tested Temporary label used only programatically for some analytics.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

actors info --input/--readme should follow the Actor's default build, not the latest tag

2 participants