fix position context after question mark token

fixes #2611

Author: Techatrix

src/analysis.zig

@@ -5054,8 +5054,9 @@ pub fn getPositionContext(
                 .test_doctest_name, .var_access, .field_access => |loc| .{ .field_access = tokenLocAppend(loc, tok) },
                 else => .{ .field_access = tokenLocAppend(curr_ctx.ctx.loc(tree) orelse tok.loc, tok) },
             },
-            .question_mark, .period_asterisk => switch (curr_ctx.ctx) {
-                .var_access, .field_access => |loc| .{ .field_access = tokenLocAppend(loc, tok) },
+            .period_asterisk => .{ .field_access = tokenLocAppend(curr_ctx.ctx.loc(tree) orelse tok.loc, tok) },
+            .question_mark => switch (curr_ctx.ctx) {
+                .field_access => |loc| .{ .field_access = tokenLocAppend(loc, tok) },
                 else => .empty,
             },
             .colon => switch (curr_ctx.ctx) {

tests/utility/position_context.zig

@@ -105,6 +105,31 @@ test "var_access.nested" {
     , .var_access, .{ .lookahead = true });
 }
 
+test "var_access.optional" {
+    try testContext(
+        \\const foo = ?<cursor>
+    , .empty, .{});
+    try testContext(
+        \\const foo = ?<loc>T<cursor></loc>
+    , .var_access, .{});
+}
+
+test "var_access.pointer" {
+    try testContext(
+        \\const foo = *<cursor>
+    , .empty, .{});
+    try testContext(
+        \\const foo = *<loc>T<cursor></loc>
+    , .var_access, .{});
+
+    try testContext(
+        \\const foo = []<cursor>
+    , .empty, .{});
+    try testContext(
+        \\const foo = []<loc>T<cursor></loc>
+    , .var_access, .{});
+}
+
 test "var_access no lookahead" {
     try testContext(
         \\const a_var =<cursor> identifier;