docs(audit): tier 6 — 'blocklist' (block list UI) → 'block list' to disambiguate#109
Conversation
…isambiguate The style_lint rule 'blocklist → denylist' (per AGENTS.md allowlist/denylist guidance) was firing on 'blocklist' references that actually mean the terminal's *block list* (the UI panel listing command blocks). That's a distinct concept from the allowlist/denylist style rule. Fixed by spacing 'blocklist' → 'block list' (two words) so the prose is unambiguous to both readers and the linter: * full-terminal-use.mdx (2x) * blocks-as-context.mdx (3x) * conversation-forking.mdx * local-agents/interacting-with-agents/index.mdx (2x) * terminal-and-agent-modes.mdx (3x) * more-features/full-screen-apps.mdx (2x) * more-features/text-selection.mdx ## Intentionally skipped (literal UI feature names / image asset) Need product verification before changing these; left as-is so prose stays accurate to the in-app UI labels: * terminal/warpify/subshells.mdx:44,46 — 'Blocklisting commands' / 'add commands to the Blocklist' (Warpify denylist feature) * privacy.mdx:85,87 — 'in the AI blocklist' (AI rule denylist UI) * secret-redaction.mdx:22 — 'Hide secrets in blocklist' setting (Settings > Privacy > Secret redaction > <literal toggle name>) * code/code-review.mdx:53 — image filename 'Blocklist-with-review-changes.png' (renaming the asset is a separate effort with git history considerations) Validation: 0 broken links / 2577 internal, lint clears 12 DEPRECATED-TERM warnings (19 → 7 remaining, all in the skipped-by-design category), 332 pages build clean. Co-Authored-By: Oz <oz-agent@warp.dev>
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
|
I'm starting a first review of this pull request. You can view the conversation on Warp. I completed the review and no human review was requested for this pull request. Comment Powered by Oz |
![oz-for-oss[bot]](https://avatars.githubusercontent.com/in/3450139?s=60&v=4){kind=small}
There was a problem hiding this comment.
Overview
This PR updates documentation references from blocklist / Blocklist to block list where the prose refers to Warp's terminal block list rather than a denylist-style control. The changes are limited to terminology in existing MDX pages and do not alter links, code examples, frontmatter, or site structure.
Concerns
No blocking concerns found. The attached spec context does not define implementation requirements for this PR, and the documentation-only diff does not introduce security-relevant behavior.
Verdict
Found: 0 critical, 0 important, 0 suggestions
Approve
Comment /oz-review on this pull request to retrigger a review (up to 3 times on the same pull request).
Powered by Oz
2 participants
Post-orchestration-launch audit, pass 3: clear the 'blocklist → denylist' lint warnings where the prose actually means the terminal's block list (the UI panel listing command blocks), not a denylist.
Background
The
style_lintruleblocklist → denylist(per AGENTS.mdallowlist/denylist (not 'whitelist'/'blocklist')) was firing on 19 occurrences. On inspection, most of those refer to the block list (the UI panel listing terminal command blocks) — a distinct concept from a denylist. Fix is to space it as 'block list' (two words) so:Changes (12 occurrences across 7 files)
agent-platform/capabilities/full-terminal-use.mdx(L110, L117)agent-platform/local-agents/agent-context/blocks-as-context.mdx(L15, L80, L81)agent-platform/local-agents/interacting-with-agents/conversation-forking.mdx(L44)agent-platform/local-agents/interacting-with-agents/index.mdx(L145, L147)agent-platform/local-agents/interacting-with-agents/terminal-and-agent-modes.mdx(L101, L102, L108)terminal/more-features/full-screen-apps.mdx(L36, L45)terminal/more-features/text-selection.mdx(L17)Intentionally skipped (literal UI feature names / image asset)
These need product verification before changing — they look like literal in-app UI labels, so updating the prose alone could create referential inaccuracy. Left as-is:
terminal/warpify/subshells.mdxL44, L46 — 'Blocklisting commands' / 'add commands to the Blocklist' (the Warpify denylist feature)support-and-community/privacy-and-security/privacy.mdxL85, L87 — telemetry event descriptions referencing the in-app 'AI blocklist' UIsupport-and-community/privacy-and-security/secret-redaction.mdxL22 — refers to a literal Settings toggle: Settings > Privacy > Secret redaction > Hide secrets in blocklistcode/code-review.mdxL53 — image filenameBlocklist-with-review-changes.png(renaming the asset is a separate effort with git history considerations)If the in-app UI has actually been renamed to 'denylist', a follow-up PR can sync both the docs and (where applicable) the image asset.
Validation
This is the third and final pass of the post-orchestration-launch audit. Sister PRs:
All three branch off
mainand edit disjoint files (or disjoint regions within shared files likeprivacy.mdx), so they can merge in any order without conflicting.Co-Authored-By: Oz oz-agent@warp.dev