Adding new Overlay and Concept nodes to the graph.

[maximiliankaul]

Preparation for mk/concepts This are the cpg-core affecting changes.

[codecov]

Codecov Report

Attention: Patch coverage is 73.68421% with 10 lines in your changes missing coverage. Please review.

Project coverage is 77.04%. Comparing base (1f16d58) to head (7f77fc0).
Report is 1 commits behind head on main.

Files with missing lines	Patch %	Lines
...c/cpg/graph/edges/collections/EdgeSingletonList.kt	25.00%	8 Missing and 1 partial ⚠️
...raunhofer/aisec/cpg/graph/edges/overlay/Overlay.kt	90.90%	1 Missing ⚠️

Additional details and impacted files

Files with missing lines	Coverage Δ
.../main/kotlin/de/fraunhofer/aisec/cpg/graph/Node.kt	87.50% <100.00%> (+0.44%)	⬆️
...otlin/de/fraunhofer/aisec/cpg/graph/OverlayNode.kt	100.00% <100.00%> (ø)
.../de/fraunhofer/aisec/cpg/graph/concepts/Concept.kt	100.00% <100.00%> (ø)
...e/fraunhofer/aisec/cpg/graph/concepts/Operation.kt	100.00% <100.00%> (ø)
...raunhofer/aisec/cpg/graph/edges/overlay/Overlay.kt	90.90% <90.90%> (ø)
...c/cpg/graph/edges/collections/EdgeSingletonList.kt	56.25% <25.00%> (+2.54%)	⬆️

... and 2 files with indirect coverage changes

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[konradweiss]

Besides some renaming and the hierarchy change suggestion this is what I had in mind.

[konradweiss]

LGTM. The only remarks I have is that I would prefer the edges to go from the overlay node to the underlay node, but that is more because it was always like this in my imagination. And then we would need a different edge label so it is not really something we have to change for this PR.

[oxisto]

LGTM. The only remarks I have is that I would prefer the edges to go from the overlay node to the underlay node, but that is more because it was always like this in my imagination. And then we would need a different edge label so it is not really something we have to change for this PR.

I can just switch the direction, if you want

[oxisto]

LGTM. The only remarks I have is that I would prefer the edges to go from the overlay node to the underlay node, but that is more because it was always like this in my imagination. And then we would need a different edge label so it is not really something we have to change for this PR.

I can just switch the direction, if you want

AH, no that doesn't work. It needs to be outgoing from an AST node, otherwise it will not end up in the persisted graph. I will keep it as is for now.

[oxisto]

Nodes can also have multiple overlay nodes now, but an overlay can only have one underlying node.