‹ BackHN Continuity

Thread

ImpactGate: A merge gate that scores the structural decay AI adds

37 points · 48 comments · sagenschneider

  1. MichaelNolan · · focus · HN ↗
    Maybe I missed it, but it look like this has just a single metric. Maybe instead of making a new project, you could try to get this metric added to a existing tool like <a href="https:&#x2F;&#x2F;dekobon.github.io&#x2F;big-code-analysis&#x2F;index.html" rel="nofollow">https:&#x2F;&#x2F;dekobon.github.io&#x2F;big-code-analysis&#x2F;index.html which already has dozens of metrics.
    1. sagenschneider · · focus · HN ↗
      Yes, I&#x27;m doing my own research on AI augmented pipelines <a href="https:&#x2F;&#x2F;blog.officefloor.net" rel="nofollow">https:&#x2F;&#x2F;blog.officefloor.net . I actually found most code quality tools look for bugs and complexity, but nothing much about cohesive erosion. The nice thing about this metric, is that it determine the files where the erosion is occurring. I turned it into a GitHub action to make it easier to access to get wider feedback on the metric. The GitHub action triggers on your merge request and tells you the files where erosion is occurring to refactor. This stops erosion before it gets too expensive to change (big refactors or rewrite). Yes, happy to work with others to get the metric into other tools.
      1. catlifeonmars · · focus · HN ↗
        What exactly is “cohesive erosion”?
        1. sagenschneider · · focus · HN ↗
          Comes from the basic Computer Science principals of High Cohesion and Low Coupling.

          High cohesion means the functionality of a component are closely related and focused on performing a single well defined task. Basically single classes for single purposes.

          Erosion of this is when classes start doing to many things, in the case of God classes.

          The Change Impact formula looks at a way of detecting when the cohesion is eroding and flagging it on a change (as the pull&#x2F;merge request itself should generally be single focus cohesive change)

Open on Hacker News to reply ↗

Unofficial Hacker News client; not affiliated with Y Combinator.