fix: replace skip with skipOnClient to avoid runtime errors

[agatha197]

follows #2743

Changes:

• Updated useControllableValue hook in EndpointSelect component to explicitly type it with string
• Changed @skip directive to @skipOnClient in GraphQL query for the endpoint field

Rationale:

• The explicit typing of useControllableValue<string> improves type safety and clarity in the codebase
• Switching from @skip to @skipOnClient directive likely aligns with updated GraphQL schema or client-side optimization strategies

Effects:

• These changes should not directly affect users
• Developers will benefit from improved type checking and potentially more efficient GraphQL query execution
• Avoid runtime errors when endpoint is empty.

How to test:

1. Remove all services
2. Enable enableLLMPlayground from config.toml
3. Visit LLM Playground page

Checklist for reviewer:

• Check if runtime error disappears.

Checklist:

• Mention to the original issue
• Documentation
• Minium required manager version
• Specific setting for review (eg., KB link, endpoint or how to setup)
• Minimum requirements to check during review
• Test case(s) to demonstrate the difference of before/after

[graphite-app]

Your org requires the Graphite merge queue for merging into main

Add the label “flow:merge-queue” to the PR and Graphite will automatically add it to the merge queue when it’s ready to merge. Or use the label “flow:hotfix” to add to the merge queue as a hot fix.

You must have a Graphite account and log in to Graphite in order to use the merge queue. Sign up using this link.

[agatha197]

• fix: replace skip with skipOnClient to avoid runtime errors #2751 👈
• main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @agatha197 and the rest of your teammates on Graphite

[github-actions]

Coverage report for ./react

St.❔	Category	Percentage	Covered / Total
🔴	Statements	5.33%	348/6533
🔴	Branches	4.82%	218/4522
🔴	Functions	3.1%	67/2163
🔴	Lines	5.22%	333/6381

Test suite run success

93 tests passing in 12 suites.

Report generated by 🧪jest coverage report action from e43f244

[ironAiken2]

LGTM

[yomybaby]

The GraphQL server performs argument type checking before evaluating directives. Therefore, even when using @skip, the argument types must be adhered to. For convenience, let's use @skipOnClient only when using the uuid argument. In all other cases, consider @skip first.

[graphite-app]

Merge activity

• Oct 20, 9:56 PM EDT: A user added this pull request to the Graphite merge queue.
• Oct 20, 9:58 PM EDT: A user merged this pull request with the Graphite merge queue.