fix: completions understand show

server/src/features/completion/completion.ts

@@ -240,6 +240,7 @@ function traverseTree(
 	accumulator: Map<string, CompletionItem[]>,
 	leaf: IScssDocument,
 	hiddenSymbols: string[] = [],
+	shownSymbols: string[] = [],
 	accumulatedPrefix = "",
 ) {
 	if (accumulator.has(leaf.uri)) {
@@ -263,6 +264,7 @@ function traverseTree(
 				document,
 				context,
 				hiddenSymbols,
+				shownSymbols,
 				accumulatedPrefix,
 			);
 			completionItems = completionItems.concat(variables);
@@ -274,6 +276,7 @@ function traverseTree(
 				document,
 				context,
 				hiddenSymbols,
+				shownSymbols,
 				accumulatedPrefix,
 			);
 			completionItems = completionItems.concat(mixins);
@@ -285,6 +288,7 @@ function traverseTree(
 				document,
 				context,
 				hiddenSymbols,
+				shownSymbols,
 				accumulatedPrefix,
 			);
 			completionItems = completionItems.concat(functions);
@@ -294,12 +298,17 @@ function traverseTree(
 			const placeholders = createPlaceholderCompletionItems(
 				scssDocument,
 				hiddenSymbols,
+				shownSymbols,
 			);
 			completionItems = completionItems.concat(placeholders);
 		}
 
 		accumulator.set(leaf.uri, completionItems);
 
+		// Reset these once we've processed the document they refered to
+		let hidden: string[] = [];
+		let shown: string[] = [];
+
 		// Check to see if we have to go deeper
 		// Don't follow uses, since we start with the document behind the first use, and symbols from further uses aren't available to us
 		// Don't follow imports, since the whole point here is to use the new module system
@@ -313,12 +322,17 @@ function traverseTree(
 				continue;
 			}
 
-			let hidden = hiddenSymbols;
 			if (
 				(child as ScssForward).hide &&
 				(child as ScssForward).hide.length > 0
 			) {
-				hidden = hiddenSymbols.concat((child as ScssForward).hide);
+				hidden = hidden.concat((child as ScssForward).hide);
+			}
+			if (
+				(child as ScssForward).show &&
+				(child as ScssForward).show.length > 0
+			) {
+				shown = shown.concat((child as ScssForward).show);
 			}
 
 			let prefix = accumulatedPrefix;
@@ -332,6 +346,7 @@ function traverseTree(
 				accumulator,
 				childDocument,
 				hidden,
+				shown,
 				prefix,
 			);
 		}

server/src/features/completion/function-completion.ts

@@ -23,6 +23,7 @@ export function createFunctionCompletionItems(
 	currentDocument: TextDocument,
 	context: CompletionContext,
 	hiddenSymbols: string[] = [],
+	shownSymbols: string[] = [],
 	prefix = "",
 ): CompletionItem[] {
 	const completions: CompletionItem[] = [];
@@ -39,6 +40,10 @@ export function createFunctionCompletionItems(
 			continue;
 		}
 
+		if (shownSymbols.length > 0 && !shownSymbols.includes(func.name)) {
+			continue;
+		}
+
 		// Client needs the namespace as part of the text that is matched,
 		// and inserted text needs to include the `.` which will otherwise
 		// be replaced (except when we're embedded in Vue, Svelte or Astro).

server/src/features/completion/mixin-completion.ts

@@ -24,6 +24,7 @@ export function createMixinCompletionItems(
 	currentDocument: TextDocument,
 	context: CompletionContext,
 	hiddenSymbols: string[] = [],
+	shownSymbols: string[] = [],
 	prefix = "",
 ): CompletionItem[] {
 	const completions: CompletionItem[] = [];
@@ -40,6 +41,10 @@ export function createMixinCompletionItems(
 			continue;
 		}
 
+		if (shownSymbols.length > 0 && !shownSymbols.includes(mixin.name)) {
+			continue;
+		}
+
 		const documentation = makeMixinDocumentation(mixin, scssDocument);
 
 		// Client needs the namespace as part of the text that is matched,

server/src/features/completion/placeholder-completion.ts

@@ -11,6 +11,7 @@ import { applySassDoc } from "../../utils/sassdoc";
 export function createPlaceholderCompletionItems(
 	scssDocument: IScssDocument,
 	hiddenSymbols: string[] = [],
+	shownSymbols: string[] = [],
 ): CompletionItem[] {
 	const completions: CompletionItem[] = [];
 
@@ -19,6 +20,10 @@ export function createPlaceholderCompletionItems(
 			continue;
 		}
 
+		if (shownSymbols.length > 0 && !shownSymbols.includes(placeholder.name)) {
+			continue;
+		}
+
 		const label = placeholder.name;
 		const filterText = placeholder.name.substring(1);
 

server/src/features/completion/variable-completion.ts

@@ -18,6 +18,7 @@ export function createVariableCompletionItems(
 	currentDocument: TextDocument,
 	context: CompletionContext,
 	hiddenSymbols: string[] = [],
+	shownSymbols: string[] = [],
 	prefix = "",
 ): CompletionItem[] {
 	const completions: CompletionItem[] = [];
@@ -65,6 +66,10 @@ export function createVariableCompletionItems(
 				continue;
 			}
 
+			if (shownSymbols.length > 0 && !shownSymbols.includes(variable.name)) {
+				continue;
+			}
+
 			if (isPrivate) {
 				sortText = label.replace(/^$[_-]/, "");
 			}