This is a stupid example to illustrate what I mean. Say you have this code:
def create_background(width: int, height: int) -> Image:
...
You tell the agent to use default values for create_background, the same as in create_screen. It changes the code to:
# Now create_background params have default values, the same as create_screen in screen.py
def create_background(width: int = DEFAULT_WIDTH, height: int = DEFAULT_HEIGHT) -> Image:
...
The unnecessary comment is a staple of vibed code, but the tone also annoys me because it leaves behind the prompt. It words comments based on what it was asked to do, not in a timeless manner.
I keep telling people in code reviews to remove unnecessary comments, and I feel I lack the vocabulary to express why this is bad.
- The user tells the agent to add feature X
- The agent adds the feature
- User: "Yeah that's good but I don't think we need Y that you added, get rid of it"
- The agent removes Y, adds comments that Y was removed, adds backwards-compatibility for saves which used Y (obviously there aren't any, since Y only existed for 5 minutes and was never deployed - the project itself might never have been deployed either), adds a test for Y's non-existence, adds text to the UI saying "Note: Y has been removed. There are no traces of Y in the project"
I think that GPT-5.5 does it the most.
reply