Skip to content

feat(code): add commit and pr attribution to system prompt#1432

Merged
adboio merged 1 commit intomainfrom
04-01-feat_code_add_commit_and_pr_attribution_to_system_prompt
Apr 3, 2026
Merged

feat(code): add commit and pr attribution to system prompt#1432
adboio merged 1 commit intomainfrom
04-01-feat_code_add_commit_and_pr_attribution_to_system_prompt

Conversation

@adboio
Copy link
Copy Markdown
Contributor

@adboio adboio commented Apr 1, 2026

Problem

we explicitly instruct the (cloud) agent to not include any attribution lines -- presumably to avoid the default claude code stuff

however, this means there's also no attribution back to posthog code

Changes

  • cloud: replaces the "no attribution" prompting with instructions to add commit trailers (see feat(code): add commit trailers for attribution #1431 ) and PR body footers
  • local: adds new commit+PR attribution instructions, which hopefully will override whatever is injected by claude code

How did you test this?

i haven't yet 🙈

@adboio adboio requested a review from a team April 1, 2026 21:57
@adboio adboio marked this pull request as ready for review April 1, 2026 21:57
@adboio adboio force-pushed the 04-01-feat_code_add_commit_trailers_for_attribution branch from eda1ca1 to 28fc5ee Compare April 2, 2026 14:59
@adboio adboio force-pushed the 04-01-feat_code_add_commit_and_pr_attribution_to_system_prompt branch from dcf3ef6 to 2b80f9a Compare April 2, 2026 15:00
Copy link
Copy Markdown
Member

@charlesvien charlesvien left a comment

Choose a reason for hiding this comment

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

LGTM

@adboio adboio force-pushed the 04-01-feat_code_add_commit_trailers_for_attribution branch from 28fc5ee to 4bbd9d3 Compare April 3, 2026 00:15
@adboio adboio force-pushed the 04-01-feat_code_add_commit_and_pr_attribution_to_system_prompt branch from 2b80f9a to 2326d8b Compare April 3, 2026 00:15
@adboio adboio force-pushed the 04-01-feat_code_add_commit_trailers_for_attribution branch 2 times, most recently from 3dd25b5 to ccbbd0c Compare April 3, 2026 00:24
@adboio adboio force-pushed the 04-01-feat_code_add_commit_and_pr_attribution_to_system_prompt branch from 2326d8b to 2b1c59e Compare April 3, 2026 00:24
@adboio adboio changed the base branch from 04-01-feat_code_add_commit_trailers_for_attribution to graphite-base/1432 April 3, 2026 00:31
@adboio adboio force-pushed the 04-01-feat_code_add_commit_and_pr_attribution_to_system_prompt branch from 2b1c59e to 39a11cb Compare April 3, 2026 00:31
@adboio adboio force-pushed the graphite-base/1432 branch from ccbbd0c to 0b92782 Compare April 3, 2026 00:31
@graphite-app graphite-app bot changed the base branch from graphite-base/1432 to main April 3, 2026 00:32
@adboio adboio force-pushed the 04-01-feat_code_add_commit_and_pr_attribution_to_system_prompt branch from 39a11cb to 99e6912 Compare April 3, 2026 00:32
@adboio adboio merged commit 18ce420 into main Apr 3, 2026
16 checks passed
Copy link
Copy Markdown
Contributor Author

adboio commented Apr 3, 2026

Merge activity

@adboio adboio deleted the 04-01-feat_code_add_commit_and_pr_attribution_to_system_prompt branch April 3, 2026 00:37
adboio added a commit that referenced this pull request Apr 3, 2026
## Problem

#1431 adds commit trailers when committing via UI, #1432 adds trailers + PR body instructions for agent

now we need to add PR body footer when creating a PR via UI

<!-- Who is this for and what problem does it solve? -->

<!-- Closes #ISSUE_ID -->

## Changes

appends "created by posthog code" to PR body when creating from UI

<!-- What did you change and why? -->

<!-- If there are frontend changes, include screenshots. -->

## How did you test this?

<!-- Describe what you tested -- manual steps, automated tests, or both. -->

<!-- If you're an agent, only list tests you actually ran. -->

PostHog/posthog#53123
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