fix #521

Author: mProjectsCode

.github/ISSUE_TEMPLATE/config.yml

@@ -1,8 +1,8 @@
 blank_issues_enabled: false
 contact_links:
-  - name: "Meta Bind thead on the official Obsidian Discord server"
-    url: "https://discord.com/channels/686053708261228577/1286803892549713921"
-    about: "Please ask your questions on the official Obsidian Discord server in the \"Meta Bind ans JS Engine\" thread." 
-  - name: "Discussions"
-    url: "https://github.com/mProjectsCode/obsidian-meta-bind-plugin/discussions"
-    about: "If you don't want to join the official Obsidian Discord server, you can use the Discussions tab of this repository to ask your questions."
+    - name: 'Meta Bind thead on the official Obsidian Discord server'
+      url: 'https://discord.com/channels/686053708261228577/1286803892549713921'
+      about: 'Please ask your questions on the official Obsidian Discord server in the "Meta Bind ans JS Engine" thread.'
+    - name: 'Discussions'
+      url: 'https://github.com/mProjectsCode/obsidian-meta-bind-plugin/discussions'
+      about: "If you don't want to join the official Obsidian Discord server, you can use the Discussions tab of this repository to ask your questions."

packages/core/src/utils/Utils.ts

@@ -324,21 +324,42 @@ export function getFolderPathFromFilePath(filePath: string): string {
  * Joins the given paths together without duplicate slashes.
  */
 export function joinPath(...paths: string[]): string {
-	let result = paths[0];
+	if (paths.length === 0) {
+		return '/';
+	}
+	if (paths.length === 1) {
+		return cleanPath(paths[0]);
+	}
+
+	let result = paths[0].startsWith('/') ? paths[0].substring(1) : paths[0];
 	for (let i = 1; i < paths.length; i++) {
+		if (paths[i] === '' || paths[i] === '/') {
+			continue;
+		}
+
 		const endsWithSlash = result.endsWith('/');
 		const startsWithSlash = paths[i].startsWith('/');
 
 		if (endsWithSlash && startsWithSlash) {
-			result = result.substring(0, result.length - 1);
+			result += paths[i].substring(1);
 		} else if (!endsWithSlash && !startsWithSlash) {
-			result += '/';
+			result += '/' + paths[i];
+		} else {
+			result += paths[i];
 		}
-
-		result += paths[i];
 	}
 
-	return result;
+	return cleanPath(result);
+}
+
+export function cleanPath(path: string): string {
+	if (path.startsWith('/')) {
+		path = path.substring(1);
+	}
+	if (path.endsWith('/')) {
+		path = path.substring(0, path.length - 1);
+	}
+	return path === '' ? '/' : path;
 }
 
 /**

tests/utils/Utils.test.ts

@@ -245,22 +245,29 @@ describe('utils', () => {
 
 		test('should handle single path correctly', () => {
 			expect(joinPath('path')).toBe('path');
-			expect(joinPath('/path')).toBe('/path');
-			expect(joinPath('path/')).toBe('path/');
+			expect(joinPath('/path')).toBe('path');
+			expect(joinPath('path/')).toBe('path');
 		});
 
 		test('should handle empty paths correctly', () => {
-			expect(joinPath('')).toBe('');
+			expect(joinPath('')).toBe('/');
 			expect(joinPath('', '')).toBe('/');
-			expect(joinPath('', 'path')).toBe('/path');
-			expect(joinPath('path', '')).toBe('path/');
+			expect(joinPath('', 'path')).toBe('path');
+			expect(joinPath('path', '')).toBe('path');
 		});
 
 		test('should handle paths with leading and trailing slashes correctly', () => {
-			expect(joinPath('/path', 'to', 'file')).toBe('/path/to/file');
+			expect(joinPath('/path', 'to', 'file')).toBe('path/to/file');
 			expect(joinPath('path', 'to', '/file')).toBe('path/to/file');
-			expect(joinPath('/path', 'to', '/file')).toBe('/path/to/file');
-			expect(joinPath('/path/', '/to/', '/file/')).toBe('/path/to/file/');
+			expect(joinPath('/path', 'to', '/file')).toBe('path/to/file');
+			expect(joinPath('/path/', '/to/', '/file/')).toBe('path/to/file');
+		});
+
+		test('should handle slashes only correctly', () => {
+			expect(joinPath('/')).toBe('/');
+			expect(joinPath('/', '/')).toBe('/');
+			expect(joinPath('/', 'path')).toBe('path');
+			expect(joinPath('path', '/')).toBe('path');
 		});
 	});