Revert "Show auto-imports in array-like tables"

This reverts commit 9e8cf33.

We are incorrectly showing auto imports when indexing tables with
properties. A test is attached

Author: JohnnyMorganz

CHANGELOG.md

@@ -6,6 +6,10 @@ The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.0.0/).
 
 ## [Unreleased]
 
+### Fixed
+
+- Reverted "Fixed auto-imports not showing up when autocompleting in array-like tables" due to it incorrectly suggesting auto-imports when indexing tables (e.g., `local x = tbl.pro|`)
+
 ## [1.56.0] - 2025-11-02
 
 ### Added
@@ -1710,6 +1714,7 @@ local y = tbl.data -- Should give "This is some special information"
 ### Added
 
 - Added configuration options to enable certain Language Server features. By default, they are all enabled:
+
   - `luau-lsp.completion.enabled`: Autocomplete
   - `luau-lsp.hover.enabled`: Hover
   - `luau-lsp.signatureHelp.enabled`: Signature Help

src/operations/Completion.cpp

@@ -761,8 +761,7 @@ std::vector<lsp::CompletionItem> WorkspaceFolder::completion(const lsp::Completi
 
     if (config.completion.suggestImports || config.completion.imports.enabled)
     {
-        if (result.context == Luau::AutocompleteContext::Expression || result.context == Luau::AutocompleteContext::Statement ||
-            result.context == Luau::AutocompleteContext::Property)
+        if (result.context == Luau::AutocompleteContext::Expression || result.context == Luau::AutocompleteContext::Statement)
         {
             suggestImports(moduleName, position, config, *textDocument, items, /* completingTypeReferencePrefix: */ false);
         }

tests/AutoImports.test.cpp

@@ -1449,12 +1449,12 @@ TEST_CASE_FIXTURE(Fixture, "auto_import_empty_require_statement")
     CHECK_EQ(item->additionalTextEdits[0].range.start.line, 1);
 }
 
-TEST_CASE_FIXTURE(Fixture, "auto_imports_shows_up_in_tables_before_equals_sign")
+TEST_CASE_FIXTURE(Fixture, "auto_imports_shows_up_in_tables_after_equals_sign")
 {
     client->globalConfig.completion.imports.enabled = true;
     auto [source, marker] = sourceWithMarker(R"(
         create({
-            |
+            Key = |
         })
     )");
 
@@ -1468,13 +1468,11 @@ TEST_CASE_FIXTURE(Fixture, "auto_imports_shows_up_in_tables_before_equals_sign")
     CHECK(getItem(result, "ReplicatedStorage"));
 }
 
-TEST_CASE_FIXTURE(Fixture, "auto_imports_shows_up_in_tables_after_equals_sign")
+TEST_CASE_FIXTURE(Fixture, "auto_imports_do_not_show_when_completion_property")
 {
     client->globalConfig.completion.imports.enabled = true;
     auto [source, marker] = sourceWithMarker(R"(
-        create({
-            Key = |
-        })
+        local x = foo.ba|
     )");
 
     auto uri = newDocument("foo.luau", source);
@@ -1484,7 +1482,7 @@ TEST_CASE_FIXTURE(Fixture, "auto_imports_shows_up_in_tables_after_equals_sign")
     params.position = marker;
 
     auto result = workspace.completion(params, nullptr);
-    CHECK(getItem(result, "ReplicatedStorage"));
+    CHECK_FALSE(getItem(result, "ReplicatedStorage"));
 }
 
 TEST_SUITE_END();