Skip to content

feat(setup): disable session URL in commit/PR attribution#13

Merged
rlorenzo merged 1 commit into
mainfrom
fix/attribution-session-url
Jul 3, 2026
Merged

feat(setup): disable session URL in commit/PR attribution#13
rlorenzo merged 1 commit into
mainfrom
fix/attribution-session-url

Conversation

@rlorenzo

@rlorenzo rlorenzo commented Jul 3, 2026

Copy link
Copy Markdown
Owner

Summary

Extends the Claude Code settings step in setup to also disable the session URL in attribution, and migrates existing configs that predate this key.

  • Apply now sets .attribution = {"commit":"", "pr":"", "sessionUrl": false} via a merge ((.attribution // {}) + {...}), so it preserves any existing attribution keys instead of replacing the object.
  • Detection offers the change when attribution is unset (fresh install, as before) or when a prior run left commit/pr blank without sessionUrl (adds just the new key).
  • Prompt description updated to "disable commit/PR/session attribution tags".

Stays opt-in behind the existing Apply these settings? prompt.

Behavior

Existing attribution Result
missing offers, sets {commit:"", pr:"", sessionUrl:false}
{commit:"", pr:""} (old setup) offers, adds sessionUrl:false
{commit:"", pr:"", sessionUrl:false} nothing to do
{commit:"My template", pr:""} custom commit respected, untouched
blank commit/pr + extra key migrated, extra key preserved
{... sessionUrl:true} user opted in, untouched

Test plan

  • bash -n setup passes
  • Pre-commit hooks pass (shellcheck, BATS)
  • Verified the detection and apply jq expressions against all six scenarios above

Add sessionUrl:false to the attribution setting so session links are
also omitted. Detection now offers the change both on fresh installs
and when a prior run left commit/pr blank without sessionUrl, adding
the key via a merge that preserves any existing attribution values.
@rlorenzo
rlorenzo merged commit ccab949 into main Jul 3, 2026
4 checks passed
@rlorenzo
rlorenzo deleted the fix/attribution-session-url branch July 3, 2026 02:43
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.

1 participant