feat: :set persistence, jump list, buffer dedup, gu/gU, Ctrl-o/Ctrl-i (#17)

* chore: add changeset for vim native batch2

* chore: release version bump

* fix(renderer-vecto): deduplicate buffers on file reopen

openBuffer now checks if a buffer with the same label already exists
before creating a new one, preventing the same file from appearing
in multiple tabs when mouse-clicked repeatedly.

* feat(core): :set persistence, jump list, buffer dedup, gu/gU parser

Author: Xuepoo

packages/core/src/editor.ts

@@ -218,8 +218,12 @@ export class VemEditorState {
   private scrollToLineCallbacks: ((line: number) => void)[] = [];
   private openFileUnderCursorCallbacks: ((path: string) => void)[] = [];
   private windowActionCallbacks: ((action: string) => void)[] = [];
+  private optionsChangedCallbacks: ((key: string, value: string) => void)[] = [];
   /** The cursor position BEFORE the last change (for g;/g,). */
   private changeList: Position[] = [];
+  /** Jump list (Ctrl-o/Ctrl-i): oldest entries at index 0. */
+  private jumpList: Position[] = [];
+  private jumpListIndex: number = -1;
   /** Number of visible lines in the viewport, set by the renderer (for H/M/L). */
   public visibleLines: number = 0;
   /** Current scroll line offset, set by the renderer (for H/M/L). */
@@ -345,6 +349,14 @@ export class VemEditorState {
     this.openFileUnderCursorCallbacks.push(callback);
   }
 
+  public onOptionsChanged(callback: (key: string, value: string) => void): void {
+    this.optionsChangedCallbacks.push(callback);
+  }
+
+  private triggerOptionsChanged(key: string, value: string): void {
+    for (const cb of this.optionsChangedCallbacks) cb(key, value);
+  }
+
   private triggerScrollToLine(line: number): void {
     for (const cb of this.scrollToLineCallbacks) {
       cb(line);
@@ -1451,6 +1463,14 @@ export class VemEditorState {
           this.addNumber(-cmd.count);
           break;
 
+        // --- <C-o>/<C-i>: jump list navigation ---
+        case '<C-o>':
+          this.goBackInJumplist();
+          break;
+        case '<C-i>':
+          this.goForwardInJumplist();
+          break;
+
         // --- m{a-zA-Z}: set mark ---
         case 'm':
           if (cmd.mark) {
@@ -1854,6 +1874,35 @@ export class VemEditorState {
     }
   }
 
+  /** Record a cursor position in the jump list. Called by moveCursorByMotion. */
+  public pushJumpList(): void {
+    this.jumpList.push({ ...this.cursor });
+    if (this.jumpList.length > 100) this.jumpList.shift();
+    this.jumpListIndex = this.jumpList.length - 1;
+  }
+
+  /** `<C-o>`: go to the previous (older) position in the jump list. */
+  private goBackInJumplist(): void {
+    if (this.jumpListIndex > 0 && this.jumpList.length > 1) {
+      this.jumpListIndex--;
+      const pos = this.jumpList[this.jumpListIndex];
+      this.cursor.line = pos.line;
+      this.cursor.character = pos.character;
+      this.desiredCol = pos.character;
+    }
+  }
+
+  /** `<C-i>`: go to the next (newer) position in the jump list. */
+  private goForwardInJumplist(): void {
+    if (this.jumpListIndex < this.jumpList.length - 1) {
+      this.jumpListIndex++;
+      const pos = this.jumpList[this.jumpListIndex];
+      this.cursor.line = pos.line;
+      this.cursor.character = pos.character;
+      this.desiredCol = pos.character;
+    }
+  }
+
   /** `gJ`: join lines without inserting a space. */
   private joinLinesNoSpace(count: number): void {
     const start = this.cursor.line;
@@ -1893,7 +1942,7 @@ export class VemEditorState {
     }
   }
 
-  private executeSetOption(option: string): void {
+  public executeSetOption(option: string): void {
     // Vim's real syntax for OS-clipboard integration: `:set clipboard=unnamed`
     // (or `unnamedplus`) routes y/d/c/x/p/P through the system clipboard
     // instead of just the internal `"` register; `:set clipboard=` reverts.
@@ -1935,14 +1984,18 @@ export class VemEditorState {
 
     if (option === 'relativenumber' || option === 'rnu') {
       this.layoutConfig = { ...this.layoutConfig, lineNumbers: 'relative' };
+      this.triggerOptionsChanged('relativenumber', 'relative');
     } else if (option === 'norelativenumber' || option === 'nornu') {
       // Falls back to absolute if numbers are on, else stays off.
       const next = this.layoutConfig.lineNumbers === 'relative' ? 'absolute' : 'none';
       this.layoutConfig = { ...this.layoutConfig, lineNumbers: next };
+      this.triggerOptionsChanged('number', next);
     } else if (option === 'number' || option === 'nu') {
       this.layoutConfig = { ...this.layoutConfig, lineNumbers: 'absolute' };
+      this.triggerOptionsChanged('number', 'absolute');
     } else if (option === 'nonumber' || option === 'nonu') {
       this.layoutConfig = { ...this.layoutConfig, lineNumbers: 'none' };
+      this.triggerOptionsChanged('number', 'none');
     } else {
       this.statusMessage = `E518: Unknown option: ${option}`;
     }
@@ -1976,6 +2029,7 @@ export class VemEditorState {
    * aborts rather than leaving the cursor somewhere half-moved.
    */
   private moveCursorByMotion(motion: string, count: number, findChar?: string): boolean {
+    this.pushJumpList();
     if (motion === 'f' || motion === 'F' || motion === 't' || motion === 'T') {
       return this.moveByFindChar(motion, count, findChar);
     }

packages/core/src/parser.ts

@@ -94,6 +94,8 @@ const normalCommands = [
   '<C-y>',
   '<C-a>',
   '<C-x>',
+  '<C-o>',
+  '<C-i>',
   'Escape',
 ];
 
@@ -105,6 +107,10 @@ const gPrefixCommands: Record<string, string> = {
   i: 'gi', // go to last insert position
   ';': 'g;', // go to previous change
   ',': 'g,', // go to next change
+  u: 'gu', // make lowercase
+  U: 'gU', // make uppercase
+  q: 'gq', // format text
+  J: 'gJ', // join without space
 };
 
 const zPrefixCommands: Record<string, string> = {

packages/renderer-vecto/src/Workspace.ts

@@ -76,6 +76,13 @@ export class VemWorkspace extends UIComponent {
    * so the caller can associate it with a file handle for save-back.
    */
   public openBuffer(text: string, label = 'untitled'): string {
+    if (label !== 'untitled') {
+      const existing = this.buffers.find((b) => b.label === label);
+      if (existing) {
+        this.syncTabs(existing.id);
+        return existing.id;
+      }
+    }
     const entry = this.makeBuffer(text, label);
     this.syncTabs(entry.id);
     return entry.id;