From bb48ebe643877d995e7cd254949f2fd123046945 Mon Sep 17 00:00:00 2001
From: Mikhail Zolotukhin <mail@genda.life>
Date: Thu, 21 Oct 2021 20:44:44 +0300
Subject: [PATCH] refactor: interface for window store

---
 src/engine/index.test.ts   |  2 +-
 src/engine/index.ts        |  6 +--
 src/engine/window_store.ts | 92 +++++++++++++++++++++++++++++---------
 3 files changed, 74 insertions(+), 26 deletions(-)

diff --git a/src/engine/index.test.ts b/src/engine/index.test.ts
index 8fb9dcd4..11d85a50 100644
--- a/src/engine/index.test.ts
+++ b/src/engine/index.test.ts
@@ -16,7 +16,7 @@ import { Rect } from "../util/rect";
 import TileLayout from "./layout/tile_layout";
 import LayoutStore from "./layout_store";
 import { EngineWindow, WindowState } from "./window";
-import WindowStore from "./window_store";
+import { WindowStore } from "./window_store";
 
 describe("arrange", () => {
   it("happens on all screens", () => {
diff --git a/src/engine/index.ts b/src/engine/index.ts
index e0634e3d..047cae93 100644
--- a/src/engine/index.ts
+++ b/src/engine/index.ts
@@ -6,7 +6,7 @@
 import MonocleLayout from "./layout/monocle_layout";
 
 import LayoutStore from "./layout_store";
-import WindowStore from "./window_store";
+import { WindowStore, WindowStoreImpl } from "./window_store";
 import { EngineWindow, EngineWindowImpl, WindowState } from "./window";
 
 import { Controller } from "../controller";
@@ -177,7 +177,7 @@ export class EngineImpl implements Engine {
     private log: Log
   ) {
     this.layouts = new LayoutStore(this.config);
-    this.windows = new WindowStore();
+    this.windows = new WindowStoreImpl();
   }
 
   public adjustLayout(basis: EngineWindow): void {
@@ -557,7 +557,7 @@ export class EngineImpl implements Engine {
   }
 
   public setMaster(window: EngineWindow): void {
-    this.windows.setMaster(window);
+    this.windows.putWindowToMaster(window);
   }
 
   public cycleLayout(step: Step): void {
diff --git a/src/engine/window_store.ts b/src/engine/window_store.ts
index a99fbfed..e562521e 100644
--- a/src/engine/window_store.ts
+++ b/src/engine/window_store.ts
@@ -7,12 +7,74 @@ import { EngineWindow } from "./window";
 
 import { DriverSurface } from "../driver/surface";
 
-export default class WindowStore {
-  public list: EngineWindow[];
+/**
+ * Window storage facility with convenient window filters built-in.
+ */
+export interface WindowStore {
+  /**
+   * Returns all visible windows on the given surface.
+   */
+  visibleWindowsOn(surf: DriverSurface): EngineWindow[];
 
-  constructor(windows?: EngineWindow[]) {
-    this.list = windows || [];
-  }
+  /**
+   * Return all visible "Tile" windows on the given surface.
+   */
+  visibleTiledWindowsOn(surf: DriverSurface): EngineWindow[];
+
+  /**
+   * Return all visible "tileable" windows on the given surface
+   * @see Window#tileable
+   */
+  visibleTileableWindowsOn(surf: DriverSurface): EngineWindow[];
+
+  /**
+   * Return all "tileable" windows on the given surface, including hidden
+   */
+  tileableWindowsOn(surf: DriverSurface): EngineWindow[];
+
+  /**
+   * Return all windows on this surface, including minimized windows
+   */
+  allWindowsOn(surf: DriverSurface): EngineWindow[];
+
+  /**
+   * Inserts the window at the beginning
+   */
+  unshift(window: EngineWindow): void;
+
+  /**
+   * Inserts the window at the end
+   */
+  push(window: EngineWindow): void;
+
+  /**
+   * Remove window from the store
+   */
+  remove(window: EngineWindow): void;
+
+  /**
+   * Move srcWin to the destWin position (before/after)
+   * @param after if true, srcWin is moved after the destWindow. If false - it is moved before.
+   */
+  move(srcWin: EngineWindow, destWin: EngineWindow, after?: boolean): void;
+
+  /**
+   * Swap windows positions
+   */
+  swap(alpha: EngineWindow, beta: EngineWindow): void;
+
+  /**
+   * Put the window into the master area.
+   * @param window window to put into the master area
+   */
+  putWindowToMaster(window: EngineWindow): void;
+}
+
+export class WindowStoreImpl implements WindowStore {
+  /**
+   * @param list window list to initialize from
+   */
+  constructor(public list: EngineWindow[] = []) {}
 
   public move(
     srcWin: EngineWindow,
@@ -25,11 +87,13 @@ export default class WindowStore {
       return;
     }
 
+    // Delete the source window
     this.list.splice(srcIdx, 1);
+    // Place the source window in before destination window or after it
     this.list.splice(after ? destIdx + 1 : destIdx, 0, srcWin);
   }
 
-  public setMaster(window: EngineWindow): void {
+  public putWindowToMaster(window: EngineWindow): void {
     const idx = this.list.indexOf(window);
     if (idx === -1) {
       return;
@@ -76,40 +140,24 @@ export default class WindowStore {
     this.list.unshift(window);
   }
 
-  /**
-   * Returns all visible windows on the given surface.
-   */
   public visibleWindowsOn(surf: DriverSurface): EngineWindow[] {
     return this.list.filter((win) => win.visibleOn(surf));
   }
 
-  /**
-   * Return all visible "Tile" windows on the given surface.
-   */
   public visibleTiledWindowsOn(surf: DriverSurface): EngineWindow[] {
     return this.list.filter((win) => win.tiled && win.visibleOn(surf));
   }
 
-  /**
-   * Return all visible "tileable" windows on the given surface
-   * @see Window#tileable
-   */
   public visibleTileableWindowsOn(surf: DriverSurface): EngineWindow[] {
     return this.list.filter((win) => win.tileable && win.visibleOn(surf));
   }
 
-  /**
-   * Return all "tileable" windows on the given surface, including hidden
-   */
   public tileableWindowsOn(surf: DriverSurface): EngineWindow[] {
     return this.list.filter(
       (win) => win.tileable && win.surface.id === surf.id
     );
   }
 
-  /**
-   * Return all windows on this surface, including minimized windows
-   */
   public allWindowsOn(surf: DriverSurface): EngineWindow[] {
     return this.list.filter((win) => win.surface.id === surf.id);
   }