Skip to content

Rename supportsToolsWhitelist to supportsToolsAllowlist#1134

Merged
pelikhan merged 2 commits intomainfrom
copilot/fix-57f24f55-296f-4ee1-a003-ff8d1ad73206
Oct 1, 2025
Merged

Rename supportsToolsWhitelist to supportsToolsAllowlist#1134
pelikhan merged 2 commits intomainfrom
copilot/fix-57f24f55-296f-4ee1-a003-ff8d1ad73206

Conversation

Copy link
Copy Markdown
Contributor

Copilot AI commented Oct 1, 2025

This PR updates terminology from the deprecated term "whitelist" to the more inclusive and appropriate term "allowlist" throughout the codebase.

Background

The Go code and compiler already used the correct supportsToolsAllowlist field name and SupportsToolsAllowlist() method. However, test data and documentation still referenced the old "whitelist" terminology, creating inconsistency.

Changes

Test Data (pkg/cli/logs_test.go)

Updated JSON test fixtures to use supports_tools_allowlist instead of supports_tools_whitelist:

{
  "engine_id": "claude",
  "supports_tools_allowlist": true,  // was: supports_tools_whitelist
  ...
}

Documentation Updates

  • docs/src/content/docs/reference/safe-outputs.md: Changed "automatically whitelisted" to "automatically allowed" for Git commands
  • docs/src/content/docs/guides/mcps.md: Updated 3 occurrences from "tool whitelisting" to "tool allow-listing"

Impact

This is a terminology-only change with no functional impact. The field name in compiled workflows (.lock.yml files) already used the correct supports_tools_allowlist name, so existing workflows remain compatible.

All tests pass and the codebase now uses consistent, inclusive terminology throughout.

Original prompt

Rename supportsToolsWhitelist to supportsToolsAllowlist


💬 Share your feedback on Copilot coding agent for the chance to win a $200 gift card! Click here to start the survey.

Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
Copilot AI changed the title [WIP] Rename supportsToolsWhitelist to supportsToolsAllowlist Rename supportsToolsWhitelist to supportsToolsAllowlist Oct 1, 2025
Copilot AI requested a review from pelikhan October 1, 2025 18:51
@pelikhan pelikhan marked this pull request as ready for review October 1, 2025 18:53
@pelikhan pelikhan merged commit 1b486ca into main Oct 1, 2025
@pelikhan pelikhan deleted the copilot/fix-57f24f55-296f-4ee1-a003-ff8d1ad73206 branch October 1, 2025 18:53
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.

2 participants