Skip to content

MPT-25923 disable Claude Code attribution in commits and PRs - #433

Merged
svazquezco merged 1 commit into
mainfrom
feature/MPT-25923/disable-claude-attribution
Sep 29, 2026
Merged

svazquezco merged 1 commit into
mainfrom
feature/MPT-25923/disable-claude-attribution

Conversation

@svazquezco

@svazquezco svazquezco commented Sep 29, 2026 •

Copy link
Copy Markdown
Collaborator

🤖 AI-generated PR — Please review carefully.

What

  • Add a shared .claude/settings.json with the Claude Code attribution settings (empty commit and pr, sessionUrl: false), so commits and PRs created with Claude Code no longer include the Claude co-author trailer or attribution text.
  • Update .gitignore to track only .claude/settings.json; .claude/settings.local.json (personal settings) and the rest of .claude/ stay ignored.
  • Document where the shared and personal Claude Code settings live in docs/contributing.md.

Same change as softwareone-platform/mpt-extension-sdk#307.

References

Jira: MPT-25923

Closes MPT-25923

  • Disable Claude Code commit and PR attribution text and session URLs in shared .claude/settings.json.
  • Track .claude/settings.json while keeping personal settings and other .claude/ files ignored.
  • Document shared and personal Claude Code settings in docs/contributing.md.

Add a shared project .claude/settings.json that clears the Claude Code
commit and PR attribution and disables the session URL, so commits and
pull requests created with Claude Code carry no co-author trailer.

Track only .claude/settings.json and keep .claude/settings.local.json
and the rest of .claude/ ignored for personal settings and worktrees.
Document the policy in docs/contributing.md.
@svazquezco
svazquezco requested a review from a team as a code owner September 29, 2026 11:46
@github-actions

github-actions Bot commented Sep 29, 2026 •

Copy link
Copy Markdown

✅ Found Jira issue key in the title: MPT-25923

Generated by 🚫 dangerJS against 8151133

@sonarqubecloud

Copy link
Copy Markdown

@coderabbitai

coderabbitai Bot commented Sep 29, 2026 •

Copy link
Copy Markdown

Review in Change Stack →

Navigate logical layers of code changes, visualize relationships, and explore their blast radius.

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Repository YAML (base), Organization UI (inherited)

Review profile: CHILL

Plan: Essentials

Run ID: 06b89d53-8618-4c1e-9a26-5512c4db2899

📥 Commits

Reviewing files that changed from the base of the PR and between 769bcb2 and 8151133.

⛔ Files ignored due to path filters (1)
  • .gitignore is excluded by !.gitignore
📒 Files selected for processing (2)
  • .claude/settings.json
  • docs/contributing.md
🔗 Linked repositories identified

CodeRabbit considers these linked repositories for cross-repo context during reviews:

Included review availability: This review used your included allowance. 0 included reviews remain after this review. Your included PR review attempts over the past 7 days set your current allowance at 5 reviews per hour.

📜 Recent review details
🧰 Additional context used
📓 Path-based instructions (4)
Review documentation changes against `docs/documentation.md` and the linked repository's `standards/documentation.md`.

⚙️ CodeRabbit configuration file

Files:

  • docs/contributing.md
For each subsequent commit in this PR, explicitly verify if previous review comments have been resolved

⚙️ CodeRabbit configuration file

Files:

  • docs/contributing.md
Topic-specific documentation must live in the matching file under [`docs/`](.).

📄 CodeRabbit inference engine (docs/documentation.md)

Files:

  • docs/contributing.md
When applicable, read the repository documentation in this order: `docs/contributing.md` — repository-specific workflow and links to shared standards Put topic-specific documentation under `docs/` instead of expanding `README.md`.

📄 CodeRabbit inference engine (AGENTS.md)

Files:

  • docs/contributing.md
🔀 Multi-repo context softwareone-platform/mpt-extension-skills

Linked repositories findings

softwareone-platform/mpt-extension-sdk

  • Open PR #307 implements the same policy: .claude/settings.json sets empty commit/pr attribution and sessionUrl: false; .gitignore tracks only that file; docs/contributing.md documents personal settings. [::softwareone-platform/mpt-extension-sdk::]

softwareone-platform/mpt-extension-skills

  • No tracked Claude settings or attribution configuration exists. Its .gitignore contains no .claude rules, and docs/contributing.md has no Claude settings guidance. [::softwareone-platform/mpt-extension-skills::]
  • Its documentation convention places repository-specific contribution guidance in docs/, consistent with adding this policy to docs/contributing.md. [::softwareone-platform/mpt-extension-skills::]
🔇 Additional comments (2)
.claude/settings.json (1)

1-7: LGTM!

docs/contributing.md (1)

28-38: LGTM!


📝 Walkthrough

Walkthrough

The change adds empty commit and PR attribution values and disables session URL attribution in the tracked Claude Code settings. The contributor guide explains which settings are shared and where to keep personal settings.

Changes

Claude Code settings

Layer / File(s) Summary
Shared settings and guidance
.claude/settings.json, docs/contributing.md
The settings file adds attribution values. The contributor guide documents the tracked shared settings file, personal settings locations, and reference links.

Priority: ⬇️ Low

Estimated code review effort: 2 (Simple) | ~8 minutes

Merge Risk: ⚪ Minimal · up to 81511

The shared settings disable Claude Code attribution as intended, while personal settings remain untracked. No actionable merge risk remains.

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
Documentation Up To Date ✅ Passed The PR changes Claude Code developer workflow settings, not application deployment configuration. It updates docs/contributing.md with the shared settings file, ignored personal settings, and releva…

Comment @coderabbitai help to get the list of available commands.

@svazquezco
svazquezco merged commit 334b2d7 into main Sep 29, 2026
8 checks passed
@svazquezco
svazquezco deleted the feature/MPT-25923/disable-claude-attribution branch September 29, 2026 15:00
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