3 ms·
Vibe-coded. Sorry. https://github.com/514-labs/dnsglobe/blob/c29802162636832e88b56bbb205e6c773c254e5b/src/dns.rs#L69 https://github.com/514-labs/dnsglobe/blob/
by OptionOfT 3mo ago
Vibe-coded. Sorry.
https://github.com/514-labs/dnsglobe/blob/c29802162636832e88b56bbb205e6c773c254e5b/src/dns.rs#L69 https://github.com/514-labs/dnsglobe/blob/c29802162636832e88...
You take the `other`, do a `to_string()` on it, which creates a String representation. Then you pass a reference to that String, and, in the case it doesn't contain `time out` or `timeout` or `refused`, the reference gets turned AGAIN into a String (i.e. new allocation), truncated to 48, and then returned.
There is no check whether that the character at the 48th byte is a character boundary.
Add to that the fact that this is a Rust project with the oldest commit created yesterday and it is using the 2021 edition.
Be better.
- Callicles 3mo agoThis was 100% vibe-coded with Claude Code and Fable. https://x.com/thatsFrScience/status/2073741209592295866 https://x.com/thatsFrScience/status/2073741209592295866 Thanks for the feedback, though, and for taking the time to look at the code. I can ship a round of cleanup.
- orangeboats 3mo agoIt's gotten to the point that the moment I see "Rust" and "TUI" together, I immediately assume it's vibe coded. The combination just seems to be vibe coders' favorite, for some reason.
- zx8080 3mo ago> The combination just seems to be vibe coders' favorite, for some reason. It's the secret key to HN front page!
- blacklite 3mo agoThat's not whose favorite it is; it's the favorite of who's doing the work. :)
- inigyou 3mo agoIt must be because if you program in Rust, you don't have bugs ;)
- rvz 3mo agoQuite mixed on this one given that the author has experience with Rust before coding agents and this is just his toy project. There is going to be a time where these vibe coded projects have silent bugs, vulnerabilities or unnecessary performance issues and the AI coding agent just lies to the user that it has none. The AI agent will be the one to introduce new issues in the codebase regardless of "tests". The new issue is now the non-technical human vibe-coding is none the wiser. We have already seen this in Codex itself. Imagine this propagated in many other code-bases.
- rickydroll 3mo agoHaving created and debugged human-written code with silent bugs, vulnerabilities, or unnecessary performance issues since my first day on the job, and pointing you to myriad companies with bugs and vulnerabilities that existed pre-AI, all I can say is "plus ça change, plus c'est la même chose."
- seamossfet 3mo agoNothing wrong with vibecoding a toy project.
- teddyh 3mo agoThere is if you present it to others without telling anyone it’s slop.
- bcjdjsndon 3mo agoFor some vague reason you can't really articulate, that's a bad thing?
- Callicles 3mo agoAddressed the feedback
- egorfine 3mo agoThe very first thing written about this tool is the programming language it's written in. Like, when I'm querying a bunch of DNS servers, it is crucial for me to know which compilable language it was written in. Like, the most important thing.