Remix.run Logo
dmitriv 3 hours ago

I am the person who approved this PR and would like to acknowledge and apologize for the mistake of turning this feature on by default without sufficient upfront validation.

There was no ill intent by evil corporation, but rather a desire to support functionality that some customers expect of VS Code w.r.t. AI-generated code. As folks mentioned here - many similar tools do this as well.

Obviously, it should not be on when disableAIFeatures is on and it should not be reporting changes that were not done by AI. I'll work on fixing those and meanwhile revert default to off in 1.119 update.

I am open to any (constructive) comments/suggestions - please feel free to reach me directly (my alias @microsoft.com) or open an issue on GitHub. Happy to answer anything here as well.

alemanek an hour ago | parent | next [-]

Changing the default behavior for all of your users with no notification is pretty unforgivable. Even if this feature worked correctly, it obviously doesn’t, this should at minimum be a prompt after upgrade to let the user confirm that this is what they want. But honestly should be opt in for those that want it.

To have it silently just start adding marketing copy to git commit messages is pretty bad. To have that added text not be visible to the user in the UI so they can remove it before commit is just much worse.

This kind of thing being released speaks to a greater disfunction over there. Not a good look at all and I am not a Microsoft or AI hater. But my commit messages are not where you move fast and break things

Aperocky 37 minutes ago | parent | next [-]

Well, the good news is commit messages are some of the most visible thing, and there are no silent modifications that are really possible.

The bad news is - where else have this happened in VS Code?

- A happy user of (n)vim

dakolli 43 minutes ago | parent | prev [-]

Then how are they going to be able to sue and take ownership stake in projects that have their clanker's authorship in it when they gain traction, or its advantageous for them to do so?

somebehemoth 2 hours ago | parent | prev | next [-]

I think the constructive criticism is best directed at whatever process you are following. That process allowed a very visible user facing change in a widely used piece of software. How did this change make it to production without some process catching the impact of this change? Was there really no internal discussion from a code review at least? This seems hard for me to believe. I expect more from Microsoft.

lightdot an hour ago | parent | prev | next [-]

> There was no ill intent by evil corporation, but rather a desire to support functionality that some customers expect of VS Code w.r.t. AI-generated code.

What metric did Microsoft use to assess that VS Code users "expect" their commits to have unsolicited messages added to them?

> Obviously, it should not be on when disableAIFeatures is on and it should not be reporting changes that were not done by AI.

Did you discuss adding these messages with your legal department?

What is Microsoft's position on adding such authorship statements to the code Microsoft did not author?

Or is Microsoft stating that using LLM assistants makes Microsoft a co-author of the code?

Does Microsoft have copyright claims on the code if LLM assistants are used at any time during it's creation?

jamesbfb 2 hours ago | parent | prev | next [-]

I think there’s a few of us who appreciate you being up front. I’d question the intent and why it was a mistake, especially when the commit[0] message reverting said functionality states “widespread criticism” citing this very HN article makes it look seemingly like the revert is due to negative PR opposed to a mistake.

[0]https://github.com/microsoft/vscode/pull/313725/commits/1e70...

l2dy an hour ago | parent [-]

Author of that PR doesn't seem to be a Microsoft employee. Keep in mind that anyone on GitHub can create PRs against VSCode.

beardbandit an hour ago | parent | next [-]

Author is absolutely from Microsoft

https://github.com/microsoft/vscode/pull/310226

https://github.com/cwebster-99

kllrnohj 30 minutes ago | parent [-]

The linked revert PR is not from Microsoft (and also isn't merged)

lightdot an hour ago | parent | prev [-]

Even if that would be so, the person who approved it certainly is.

anvuong 8 minutes ago | parent | prev | next [-]

Don't you understand that the default shouldn't be changed at all in this case? It improves nothing and affects every single user. If an org/project wants this behavior then it can enforce this flag for its contributions. The only valid reason for this change is someone's performance somewhere in Microsoft is dependent on VS Copilot usage metric.

kgeist an hour ago | parent | prev | next [-]

Interesting case:

- a project manager vibe-coded the change without thinking it through at all

- the PR was reviewed by an LLM

- an actual engineer gave LGTM without really reviewing the changes, trusting the LLM

Did I get this right?

schwede 2 hours ago | parent | prev | next [-]

Why does the commit editor hide the coauthored message? Why not pre-populate the text field and users take or leave it when committing?

jdlshore 21 minutes ago | parent [-]

Co-Authored-By is normally a trailer, and trailers aren’t part of the commit message. It’s likely the commit editor isn’t set up to show trailers. They’re not exactly obscure, but it does seem that they’re relatively unknown.

jbxntuehineoh 17 minutes ago | parent | prev | next [-]

thank you for doing this, it gave me the push I needed to finally switch to zed. vscode has really been going downhill for a while now. it's sad to watch, it used to be a really nice editor

mellosouls an hour ago | parent | prev | next [-]

Thanks for facing this head-on here; mistakes happen.

I think the default to on should also be reconsidered regardless. The assessment (co-authored by AI) may be valid but the assumption the user wants that advertising is exactly that, an assumption, and a dubious one at that.

nhinck2 an hour ago | parent | prev | next [-]

> a desire to support functionality that some customers expect of VS Code w.r.t. AI-generated code

Literally who?

teunispeters an hour ago | parent | prev | next [-]

Have it as an add-on said customers can add. Opt-in, not opt-out. No AI without consent.

2 hours ago | parent | prev | next [-]
[deleted]
aaaronic an hour ago | parent | prev | next [-]

Brave man. RIP your inbox.

an hour ago | parent [-]
[deleted]
p-e-w 2 hours ago | parent | prev | next [-]

I appreciate you acknowledging that this was a mistake, but as you surely know from your own experience with other people’s mistakes, some mistakes are so egregious that they cast doubt on the intentions of the people involved even if they are corrected later.

To me, “let’s add false attribution to every commit by default without informing the user” falls squarely into that category. I don’t think I’ve ever worked in an environment where something like that wouldn’t have been red-flagged in three seconds by anyone who took even a casual glance. I’d honestly be embarrassed if such a proposal even made it into a public pull request for my organization, nevermind that pull request getting merged.

jibal an hour ago | parent | prev [-]

[dead]