Live data from Hacker News

Copilot edited an ad into my PR

notes.zachmanson.com

511–520 of 704 posts

Re: Copilot edited an ad into my PR

#511

Earlier quoted context omitted.

Yeah but at least a dozen Microsoft employees went on a seemingly scripted blitz on X about how they’re ready to start listening to feedback and… * checks notes * Only have copilot shoehorned into most things instead of everything. And some shit about windows developers which isn’t exactly going to fix the glaring issues with the OS itself.

> Yeah but at least a dozen Microsoft employees went on a seemingly scripted blitz on X about how they’re ready to start listening to feedback and… So what was the purpose of all that telemetry they collected then? Because it doesn't seem to have made the OS like what the users want it to be.

Data Gnomes

1) collect data

2) ???

3) profit

Re: Copilot edited an ad into my PR

#514
post #361

I asked copilot how developers would react if AI agents put ads in their PRs. >Developers would react extremely negatively. This would be seen as 1. A massive breach of trust. 2. Unprofessional and disruptive. 3. A security/integrity concern. 4. Career-ending for the product. The backlash would likely be swift and severe. Sometimes AI can be right.

It’s not really ads, it’s more like "Sent from my iPhone"-style sentences at the end of PR texts.

And everyone thought they were cool! Mac zeolots still put "Made with a Mac" on their webpages.

Re: Copilot edited an ad into my PR

#515
post #393

Earlier quoted context omitted.

I don't quite see the benefit of this, personally. Whoever is submitting the code is still responsible for it, why would the reviewer care if you wrote it with your fingers or if an LLM wrote (parts of) it? The quality+understanding bar shouldn't change just because "oh idk claude wrote this part". You don't get extra leeway just because you saved your own time writing the code - that fact doesn't benefit me/the proj…

You're quite right that the quality of the code is all that matters in a PR. My point is more historical. AI is a very new tool, and as such the quality of the code it produces depends both on the quality of the tool, and how you've wielded it. I want to be able to track how well I've been using the tool, to see what techniques produce better results, to see if I'm getting better. There's a lot more to AI coding than…

Just curious, what metrics would you use to track how well your results are?

Re: Copilot edited an ad into my PR

#516
post #415

Earlier quoted context omitted.

[flagged]

Please don't attack people for showing up to engage in discussion like this. I'm sure you don't intend to, but it quickly becomes part of mob behavior. We don't want that on HN for obvious reasons, and I'm sure nobody intends it, exactly, but it happens all too easily anyhow. https://news.ycombinator.com/newsguidelines.html

Absolutely my bad it came across like this.

Re: Copilot edited an ad into my PR

#517

Earlier quoted context omitted.

In my project's readme I put this text: "There is no commit by an agent user, for two reasons: * If an agent commits locally during development, the code is reviewed and often thoroughly modified and rearranged by a human. * I don't want to push unreviewed code to the repo, so I have set up a git hook refusing to push commits done by an LLM agent." It's not that I want to hide the use of llms, I just modified code a…

> * I don't want to push unreviewed code to the repo, so I have set up a git hook refusing to push commits done by an LLM agent." Seems... Not that useful? Why would someone make commits in your local projects without you knowing about it? That git hook only works on your own machine, so you're trying to prevent yourself from pushing code you haven't reviewed, but the only way that can happen is if you use an agent l…

The agents run in a container and have an other git identity configured. It happens that agents commit code and I don't want to push it accidentally from outside the container, which is where I work.

Re: Copilot edited an ad into my PR

#518

Earlier quoted context omitted.

It's not a self-own, it's honest disclosure. It's unethical (if not outright fraudulent) to publish LLM work as if it were your own. Claude setting itself as coauthor is a good way to address this problem, and it doing so by default is a very good thing.

I think it depends a lot if you reviewed it as carefully as you would your own code. Of course most people don’t do that

Not just review but how you worked with the AI.

If you gave it four words and waited and hour maybe you're not the author. But that's not how these tools are best used anyway.

Re: Copilot edited an ad into my PR

#519

Tim from the Copilot coding agent team here. We've now disabled these tips in pull requests created by or touched by Copilot, so you won't see this happen again for future PRs. We've been including product tips in PRs created by Copilot coding agent. The goal was to help developers learn new ways to use the agent in their workflow. But hearing the feedback here, and on reflection, this was the wrong judgement call. W…

> We won't do something like this again.

This has just as much value as when an LLM claims it won't make a certain mistake again, and for exactly the same reason.

Re: Copilot edited an ad into my PR

#520
post #361

I asked copilot how developers would react if AI agents put ads in their PRs. >Developers would react extremely negatively. This would be seen as 1. A massive breach of trust. 2. Unprofessional and disruptive. 3. A security/integrity concern. 4. Career-ending for the product. The backlash would likely be swift and severe. Sometimes AI can be right.

It’s not really ads, it’s more like "Sent from my iPhone"-style sentences at the end of PR texts.

the difference is "sent from my iPhone" is on YOUR outgoing email. you opted into that default. this is copilot editing someone else's PR description with promotional text for third party tools. that's not a signature, that's injection. imagine if gcc started appending "compiled with gcc, try our new optimization flags" to your README every time you built a project.
Post reply on HN