Feature: sync links on file rename (foambubble#969)

* basic implementation of file rename support

* tweaks to various tests

* make lint happy again

* Improved reporting

* added setting related to file sync

* added documentation in readme

Author: riccardoferretti

assets/screenshots/feature-link-sync.gif

@@ -27,6 +27,12 @@ Foam helps you create the connections between your notes, and your placeholders
 
 ![Link Autocompletion](./assets/screenshots/feature-link-autocompletion.gif)
 
+### Sync links on file rename
+
+Foam updates the links to renamed files, so your notes stay consistent.
+
+![Sync links on file rename](./assets/screenshots/feature-link-sync.gif)
+
 ### Unique identifiers across directories
 
 Foam supports files with the same name in multiple directories.

packages/foam-vscode/README.md

@@ -255,6 +255,11 @@
             "Disable wikilink definitions generation"
           ]
         },
+        "foam.links.sync.enable": {
+          "description": "Enable synching links when moving/renaming notes",
+          "type": "boolean",
+          "default": true
+        },
         "foam.links.hover.enable": {
           "description": "Enable displaying note content on hover links",
           "type": "boolean",
@@ -418,7 +423,8 @@
     "tsdx": "^0.13.2",
     "tslib": "^2.0.0",
     "typescript": "^3.9.5",
-    "vscode-test": "^1.3.0"
+    "vscode-test": "^1.3.0",
+    "wait-for-expect": "^3.0.2"
   },
   "dependencies": {
     "dateformat": "^3.0.3",
@@ -440,4 +446,4 @@
     "unist-util-visit": "^2.0.2",
     "yaml": "^1.10.0"
   }
-}
+}

packages/foam-vscode/assets/screenshots/feature-link-sync.gif

@@ -71,5 +71,13 @@ describe('Foam URI', () => {
     expect(URI.file('/my/file.markdown').resolve('../hello')).toEqual(
       URI.file('/hello.markdown')
     );
+    expect(
+      URI.file('/path/to/a/note.md').resolve('../another-note.md')
+    ).toEqual(URI.file('/path/to/another-note.md'));
+    expect(
+      URI.file('/path/to/a/note.md').relativeTo(
+        URI.file('/path/to/another/note.md').getDirectory()
+      )
+    ).toEqual(URI.file('../a/note.md'));
   });
 });

packages/foam-vscode/package.json

@@ -167,4 +167,23 @@ describe('Identifier computation', () => {
     const identifier = FoamWorkspace.getShortestIdentifier(needle, haystack);
     expect(identifier).toEqual('project/car/todo');
   });
+
+  it('should ignore elements from the exclude list', () => {
+    const workspace = new FoamWorkspace();
+    const noteA = createTestNote({ uri: '/path/to/note-a.md' });
+    const noteB = createTestNote({ uri: '/path/to/note-b.md' });
+    const noteC = createTestNote({ uri: '/path/to/note-c.md' });
+    const noteD = createTestNote({ uri: '/path/to/note-d.md' });
+    const noteABis = createTestNote({ uri: '/path/to/another/note-a.md' });
+
+    workspace
+      .set(noteA)
+      .set(noteB)
+      .set(noteC)
+      .set(noteD);
+    expect(workspace.getIdentifier(noteABis.uri)).toEqual('another/note-a');
+    expect(
+      workspace.getIdentifier(noteABis.uri, [noteB.uri, noteA.uri])
+    ).toEqual('note-a');
+  });
 });

packages/foam-vscode/src/core/model/uri.test.ts

@@ -5,7 +5,6 @@ import { isSome } from '../utils';
 import { Emitter } from '../common/event';
 import { ResourceProvider } from './provider';
 import { IDisposable } from '../common/lifecycle';
-import { Logger } from '../utils/log';
 
 export class FoamWorkspace implements IDisposable {
   private onDidAddEmitter = new Emitter<Resource>();
@@ -83,17 +82,25 @@ export class FoamWorkspace implements IDisposable {
    *
    * @param forResource the resource to compute the identifier for
    */
-  public getIdentifier(forResource: URI): string {
+  public getIdentifier(forResource: URI, exclude?: URI[]): string {
     const amongst = [];
     const basename = forResource.getBasename();
     for (const res of this._resources.values()) {
-      // Just a quick optimization to only add the elements that might match
-      if (res.uri.path.endsWith(basename)) {
-        if (!res.uri.isEqual(forResource)) {
-          amongst.push(res.uri);
-        }
+      // skip elements that cannot possibly match
+      if (!res.uri.path.endsWith(basename)) {
+        continue;
+      }
+      // skip self
+      if (res.uri.isEqual(forResource)) {
+        continue;
       }
+      // skip exclude list
+      if (exclude && exclude.find(ex => ex.isEqual(res.uri))) {
+        continue;
+      }
+      amongst.push(res.uri);
     }
+
     let identifier = FoamWorkspace.getShortestIdentifier(
       forResource.path,
       amongst.map(uri => uri.path)

packages/foam-vscode/src/core/model/workspace.test.ts

@@ -96,7 +96,7 @@ describe('MarkdownLink', () => {
   });
 
   describe('rename wikilink', () => {
-    it.skip('should rename the target only', () => {
+    it('should rename the target only', () => {
       const link = parser.parse(
         getRandomURI(),
         `this is a [[wikilink#section]]`

packages/foam-vscode/src/core/model/workspace.ts

@@ -2,23 +2,23 @@ import { ResourceLink } from '../model/note';
 
 export abstract class MarkdownLink {
   private static wikilinkRegex = new RegExp(
-    /\[\[([^#\|]+)?#?([^\|]+)?\|?(.*)?\]\]/
+    /\[\[([^#|]+)?#?([^|]+)?\|?(.*)?\]\]/
   );
   private static directLinkRegex = new RegExp(
     /\[([^\]]+)\]\(([^#]*)?#?([^\]]+)?\)/
   );
 
   public static analyzeLink(link: ResourceLink) {
     if (link.type === 'wikilink') {
-      const [_, target, section, alias] = this.wikilinkRegex.exec(link.rawText);
+      const [, target, section, alias] = this.wikilinkRegex.exec(link.rawText);
       return {
         target: target?.replace(/\\/g, ''),
         section,
         alias,
       };
     }
     if (link.type === 'link') {
-      const [_, alias, target, section] = this.directLinkRegex.exec(
+      const [, alias, target, section] = this.directLinkRegex.exec(
         link.rawText
       );
       return { target, section, alias };

packages/foam-vscode/src/core/services/markdown-link.test.ts

@@ -1,5 +1,4 @@
 import { createMarkdownParser, ParserPlugin } from './markdown-parser';
-import { ResourceLink } from '../model/note';
 import { Logger } from '../utils/log';
 import { URI } from '../model/uri';
 import { Range } from '../model/range';