fix: handle analysis of decls more correctly

[neoto]

Handles references to aliased decls more correctly by distinguishing between containers and runtime types. Compared to
#3189 which special cases renames, this should address the root cause, fixing both renaming and highlighting symbols. I also added regression tests that fail without the changes on this branch.

Closes #3184

[Techatrix]

I prefer the behavior provided by #3189 over this solution. Because of alises, renaming should not necessarily behave identical to references. Consider this example:

const S = struct {
    fn foo() void {}
};
comptime {
    const foo = S.foo;
    foo();
}

Works as expected:

1. Renaming foo() should not rename fn foo and S.foo

Doesn't work as expected:

2. Asking for references on foo() should also provide fn foo and S.foo
3. Asking for references on fn foo should also provide const foo or foo()
4. Renaming fn foo should also rename const foo or foo()

The preferred behavior of the second item could be debated but 3 & 4 I feel more confident about.

[neoto]

Aight. That still leaves highlighting borked, which that PR doesn't fix