Skip to content
This repository was archived by the owner on May 20, 2026. It is now read-only.

Disable Claude co-attribution in commits/PRs#4788

Merged
TylerLeonhardt merged 6 commits into
mainfrom
copilot/turn-off-claude-coattribution
Mar 30, 2026
Merged

Disable Claude co-attribution in commits/PRs#4788
TylerLeonhardt merged 6 commits into
mainfrom
copilot/turn-off-claude-coattribution

Conversation

Copilot AI commented Mar 27, 2026

Copy link
Copy Markdown
Contributor

When using Claude in Copilot, we shouldn't be attributing PRs to Claude.

The Claude Agent SDK supports disabling the "Co-authored-by: Claude" attribution added to git commits and pull requests, and this PR uses the attribution settings API with empty strings for commit and pr to suppress this attribution on Claude agent-authored code.

https://code.claude.com/docs/en/settings#attribution-settings


🔒 GitHub Advanced Security automatically protects Copilot coding agent pull requests. You can protect all pull requests by enabling Advanced Security for your repositories. Learn more about Advanced Security.

Copilot AI changed the title [WIP] Update Claude integration to disable co-attribution Add setting to control Claude co-attribution in commits/PRs Mar 27, 2026
Copilot AI requested a review from pierceboggan March 27, 2026 22:49
@pierceboggan

Copy link
Copy Markdown
Contributor

@copilot I don't want you to add a new setting, I want you to just disable it in our agent SDK configuration for Claude.

Copilot AI commented Mar 27, 2026

Copy link
Copy Markdown
Contributor Author

@copilot I don't want you to add a new setting, I want you to just disable it in our agent SDK configuration for Claude.

Done — removed the setting and hardcoded includeCoAuthoredBy: false in the SDK settings object in claudeCodeAgent.ts. Reverted all changes to package.json, package.nls.json, and configurationService.ts. See 54432ac.

Copilot AI changed the title Add setting to control Claude co-attribution in commits/PRs Disable Claude co-attribution in commits/PRs Mar 27, 2026
@TylerLeonhardt

Copy link
Copy Markdown
Member

Copilot AI commented Mar 28, 2026

Copy link
Copy Markdown
Contributor Author

@copilot bro read the docs https://code.claude.com/docs/en/settings#attribution-settings

Switched to the attribution API with empty strings for commit and pr instead of the deprecated includeCoAuthoredBy. See eaad1d8.

Warning

Firewall rules blocked me from connecting to one or more addresses (expand for details)

I tried to connect to the following addresses, but was blocked by firewall rules:

  • code.claude.com
    • Triggering command: /home/REDACTED/work/_temp/ghcca-node/node/bin/node /home/REDACTED/work/_temp/ghcca-node/node/bin/node --enable-source-maps /home/REDACTED/work/_temp/copilot-developer-action-main/dist/index.js (dns block)
  • docs.claude.com
    • Triggering command: /home/REDACTED/work/_temp/ghcca-node/node/bin/node /home/REDACTED/work/_temp/ghcca-node/node/bin/node --enable-source-maps /home/REDACTED/work/_temp/copilot-developer-action-main/dist/index.js (dns block)

If you need me to access, download, or install something from one of these locations, you can either:

@TylerLeonhardt TylerLeonhardt marked this pull request as ready for review March 28, 2026 01:51
Copilot AI review requested due to automatic review settings March 28, 2026 01:51

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR updates the Claude Code session configuration in the Copilot Chat extension to suppress the Claude Agent SDK’s default “Co-authored-by: Claude” attribution on commits and pull requests.

Changes:

  • Sets Claude Agent SDK settings.attribution for sessions to disable commit attribution.
  • Sets Claude Agent SDK settings.attribution for sessions to disable PR attribution.

Comment thread src/extension/chatSessions/claude/node/claudeCodeAgent.ts
@TylerLeonhardt TylerLeonhardt added this pull request to the merge queue Mar 30, 2026
Merged via the queue into main with commit dcffd95 Mar 30, 2026
20 checks passed
@TylerLeonhardt TylerLeonhardt deleted the copilot/turn-off-claude-coattribution branch March 30, 2026 20:28
@Frulfump

Copy link
Copy Markdown

The PR description doesn't mention why this is being done (and no reference to an issue). Isn't this removing transparency around AI usage?

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants