Skip to content

feat(dashboard): frame ACP as optional advanced setup - #11206

Merged
diegosouzapw merged 9 commits into
cheaper-inference:release/v3.8.50from
ignamiranda:contributor/feat/beginner-ux-acp-agents
Aug 23, 2026
Merged

diegosouzapw merged 9 commits into
cheaper-inference:release/v3.8.50from
ignamiranda:contributor/feat/beginner-ux-acp-agents

Conversation

@ignamiranda

@ignamiranda ignamiranda commented Aug 23, 2026 •

Copy link
Copy Markdown
Contributor

What

  • lead the ACP concept card with the task outcome instead of protocol mechanics
  • mark ACP as advanced and optional in the ACP-specific card
  • add focused coverage so the warning never appears on Code or Agent cards

Why

ACP is useful for long-running backend agent tasks, but most users do not need it during initial setup. The current copy presents the protocol before the outcome and can make an optional integration look mandatory.

Validation

  • npx vitest run tests/unit/ui/CliConceptCard.test.tsx
  • npm run lint
  • npm run typecheck:core
  • npm run i18n:check-ui-coverage

Part of #11167.

⚠️ base-red inherited: #9985

diegosouzapw and others added 8 commits August 23, 2026 02:15
…per-inference#11208)

The sponsor banner landed in en/pt without the vi copy, breaking the strict
Vietnamese parity gate (3 failing tests in i18n-vi-completeness: key parity,
ICU placeholder parity, ICU parse regression) — red on every release-branch CI
run since.

Adds the 5 missing vi translations (title, description, cta, partnerLinkNote,
dismissAriaLabel). The basereds-sse-vi worktree that owned this debt has been
idle for 4 days, so this drains it here.

Verified: i18n-vi-completeness 5/5, i18n:check-ui-coverage PASS (vi now
100.0%), i18n:check-value-drift PASS, prettier clean.

Co-authored-by: Xiangzhe <bakryun0718@proton.me>
…heaper-inference#11139)

Validated on the combined batch board over tip 17897cc: gates clean (changelog, file-size 159 frozen, complexity 2628<=2774, cognitive 1184<=1223, dead-code 409<=416, provider-consistency 267/350/0), typecheck:core clean, 83/83 PR suites + neighbors green (mcp-route-scope-carveout 15 tests with bug-injection proof, management-auth-hardening per-route call-shape pinning). The route layer now honors the cheaper-inference#9159 mcp:connect carve-out exactly like the policy layer — audit/audit-stats deliberately stay manage-only, oma_ tokens still require admin. Thank you @HouMinXi!
…leness (cheaper-inference#11141)

Validated on the combined batch board + this branch: oauth-400-recovery + quota-connection-recovery 22/22; file-size gate green with the test/route.ts 1215 frozen entry (dated annotation for the +190 rebuilt-probe lines, owner-authorized). Unknown expiry + refresh token now means proactive refresh before probing, a hard 400 gets exactly one refresh+retry with fresh-token assertion, and the recovery tick clears stale error labels — the four dead-for-weeks antigravity connections from the production report would have self-healed. TDD with bug-injection proof on each leg. Thank you @HouMinXi!
cheaper-inference#11140)

Validated on the combined batch board + this branch: context7-provider 33/33 + search-registry 45/45 (propagated to 17 providers), docs-counts green (350 everywhere: PROVIDER_REFERENCE regenerated, SVGs, README, AGENTS.md, package.json, llm.txt + 42 mirrors synced), provider-consistency OK. Hardening reviewed: canonical isValidContext7LibraryId shared by normalizer+executor, byte-capped error bodies, validated baseUrl with public fallback, anonymous-tier quota sentinel, excluded from auto-select walks. Thank you @HouMinXi — 15 forge-review cycles show!
…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!
@diegosouzapw
diegosouzapw merged commit 7e57354 into cheaper-inference:release/v3.8.50 Aug 23, 2026
4 of 7 checks passed
muhamadgalihsaputra pushed a commit to niyatna/NiyatnaRoute that referenced this pull request Sep 27, 2026
…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!
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants