Fixing bug on collection deletion

Extending Admin Interface
Adding cleanup procedure, that clears undeleted collection data

package-lock.json

@@ -1,6 +1,6 @@
 {
   "name": "@hibas123/realtimedb",
-  "version": "2.0.0-beta.0",
+  "version": "2.0.1-0",
   "lockfileVersion": 1,
   "requires": true,
   "dependencies": {

package.json

@@ -1,8 +1,9 @@
 {
   "name": "@hibas123/realtimedb",
-  "version": "2.0.0-beta.0",
+  "version": "2.0.0-beta.4",
   "description": "",
   "main": "lib/index.js",
+  "private": true,
   "scripts": {
     "start": "node lib/index.js",
     "build": "tsc",

src/connection.ts

@@ -2,7 +2,7 @@ import * as WebSocket from "ws";
 import { Server, IncomingMessage } from "http";
 import { DatabaseManager } from "./database/database";
 import Logging from "@hibas123/logging";
-import Query from "./database/query";
+import { Query, CollectionQuery, DocumentQuery } from "./database/query";
 import Session from "./database/session";
 import nanoid = require("nanoid");
 
@@ -100,99 +100,41 @@ export class ConnectionManager {
       const handler = new Map<string, ((data: any) => void)>();
       type QueryData = { id: string, type: QueryTypes, path: string[], data: any, options: any };
 
-      const queryHandler = new Map<string, (query: Query, permissions: { read: boolean, write: boolean }, data?: any) => Promise<any>>();
-
-      const noperm = new Error("No permisison!");
-
-      if (session.root) {
-         queryHandler.set("collections", (query, perm, data) => {
-            return query.getCollections();
-         })
-
-         queryHandler.set("delete-collection", (query, perm, data) => {
-            return query.deleteCollection();
-         })
-      }
-
-      queryHandler.set("keys", (query, perm, data) => {
-         if (!perm.read)
-            throw noperm;
-         return query.keys();
-      })
-
-      queryHandler.set("get", (query, perm, data) => {
-         if (!perm.read)
-            throw noperm;
-         return query.get();
-      })
-
-      queryHandler.set("set", (query, perm, data) => {
-         if (!perm.write)
-            throw noperm;
-         return query.set(data, {});
-      })
-
-      queryHandler.set("update", (query, perm, data) => {
-         if (!perm.write)
-            throw noperm;
-         return query.update(data);
-      })
-
-      queryHandler.set("push", (query, perm, data) => {
-         if (!perm.write)
-            throw noperm;
-         return query.push(data);
-      })
-
-      queryHandler.set("delete", (query, perm, data) => {
-         if (!perm.write)
-            throw noperm;
-         return query.delete();
-      })
-
-      queryHandler.set("subscribe", async (query, perm, data) => {
-         if (!perm.read)
-            throw noperm;
-
-         let subscriptionID = nanoid();
-
-         query.subscribe((data) => {
-            socket.send(JSON.stringify({ ns: "event", data: { id: subscriptionID, data } }));
-         }, data.types, data.options);
-
-         return StoreQuery(subscriptionID);
-      })
-
-      queryHandler.set("unsubscribe", async (query, perm, data) => {
-         query.unsubscribe();
-         return DeleteQuery(true);
-      })
-
       handler.set("query", async ({ id, type, path, data }: QueryData) => {
          //TODO: Handle case with no id, type, path
-         Logging.debug(`Request with id '${id}' from type '${type}' and path '${path}' with data`, data)
+         Logging.debug(`Request with id '${id}' from type '${type}' and path '${path.join("/")}' with data`, data)
 
          try {
             if (!db)
                throw new Error("Database not found!");
             else {
-               let handler = queryHandler.get(type);
-               if (!handler) {
-                  throw new Error("Invalid Request!");
-               } else {
-                  const query = stored.get(id) || db.getQuery(path || [], session.sessionid);
-                  const perms = db.rules.hasPermission(path, session);
+               let isDoc = path.length % 2 === 0;
+               let handler = isDoc ? documentHandler.get(type) : collectionHandler.get(type);
+
+               if (!handler && session.root)
+                  handler = rootHandler.get(type);
+
+               if (!handler)
+                  throw new Error("Invalid Request!");
+
+               let query = db.getQuery(path || [], session.sessionid, isDoc ? "document" : "collection");
+               let res = await handler({
+                  id,
+                  data,
+                  socket,
+                  query: query as any // We know it is the right one
+               })
 
-                  let res = await handler(query, perms, data);
                if (res && typeof res === "object" && res[StoreSym] !== undefined) {
                   if (res[StoreSym])
                      stored.set(id, query);
                   else
                      stored.delete(id);
+
+                  res = res.result;
                }
                answer(id, res);
             }
-            }
          } catch (err) {
             // Logging.error(err);
             Logging.debug("Sending error:", err);
@@ -213,10 +155,103 @@ export class ConnectionManager {
          }
       })
 
-      socket.on("disconnect", () => {
-         stored.forEach(query => query.unsubscribe());
+      socket.on("close", () => {
+         Logging.log(`${session.sessionid} has disconnected!`);
+         Logging.debug("Clearing stored:", stored);
+         stored.forEach(query => (query as DocumentQuery | CollectionQuery).unsubscribe());
          stored.clear();
          socket.removeAllListeners();
       })
    }
 }
+
+type QueryHandler<T extends Query> = (api: {
+   id: string;
+   query: T;
+   // storedQuery(id: string): T  | undefined;   
+   socket: WebSocket;
+   data: any;
+}) => any;
+
+const NoPermissionError = new Error("No permisison!");
+
+const rootHandler = new Map<string, QueryHandler<Query>>();
+rootHandler.set("collections", ({ query }) => {
+   return CollectionQuery.fromQuery(query).collections();
+})
+
+rootHandler.set("delete-collection", ({ query }) => {
+   return CollectionQuery.fromQuery(query).deleteCollection();
+})
+
+
+const documentHandler = new Map<string, QueryHandler<DocumentQuery>>();
+documentHandler.set("get", ({ query }) => {
+   return query.get();
+})
+
+documentHandler.set("set", ({ query, data }) => {
+   return query.set(data, {});
+})
+
+documentHandler.set("update", ({ query, data }) => {
+   return query.update(data);
+})
+
+documentHandler.set("delete", ({ query }) => {
+   return query.delete();
+})
+
+documentHandler.set("snapshot", async ({ query, data, id, socket }) => {
+   let res = await query.snapshot((data) => {
+      socket.send(JSON.stringify({
+         ns: "snapshot", data: { id, data }
+      }));
+   });
+
+   return StoreQuery(res);
+})
+
+documentHandler.set("unsubscribe", async ({ query }) => {
+   query.unsubscribe();
+   return DeleteQuery(true);
+})
+
+
+const collectionHandler = new Map<string, QueryHandler<CollectionQuery>>();
+collectionHandler.set("keys", ({ query }) => {
+   return query.keys();
+})
+
+
+collectionHandler.set("add", ({ query, data }) => {
+   return query.add(data);
+})
+
+collectionHandler.set("get", ({ query, data }) => {
+   if (data.where)
+      query.where = data.where;
+   if (data.limit)
+      query.limit = data.limit;
+   return query.get();
+})
+
+collectionHandler.set("snapshot", async ({ query, id, socket, data }) => {
+   if (data.where)
+      query.where = data.where;
+   if (data.limit)
+      query.limit = data.limit;
+
+   let res = await query.snapshot((data) => {
+      socket.send(JSON.stringify({
+         ns: "snapshot", data: { id, data }
+      }));
+   });
+
+   return StoreQuery(res);
+})
+
+collectionHandler.set("unsubscribe", async ({ query }) => {
+   query.unsubscribe();
+   return DeleteQuery(true);
+})

src/database/database.ts

@@ -2,7 +2,8 @@ import { Rules } from "./rules";
 import Settings from "../settings";
 import getLevelDB, { LevelDB, deleteLevelDB } from "../storage";
 import DocumentLock from "./lock";
-import Query from "./query";
+import { DocumentQuery, CollectionQuery, Query } from "./query";
+import Logging from "@hibas123/nodelogging";
 
 export class DatabaseManager {
    static databases = new Map<string, Database>();
@@ -104,11 +105,117 @@ export class Database {
    }
 
 
-   getQuery(path: string[], sender: string) {
+   getQuery(path: string[], sender: string, type: "document" | "collection" | "any") {
+      if (type === "document")
+         return new DocumentQuery(this, path, sender);
+      else if (type === "collection")
+         return new CollectionQuery(this, path, sender);
+      else
          return new Query(this, path, sender);
    }
 
    async stop() {
       await this.data.close();
    }
+
+   public async runCleanup() {
+      const should = await new Promise<Set<string>>((yes, no) => {
+         const stream = this.collections.iterator({
+            keyAsBuffer: false,
+            valueAsBuffer: false
+         })
+
+         const collections = new Set<string>();
+         const onValue = (err: Error, key: string, value: string) => {
+            if (err) {
+               Logging.error(err);
+               stream.end((err) => Logging.error(err))
+               no(err);
+            }
+
+            if (!key && !value) {
+               yes(collections);
+            } else {
+               collections.add(value)
+               stream.next(onValue);
+            }
+         }
+
+         stream.next(onValue);
+      })
+
+      const existing = await new Promise<Set<string>>((yes, no) => {
+         const stream = this.data.iterator({
+            keyAsBuffer: false,
+            values: false
+         })
+
+         const collections = new Set<string>();
+         const onValue = (err: Error, key: string, value: Buffer) => {
+            if (err) {
+               Logging.error(err);
+               stream.end((err) => Logging.error(err))
+               no(err);
+            }
+
+            if (!key && !value) {
+               yes(collections);
+            } else {
+               let coll = key.split("/")[0];
+               collections.add(coll)
+               stream.next(onValue);
+            }
+         }
+
+         stream.next(onValue);
+      })
+
+      const toDelete = new Set<string>();
+      existing.forEach(collection => {
+         if (!should.has(collection))
+            toDelete.add(collection);
+      })
+
+      for (let collection of toDelete) {
+         const batch = this.data.batch();
+
+         let gt = Buffer.from(collection + "/ ");
+         gt[gt.length - 1] = 0;
+
+         let lt = Buffer.alloc(gt.length);
+         lt.set(gt);
+         lt[gt.length - 1] = 0xFF;
+
+         await new Promise<void>((yes, no) => {
+            const stream = this.data.iterator({
+               keyAsBuffer: false,
+               values: false,
+               gt,
+               lt
+            })
+
+            const onValue = (err: Error, key: string, value: Buffer) => {
+               if (err) {
+                  Logging.error(err);
+                  stream.end((err) => Logging.error(err))
+                  no(err);
+               }
+
+               if (!key && !value) {
+                  yes();
+               } else {
+                  batch.del(key);
+                  stream.next(onValue);
+               }
+            }
+
+            stream.next(onValue);
+         })
+
+         await batch.write();
+      }
+
+      return Array.from(toDelete.values());
+   }
+
 }

src/database/lock.ts

@@ -19,7 +19,7 @@ export default class DocumentLock {
 
       return () => {
          if (l.length > 0)
-            setImmediate(() => l.shift());
+            setImmediate(() => l.shift()());
          else
             this.locks.delete(key)
       }

src/database/query.ts

@@ -14,7 +14,7 @@ interface ISubscribeOptions {
    existing: boolean;
 }
 
-export default class Query {
+export class Query {
    /**
     * Returns true if the path only contains valid characters and false if it doesn't
     * @param path Path to be checked
@@ -23,19 +23,17 @@ export default class Query {
       return path.every(e => (e.match(/[^a-zA-Z0-9_\-\<\>]/g) || []).length === 0);
    }
 
-   constructor(private database: Database, private path: string[], private sender: string) {
+   constructor(protected database: Database, protected path: string[], protected sender: string) {
       if (path.length > 10) {
          throw new Error("Path is to long. Path is only allowed to be 10 Layers deep!");
       }
       if (!this.validatePath(path)) {
          throw new Error("Path can only contain a-z A-Z 0-9  '-' '-' '<' and '>' ");
       }
-
-      this.onChange = this.onChange.bind(this);
    }
 
 
-   private async resolve(path: string[], create = false): Promise<{ collection: string, document: string, collectionKey: string }> {
+   protected async resolve(path: string[], create = false): Promise<{ collection: string, document: string, collectionKey: string }> {
       path = [...path]; // Create modifiable copy
       let collectionID: string = undefined;
       let documentKey = path.length % 2 === 0 ? path.pop() : undefined;
@@ -60,15 +58,17 @@ export default class Query {
       };
    }
 
-   private getKey(collection: string, document?: string) {
+   protected getKey(collection: string, document?: string) {
       return `${collection || ""}/${document || ""}`;
    }
 
-   private getDoc(collection: string, document: string) {
-      return this.database.data.get(this.getKey(collection, document), { asBuffer: true }).then(res => decode(res as Buffer)).catch(resNull);
+   protected getDoc(collection: string, document: string) {
+      return this.database.data
+         .get(this.getKey(collection, document), { asBuffer: true })
+         .then(res => decode<any>(res as Buffer)).catch(resNull);
    }
 
-   private sendChange(collection: string, document: string, type: ChangeTypes, data: any) {
+   protected sendChange(collection: string, document: string, type: ChangeTypes, data: any) {
       let change: Change = {
          type,
          document,
@@ -77,6 +77,7 @@ export default class Query {
       }
 
       let s = this.database.changes.get(this.getKey(collection, document))
+
       if (s)
          s.forEach(e => setImmediate(() => e(change)))
       s = this.database.changes.get(this.getKey(collection))
@@ -85,46 +86,31 @@ export default class Query {
 
    }
 
+   protected static getConstructorParams(query: Query): [Database, string[], string] {
+      return [query.database, query.path, query.sender];
+   }
+}
+
+interface UpdateData {
+   [path: string]: {
+      type: "value" | "timestamp" | "increment" | "push";
+      value: any;
+   }
+}
+export class DocumentQuery extends Query {
+   constructor(database: Database, path: string[], sender: string) {
+      super(database, path, sender);
+      this.onChange = this.onChange.bind(this);
+   }
+
    public async get() {
       let { collection, document } = await this.resolve(this.path);
 
       if (!collection || !document) {
          return null;
       }
-      const lock = await this.database.locks.lock(collection, document);
-      return this.getDoc(collection, document).finally(() => lock())
-   }
 
-   public async keys() {
-      let { collection, document } = await this.resolve(this.path);
-      if (document)
-         throw new Error("Keys only works on collections!");
-      if (!collection)
-         throw new Error("There must be a collection");
-
-
-      let gt = Buffer.from(this.getKey(collection) + " ");
-      gt[gt.length - 1] = 0;
-
-      let lt = Buffer.alloc(gt.length);
-      lt.set(gt);
-      lt[gt.length - 1] = 0xFF;
-
-      return new Promise<string[]>((yes, no) => {
-         let keys = [];
-         const stream = this.database.data.createKeyStream({
-            gt,
-            lt,
-            keyAsBuffer: false
-         })
-         stream.on("data", (key: string) => {
-            let s = key.split("/", 2);
-            if (s.length > 1)
-               keys.push(s[1]);
-         });
-         stream.on("end", () => yes(keys));
-         stream.on("error", no);
-      });
+      return this.getDoc(collection, document)
    }
 
    public async set(data: any, { merge = false }) {
@@ -141,7 +127,7 @@ export default class Query {
 
       const lock = await this.database.locks.lock(collection, document);
 
-      let isNew = await this.database.data.get(this.getKey(collection, document)).catch(resNull).then(e => e !== null);
+      let isNew = !(await this.getDoc(collection, document))
 
       return this.database.data
          .put(this.getKey(collection, document), encode(data))
@@ -149,11 +135,79 @@ export default class Query {
          .finally(() => lock())
    }
 
-   public async push(value: any) {
-      let id = nanoid(ALPHABET, 32);
-      let q = new Query(this.database, [...this.path, id], this.sender);
-      await q.set(value, {});
-      return id;
+   public async update(updateData: UpdateData) {
+      let { collection, document } = await this.resolve(this.path, true);
+      if (!collection) {
+         throw new Error("There must be a collection!")
+      }
+
+      if (!document) {
+         throw new Error("There must be a document key!")
+      }
+
+      // Logging.debug(updateData);
+
+      const lock = await this.database.locks.lock(collection, document);
+      try {
+         let data = await this.getDoc(collection, document);
+         let isNew = false
+         if (!data) {
+            isNew = true;
+            data = {};
+         }
+
+         for (let path in updateData) {
+            const toUpdate = updateData[path];
+            let d = data;
+            let parts = path.split(".");
+            while (parts.length > 1) {
+               let seg = parts.shift();
+               if (!data[seg])
+                  data[seg] = {}
+               d = data[seg];
+            }
+
+            const last = parts[0];
+
+            // Logging.debug(parts, last, d)
+
+            switch (toUpdate.type) {
+               case "value":
+                  d[last] = toUpdate.value;
+                  break;
+               case "increment":
+                  if (d[last] === undefined || d[last] === null)
+                     d[last] = toUpdate.value;
+                  else if (typeof d[last] !== "number") {
+                     throw new Error("Field is no number!");
+                  } else {
+                     d[last] += toUpdate.value;
+                  }
+                  break;
+               case "timestamp":
+                  d[last] = new Date().valueOf();
+                  break;
+               case "push":
+                  if (d[last] === undefined || d[last] === null)
+                     d[last] = [toUpdate.value];
+                  else if (Array.isArray(d[last])) {
+                     d[last].push(toUpdate.value);
+                  } else {
+                     throw new Error("Field is not array!");
+                  }
+                  break;
+               default:
+                  throw new Error("Invalid update type: " + toUpdate.type);
+            }
+         }
+
+         this.database.data
+            .put(this.getKey(collection, document), encode(data))
+            .then(() => this.sendChange(collection, document, isNew ? "added" : "modified", data))
+      } finally {
+         lock();
+      }
+      //TODO: Implement
    }
 
    public async delete() {
@@ -167,7 +221,6 @@ export default class Query {
          throw new Error("There must be a document key!")
       }
 
-
       const lock = await this.database.locks.lock(collection, document);
 
       return await this.database.data
@@ -176,11 +229,294 @@ export default class Query {
          .finally(() => lock())
    }
 
-   public async update(data: any) {
-      //TODO: Implement
+
+
+   private subscription: {
+      key: string,
+      onChange: (change: DocRes & { type: ChangeTypes }) => void
+   };
+
+   async snapshot(onChange: (change: DocRes & { type: ChangeTypes }) => void) {
+      if (this.subscription)
+         throw new Error("This query is already subscribed!");
+      let { collection, document } = await this.resolve(this.path);
+
+      let data = await this.getDoc(collection, document);
+      let key = this.getKey(collection, document);
+      this.subscription = {
+         key,
+         onChange
+      }
+      let s = this.database.changes.get(key);
+      if (!s) {
+         s = new Set();
+         this.database.changes.set(key, s);
       }
 
-   public async getCollections() {
+      s.add(this.onChange);
+
+      return data;
+   }
+
+   onChange(change: Change) {
+      // if(change.sender === this.sender)
+      //    return
+      this.subscription.onChange({
+         id: change.document,
+         data: change.data,
+         type: change.type
+      })
+   }
+
+   unsubscribe() {
+      if (!this.subscription)
+         return;
+      let s = this.database.changes.get(this.subscription.key);
+      s.delete(this.onChange);
+      if (s.size <= 0)
+         this.database.changes.delete(this.subscription.key);
+
+      this.subscription = undefined;
+   }
+
+   public static fromQuery(query: Query) {
+      return new DocumentQuery(...Query.getConstructorParams(query));
+   }
+}
+
+type FieldPath = string;
+type WhereFilterOp =
+   | '<'
+   | '<='
+   | '=='
+   | '>='
+   | '>'
+   | 'array-contains'
+   | 'in'
+   | 'array-contains-any';
+
+interface IQueryWhere {
+   fieldPath: FieldPath,
+   opStr: WhereFilterOp,
+   value: any
+}
+
+
+interface DocRes {
+   id: string;
+   data: any;
+}
+
+export class CollectionQuery extends Query {
+   constructor(database: Database, path: string[], sender: string) {
+      super(database, path, sender);
+      this.onChange = this.onChange.bind(this);
+   }
+
+
+   public where: IQueryWhere[] = [];
+   public limit: number = -1;
+
+   public async add(value: any) {
+      let id = nanoid(ALPHABET, 32);
+      let q = new DocumentQuery(this.database, [...this.path, id], this.sender);
+      await q.set(value, {});
+      return id;
+   }
+
+   private getStreamOptions(collection: string) {
+      let gt = Buffer.from(this.getKey(collection) + " ");
+      gt[gt.length - 1] = 0;
+
+      let lt = Buffer.alloc(gt.length);
+      lt.set(gt);
+      lt[gt.length - 1] = 0xFF;
+
+      return {
+         gt,
+         lt
+      }
+   }
+
+   public async keys() {
+      let { collection, document } = await this.resolve(this.path);
+      if (document)
+         throw new Error("Keys only works on collections!");
+      if (!collection)
+         throw new Error("There must be a collection");
+
+      return new Promise<string[]>((yes, no) => {
+         let keys = [];
+         const stream = this.database.data.createKeyStream({
+            ...this.getStreamOptions(collection),
+            keyAsBuffer: false
+         })
+         stream.on("data", (key: string) => {
+            let s = key.split("/", 2);
+            if (s.length > 1)
+               keys.push(s[1]);
+         });
+         stream.on("end", () => yes(keys));
+         stream.on("error", no);
+      });
+   }
+
+   private getFieldValue(data: any, path: FieldPath) {
+      let parts = path.split(".");
+      let d = data;
+      while (parts.length > 0) {
+         let seg = parts.shift();
+
+         d = data[seg];
+         if (d === undefined || d === null)
+            break; // Undefined/Null has no other fields!
+      }
+      return d;
+   }
+
+   private fitsWhere(data: any): boolean {
+      if (this.where.length > 0) {
+         return this.where.every(where => {
+            let val = this.getFieldValue(data, where.fieldPath);
+            switch (where.opStr) {
+               case "<":
+                  return val < where.value;
+               case "<=":
+                  return val <= where.value;
+               case "==":
+                  return val == where.value;
+               case ">=":
+                  return val >= where.value;
+               case ">":
+                  return val > where.value;
+               case "array-contains":
+                  if (Array.isArray(val)) {
+                     return val.some(e => e === where.value);
+                  }
+
+                  break;
+               // case "array-contains-any":
+               // case "in":
+
+               default:
+                  throw new Error("Invalid where operation " + where.opStr);
+            }
+         })
+      }
+      return true;
+   }
+
+   async get() {
+      let { collection, document } = await this.resolve(this.path);
+      if (document)
+         throw new Error("Keys only works on collections!");
+      if (!collection)
+         throw new Error("There must be a collection");
+      return new Promise<DocRes[]>((yes, no) => {
+         const stream = this.database.data.iterator({
+            ...this.getStreamOptions(collection),
+            keyAsBuffer: false,
+            valueAsBuffer: true
+         })
+
+         let values: DocRes[] = [];
+
+         const onValue = (err: Error, key: string, value: Buffer) => {
+            if (err) {
+               no(err);
+               stream.end(err => Logging.error(err));
+            }
+            else {
+               if (!key && !value) {
+                  // END
+                  Logging.debug("Checked all!")
+                  yes(values);
+               } else {
+                  let s = key.split("/", 2);
+                  if (s.length <= 1)
+                     return;
+
+                  const id = s[1];
+
+                  let data = decode(value);
+                  if (this.fitsWhere(data)) {
+                     Logging.debug("Found fitting")
+                     if (this.limit < 0 || value.length < this.limit) {
+                        values.push({
+                           id,
+                           data
+                        });
+
+                     }
+                     else {
+                        stream.end((err) => err ? no(err) : yes(values))
+                        return;
+                     }
+                  }
+
+                  stream.next(onValue);
+               }
+            }
+         }
+
+         stream.next(onValue)
+      })
+   }
+
+   private subscription: {
+      key: string,
+      onChange: (change: (DocRes & { type: ChangeTypes })[]) => void
+   };
+
+   async snapshot(onChange: (change: (DocRes & { type: ChangeTypes })[]) => void) {
+      if (this.subscription)
+         throw new Error("This query is already subscribed!");
+      let { collection, document } = await this.resolve(this.path);
+
+      let data = await this.get();
+
+      let key = this.getKey(collection, document);
+      this.subscription = {
+         key,
+         onChange
+      }
+      let s = this.database.changes.get(key);
+      if (!s) {
+         s = new Set();
+         this.database.changes.set(key, s);
+      }
+
+      s.add(this.onChange);
+
+      return data;
+   }
+
+   onChange(change: Change) {
+      // if(change.sender === this.sender)
+      //    return
+
+      if (this.fitsWhere(change.data)) {
+         this.subscription.onChange([{
+            id: change.document,
+            data: change.data,
+            type: change.type
+         }])
+      }
+   }
+
+   unsubscribe() {
+      if (!this.subscription)
+         return;
+      let s = this.database.changes.get(this.subscription.key);
+      s.delete(this.onChange);
+      if (s.size <= 0)
+         this.database.changes.delete(this.subscription.key);
+
+      this.subscription = undefined;
+   }
+
+
+   public async collections() {
       return new Promise<string[]>((yes, no) => {
          let keys = [];
          const stream = this.database.data.createKeyStream({ keyAsBuffer: false })
@@ -197,12 +533,15 @@ export default class Query {
          throw new Error("There can be no document defined on this operation");
       }
 
+      //TODO: Lock whole collection!
+
       let batch = this.database.data.batch();
       try {
          if (collection) {
-            let keys = await this.keys();
-            for (let key in keys) {
-               batch.del(key);
+            let documents = await this.keys();
+            // Logging.debug("To delete:", documents)
+            for (let document of documents) {
+               batch.del(this.getKey(collection, document));
             }
             await batch.write();
             batch = undefined;
@@ -214,80 +553,7 @@ export default class Query {
       }
    }
 
-
-
-   private subscription: {
-      key: string;
-      type: Set<ChangeTypes>;
-      send: (data: Omit<Change, "sender">) => void;
-   };
-   async subscribe(send: (data: Omit<Change, "sender">) => void, type: ChangeTypes[] | undefined, _options: Partial<ISubscribeOptions>) {
-      let options: ISubscribeOptions = {
-         existing: true,
-         ..._options
-      }
-
-      let { collection, document } = await this.resolve(this.path);
-
-      let key = this.getKey(collection, document);
-      let s = this.database.changes.get(key) || new Set();
-      s.add(this.onChange)
-      this.database.changes.set(key, s);
-
-      this.subscription = {
-         key,
-         send,
-         type: new Set(type || [])
-      };
-
-      Logging.debug("Existing?", options.existing)
-
-      if (options.existing) {
-         if (document) {
-            send({
-               document: document,
-               type: "added",
-               data: await this.get(),
-            })
-         } else {
-            await this.keys().then(async documents => {
-               for (let document of documents) {
-                  const data = await this.getDoc(collection, document);
-                  if (data)
-                     send({
-                        type: "added",
-                        document,
-                        data
-                     })
-               }
-            })
-         }
-      }
-
-      Logging.debug("Subscribed");
-   }
-
-   async onChange(change: Change) {
-      const { type, send } = this.subscription;
-
-      // Events from the sender are handled locally
-      if (change.sender !== this.sender && type.has(change.type)) {
-         let c = { ...change };
-         delete c.sender;
-         send(c)
-      }
-   }
-
-   unsubscribe() {
-      if (!this.subscription) {
-         const { key } = this.subscription;
-         let s = this.database.changes.get(key);
-         if (s) {
-            s.delete(this.onChange);
-            if (s.size <= 0)
-               this.database.changes.delete(key);
-         }
-         this.subscription = undefined;
-      }
+   public static fromQuery(query: Query) {
+      return new CollectionQuery(...Query.getConstructorParams(query));
    }
 }

src/web/helper/form.ts

@@ -1,4 +1,4 @@
-import getTemplate from "./hb";
+import { getTemplate } from "./hb";
 import { Context } from "vm";
 
 interface IFormConfigField {

src/web/helper/hb.ts

@@ -37,8 +37,22 @@ Handlebars.registerHelper('ifCond', function (v1, operator, v2, options) {
 
 
 const cache = new Map<string, Handlebars.TemplateDelegate>();
+const htmlCache = new Map<string, string>();
 
-export default function getTemplate(name: string) {
+export function getView(name: string) {
+   let tl: string;
+   if (!config.dev)
+      tl = htmlCache.get(name);
+
+   if (!tl) {
+      tl = readFileSync(`./views/${name}.html`).toString();
+      htmlCache.set(name, tl);
+   }
+
+   return tl;
+}
+
+export function getTemplate(name: string) {
    let tl: Handlebars.TemplateDelegate;
    if (!config.dev)
       tl = cache.get(name);

src/web/helper/table.ts

@@ -1,5 +1,5 @@
 import { Context } from "koa";
-import getTemplate from "./hb";
+import { getTemplate } from "./hb";
 
 export default function getTable(title: string, data: any[], ctx: Context) {
    let table: string[][] = [];

src/web/v1/admin.ts

@@ -7,6 +7,7 @@ import { DatabaseManager } from "../../database/database";
 import { MP } from "../../database/query";
 import config from "../../config";
 import Logging from "@hibas123/logging";
+import { getView } from "../helper/hb";
 
 const AdminRoute = new Router();
 
@@ -17,6 +18,11 @@ AdminRoute.use(async (ctx, next) => {
    return next();
 })
 
+AdminRoute.get("/", async ctx => {
+   //TODO: Main Interface
+   ctx.body = getView("admin");
+});
+
 AdminRoute.get("/settings", async ctx => {
    let res = await new Promise<string[][]>((yes, no) => {
       const stream = Settings.db.createReadStream({
@@ -51,11 +57,11 @@ AdminRoute.get("/data", async ctx => {
          keys: true,
          values: true,
          valueAsBuffer: true,
-         keyAsBuffer: false
+         keyAsBuffer: false,
+         limit: 1000
       });
       let res = [["key", "value"]];
       stream.on("data", ({ key, value }: { key: string, value: Buffer }) => {
-         Logging.debug("Admin Key:", key);
          res.push([key, key.split("/").length > 2 ? value.toString() : JSON.stringify(MP.decode(value))]);
       })
 
@@ -114,6 +120,48 @@ AdminRoute
       ctx.body = "Success";
    })
 
+AdminRoute.get("/collections", async ctx => {
+   const { database } = ctx.query;
+   let db = DatabaseManager.getDatabase(database);
+   if (!db)
+      throw new BadRequestError("Database not found");
+
+   let res = await new Promise<string[]>((yes, no) => {
+
+      const stream = db.collections.createKeyStream({
+         keyAsBuffer: false,
+         limit: 1000
+      });
+      let res = [];
+      stream.on("data", (key: string) => {
+         res.push(key);
+      })
+
+      stream.on("error", no);
+      stream.on("end", () => yes(res))
+   })
+
+   if (ctx.query.view) {
+      return getTable("Databases", res, ctx);
+   } else {
+      ctx.body = res;
+   }
+})
+
+AdminRoute.get("/collections/cleanup", async ctx => {
+   const { database } = ctx.query;
+   let db = DatabaseManager.getDatabase(database);
+   if (!db)
+      throw new BadRequestError("Database not found");
+
+   let deleted = await db.runCleanup();
+   if (ctx.query.view) {
+      return getTable("Databases", deleted, ctx);
+   } else {
+      ctx.body = deleted;
+   }
+})
+
 AdminRoute.get("/database/new", getForm("/v1/admin/database", "New/Change Database", {
    name: { label: "Name", type: "text", },
    accesskey: { label: "Access Key", type: "text" },

views/admin.html

@@ -0,0 +1,112 @@
+<!DOCTYPE html>
+<html lang="en">
+
+<head>
+   <meta charset="UTF-8">
+   <meta name="viewport" content="width=device-width, initial-scale=1.0">
+   <meta http-equiv="X-UA-Compatible" content="ie=edge">
+   <title>Admin Interface</title>
+   <link rel="stylesheet" href="https://unpkg.com/@hibas123/theme/out/base.css">
+   <link rel="stylesheet" href="https://unpkg.com/@hibas123/theme/out/light.css">
+
+   <script src="https://unpkg.com/handlebars/dist/handlebars.min.js"></script>
+
+   <style>
+      #message {
+         visibility: hidden;
+         background-color: lightgreen;
+         border: 1px solid lime;
+         border-radius: .5rem;
+         padding: 1rem;
+         font-size: 1.5rem;
+         margin-bottom: 1rem;
+      }
+
+      .grid {
+         display: grid;
+         height: 100vh;
+         grid-template-columns: 360px auto;
+      }
+
+      #content {
+         position: absolute;
+         top: 0;
+         left: 0;
+         width: 100%;
+         height: 100%;
+         border: 0;
+      }
+   </style>
+</head>
+
+<body>
+   <div class="grid">
+      <div style="border-right: 1px solid darkgrey; padding: 1rem;">
+         <h2>Navigation: </h2>
+         <ul class="list list-clickable">
+            <li onclick="loadView('settings');">Settings</li>
+            <li onclick="loadView('database', {full:true});">Databases</li>
+            <li onclick="loadView('database/new');">New Database</li>
+         </ul>
+         Databases:
+         <div id="dbs" class="list list-clickable" style="margin: 1rem;"></div>
+
+      </div>
+      <div style="position:relative;">
+         <iframe id="content"></iframe>
+      </div>
+   </div>
+
+   <template>
+
+   </template>
+
+   <script>
+      const key = new URL(window.location.href).searchParams.get("key");
+      const content = document.getElementById("content");
+      const base = new URL(window.location.href).host;
+
+      function getUrl(name, params, view = true) {
+         const url = new URL(window.location.href);
+         url.pathname = "/v1/admin/" + name;
+         for (let key in params || {})
+            url.searchParams.set(key, params[key]);
+
+         url.searchParams.set("key", key);
+         if (view)
+            url.searchParams.set("view", "true");
+
+         return url.href;
+      }
+
+      function loadView(name, params) {
+         content.src = getUrl(name, params);
+      }
+
+      loadView("settings")
+
+      const dbsul = document.getElementById("dbs");
+      function reloadDBs() {
+         fetch(getUrl("database", {}, false))
+            .then(res => res.json())
+            .then(databases => databases.map(database => `
+<div class="card margin elv-4">
+   <h3>${database}</h3>
+   <button class=btn onclick="loadView('data', {database:'${database}'})">Data</button>
+   <button class=btn onclick="loadView('collections', {database:'${database}'})">Collections</button>
+   <button class=btn onclick="loadView('collections/cleanup', {database:'${database}'})">Clean</button>
+</div>`
+            ))
+            .then(d => d.join("\n"))
+            .then(d => dbsul.innerHTML = d)
+            .catch(console.error)
+
+
+      }
+
+      reloadDBs();
+      setInterval(reloadDBs, 5000);
+   </script>
+</body>
+
+</html>