Skip to content

Commit cf23892

Browse files
Dumbrisclaude
andcommitted
docs(web): align setup.md and test prose with the Review & connect flow
Codex review follow-up: docs/setup.md described the Claude Desktop bridge registration as one-click "Connect"; it is now review-then- confirm. Test comments updated to match the renamed row action. Spec 046/075 texts intentionally left untouched — they are point-in- time records (046 FR-005 already required the diff preview this branch implements). Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
1 parent b8485fa commit cf23892

3 files changed

Lines changed: 6 additions & 6 deletions

File tree

docs/setup.md

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -254,7 +254,7 @@ Add mcpproxy as a remote MCP server via Settings → Connectors → Add Custom C
254254

255255
#### Option A: Free Plan — JSON Configuration
256256

257-
> **💡 One-click:** mcpproxy's built-in **Connect** wizard (Web UI / tray) can write this bridge configuration for you automatically — pick **Claude Desktop** and click **Connect**. It registers the `npx -y mcp-remote` bridge shown below (Node.js required). The manual steps remain available if you prefer to edit the file yourself.
257+
> **💡 Built-in wizard:** mcpproxy's **Connect** wizard (Web UI / tray) can write this bridge configuration for you — pick **Claude Desktop**, click **Review & connect** to see the exact entry that will be written (a timestamped backup is created first), then confirm with **Connect**. It registers the `npx -y mcp-remote` bridge shown below (Node.js required). The manual steps remain available if you prefer to edit the file yourself.
258258
259259
1. Create the config file if it doesn't exist:
260260

frontend/tests/unit/connect-modal.spec.ts

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -151,7 +151,7 @@ describe('ConnectModal', () => {
151151
await wrapper.setProps({ show: true })
152152
await flushPromises()
153153

154-
// A real one-click Connect button must be offered (not greyed out).
154+
// A real Review & connect button must be offered (not greyed out).
155155
const connectButton = wrapper.find('button.btn-primary.btn-xs')
156156
expect(connectButton.exists()).toBe(true)
157157
expect(connectButton.text()).toContain('Review & connect')
@@ -290,7 +290,7 @@ describe('ConnectModal', () => {
290290
expect(codexRow.exists()).toBe(true)
291291
expect(codexRow.text()).toContain('Disconnect')
292292

293-
// cursor is genuinely not connected -> Connect button still offered.
293+
// cursor is genuinely not connected -> Review & connect button still offered.
294294
const connectButtons = wrapper.findAll('button.btn-primary.btn-xs')
295295
expect(connectButtons.length).toBe(1)
296296
expect(connectButtons[0].text()).toContain('Review & connect')
@@ -326,7 +326,7 @@ describe('ConnectModal', () => {
326326
expect(wrapper.find('[data-test="connect-denied-banner"]').exists()).toBe(false)
327327
// Explicit, no-eager-read affordance to verify access on demand.
328328
expect(wrapper.find('[data-test="connect-check-access"]').exists()).toBe(true)
329-
// Connect remains offered.
329+
// Review & connect remains offered.
330330
expect(wrapper.find('button.btn-primary.btn-xs').text()).toContain('Review & connect')
331331
})
332332

@@ -433,7 +433,7 @@ describe('ConnectModal', () => {
433433
await wrapper.setProps({ show: true })
434434
await flushPromises()
435435

436-
// Spec 078 US1: click Connect → preview panel → confirm writes.
436+
// Spec 078 US1: click Review & connect → preview panel → confirm (Connect) writes.
437437
await wrapper.find('[data-test="connect-start-cursor"]').trigger('click')
438438
await flushPromises()
439439
await wrapper.find('[data-test="connect-preview-confirm-cursor"]').trigger('click')

frontend/tests/unit/onboarding-wizard-connected-merge.spec.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -99,7 +99,7 @@ describe('OnboardingWizard connected_client_ids merge', () => {
9999
expect(codexRow.text()).toContain('Connected')
100100
expect(wrapper.find('[data-test="connect-codex"]').exists()).toBe(false)
101101

102-
// cursor genuinely not connected -> Connect button still offered.
102+
// cursor genuinely not connected -> Review & connect button still offered.
103103
const cursorRow = wrapper.find('[data-test="client-row-cursor"]')
104104
expect(cursorRow.exists()).toBe(true)
105105
expect(wrapper.find('[data-test="connect-cursor"]').exists()).toBe(true)

0 commit comments

Comments
 (0)