Copy and naming pass: providers, add flows, API keys, policies#1267
Conversation
Greptile SummaryThis is a copy and naming consistency pass across providers, add flows, API keys, and policies pages. All changes are text-only; no logic, data, or component structure was modified.
Confidence Score: 5/5All changes are UI copy only — no logic, API calls, or data mutations were modified, so there is no runtime risk. Every diff line is a string literal or label map entry. The one structural change in secrets.tsx (adding per-entry descriptions and a fallback title-caser) is straightforward and well-bounded. No files require special attention. Important Files Changed
Flowchart%%{init: {'theme': 'neutral'}}%%
flowchart TD
A[Provider key from API] --> B{key in PROVIDER_LABELS?}
B -- Yes --> C[Use mapped label\ne.g. 'Encrypted store']
B -- No --> D[Fallback: split on - or _\ncapitalize each word]
C --> E{key === 'encrypted'?}
D --> E
E -- Yes --> F[Description: encrypted DB copy\nBadge: 'default']
E -- No --> G[Description: external provider copy\nBadge: 'provider']
%%{init: {'theme': 'base', 'themeVariables': {"darkMode": true, "background": "#0d1117", "primaryColor": "#21262d", "primaryTextColor": "#e6edf3", "primaryBorderColor": "#8b949e", "lineColor": "#8b949e", "textColor": "#e6edf3", "edgeLabelBackground": "#161b22", "actorBkg": "#21262d", "actorBorder": "#8b949e", "actorTextColor": "#e6edf3", "actorLineColor": "#8b949e", "signalColor": "#8b949e", "signalTextColor": "#e6edf3", "noteBkgColor": "#373320", "noteBorderColor": "#d4a72c", "noteTextColor": "#f0e6c0", "labelBoxBkgColor": "#21262d", "labelBoxBorderColor": "#8b949e", "labelTextColor": "#e6edf3", "loopTextColor": "#e6edf3", "activationBkgColor": "#30363d", "activationBorderColor": "#8b949e"}}}%%
flowchart TD
A[Provider key from API] --> B{key in PROVIDER_LABELS?}
B -- Yes --> C[Use mapped label\ne.g. 'Encrypted store']
B -- No --> D[Fallback: split on - or _\ncapitalize each word]
C --> E{key === 'encrypted'?}
D --> E
E -- Yes --> F[Description: encrypted DB copy\nBadge: 'default']
E -- No --> G[Description: external provider copy\nBadge: 'provider']
Reviews (4): Last reviewed commit: "Clarify provider and integration copy" | Re-trigger Greptile |
| <PageHeader | ||
| title="Policies" | ||
| description="Override default approval behavior for tools. The most restrictive matched action wins. Blocked tools are hidden from agent search and fail at invoke." | ||
| description="Decide which tools run automatically, which ask for approval, and which are blocked. The most restrictive matching rule wins; blocked tools are hidden from agents." |
There was a problem hiding this comment.
"fail at invoke" behavior dropped from description
The old description stated "Blocked tools are hidden from agent search and fail at invoke." The new description only says "blocked tools are hidden from agents," omitting what happens when a blocked tool is invoked directly (e.g., via API or explicit tool call). A user who sees the block is silent-to-agents might assume invocation still works through another path and be surprised by an error at runtime.
Cloudflare previewTorn down — the PR is closed. |
Deploying with
|
| Status | Name | Latest Commit | Preview URL | Updated (UTC) |
|---|---|---|---|---|
| ✅ Deployment successful! View logs |
executor-marketing | 97bacb8 | Commit Preview URL Branch Preview URL |
Jul 02 2026, 08:23 PM |
Deploying with
|
| Status | Name | Latest Commit | Updated (UTC) |
|---|---|---|---|
| ✅ Deployment successful! View logs |
executor-cloud | 97bacb8 | Jul 02 2026, 08:26 PM |
@executor-js/cli
@executor-js/config
@executor-js/execution
@executor-js/sdk
@executor-js/codemode-core
@executor-js/runtime-quickjs
@executor-js/plugin-file-secrets
@executor-js/plugin-graphql
@executor-js/plugin-keychain
@executor-js/plugin-mcp
@executor-js/plugin-onepassword
@executor-js/plugin-openapi
executor
commit: |
826e586 to
e62899a
Compare
ac34707 to
9566a11
Compare
45539b1 to
97559dc
Compare
9566a11 to
9c567e5
Compare
97559dc to
4d35a3c
Compare
26755fb to
51ed6e9
Compare
4d35a3c to
fcc2b6e
Compare
51ed6e9 to
7828740
Compare
fcc2b6e to
e502c2f
Compare
7828740 to
07590a7
Compare
e502c2f to
074d005
Compare
07590a7 to
97bacb8
Compare
Several copy problems surfaced in a first-run walkthrough. The Providers page rendered the built-in store as encrypted encrypted provider because the label map was keyed default while the runtime provider key is encrypted. Add flows disagreed on the noun (Add MCP Source, Add GraphQL Source, Add OpenAPI Integration). API keys copy explained one acronym with another (PATs). The policies description led with matching mechanics before saying what policies are for.
The built-in provider now renders as Encrypted store with a default badge and a plain-language description; add flows all say Add ... integration; PATs is now personal access tokens; the policies header leads with what the user decides (run automatically, ask for approval, block) before the precedence rule.
Verified in the browser on each page. Typecheck is green.
Stacked on #1266.