Inspect - don't exclude valid scope members declared after cursor position about powerquery-parser HOT 3 CLOSED

I'm wondering if I'm misunderstanding the purpose of scope.
I tried removing the position check in inspectIdentifier, but that ended up breaking a number of valid test cases. So clearly a different or smarter approach is required.

case 1: (x|, y) => z

Removing the position check added y to the current scope, which isn't what we want.

If scope is for intellisense, then x shouldn't be included. If scope is used to represent a hierarchy of symbols, then we need an easy way to determine which scope members should be excluded from intellisense results (i.e. what assignmentKeyNodeIdMap is doing now).

case 2: s|ection foo; x = 1; y = 2;

Since the cursor is on the section keyword/declaration, we shouldn't have any members in scope.

from powerquery-parser.

I resolved case 1 by not calling inspectIdentifier when processing the section.

For case 2, I updated the related test cases and added TODO comments.

I think this scenario is related to #56 in that there are certain cursor positions where we should be returning an empty scope, or providing additional context so the intellisense layer knows that the current list of identifiers shouldn't be included in suggestions.

from powerquery-parser.

Should be resolved in #70

from powerquery-parser.