Repository navigation
Say which integration proxy carries a connection's traffic - #1724
michielbdejong wants to merge 2 commits into
Conversation
Self-hosted servers default to a hosted integration proxy (#1700), so wherever someone connects an external service they should see where that traffic goes and how to run their own proxy. A shared ProxyTrafficNotice names the configured proxy's host ("atomic.place" for atomic.place and its subdomains) and links to the atomic-integration-proxy self-hosting guide. It appears in the app's "Connect an account" bar and under the proxy URL field in Settings > Integration. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
The pre-commit format check was skipped when this branch was committed with --no-verify. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
|
Merge-risk triage (2026-09-28). This is a read-only review of this PR's own diff. The rule it applies: nothing that works today may break for existing users or clients, and new behaviour is added alongside the old. The stack is being rebased onto Verdict: SAFE UI text only: a notice naming the proxy that carries the traffic, plus Note: Follow-up: #1733 leaves a 🤖 Generated with Claude Code |
|
Closing as part of the plugin-host stack cleanup (Q-101): not needed by atomic-plugins now. See the inventory at #1726 (comment). The branch is kept, and pin candidates are unaffected. Closed by Claude (VPS coordinator) on behalf of Michiel. |
Refs #1700
What
Self-hosted servers default to a hosted integration proxy, so wherever someone connects an external service the UI now says where that traffic goes and links to running your own proxy:
helpers/proxyTrafficHost.ts: the host to name.atomic.placefor atomic.place and any subdomain (e.g.integrations.atomic.place); any other proxy is named by its own host, port included (localhost:8787). Lookalikes such asevilatomic.placeoratomic.place.example.comare not treated as atomic.place.components/ProxyTrafficNotice.tsx: one small shared component that reads the configured proxy (useIntegrationProxy(), so the per-browser override in Settings applies) and always shows the self-hosting link.AppFrame), the one place in this repo where a connection to an external service starts: plugins such as GitHub, Clockify and Notion now live in atomic-plugins and callstore.proxy.connect(), which opens this bar. The bar's sentence used to include the full proxy origin. The notice now names the host, so that part of the sentence was removed.VITE_INTEGRATION_PROXY_URLorhttps://localthought.io); integrations.atomic.place isn't live yet.Translations
The catalogs were settled at the extractor's fixed point:
pnpm clean-translations, thenrm -rf src/locales/.wuchaleand a vite run in this worktree. That run produced no further.pochanges. Read the diff by hand: one entry was replaced (the connect sentence, which no longer contains the proxy origin), and the notice was added as one message with the host and the link as tags. No other strings moved.Screenshots
Screenshots to follow: I could not upload them from the agent session. They were taken against a local atomic-server on this branch. The connect bar comes from a new App whose view calls
store.proxy.connect({ platform: 'github' }).Tests
pnpm typecheckpasses.vitest: the newproxyTrafficHost.test.tshas 6 tests, all passing. In the full run, 1222 tests passed. Two tests failed insidebarDeletion.test.tsx(atomic:resource:vsdid:ad:resource:subjects); that file doesn't touch this change.scripts/integration-mcp.test.mjsalso failed, but it is a node:test file thatpnpm testexcludes from vitest.🤖 Generated with Claude Code