https://scanaislop.com/postgres/postgres ( @ 5ab239c )
- Errors: 0 ; Warnings: 6 --> "56 / 100 Failing" ???
61–70 of 70 posts
https://scanaislop.com/postgres/postgres ( @ 5ab239c )
- Errors: 0 ; Warnings: 6 --> "56 / 100 Failing" ???
Nice job. Somewhat related, but one "AI smell" (if you can call it that) is LLMs' desire to add layers of redundant safeguards with things like null coalescing operators and defensive fallbacks so that there’s always some default value. LLMs often seem to struggle with the distinction between happy/unhappy paths and often end up treating everything as optional and "safe". I’m a much bigger fan of surfacing problems l…
Nice job. Somewhat related, but one "AI smell" (if you can call it that) is LLMs' desire to add layers of redundant safeguards with things like null coalescing operators and defensive fallbacks so that there’s always some default value. LLMs often seem to struggle with the distinction between happy/unhappy paths and often end up treating everything as optional and "safe". I’m a much bigger fan of surfacing problems l…
Happy to share your AGENTS.md?
https://gist.github.com/scpedicini/1407c776b82945b723199a91a...
Why is the Postgres status: "56 / 100 Failing" ??? https://scanaislop.com/postgres/postgres ( @ 5ab239c ) - Errors: 0 ; Warnings: 6 --> "56 / 100 Failing" ???
I've tried it on my repository; for go if function is in format `func(...) (T, bool)` and you use it as `object, _ := func(...)` it reports as `Error return value is being ignored` even though it's clearly not an error being ignored (and ignoring the bool ok is a common pattern in go).
I’m eager to test this out. I have agent instructions to try to limit the worst of this already, but patterns still sneak through. I have a review agent run after every single edit looking for all of the following if you need more ideas for checks: - DRY principle violations, multiple definitions of the same helpers or utilities. - Changes that deviate from existing patterns and architecture already in the code, espe…
Would be nice to have a way to tell the scan that some parts are intentional, like #[allow(...)] in rust