bump

- Fixed the inexplicable behaviour when retrieving chunks from the network.
    - Chunk manager has been layered to responsible its own areas and duties. e.g., `DatabaseWriteLayer`, `DatabaseReadLayer`, `NetworkLayer`, `CacheLayer`,and `ArrivalWaitLayer`.
        - All layers have test now!
        - `LayeredChunkManager` has been implemented to manage these layers. Also tested.
    - `EntryManager` has been mostly rewritten, and also tested.

- Now we can configure `Never warn` for remote storage size notification, again.

devs.md

@@ -52,6 +52,7 @@ Hence, the new feature should be implemented as follows:
 ### Commands
 
 ```bash
+npm run test:unit    # Run unit tests with vitest (or `npm run test:unit:coverage` for coverage)
 npm run check        # TypeScript and svelte type checking
 npm run dev          # Development build with auto-rebuild (uses .env for test vault paths)
 npm run build        # Production build
@@ -67,8 +68,11 @@ npm test             # Run vitest tests (requires Docker services)
 
 ### Testing Infrastructure
 
-- **Deno Tests**: Unit tests for platform-independent code (e.g., `HashManager.test.ts`)
-- **Vitest** (`vitest.config.ts`): E2E test by Browser-based-harness using Playwright
+- ~~**Deno Tests**: Unit tests for platform-independent code (e.g., `HashManager.test.ts`)~~
+    - This is now obsolete, migrated to vitest.
+- **Vitest** (`vitest.config.ts`): E2E test by Browser-based-harness using Playwright, unit tests.
+    - Unit tests should be `*.unit.spec.ts` and placed alongside the implementation file (e.g., `ChunkFetcher.unit.spec.ts`).
+
 - **Docker Services**: Tests require CouchDB, MinIO (S3), and P2P services:
     ```bash
     npm run test:docker-all:start  # Start all test services

manifest.json

