Fix resolution issue

Author: AJenbo

src/text_scan.rs

@@ -432,7 +432,14 @@ fn same_line_continuation_prefix(trimmed: &str) -> Option<&str> {
     let mut saw_closer = false;
     for (idx, ch) in trimmed.char_indices() {
         match ch {
-            ')' | '}' | ']' => {
+            // Only `)`/`}` are recognized here because the backward balance
+            // scan below (and in `collapse_continuation_lines`) only tracks
+            // paren/brace depth, not bracket depth. Treating `]` as a
+            // trigger would let this collapse a multi-line array subscript
+            // (`$config[\n    'key',\n]->foo`) while never accounting for
+            // the `[`, so the walk stops one line too early and silently
+            // drops the base expression instead of producing no match.
+            ')' | '}' => {
                 saw_closer = true;
                 end = idx + ch.len_utf8();
             }

tests/integration/completion_multiline_chains.rs

@@ -559,3 +559,38 @@ async fn test_multiline_chain_same_line_after_closure_arg_close() {
         "Should offer Collection::all() after same-line closure close, got: {names:?}"
     );
 }
+
+#[tokio::test]
+async fn test_multiline_chain_same_line_after_bracket_close_does_not_misresolve() {
+    // Bracket depth (`[`/`]`) is not tracked by the backward balance scan
+    // used to reconstruct the base expression, so a chain operator that
+    // follows a closing `]` on its own line must not be treated as a
+    // same-line continuation: doing so would drop everything before the
+    // bracket close (e.g. the `$this->items[` base) and hand the resolver
+    // a malformed expression instead of the real receiver. Falling back to
+    // "no completions" is correct here; resolving to the wrong receiver's
+    // members would not be.
+    let backend = create_test_backend();
+    let uri = Url::parse("file:///multiline_bracket_same_line.php").unwrap();
+    let text = concat!(
+        "<?php\n",
+        "class Box {\n",
+        "    public function open(): string { return ''; }\n",
+        "}\n",
+        "class Container {\n",
+        "    /** @var array<string, Box> */\n",
+        "    public array $items = [];\n",
+        "    public function run(): void {\n",
+        "        $this->items[\n",
+        "            'key'\n",
+        "        ]->o\n",
+        "    }\n",
+        "}\n",
+    );
+
+    let names = complete_at(&backend, &uri, text, 10, 12).await;
+    assert!(
+        !names.iter().any(|n| n.starts_with("open(")),
+        "Should not resolve Box::open() from a malformed collapsed expression, got: {names:?}"
+    );
+}