fix(no-navigation-without-resolve): Detecting asset() function as well (#1365)

Author: marekdedic

.changeset/poor-frogs-build.md

@@ -0,0 +1,5 @@
+---
+'eslint-plugin-svelte': patch
+---
+
+fix(no-navigation-without-resolve): Detecting asset() function as well

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

@@ -125,6 +125,9 @@ function extractResolveReferences(
 for (const { node } of referenceTracker.iterateEsmReferences({
 '$app/paths': {
 [ReferenceTracker.ESM]: true,
+asset: {
+[ReferenceTracker.READ]: true
+},
 resolve: {
 [ReferenceTracker.READ]: true
 }

packages/eslint-plugin-svelte/tests/fixtures/rules/no-navigation-without-resolve/invalid/link-partial-asset01-errors.yaml

@@ -0,0 +1,8 @@
+- message: Found a link with a url that isn't resolved.
+ line: 5
+ column: 9
+ suggestions: null
+- message: Found a link with a url that isn't resolved.
+ line: 6
+ column: 9
+ suggestions: null

packages/eslint-plugin-svelte/tests/fixtures/rules/no-navigation-without-resolve/invalid/link-partial-asset01-input.svelte

@@ -0,0 +1,6 @@
+<script>
+import { asset } from '$app/paths';
+</script>
+
+<a href={asset('/foo') + '/bar'}>Click me!</a>
+<a href={'/foo' + asset('/bar')}>Click me!</a>

packages/eslint-plugin-svelte/tests/fixtures/rules/no-navigation-without-resolve/valid/link-asset-aliased01-input.svelte

@@ -0,0 +1,5 @@
+<script>
+import { asset as alias } from '$app/paths';
+</script>
+
+<a href={alias('/foo/')}>Click me!</a>;

packages/eslint-plugin-svelte/tests/fixtures/rules/no-navigation-without-resolve/valid/link-asset01-input.svelte

@@ -0,0 +1,8 @@
+<script>
+import { asset } from '$app/paths';
+
+const value = asset('/foo/');
+</script>
+
+<a href={asset('/foo/')}>Click me!</a>
+<a href={value}>Click me!</a>