feat: [email protected]

BREAKING CHANGE:

- Before, change events (incl. add, update, remove) have only been triggered when using the custom APIs like `.add()` or `.update()`. Now they get always triggered, including for changes replicated into the database. hoodiehq/pouchdb-hoodie-api@1958c42

- the order of when the methods’ promises resolve and the events get triggered cannot be guaranteed as we rely on PouchDB’s .changes(). We would love to enforce promises to resolve after changes get emitted, but the required complexity to do that is not worth it.

- separate methods like `require(pouchdb-hoodie-api/add)` can no longer be required directly

  The reason for that is that all the methods now also accept a `prefix` argument which by default is null. That way we don’t need to implement each method twice, once for `store.add`, and once for `store.withIdPrefix(test/).add`

- We no longer map PouchDB’s `._id` property to `.id`, instead we pass trough docs from PouchDB 1:1. Also the timestamps are now all namespaced with `.hoodie` (`doc.createdAt` becomes `doc.hoodie.createdAt`)

Author: gr2m

index.js

@@ -5,8 +5,6 @@ var EventEmitter = require('events').EventEmitter
 var merge = require('lodash/merge')
 
 var subscribeToSyncEvents = require('./lib/subscribe-to-sync-events')
-var syncWrapper = require('./lib/sync-wrapper')
-var scoped = require('./lib/scoped/')
 var isPersistent = require('./lib/is-persistent')
 
 function Store (dbName, options) {
@@ -40,7 +38,6 @@ function Store (dbName, options) {
 
   var state = {
     objectTypeById: {},
-    scopedApis: {},
     db: db
   }
 
@@ -52,7 +49,6 @@ function Store (dbName, options) {
   })
 
   var api = merge(
-    scoped.bind(null, state, storeApi),
     {
       db: storeApi.db,
       add: storeApi.add,
@@ -66,12 +62,13 @@ function Store (dbName, options) {
       removeAll: storeApi.removeAll,
       on: storeApi.on,
       one: storeApi.one,
-      off: storeApi.off
+      off: storeApi.off,
+      withIdPrefix: storeApi.withIdPrefix
     },
     {
-      push: syncWrapper.bind(syncApi, 'push'),
-      pull: syncWrapper.bind(syncApi, 'pull'),
-      sync: syncWrapper.bind(syncApi, 'sync'),
+      push: syncApi.push,
+      pull: syncApi.pull,
+      sync: syncApi.sync,
       connect: syncApi.connect,
       disconnect: syncApi.disconnect,
       isConnected: syncApi.isConnected,
@@ -93,7 +90,7 @@ Store.defaults = function (defaultOpts) {
 
     options = merge({}, defaultOpts, options)
 
-    return Store(dbName, options)
+    return new Store(dbName, options)
   }
 
   return CustomStore

lib/reset.js

@@ -3,8 +3,6 @@ module.exports = reset
 var merge = require('lodash/merge')
 
 var subscribeToSyncEvents = require('./subscribe-to-sync-events')
-var syncWrapper = require('./sync-wrapper')
-var scoped = require('./scoped/')
 
 function reset (dbName, CustomPouchDB, state, api, clear, emitter, remoteBaseUrl, remote, options) {
   if (options) {
@@ -34,7 +32,6 @@ function reset (dbName, CustomPouchDB, state, api, clear, emitter, remoteBaseUrl
 
     merge(
       api,
-      scoped.bind(null, state, storeApi),
       {
         db: storeApi.db,
         add: storeApi.add,
@@ -48,12 +45,13 @@ function reset (dbName, CustomPouchDB, state, api, clear, emitter, remoteBaseUrl
         removeAll: storeApi.removeAll,
         on: storeApi.on,
         one: storeApi.one,
-        off: storeApi.off
+        off: storeApi.off,
+        withIdPrefix: storeApi.storeApi
       },
       {
-        push: syncWrapper.bind(syncApi, 'push'),
-        pull: syncWrapper.bind(syncApi, 'pull'),
-        sync: syncWrapper.bind(syncApi, 'sync'),
+        push: syncApi.push,
+        pull: syncApi.pull,
+        sync: syncApi.sync,
         connect: syncApi.connect,
         disconnect: syncApi.disconnect,
         isConnected: syncApi.isConnected