[stable34] fix(autosave): only save after changes in the editor

src/components/Editor.vue

@@ -296,6 +296,7 @@ export default defineComponent({
 
 		const { document, saveService } = provideSaveService(
 			connection,
+			editor,
 			syncService,
 			serialize,
 			ydoc,
@@ -618,8 +619,8 @@ export default defineComponent({
 			// Save and push unsaved changes from offline editing session.
 			Promise.all([this.whenSynced, this.editorReady]).then(() => {
 				if (this.dirty) {
-					// the update will trigger an autosave
 					this.syncProvider.sendUpdateFromDoc('offline', this.ydoc)
+					this.saveService.autosave()
 				}
 			})
 			this.updateUser(session)

src/composables/useSaveService.ts

@@ -3,6 +3,8 @@
  * SPDX-License-Identifier: AGPL-3.0-or-later
  */
 
+import type { Editor } from '@tiptap/core'
+import type { Transaction } from '@tiptap/pm/state'
 import type { InjectionKey, ShallowRef } from 'vue'
 import type { Doc } from 'yjs'
 import type { SaveData } from '../apis/save.ts'
@@ -18,13 +20,15 @@ const saveServiceKey = Symbol('text:save') as InjectionKey<SaveService>
 /**
  *
  * @param connection to the api
+ * @param editor the tiptap instance
  * @param syncService mostly used for the event bus and events
  * @param serialize to extract the document markdown content
  * @param ydoc to extract the document state from
  * @param setDirty set the dirty state for the editor
  */
 export function provideSaveService(
 	connection: ShallowRef<Connection | undefined>,
+	editor: Editor,
 	syncService: SyncService,
 	serialize: () => string,
 	ydoc: Doc,
@@ -49,11 +53,15 @@ export function provideSaveService(
 		getSaveData,
 	})
 
-	syncService.bus.on('changesPushed', saveService.autosave)
-	syncService.bus.on('close', saveService.clear)
-	onUnmounted(() => {
-		syncService.bus.off('changesPushed', saveService.autosave)
-		syncService.bus.off('close', saveService.clear)
+	const autosaveOnUpdate = ({ transaction }: { transaction: Transaction }) => {
+		if (transaction.getMeta('addToHistory') !== false) {
+			saveService.autosave()
+		}
+	}
+
+	editor.on('update', autosaveOnUpdate)
+	editor.on('destroy', () => {
+		editor.off('update', autosaveOnUpdate)
 	})
 
 	/**