Skip to content

feat: Limit tools to discovery, dynamic Actor management, and help; simplify Actor input schema; return all dataset items at once with only relevant fields in outputs #158

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
merged 6 commits into from
Jul 7, 2025
Merged

feat: Limit tools to discovery, dynamic Actor management, and help; simplify Actor input schema; return all dataset items at once with only relevant fields in outputs #158

merged 6 commits into from
Jul 7, 2025

Conversation

@MQ37
Copy link
Contributor

@MQ37 MQ37 commented Jul 4, 2025 •
edited
Loading

Currently only these tools are enabled:

  • get-actor-details
  • apify-actor-help-tool
  • search-actors
  • add-actor
  • plus default Actors

Changes:

  • improved internal tool output format - still JSON with only relevant fields and complex props transformed to string (for example pricing info)
  • simplified Actor tool input schema - only required fields and integer fields if required defined else dump whole input schema
  • simplified Actor tool output - return all output dataset items at once with only fields defined in dataset views if dataset views defined else dump all fields

closes #153
closes #152
closes #121
closes #159

MQ37 added 2 commits July 4, 2025 15:50
@MQ37
Disable internal tools except actor details, help tool, search actors…
5f44003 
…. Improve help tool, search actors and get actor details output.
@MQ37
Return only relevant output dataset properies if defined in storage v…
4696373 
…iews defiintion, if dump all properties. Fix input schema for Actors without any input schema.
@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 4, 2025
@MQ37 MQ37 changed the title feat: Disable tools not related to discovery, dynamic adding and calling of Actors and help tool. Return only properties mentioned in the Actor definition storage views if defined, otherwise dump everything. feat: Disable tools not related to discovery, dynamic adding and calling of Actors and help tool. Return only properties mentioned in the Actor definition storage views if defined, otherwise dump everything. Improve output of tools by returning only relevant fields and transforming complex properties into human readable strings. Jul 4, 2025
@MQ37 MQ37 marked this pull request as ready for review July 7, 2025 09:41
@MQ37 MQ37 marked this pull request as draft July 7, 2025 09:42
@MQ37 MQ37 changed the title feat: Disable tools not related to discovery, dynamic adding and calling of Actors and help tool. Return only properties mentioned in the Actor definition storage views if defined, otherwise dump everything. Improve output of tools by returning only relevant fields and transforming complex properties into human readable strings. feat: Limit tools to discovery, dynamic Actor management, and help; simplify Actor input schema; return all dataset items at once with only relevant fields in outputs Jul 7, 2025
@MQ37 MQ37 marked this pull request as ready for review July 7, 2025 12:37
@MQ37 MQ37 requested a review from MichalKalita July 7, 2025 12:38
MichalKalita
MichalKalita suggested changes Jul 7, 2025
View reviewed changes
MQ37 and others added 2 commits July 7, 2025 15:49
@MQ37 @MichalKalita
Update tests/integration/suite.ts
7bfe929 
Co-authored-by: Michal Kalita <kalita.michal@gmail.com>
@MQ37
split long one liner
5e9e145
@MQ37 MQ37 merged commit dd7a924 into master Jul 7, 2025
2 checks passed
@MQ37 MQ37 deleted the feat/fewer-tools--improve-tool-output branch July 7, 2025 14:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

1 more reviewer

@MichalKalita MichalKalita MichalKalita approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

@MQ37 MQ37

Labels

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

Projects

None yet

Milestone

No milestone

2 participants

@MQ37 @MichalKalita