| ▲ | rkochanowski an hour ago | |
Recently there was a popular article on HN saying that sometimes code duplication is better than abstraction, so I assume that this question is not a joke. While testing this tool, one detected duplication was interesting for a use case. Permission check logic was duplicated and placed in different distant places in the codebase. The code was similar, but not identical, the logic was not the same. One version had stricter checks. I analyzed this with the coding agent, and we found out that both versions are used for the same thing, which means that in some cases validation is insufficient. Having only a single validation place, this bug could be prevented or easily detected. | ||