Various improvements to the import system

[oxisto]

Unfortunately this PR is beginning to grow into something too big, so we probably want to merge it soon, but the problem is that things in the import / scope system are really tightly coupled. Hopefully a few things are now more de-coupled.

• Instead of caching importedSymbols statically once (or twice), instead we resolve the import declaration back to the namespace (declaration) and update the symbol on-chance. This still not the perfect solution that I want, but it's a step into the right direction.
• This also speeds up the ResolveMemberExpressionAmbiguityPass.
• Rework resolveParentAlias to take scope information into account

[codecov]

Codecov Report

Attention: Patch coverage is 78.51852% with 29 lines in your changes missing coverage. Please review.

Project coverage is 78.22%. Comparing base (541dea1) to head (1de14ed).
Report is 1 commits behind head on main.

Files with missing lines	Patch %	Lines
...ain/kotlin/de/fraunhofer/aisec/cpg/ScopeManager.kt	72.72%	0 Missing and 6 partials ⚠️
...n/de/fraunhofer/aisec/cpg/passes/ImportResolver.kt	76.92%	2 Missing and 4 partials ⚠️
...ofer/aisec/cpg/frontends/cxx/DeclarationHandler.kt	58.33%	5 Missing ⚠️
...sec/cpg/graph/declarations/ParameterDeclaration.kt	50.00%	3 Missing and 1 partial ⚠️
.../fraunhofer/aisec/cpg/graph/edges/scopes/Import.kt	75.00%	3 Missing and 1 partial ⚠️
...ec/cpg/graph/edges/collections/UnwrappedEdgeSet.kt	33.33%	2 Missing ⚠️
...kotlin/de/fraunhofer/aisec/cpg/graph/Extensions.kt	0.00%	1 Missing ⚠️
...raunhofer/aisec/cpg/graph/scopes/NamespaceScope.kt	85.71%	0 Missing and 1 partial ⚠️

Additional details and impacted files

Files with missing lines	Coverage Δ
...e/fraunhofer/aisec/cpg/graph/DeclarationBuilder.kt	94.59% <100.00%> (-0.04%)	⬇️
...isec/cpg/graph/declarations/FunctionDeclaration.kt	77.10% <100.00%> (-0.28%)	⬇️
.../aisec/cpg/graph/declarations/ImportDeclaration.kt	100.00% <100.00%> (ø)
.../de/fraunhofer/aisec/cpg/graph/edges/Extensions.kt	89.47% <100.00%> (+1.97%)	⬆️
...tlin/de/fraunhofer/aisec/cpg/graph/scopes/Scope.kt	82.66% <100.00%> (+0.97%)	⬆️
...isec/cpg/graph/scopes/StructureDeclarationScope.kt	68.75% <ø> (ø)
...er/aisec/cpg/graph/scopes/ValueDeclarationScope.kt	50.00% <ø> (ø)
...cpg/passes/ResolveMemberExpressionAmbiguityPass.kt	96.15% <100.00%> (-0.63%)	⬇️
...lin/de/fraunhofer/aisec/cpg/passes/TypeResolver.kt	95.45% <ø> (-0.11%)	⬇️
...fer/aisec/cpg/frontends/cxx/CXXLanguageFrontend.kt	77.65% <ø> (-0.07%)	⬇️
... and 12 more

... and 3 files with indirect coverage changes

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

[konradweiss]

LGTM

[lshala]

Besides the minor point, it LGTM