@@ -1,7 +1,7 @@
 {
     "id": "obsidian-livesync",
     "name": "Self-hosted LiveSync",
-    "version": "0.25.45",
+    "version": "0.25.47",
     "minAppVersion": "0.9.12",
     "description": "Community implementation of self-hosted livesync. Reflect your vault changes to some other devices immediately. Please make sure to disable other synchronize solutions to avoid content corruption or duplication.",
     "author": "vorotamoroz",

package.json

@@ -1,6 +1,6 @@
 {
     "name": "obsidian-livesync",
-    "version": "0.25.45",
+    "version": "0.25.47",
     "description": "Reflect your vault changes to some other devices immediately. Please make sure to disable other synchronize solutions to avoid content corruption or duplication.",
     "main": "main.js",
     "type": "module",

src/common/events.ts

@@ -24,7 +24,7 @@ export const EVENT_REQUEST_RUN_FIX_INCOMPLETE = "request-run-fix-incomplete";
 
 export const EVENT_ANALYSE_DB_USAGE = "analyse-db-usage";
 export const EVENT_REQUEST_PERFORM_GC_V3 = "request-perform-gc-v3";
-export const EVENT_REQUEST_CHECK_REMOTE_SIZE = "request-check-remote-size";
+// export const EVENT_REQUEST_CHECK_REMOTE_SIZE = "request-check-remote-size";
 // export const EVENT_FILE_CHANGED = "file-changed";
 
 declare global {
@@ -44,7 +44,6 @@ declare global {
         [EVENT_REQUEST_RUN_DOCTOR]: string;
         [EVENT_REQUEST_RUN_FIX_INCOMPLETE]: undefined;
         [EVENT_ANALYSE_DB_USAGE]: undefined;
-        [EVENT_REQUEST_CHECK_REMOTE_SIZE]: undefined;
         [EVENT_REQUEST_PERFORM_GC_V3]: undefined;
     }
 }

src/features/HiddenFileSync/CmdHiddenFileSync.ts

@@ -92,7 +92,7 @@ export class HiddenFileSync extends LiveSyncCommands {
         return this.plugin.kvDB;
     }
     getConflictedDoc(path: FilePathWithPrefix, rev: string) {
-        return this.plugin.managers.conflictManager.getConflictedDoc(path, rev);
+        return this.localDatabase.managers.conflictManager.getConflictedDoc(path, rev);
     }
     onunload() {
         this.periodicInternalFileScanProcessor?.disable();
@@ -244,13 +244,23 @@ export class HiddenFileSync extends LiveSyncCommands {
             if (this.isThisModuleEnabled()) {
                 //system file
                 const filename = this.getPath(doc);
-                if (await this.services.vault.isTargetFile(filename)) {
-                    // this.procInternalFile(filename);
-                    await this.processReplicationResult(doc);
+                const unprefixedPath = stripAllPrefixes(filename);
+                // No need to check via vaultService
+                // if (!await this.services.vault.isTargetFile(unprefixedPath)) {
+                //     this._log(`Skipped processing sync file:${unprefixedPath} (Not target)`, LOG_LEVEL_VERBOSE);
+                //     return true;
+                // }
+                if (!(await this.isTargetFile(stripAllPrefixes(unprefixedPath)))) {
+                    this._log(
+                        `Skipped processing sync file:${unprefixedPath} (Not Hidden File Sync target)`,
+                        LOG_LEVEL_VERBOSE
+                    );
+                    // We should return true, we made sure that document is a internalMetadata.
                     return true;
-                } else {
-                    this._log(`Skipped (Not target:${filename})`, LOG_LEVEL_VERBOSE);
-                    return false;
+                }
+                if (!(await this.processReplicationResult(doc))) {
+                    this._log(`Failed to process sync file:${unprefixedPath}`, LOG_LEVEL_NOTICE);
+                    // Do not yield false, this file had been processed.
                 }
             }
             return true;
@@ -700,7 +710,7 @@ Offline Changed files: ${processFiles.length}`;
                         revFrom._revs_info
                             ?.filter((e) => e.status == "available" && Number(e.rev.split("-")[0]) < conflictedRevNo)
                             .first()?.rev ?? "";
-                    const result = await this.plugin.managers.conflictManager.mergeObject(
+                    const result = await this.localDatabase.managers.conflictManager.mergeObject(
                         doc.path,
                         commonBase,
                         doc._rev,

src/main.ts

@@ -18,7 +18,7 @@ import type { IObsidianModule } from "./modules/AbstractObsidianModule.ts";
 import { ModuleDev } from "./modules/extras/ModuleDev.ts";
 import { ModuleMigration } from "./modules/essential/ModuleMigration.ts";
 
-import { ModuleCheckRemoteSize } from "./modules/essentialObsidian/ModuleCheckRemoteSize.ts";
+// import { ModuleCheckRemoteSize } from "./modules/essentialObsidian/ModuleCheckRemoteSize.ts";
 import { ModuleConflictResolver } from "./modules/coreFeatures/ModuleConflictResolver.ts";
 import { ModuleInteractiveConflictResolver } from "./modules/features/ModuleInteractiveConflictResolver.ts";
 import { ModuleLog } from "./modules/features/ModuleLog.ts";
@@ -64,6 +64,7 @@ import { onLayoutReadyFeatures } from "./serviceFeatures/onLayoutReady.ts";
 import type { ServiceModules } from "./types.ts";
 import { useTargetFilters } from "@lib/serviceFeatures/targetFilter.ts";
 import { setNoticeClass } from "@lib/mock_and_interop/wrapper.ts";
+import { useCheckRemoteSize } from "./lib/src/serviceFeatures/checkRemoteSize.ts";
 
 export default class ObsidianLiveSyncPlugin
     extends Plugin
@@ -177,7 +178,7 @@ export default class ObsidianLiveSyncPlugin
         this._registerModule(new ModuleRedFlag(this));
         this._registerModule(new ModuleInteractiveConflictResolver(this, this));
         this._registerModule(new ModuleObsidianGlobalHistory(this, this));
-        this._registerModule(new ModuleCheckRemoteSize(this));
+        // this._registerModule(new ModuleCheckRemoteSize(this));
         // Test and Dev Modules
         this._registerModule(new ModuleDev(this, this));
         this._registerModule(new ModuleReplicateTest(this, this));
@@ -238,13 +239,6 @@ export default class ObsidianLiveSyncPlugin
         return this.services.database.localDatabase;
     }
 
-    /**
-     * @obsolete Use services.database.managers instead. The database managers, including entry manager, revision manager, etc.
-     */
-    get managers() {
-        return this.services.database.managers;
-    }
-
     /**
      * @obsolete Use services.database.localDatabase instead. Get the PouchDB database instance. Note that this is not the same as the local database instance, which is a wrapper around the PouchDB database.
      * @returns The PouchDB database instance.
@@ -421,6 +415,7 @@ export default class ObsidianLiveSyncPlugin
         }
         // enable target filter feature.
         useTargetFilters(this);
+        useCheckRemoteSize(this);
     }
 
     constructor(app: App, manifest: PluginManifest) {

src/modules/features/ModuleSetupObsidian.ts

@@ -1,4 +1,4 @@
-import { type ObsidianLiveSyncSettings, LOG_LEVEL_NOTICE } from "../../lib/src/common/types.ts";
+import { type ObsidianLiveSyncSettings, LOG_LEVEL_NOTICE, LOG_LEVEL_VERBOSE } from "../../lib/src/common/types.ts";
 import { configURIBase } from "../../common/types.ts";
 // import { PouchDB } from "../../lib/src/pouchdb/pouchdb-browser.js";
 import { fireAndForget } from "../../lib/src/common/utils.ts";
@@ -25,16 +25,24 @@ export class ModuleSetupObsidian extends AbstractModule {
     private _setupManager!: SetupManager;
     private _everyOnload(): Promise<boolean> {
         this._setupManager = this.core.getModule(SetupManager);
-        this.registerObsidianProtocolHandler("setuplivesync", async (conf: any) => {
-            if (conf.settings) {
-                await this._setupManager.onUseSetupURI(
-                    UserMode.Unknown,
-                    `${configURIBase}${encodeURIComponent(conf.settings)}`
-                );
-            } else if (conf.settingsQR) {
-                await this._setupManager.decodeQR(conf.settingsQR);
-            }
-        });
+        try {
+            this.registerObsidianProtocolHandler("setuplivesync", async (conf: any) => {
+                if (conf.settings) {
+                    await this._setupManager.onUseSetupURI(
+                        UserMode.Unknown,
+                        `${configURIBase}${encodeURIComponent(conf.settings)}`
+                    );
+                } else if (conf.settingsQR) {
+                    await this._setupManager.decodeQR(conf.settingsQR);
+                }
+            });
+        } catch (e) {
+            this._log(
+                "Failed to register protocol handler. This feature may not work in some environments.",
+                LOG_LEVEL_NOTICE
+            );
+            this._log(e, LOG_LEVEL_VERBOSE);
+        }
         this.addCommand({
             id: "livesync-setting-qr",
             name: "Show settings as a QR code",

src/modules/services/ObsidianServiceHub.ts

@@ -59,6 +59,7 @@ export class ObsidianServiceHub extends InjectableServiceHub<ObsidianServiceCont
             path: path,
             vault: vault,
             setting: setting,
+            API: API,
         });
         const keyValueDB = new ObsidianKeyValueDBService(context, {
             appLifecycle: appLifecycle,
@@ -77,7 +78,6 @@ export class ObsidianServiceHub extends InjectableServiceHub<ObsidianServiceCont
         const replication = new ObsidianReplicationService(context, {
             APIService: API,
             appLifecycleService: appLifecycle,
-            databaseEventService: databaseEvents,
             replicatorService: replicator,
             settingService: setting,
             fileProcessingService: fileProcessing,

updates.md

@@ -3,6 +3,47 @@ Since 19th July, 2025 (beta1 in 0.25.0-beta1, 13th July, 2025)
 
 The head note of 0.25 is now in [updates_old.md](https://github.com/vrtmrz/obsidian-livesync/blob/main/updates_old.md). Because 0.25 got a lot of updates, thankfully, compatibility is kept and we do not need breaking changes! In other words, when get enough stabled. The next version will be v1.0.0. Even though it my hope.
 
+## 0.25.47
+
+27th February, 2026
+
+Phew, the financial year is still not over yet, but I have got some time to work on the plug-in again!
+
+### Fixed and refactored
+
+- Fixed the inexplicable behaviour when retrieving chunks from the network.
+    - The chunk manager has been layered to be responsible for its own areas and duties. e.g., `DatabaseWriteLayer`, `DatabaseReadLayer`, `NetworkLayer`, `CacheLayer`, and `ArrivalWaitLayer`.
+        - All layers have been tested now!
+        - `LayeredChunkManager` has been implemented to manage these layers. Also tested.
+    - `EntryManager` has been mostly rewritten and also tested.
+
+- Now we can configure `Never warn` for remote storage size notification again.
+
+### Tests
+
+- The following test has been added:
+    - `ConflictManager`.
+
+## 0.25.46
+
+26th February, 2026
+
+### Fixed
+
+- Unexpected errors no longer occurred when the plug-in was unloaded.
+- Hidden File Sync now respects selectors.
+- Registering protocol-handlers now works safely without causing unexpected errors.
+
+### Refactored
+
+- `ModuleCheckRemoteSize` has been ported to a serviceFeature, and tests have also been added.
+- Some unnecessary things have been removed.
+- LiveSyncManagers has now explicit dependencies.
+- LiveSyncLocalDB is now responsible for LiveSyncManagers, not accepting the managers as dependencies.
+    - This is to avoid circular dependencies and clarify the ownership of the managers.
+- ChangeManager has been refactored. This had a potential issue, so something had been fixed, possibly.
+- Some tests have been ported from Deno's test runner to Vitest to accumulate coverage.
+
 ## 0.25.45
 
 25th February, 2026
@@ -12,11 +53,10 @@ As a result of recent refactoring, we are able to write tests more easily now!
 ### Refactored
 
 - `ModuleTargetFilter`, which was responsible for checking if a file is a target file, has been ported to a serviceFeature.
-  - And also tests have been added. The middleware-style-power.
+    - And also tests have been added. The middleware-style-power.
 - `ModuleObsidianAPI` has been removed and implemented in `APIService` and `RemoteService`.
 - Now `APIService` is responsible for the network-online-status, not `databaseService.managers.networkManager`.
 
-
 ## 0.25.44
 
 24th February, 2026
@@ -34,7 +74,7 @@ However, as this update is very substantial, please do feel free to let me know
 ### Improved
 
 - Now we can configure network-error banners as icons, or hide them completely with the new `Network Warning Style` setting in the `General` pane of the settings dialogue. (#770, PR #804)
-  - Thanks so much to @A-wry!
+    - Thanks so much to @A-wry!
 
 ### Refactored
 
@@ -71,8 +111,8 @@ However, as this update is very substantial, please do feel free to let me know
 #### Dependencies:
 
 - Bumped dependencies simply to a point where they can be considered problem-free (by human-powered-artefacts-diff).
-  - Svelte, terser, and more something will be bumped later. They have a significant impact on the diff and paint it totally.
-  - You may be surprised, but when I bump the library, I am actually checking for any unintended code.
+    - Svelte, terser, and more something will be bumped later. They have a significant impact on the diff and paint it totally.
+    - You may be surprised, but when I bump the library, I am actually checking for any unintended code.
 
 ## 0.25.43
 

vitest.config.common.ts

@@ -89,7 +89,6 @@ export default defineConfig({
     ],
     resolve: {
         alias: {
-            obsidian: path.resolve(__dirname, "./test/harness/obsidian-mock.ts"),
             "@": path.resolve(__dirname, "./src"),
             "@lib": path.resolve(__dirname, "./src/lib/src"),
             src: path.resolve(__dirname, "./src"),

vitest.config.ts

@@ -1,6 +1,7 @@
 import { defineConfig, mergeConfig } from "vitest/config";
 import { playwright } from "@vitest/browser-playwright";
 import viteConfig from "./vitest.config.common";
+import path from "path";
 import dotenv from "dotenv";
 import { grantClipboardPermissions, openWebPeer, closeWebPeer, acceptWebPeer } from "./test/lib/commands";
 const defEnv = dotenv.config({ path: ".env" }).parsed;
@@ -12,6 +13,11 @@ const headless = !debuggerEnabled && !enableUI;
 export default mergeConfig(
     viteConfig,
     defineConfig({
+        resolve: {
+            alias: {
+                obsidian: path.resolve(__dirname, "./test/harness/obsidian-mock.ts"),
+            },
+        },
         test: {
             env: env,
             testTimeout: 40000,

vitest.config.unit.ts

@@ -1,16 +1,30 @@
 import { defineConfig, mergeConfig } from "vitest/config";
 import viteConfig from "./vitest.config.common";
 
+const importOnlyFiles = ["**/encryption/encryptHKDF.ts"];
 export default mergeConfig(
     viteConfig,
     defineConfig({
+        resolve: {
+            alias: {
+                obsidian: "", // prevent accidental imports of obsidian types in unit tests,
+            },
+        },
         test: {
             name: "unit-tests",
             include: ["**/*unit.test.ts", "**/*.unit.spec.ts"],
             exclude: ["test/**"],
             coverage: {
                 include: ["src/**/*.ts"],
-                exclude: ["**/*.test.ts", "src/lib/**/*.test.ts", "**/_*", "src/lib/apps", "src/lib/src/cli"],
+                exclude: [
+                    "**/*.test.ts",
+                    "src/lib/**/*.test.ts",
+                    "**/_*",
+                    "src/lib/apps",
+                    "src/lib/src/cli",
+                    "**/*_obsolete.ts",
+                    ...importOnlyFiles,
+                ],
                 provider: "v8",
                 reporter: ["text", "json", "html"],
             },