Skip to content

fix: remove gameable Runs succeeded stat from actor search/details - #1182

Merged
MQ37 merged 1 commit into
masterfrom
investigate/runs-succeeded-metric
Jul 31, 2026
Merged

fix: remove gameable Runs succeeded stat from actor search/details#1182
MQ37 merged 1 commit into
masterfrom
investigate/runs-succeeded-metric

Conversation

@MQ37

@MQ37 MQ37 commented Jul 30, 2026

Copy link
Copy Markdown
Contributor

Adresses https://apify.slack.com/archives/C04URRT3934/p1785403187603329

What

Removes the per-Actor "Runs succeeded" success-rate stat from search-actors and fetch-actor-details (markdown card, structured output, and JSON schema), plus the now-stale dev-harness mock fixture and test assertions.

Why

Reported in Slack: this metric is gamed by developers and was already pulled from public Actor pages on apify.com. MCP/API/CLI should mirror that. This PR covers the MCP surface only, per team decision — API/CLI and a follow-up "show verified creators" feature (blocked on apify-core exposing that field publicly) are tracked separately.

Testing

pnpm run type-check, pnpm run lint, pnpm run test:unit (89 files, 1245 passed / 1 skipped), pnpm run format (no diff), pnpm run check:agents — all clean. Confirmed via grep that successRate/"Runs succeeded" has zero remaining references in src/ and tests/, and that the widget/apps-mode path never carried this field (nothing to change there). Independently re-verified by a fresh subagent (PASS).

@github-actions github-actions Bot added t-ai Issues owned by the AI team. tested Temporary label used only programatically for some analytics. labels Jul 30, 2026
@MQ37
MQ37 requested a review from vojtechj-apify July 30, 2026 10:37
@MQ37

MQ37 commented Jul 30, 2026

Copy link
Copy Markdown
Contributor Author

FYI @jirispilka @RobertCrupa

@MQ37
MQ37 merged commit 7142233 into master Jul 31, 2026
13 checks passed
@MQ37
MQ37 deleted the investigate/runs-succeeded-metric branch July 31, 2026 10:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants