Merge pull request #865 from trey-wallis/dev

* chore: bump version to 8.13.0

* feat: [WIP] make individual cell types

* feat: [WIP] fix errors resulting from changing types

* fix: resolve crash on selecting tag

* refactor: [WIP] replace undo/redo logic with jsondiffpatch

* refactor: rename to finishExecute

* fix: resolve double call to onExecute

* test: remove unnecessary checks

* refactor: move errors into folder

* fix: update dependencies paths

* fix: resolve previous state being mutated

* fix: handle undefined state of json diff of states

* refactor: clean up code

* refactor: update links module

* fix: resolve text links not updating

* fix: resolve views not updating

* fix: resolve refresh event values not being handled properly

* fix: use largest wiki link

* fix: only update type of cells for the column id

* fix: display empty string if path is empty

* refactor: remove log statements

* fix: resolve embed and file cell updating of links

* fix: resolve crash when tag or mult-tag are selected

* refactor: don't display file extension on source

* refactor: don't display disabled message in source or source file column

* feat: add right click copy to all cells

* fix: resolve eslint errors

Author: decaf-dev

bun.lockb

@@ -9,5 +9,5 @@
 	"fundingUrl": {
 		"Buymeacoffee": "https://www.buymeacoffee.com/treywallis"
 	},
-	"version": "8.12.2"
+	"version": "8.13.0"
 }

manifest.json

@@ -1,74 +1,75 @@
 {
-	"name": "obsidian-dataloom",
-	"version": "8.12.2",
-	"description": "Weave together data from diverse sources into a cohesive table view. Inspired by Excel Spreadsheets and Notion.so.",
-	"main": "main.js",
-	"scripts": {
-		"dev": "node esbuild.config.mjs",
-		"dev-with-tools": "node esbuild.config.mjs tools",
-		"build": "tsc -noEmit -skipLibCheck && node esbuild.config.mjs production",
-		"lint": "eslint src --ext ts,tsx --report-unused-disable-directives --max-warnings 0",
-		"test": "jest",
-		"version": "node version-bump.mjs && git add manifest.json versions.json"
-	},
-	"keywords": [],
-	"author": "Trey Wallis",
-	"license": "GPL-3.0+",
-	"devDependencies": {
-		"@babel/preset-env": "^7.19.0",
-		"@babel/preset-react": "^7.18.6",
-		"@babel/preset-typescript": "^7.18.6",
-		"@simbathesailor/use-what-changed": "^2.0.0",
-		"@testing-library/jest-dom": "^5.16.2",
-		"@testing-library/react": "^13.4.0",
-		"@testing-library/user-event": "^14.4.3",
-		"@types/codemirror": "^5.60.7",
-		"@types/jest": "^29.0.0",
-		"@types/lodash": "^4.14.180",
-		"@types/markdown-it": "^12.2.3",
-		"@types/mock-fs": "^4.13.1",
-		"@types/node": "^18.7.15",
-		"@types/papaparse": "^5.3.7",
-		"@types/react": "^18.0.18",
-		"@types/react-dom": "^18.0.6",
-		"@types/uuid": "^8.3.4",
-		"@typescript-eslint/eslint-plugin": "^5.59.8",
-		"@typescript-eslint/parser": "^5.59.8",
-		"babel-jest": "^29.0.2",
-		"builtin-modules": "^3.2.0",
-		"esbuild": "^0.15.7",
-		"eslint": "^8.41.0",
-		"eslint-config-react-app": "^7.0.1",
-		"eslint-plugin-react-hooks": "^4.6.0",
-		"eslint-plugin-react-refresh": "^0.4.1",
-		"jest": "^29.5.0",
-		"jest-date-mock": "^1.0.8",
-		"jest-environment-jsdom": "^29.7.0",
-		"jsdom": "^22.1.0",
-		"obsidian": "^1.4.11",
-		"react-test-renderer": "^18.2.0",
-		"renamer": "^4.0.0",
-		"ts-jest": "^29.0.5",
-		"tslib": "^2.4.0",
-		"typescript": "^5.1.3"
-	},
-	"dependencies": {
-		"@reduxjs/toolkit": "^1.8.5",
-		"codemirror": "^6.0.1",
-		"dayjs": "^1.11.10",
-		"fuzzysort": "^2.0.4",
-		"htmlparser2": "^9.0.0",
-		"lucide": "^0.241.0",
-		"markdown-it": "^13.0.1",
-		"markdown-table": "^3.0.3",
-		"papaparse": "^5.4.1",
-		"react": "^18.2.0",
-		"react-dom": "^18.2.0",
-		"react-redux": "^8.0.4",
-		"react-virtuoso": "^4.6.0",
-		"recoil": "^0.7.7",
-		"runtypes": "^6.7.0",
-		"ts-node": "^10.9.1",
-		"uuid": "^9.0.0"
-	}
+  "name": "obsidian-dataloom",
+  "version": "8.13.0",
+  "description": "Weave together data from diverse sources into a cohesive table view. Inspired by Excel Spreadsheets and Notion.so.",
+  "main": "main.js",
+  "scripts": {
+    "dev": "node esbuild.config.mjs",
+    "dev-with-tools": "node esbuild.config.mjs tools",
+    "build": "tsc -noEmit -skipLibCheck && node esbuild.config.mjs production",
+    "lint": "eslint src --ext ts,tsx --report-unused-disable-directives --max-warnings 0",
+    "test": "jest",
+    "version": "node version-bump.mjs && git add manifest.json versions.json"
+  },
+  "keywords": [],
+  "author": "Trey Wallis",
+  "license": "GPL-3.0+",
+  "devDependencies": {
+    "@babel/preset-env": "^7.19.0",
+    "@babel/preset-react": "^7.18.6",
+    "@babel/preset-typescript": "^7.18.6",
+    "@simbathesailor/use-what-changed": "^2.0.0",
+    "@testing-library/jest-dom": "^5.16.2",
+    "@testing-library/react": "^13.4.0",
+    "@testing-library/user-event": "^14.4.3",
+    "@types/codemirror": "^5.60.7",
+    "@types/jest": "^29.0.0",
+    "@types/lodash": "^4.14.180",
+    "@types/markdown-it": "^12.2.3",
+    "@types/mock-fs": "^4.13.1",
+    "@types/node": "^18.7.15",
+    "@types/papaparse": "^5.3.7",
+    "@types/react": "^18.0.18",
+    "@types/react-dom": "^18.0.6",
+    "@types/uuid": "^8.3.4",
+    "@typescript-eslint/eslint-plugin": "^5.59.8",
+    "@typescript-eslint/parser": "^5.59.8",
+    "babel-jest": "^29.0.2",
+    "builtin-modules": "^3.2.0",
+    "esbuild": "^0.15.7",
+    "eslint": "^8.41.0",
+    "eslint-config-react-app": "^7.0.1",
+    "eslint-plugin-react-hooks": "^4.6.0",
+    "eslint-plugin-react-refresh": "^0.4.1",
+    "jest": "^29.5.0",
+    "jest-date-mock": "^1.0.8",
+    "jest-environment-jsdom": "^29.7.0",
+    "jsdom": "^22.1.0",
+    "obsidian": "^1.4.11",
+    "react-test-renderer": "^18.2.0",
+    "renamer": "^4.0.0",
+    "ts-jest": "^29.0.5",
+    "tslib": "^2.4.0",
+    "typescript": "^5.1.3"
+  },
+  "dependencies": {
+    "@reduxjs/toolkit": "^1.8.5",
+    "codemirror": "^6.0.1",
+    "dayjs": "^1.11.10",
+    "fuzzysort": "^2.0.4",
+    "htmlparser2": "^9.0.0",
+    "jsondiffpatch": "^0.5.0",
+    "lucide": "^0.241.0",
+    "markdown-it": "^13.0.1",
+    "markdown-table": "^3.0.3",
+    "papaparse": "^5.4.1",
+    "react": "^18.2.0",
+    "react-dom": "^18.2.0",
+    "react-redux": "^8.0.4",
+    "react-virtuoso": "^4.6.0",
+    "recoil": "^0.7.7",
+    "runtypes": "^6.7.0",
+    "ts-node": "^10.9.1",
+    "uuid": "^9.0.0"
+  }
 }

package.json

@@ -18,4 +18,4 @@ export const EXTENSION_REGEX = new RegExp(/\.[a-z]*$/);
  * @example
  * [[my-file|]]
  */
-export const WIKI_LINK_REGEX = new RegExp(/\[\[([^|\]]+)(?:\|([\w-]+))?\]\]/g);
+export const WIKI_LINK_REGEX = new RegExp(/\[\[([^|\]]+)(\|([\w-]*))?\]\]/g);

src/data/constants.ts

@@ -1,5 +1,5 @@
 import { App, TFile } from "obsidian";
-import { splitFileExtension } from "./utils";
+import { EXTENSION_REGEX } from "./constants";
 
 export const createFolder = async (app: App, folderPath: string) => {
 	try {
@@ -36,3 +36,16 @@ export const createFile = async (
 		throw err;
 	}
 };
+
+export const splitFileExtension = (
+	filePath: string
+): [string, string] | null => {
+	if (filePath.match(EXTENSION_REGEX)) {
+		const periodIndex = filePath.lastIndexOf(".");
+		return [
+			filePath.substring(0, periodIndex),
+			filePath.substring(periodIndex),
+		];
+	}
+	return null;
+};

src/data/file-operations.ts

@@ -36,10 +36,10 @@ export const createLoomFile = async (
 		//This is a bug in Obsidian.
 		//TODO report this bug the Obsidian team
 		const formattedPath = removeLeadingPeriod(filePath);
-		const loomState = createLoomState(
+		const loomState = createLoomState(1, 1, {
 			pluginVersion,
-			defaultFrozenColumnCount
-		);
+			frozenColumnCount: defaultFrozenColumnCount,
+		});
 		const serializedState = serializeState(loomState);
 
 		const file = await createFile(app, formattedPath, serializedState);