feat(no-navigation-without-resolve): added support for ResolvedPathname types

Author: marekdedic

.changeset/icy-mammals-cover.md

@@ -0,0 +1,5 @@
+---
+'eslint-plugin-svelte': patch
+---
+
+feat(no-navigation-without-resolve): added support for ResolvedPathname types

packages/eslint-plugin-svelte/src/rules/no-navigation-without-resolve.ts

@@ -5,6 +5,7 @@ import { FindVariableContext } from '../utils/ast-utils.js';
 import { findVariable } from '../utils/ast-utils.js';
 import type { RuleContext } from '../types.js';
 import type { AST } from 'svelte-eslint-parser';
+import { type TSTools, getTypeScriptTools } from 'src/utils/ts-utils/index.js';
 
 export default createRule('no-navigation-without-resolve', {
 	meta: {
@@ -48,6 +49,8 @@ export default createRule('no-navigation-without-resolve', {
 		]
 	},
 	create(context) {
+		const tsTools = getTypeScriptTools(context);
+
 		let resolveReferences: Set<TSESTree.Identifier> = new Set<TSESTree.Identifier>();
 
 		const ignoreGoto = context.options[0]?.ignoreGoto ?? false;
@@ -66,7 +69,7 @@ export default createRule('no-navigation-without-resolve', {
 				} = extractFunctionCallReferences(referenceTracker);
 				if (!ignoreGoto) {
 					for (const gotoCall of gotoCalls) {
-						checkGotoCall(context, gotoCall, resolveReferences);
+						checkGotoCall(context, gotoCall, resolveReferences, tsTools);
 					}
 				}
 				if (!ignorePushState) {
@@ -75,6 +78,7 @@ export default createRule('no-navigation-without-resolve', {
 							context,
 							pushStateCall,
 							resolveReferences,
+							tsTools,
 							'pushStateWithoutResolve'
 						);
 					}
@@ -85,6 +89,7 @@ export default createRule('no-navigation-without-resolve', {
 							context,
 							replaceStateCall,
 							resolveReferences,
+							tsTools,
 							'replaceStateWithoutResolve'
 						);
 					}
@@ -135,7 +140,8 @@ export default createRule('no-navigation-without-resolve', {
 						!isResolveCall(
 							new FindVariableContext(context),
 							node.value[0].expression,
-							resolveReferences
+							resolveReferences,
+							tsTools
 						))
 				) {
 					context.report({ loc: node.value[0].loc, messageId: 'linkWithoutResolve' });
@@ -223,13 +229,14 @@ function extractFunctionCallReferences(referenceTracker: ReferenceTracker): {
 function checkGotoCall(
 	context: RuleContext,
 	call: TSESTree.CallExpression,
-	resolveReferences: Set<TSESTree.Identifier>
+	resolveReferences: Set<TSESTree.Identifier>,
+	tsTools: TSTools | null
 ): void {
 	if (call.arguments.length < 1) {
 		return;
 	}
 	const url = call.arguments[0];
-	if (!isResolveCall(new FindVariableContext(context), url, resolveReferences)) {
+	if (!isResolveCall(new FindVariableContext(context), url, resolveReferences, tsTools)) {
 		context.report({ loc: url.loc, messageId: 'gotoWithoutResolve' });
 	}
 }
@@ -238,6 +245,7 @@ function checkShallowNavigationCall(
 	context: RuleContext,
 	call: TSESTree.CallExpression,
 	resolveReferences: Set<TSESTree.Identifier>,
+	tsTools: TSTools | null,
 	messageId: string
 ): void {
 	if (call.arguments.length < 1) {
@@ -246,7 +254,7 @@ function checkShallowNavigationCall(
 	const url = call.arguments[0];
 	if (
 		!expressionIsEmpty(url) &&
-		!isResolveCall(new FindVariableContext(context), url, resolveReferences)
+		!isResolveCall(new FindVariableContext(context), url, resolveReferences, tsTools)
 	) {
 		context.report({ loc: url.loc, messageId });
 	}
@@ -257,7 +265,8 @@ function checkShallowNavigationCall(
 function isResolveCall(
 	ctx: FindVariableContext,
 	node: TSESTree.CallExpressionArgument,
-	resolveReferences: Set<TSESTree.Identifier>
+	resolveReferences: Set<TSESTree.Identifier>,
+	tsTools: TSTools | null
 ): boolean {
 	if (
 		node.type === 'CallExpression' &&
@@ -268,9 +277,13 @@ function isResolveCall(
 	) {
 		return true;
 	}
-	if (node.type !== 'Identifier') {
+	if (node.type !== 'Identifier' || tsTools === null) {
 		return false;
 	}
+	const tsNode = tsTools.service.esTreeNodeToTSNodeMap.get(node);
+	console.log(tsNode);
+	console.log(tsTools.service.program.getTypeChecker().getTypeAtLocation(tsNode));
+	/*
 	const variable = ctx.findVariable(node);
 	if (
 		variable === null ||
@@ -281,6 +294,7 @@ function isResolveCall(
 		return false;
 	}
 	return isResolveCall(ctx, variable.identifiers[0].parent.init, resolveReferences);
+	*/
 }
 
 function expressionIsEmpty(url: TSESTree.CallExpressionArgument): boolean {