Repository navigation
Beginner UX: guided endpoint connection header - #11228
Merged
diegosouzapw merged 9 commits intoAug 23, 2026
Merged
diegosouzapw merged 9 commits into
diegosouzapw merged 9 commits into
Conversation
…inference#11204) Validated on the combined batch board: gates clean (changelog, file-size 159 frozen, complexity 2620<=2774, cognitive 1180<=1223, dead-code 409<=416), typecheck:core clean, i18n suites 16/16 after the vi translation pass. Two-string reframe that stops Embedded Services reading as required infrastructure. Thank you @ignamiranda!
…rence#11195) Validated on the combined batch board: gates + typecheck clean, i18n suites 16/16. Pre-merge: the vi.json request-flow strings arrived in English — translated to Vietnamese on the branch (vi is the locale that requires real translations, not markers). Purpose-first API-key orientation with the request-path visual, controls unchanged. Part of cheaper-inference#11167. Thank you @ignamiranda!
…nce#11206) Validated on the combined batch board (CliConceptCard vitest 9/9 — the ACP warning never leaks onto Code/Agent cards) + this branch: i18n suites 16/16 with the Vietnamese translation of the new acp warning added here. ACP now reads as optional/advanced before its mechanics. Thank you @ignamiranda!
…eaper-inference#11215) Validated on the combined batch board (resilience-connections static test green) + this branch: i18n suites 16/16 with Vietnamese translations of the reassurance/legend strings added here. Plain-language reassurance + three-state legend lead the page; the technical table stays as advanced detail. Thank you @ignamiranda!
…aper-inference#11224) Validated on the combined batch board (health-page static test green, new testing-library deps installed for the component test) + this branch: i18n suites 16/16 with Vietnamese translations of the verdict/diagnostics strings added here. Plain-language verdict leads; advanced diagnostics collapse behind a toggle. Thank you @ignamiranda!
…rence#11227) Validated on the combined batch board (batch-page static test green) + this branch: i18n suites 16/16 with Vietnamese translations of the batch header/step strings added here. Stable outcome header with the 3-step flow; the collapsible concept card stays as advanced detail. Thank you @ignamiranda!
diegosouzapw
merged commit Aug 23, 2026
8a42aee
into
cheaper-inference:release/v3.8.50
5 of 7 checks passed
This was referenced Aug 23, 2026
Collaborator
|
Heads-up @ignamiranda — we had to surgically revert the `EndpointPageClient.tsx` hunk of this PR in #11256 (base-red #9985): the hunk replaced the `groupModels.map` callback's `return (` inside `ProviderModelsModal` with the guided-header JSX, so the file stopped parsing and every Build App / Docker publish run went red. The rest of the PR (health verdict header, resilience reassurance, i18n keys) is intact and live. The guided endpoint connection header is still wanted — it just needs to land in the page-level return of `EndpointPageClient` (the main component body), not inside the modal. If you open a follow-up, please run `npm run build` (or at least `npx prettier --check` + the endpoint vitest suite) before pushing — the Fast Quality Gates did not catch this one because the parse error only surfaces at build time. Thanks again for the beginner-UX wave! |
diegosouzapw
pushed a commit
that referenced
this pull request
Aug 23, 2026
…n main return (#11228 follow-up) The #11228 hunk spliced the guided endpoint header into ProviderModelsModal's renderModelGroup map callback, replacing its return statement and leaving an unclosed <div> — dashboard typecheck failed on the release tip. Restore the modal's map return and render the header (title/subtitle//v1 URL/test link/ advanced-protocols note) at the top of the main component return, using the i18n keys the PR added (endpoint.title/subtitle/testEndpoint/advancedProtocols) and the existing displayBaseUrl variable. Also untrack two docs/superpowers planning files that leaked in via #11213; they belong to _tasks/ (copies preserved there) and their tracked presence trips check:tracked-artifacts on every commit. Validated: check:dashboard-typecheck OK (220 pre-existing, all within frozen baseline; zero syntax errors in EndpointPageClient.tsx).
diegosouzapw
added a commit
that referenced
this pull request
Aug 23, 2026
…e-red #9985) (#11256) The squash merge of #11228 applied its endpoint-header hunk inside ProviderModelsModal, replacing the groupModels.map callback's return statement with the page-level guided header JSX. The file no longer parsed (Turbopack: 3 errors at line 2397; release-green reported the same defect as '1 ESLint error'), red-ing every Build App / Docker publish run since 8a42aee. Surgical revert of that single hunk: the file is byte-identical to its pre-#11228 state. The rest of #11228 (health page verdict header, resilience reassurance, i18n keys) parses fine and stays. Validation: prettier parse OK; EndpointPageClient.test.tsx 4/4 (the pre-existing jsdom render suite imports the component, so it is the permanent regression guard); diff vs pre-#11228 empty. Refs #9985 Co-authored-by: Xiangzhe <bakryun0718@proton.me>
This was referenced Aug 23, 2026
muhamadgalihsaputra
pushed a commit
to niyatna/NiyatnaRoute
that referenced
this pull request
Sep 27, 2026
Validated on the combined batch board + this branch: i18n suites 16/16 with Vietnamese translations of the new endpoint strings added here. Guided connection header with the /v1 URL and test action leads; advanced protocols demoted. Thank you @ignamiranda!
muhamadgalihsaputra
pushed a commit
to niyatna/NiyatnaRoute
that referenced
this pull request
Sep 27, 2026
…n main return (cheaper-inference#11228 follow-up) The cheaper-inference#11228 hunk spliced the guided endpoint header into ProviderModelsModal's renderModelGroup map callback, replacing its return statement and leaving an unclosed <div> — dashboard typecheck failed on the release tip. Restore the modal's map return and render the header (title/subtitle//v1 URL/test link/ advanced-protocols note) at the top of the main component return, using the i18n keys the PR added (endpoint.title/subtitle/testEndpoint/advancedProtocols) and the existing displayBaseUrl variable. Also untrack two docs/superpowers planning files that leaked in via cheaper-inference#11213; they belong to _tasks/ (copies preserved there) and their tracked presence trips check:tracked-artifacts on every commit. Validated: check:dashboard-typecheck OK (220 pre-existing, all within frozen baseline; zero syntax errors in EndpointPageClient.tsx).
muhamadgalihsaputra
pushed a commit
to niyatna/NiyatnaRoute
that referenced
this pull request
Sep 27, 2026
…erence#11228 (base-red cheaper-inference#9985) (cheaper-inference#11256) The squash merge of cheaper-inference#11228 applied its endpoint-header hunk inside ProviderModelsModal, replacing the groupModels.map callback's return statement with the page-level guided header JSX. The file no longer parsed (Turbopack: 3 errors at line 2397; release-green reported the same defect as '1 ESLint error'), red-ing every Build App / Docker publish run since 76f9b2f. Surgical revert of that single hunk: the file is byte-identical to its pre-cheaper-inference#11228 state. The rest of cheaper-inference#11228 (health page verdict header, resilience reassurance, i18n keys) parses fine and stays. Validation: prettier parse OK; EndpointPageClient.test.tsx 4/4 (the pre-existing jsdom render suite imports the component, so it is the permanent regression guard); diff vs pre-cheaper-inference#11228 empty. Refs cheaper-inference#9985 Co-authored-by: Xiangzhe <bakryun0718@proton.me>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Pitch #8 (Endpoint guided connection, Wave 3) — stable header with endpoint URL (/v1), test action, advanced protocols demoted. Branch: contributor/feat/beginner-ux-endpoint. Targets release/v3.8.50, inherits base-red #9985. 2 files, 16 insertions. Static test included.