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

AI PR adds auto generated comments to whole Spring Boot Project

github.com

Post reply on HN
Print view View original

11–20 of 48 posts

Re: AI PR adds auto generated comments to whole Spring Boot Project

#11

Post by ben_w »

lelandfe wrote: post #7

Given that all the comments I'm seeing are of the “‘makeFooReady’ makes foo ready” ilk, I wonder how these tools fare on incorrectly-named things. Would it pick up on ‘asyncRemoveFoo’ being neither async nor a function that removes foo?

Reminds me of something I learned of over a decade ago:

   isUserAMonkey

   Added in API level 8

   public static boolean isUserAMonkey ()

   Returns "true" if the user interface is currently being messed with by a monkey.
- https://developer.android.com/reference/android/app/Activity...

Re: AI PR adds auto generated comments to whole Spring Boot Project

#13

Post by no_wizard »

ben_w wrote: post #11
lelandfe wrote: post #7

Given that all the comments I'm seeing are of the “‘makeFooReady’ makes foo ready” ilk, I wonder how these tools fare on incorrectly-named things. Would it pick up on ‘asyncRemoveFoo’ being neither async nor a function that removes foo?

Reminds me of something I learned of over a decade ago: isUserAMonkey Added in API level 8 public static boolean isUserAMonkey () Returns "true" if the user interface is currently being messed with by a monkey. - https://developer.android.com/reference/android/app/Activity...

I wonder why this exists.

What is this code for. How would they even know its a monkey?!

I'm sure its a joke of sorts, or clever moniker if you want to be charitable, but unclear to me in any case

Re: AI PR adds auto generated comments to whole Spring Boot Project

#14

Post by dartos »

no_wizard wrote: post #4
mateusfreira wrote: post #2

I struggle to see how any open-source project would accept this kind os PRs, the tools seems ok for research, but not a good ideia to push PRs like that open source projects.

I have to say, the maintainers handled it with grace though. Didn't sass the company for it or anything, politely declined the PR, closed it, locked it and moved on. It reflects back on them really well. I've never used Spring Boot, but I have to give them credit for this. I don't know that I'd have similar restraint

Spring boot is one of the most enterprise-y frameworks I’ve used.

I’d be surprised if their team wasn’t incredibly professional.

Re: AI PR adds auto generated comments to whole Spring Boot Project

#15

Post by spiderxxxx »

shadowgovt wrote: post #8

To be honest, I see two positives regarding what Codemaker has provided here. One is that (glancing through the comments) they're actually significantly more thorough than what is already there and they're at the level of thoroughness that a new user may actually want. Phil Webb is quite right; this is the level of detail someone new to the codebase could use, and if you could generate it dynamically on the fly as a…

> /* > * Determines the GitHub tag for the given project. > * @param project the project for which to determine the GitHub tag > * @return the GitHub tag for the project > */ > private String determineGitHubTag(Project project) {

Here's a sample of the crap you get, for 3000+ files, multiple comments per file, all this inane, that anyone with two brain cells could figure out just by reading the name of the function. If the project didn't already have these sort of comments, then adding them now is a choice that the maintainer of standards would have to make, not some random contributor.

Re: AI PR adds auto generated comments to whole Spring Boot Project

#16

Post by Tommah »

timetraveller26 wrote: post #12

As said in the PR comments, this is Codemaker AI advertisement, no good will. And funnily enough, bad advertisement. How could a startup that focuses on code think that making a PR with 3,158 changed files is acceptable?

Maybe they just wanted 3,158 Hacktoberfest T-shirts.

Re: AI PR adds auto generated comments to whole Spring Boot Project

#18

Post by [deleted] »

jlund-molfese wrote: post #9
lelandfe wrote: post #7

Given that all the comments I'm seeing are of the “‘makeFooReady’ makes foo ready” ilk, I wonder how these tools fare on incorrectly-named things. Would it pick up on ‘asyncRemoveFoo’ being neither async nor a function that removes foo?

Auto-generated comments ("foo takes bar and returns baz") have been a thing in Java IDEs for a very long time...but most people just consider them to be clutter and don't use them. A product that scans your repo and identifies mis-named functions would be compelling, though.

[deleted]

Re: AI PR adds auto generated comments to whole Spring Boot Project

#19

Post by r3trohack3r »

no_wizard wrote: post #13
ben_w wrote: post #11

Earlier quoted context omitted.

Reminds me of something I learned of over a decade ago: isUserAMonkey Added in API level 8 public static boolean isUserAMonkey () Returns "true" if the user interface is currently being messed with by a monkey. - https://developer.android.com/reference/android/app/Activity...

I wonder why this exists. What is this code for. How would they even know its a monkey?! I'm sure its a joke of sorts, or clever moniker if you want to be charitable, but unclear to me in any case

A quick Kagi search revealed that this checks if the user is a simulated user in for stress tests:

https://developer.android.com/studio/test/other-testing-tool...

My guess is that the name shares its etymology with other Monkey references in tech, like Chaos Monkey. Where a being that has no understanding of the purpose of anything in the environment is given access to the environment and allowed to wreak havoc as a form of resilience testing.

This monkey appears to generate system events and user input events in a pseudo-random (deterministic) way to help with fuzzy-like testing of an app.

Re: AI PR adds auto generated comments to whole Spring Boot Project

#20

Post by timeon »

timetraveller26 wrote: post #12

As said in the PR comments, this is Codemaker AI advertisement, no good will. And funnily enough, bad advertisement. How could a startup that focuses on code think that making a PR with 3,158 changed files is acceptable?

I wonder if being code maintainer is going to suck with all the generated PRs.
Post reply on HN