Hacker News Board Hacker News, browsed like a classic message board
Live data from Hacker News

Claude Session URL appended to commit messages and PR descriptions by default

github.com

Post reply on HN
Print view View original

251–257 of 257 posts

Re: Claude Session URL appended to commit messages and PR descriptions by default

#251

Post by opwizardx »

joshstrange wrote: post #211
1718627440 wrote: post #210

Earlier quoted context omitted.

Then just copy the files into another directory, or maybe include it in a VCS?

Yep, that’s what I’m doing, a cron to copy them off to the side daily

Same worry got me to build pond for myself: it imports all of my sessions on a cron schedule every 5 mins into one store, and my agents can access it over MCP. Whenever I need a session from a while ago now, I just know it's in pond, because all of my sessions for about the last 1.5 years are collected in there. It also happens really often that when my agents cannot find something, they just call pond search themselves. Faster that way.

Codex, Pi, Hermes, Openclaw and bunch of others a getting pulled in automatically too. Repo: https://github.com/tenequm/pond

Re: Claude Session URL appended to commit messages and PR descriptions by default

#252

Post by ericyd »

oniony wrote: post #90
ericyd wrote: post #53

Earlier quoted context omitted.

I'm not sure i see it as an ad. "Sent from my iPhone" is an ad because the iPhone is not a meaningful part of sending the email. Claude is a meaningful part of the commit though if you're using it, so to me it's more like clarity or transparency than an advertisement.

My spade is a meaningful part of digging a hole when I plant a tree, but I still don't want a "Spear and Jackson" sign on my tree when I'm done.

If my tree kept a log of all the activities that led to it being a tree, I wouldn't mind if that log included "Spear and Jackson". To me the closer analogy with software would be if the final web app interface was stamped with "built with Claude".

Re: Claude Session URL appended to commit messages and PR descriptions by default

#253

Post by ex-aws-dude »

ketzu wrote: post #25

I like claude code, I use it quite a bit for things, and I don't mind the attribution (I do not plan to deceive people of me using AI or not for a change) and I think a link back to the session is valueable after the fact. At least for me, the session contains a lot of discussion, back and forth, changes from me and more. (Also, if the feature is not on by default, it probably might not exist.) However, I think claud…

> I do not plan to deceive people of me using AI or not for a change

You don't have to worry about that

Re: Claude Session URL appended to commit messages and PR descriptions by default

#254

Post by nicbou »

nonethewiser wrote: post #40
jlawrence6809 wrote: post #6

This is probably a controversial take, but I love this. Back in the early days of claude code I had to write a script (well, claude wrote it) to pull out the session id and append it to every commit. It's a lifesaver when you are trying to debug an old commit to pull up the chat session that actually wrote it.

It’s reasonable. I think its the beginning of a larger trend where Anthropic makes its infeasible to use Claude without Claude getting credit. Not for ethical reasons, but business reasons. Fingerprinting supports this as well. Im speculating. And these are little things that are reasonable by itself. They are ethical. But I dont think Anthropic wants Claude to ultimately be a tool that produces some output that just…

The irony of Claude wanting attribution when its work is being used by someone else

Re: Claude Session URL appended to commit messages and PR descriptions by default

#255

Post by othmanosx »

_zoltan_ wrote: post #247
othmanosx wrote: post #246

Earlier quoted context omitted.

Curious to know, do you read the code first or the summary? How much value do you think AI generated summaries are adding to the review process?

I read the summary which for me focuses on intent: What's the current state, why it's not good, what is being changed and how, and then look at performance numbers. Then I read the code but only after it got scrutinized by pr review skills on multiple models and those feedback got marked completed by the author.

Yeah, same here, but I do not pay that much attention to the PR description since it's all written by the same AI, it just narrates its own code. I prefer to skim it then jump to the code. The problem thesis is already in the ticket, so we could just reference it in the PR without duplication. The current state and how it's being changed are already the code diff itself under the changes tab, right? Reading it straight makes me question the code better since the AI is usually overconfident in its writing, so I don't focus on it unless I face a hard blocker or constraint that should be mentioned there.

I also do this with my swarm of review agents, asking them to review the code without context, that way, they produce more high-quality feedback that is backed by self-sourcing the context from the codebase instead of relying on the AI-provided PR description that might justify a code change that others might disagree with otherwise. it works well especially with code changes that could miss other parts of the codebase during refactoring or implementing features that touch multiple domains.

Re: Claude Session URL appended to commit messages and PR descriptions by default

#256

Post by gitaarik »

exe34 wrote: post #235
gitaarik wrote: post #232

Earlier quoted context omitted.

Isn't that implied when it says coauthored by Claude? Would be waste of chars to name everyone else

Where can I see the list of everybody they stole IP from? Just saying "that guy stole it for me" isn't really giving credit to the real owners/creators.

Ok, so what do you propose?

Re: Claude Session URL appended to commit messages and PR descriptions by default

#257

Post by tepmoc »

nijave wrote: post #248
tepmoc wrote: post #239

Earlier quoted context omitted.

I found both models produce word salad for PR bodies by default. Same thing with commits messages. I had to create hooks that invoke skills when pushing new commit and open PR body text.

I made a skill which is maybe why GPT is doing okay but Claude seems to straight up ignore it (it invokes/reads it but doesn't follow the directions)

I found using just skill isnt enogh. You have to bounce its turn and inject skill text to actually follow instructions 100%.

Because even if you use PreToolUse hook and don't bounce it, it will never work since skill text is injected when along with text that already perpared to it need to be edited.

Post reply on HN