Skip to content

Conversation

MQ37
Copy link
Contributor

@MQ37 MQ37 commented Oct 4, 2025

close: #1974

@MQ37 MQ37 requested a review from TC-MO as a code owner October 4, 2025 08:53
@github-actions github-actions bot added the t-ai Issues owned by the AI team. label Oct 4, 2025
@MQ37 MQ37 requested a review from jirispilka October 4, 2025 08:53
@apify-service-account
Copy link

Preview for this PR was built for commit 32a462b and is ready at https://pr-1973.preview.docs.apify.com!

@TC-MO
Copy link
Contributor

TC-MO commented Oct 6, 2025

We definitely cannot have two admonitions next to each other, it visually way to distracting and just not a good practice. But I think I may have a solution (tho a little more involved)

What are your thoughts about doing a slight reshuffle of the order of headings. I would propose the following

Available tools

Table

get-actor-output admonitions

Dynamic tool discovery (as it to me feels like a extension of all available tools)

All of it's content

Dynamic tool discovery admonition

Thoughts @jirispilka @MQ37 ?

@apify-service-account
Copy link

Preview for this PR was built for commit 03296f35 and is ready at https://pr-1973.preview.docs.apify.com!

@jirispilka
Copy link
Contributor

jirispilka commented Oct 6, 2025

@TC-MO thanks for the suggestion. I decided to fix it right away based on it. Please check it out. Thanks

@TC-MO
Copy link
Contributor

TC-MO commented Oct 6, 2025

Thanks! I think you've left accidentally first Dynamic tool discovery heading & content
obraz

Also what do you think about making Dynamic tool discovery H4?

@apify-service-account
Copy link

Preview for this PR was built for commit be13f96b and is ready at https://pr-1973.preview.docs.apify.com!

@jirispilka
Copy link
Contributor

oh, right, thank you!

@TC-MO
Copy link
Contributor

TC-MO commented Oct 6, 2025

Yeah i think it works much better, thank you!

Copy link
Contributor

@TC-MO TC-MO left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@jirispilka jirispilka merged commit 09de25b into master Oct 6, 2025
9 checks passed
@jirispilka jirispilka deleted the docs/mcp-dynamic-tool-discovery-note branch October 6, 2025 13:03
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.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

MCP tools documentation (mixed content in the columns), see the last row in the screenshot
4 participants