Fixed apps incorrectly using local filesystem path from artifacts path (#4946)

## Changes
Fixed apps incorrectly using local filesystem path from artifacts path

## Why
Fixes #4924

## Tests
Added an acceptance test

<!-- If your PR needs to be included in the release notes for next
release,
add a separate entry in NEXT_CHANGELOG.md as part of your PR. -->

Author: andrewnester

acceptance/bundle/apps/artifact_and_app_same_path/databricks.yml

@@ -0,0 +1,13 @@
+bundle:
+  name: test-bundle
+
+artifacts:
+  my_artifact:
+    type: whl
+    path: ./src/app
+
+resources:
+  apps:
+    my_app:
+      name: my-app
+      source_code_path: ./src/app

acceptance/bundle/apps/artifact_and_app_same_path/out.test.toml

@@ -0,0 +1,6 @@
+
+>>> [CLI] bundle validate -o json
+/Workspace/Users/[USERNAME]/.bundle/test-bundle/default/files/src/app
+
+>>> [CLI] bundle validate -o json
+[TEST_TMP_DIR]/src/app

acceptance/bundle/apps/artifact_and_app_same_path/output.txt

@@ -0,0 +1,2 @@
+trace $CLI bundle validate -o json | jq -r '.resources.apps.my_app.source_code_path'
+trace $CLI bundle validate -o json | jq -r '.artifacts.my_artifact.path'

acceptance/bundle/apps/artifact_and_app_same_path/script

@@ -0,0 +1,5 @@
+RecordRequests = false
+
+Ignore = [
+    '.databricks',
+]

acceptance/bundle/apps/artifact_and_app_same_path/src/app/test.py

@@ -47,6 +47,14 @@ func (err ErrIsNotNotebook) Error() string {
 	return fmt.Sprintf("file at %s is not a notebook", err.path)
 }
 
+// seenKey is the cache key for the seen map in translateContext.
+// It includes both the local path and the translation mode to prevent
+// cross-mode cache collisions (e.g. artifact vs. workspace path translations).
+type seenKey struct {
+	path string
+	mode paths.TranslateMode
+}
+
 type translatePaths struct{}
 
 type translatePathsDashboards struct{}
@@ -76,9 +84,9 @@ func (m *translatePathsDashboards) Name() string {
 type translateContext struct {
 	b *bundle.Bundle
 
-	// seen is a map of local paths to their corresponding remote paths.
-	// If a local path has already been successfully resolved, we do not need to resolve it again.
-	seen map[string]string
+	// seen is a map of (local path, translation mode) pairs to their corresponding remote paths.
+	// If a local path has already been successfully resolved for a given mode, we do not need to resolve it again.
+	seen map[seenKey]string
 
 	// remoteRoot is the root path of the remote workspace.
 	// It is equal to ${workspace.file_path} for regular deployments.
@@ -135,7 +143,8 @@ func (t *translateContext) rewritePath(
 
 	// Local path is relative to the directory the resource was defined in.
 	localPath := filepath.Join(dir, input)
-	if interp, ok := t.seen[localPath]; ok {
+	key := seenKey{path: localPath, mode: opts.Mode}
+	if interp, ok := t.seen[key]; ok {
 		return interp, nil
 	}
 
@@ -181,7 +190,7 @@ func (t *translateContext) rewritePath(
 		return "", err
 	}
 
-	t.seen[localPath] = interp
+	t.seen[key] = interp
 	return interp, nil
 }
 
@@ -337,7 +346,7 @@ func applyTranslations(ctx context.Context, b *bundle.Bundle, t *translateContex
 func (m *translatePaths) Apply(ctx context.Context, b *bundle.Bundle) diag.Diagnostics {
 	t := &translateContext{
 		b:                       b,
-		seen:                    make(map[string]string),
+		seen:                    make(map[seenKey]string),
 		skipLocalFileValidation: b.SkipLocalFileValidation,
 	}
 
@@ -354,7 +363,7 @@ func (m *translatePaths) Apply(ctx context.Context, b *bundle.Bundle) diag.Diagn
 func (m *translatePathsDashboards) Apply(ctx context.Context, b *bundle.Bundle) diag.Diagnostics {
 	t := &translateContext{
 		b:                       b,
-		seen:                    make(map[string]string),
+		seen:                    make(map[seenKey]string),
 		skipLocalFileValidation: b.SkipLocalFileValidation,
 	}