fix: patchdb should not bundle rxjs but instead rely on the version in the app

This commit is contained in:
waterplea
2022-07-24 16:58:04 +03:00
committed by Matt Hill
parent ec35e5f14c
commit c7006e0eb2
3 changed files with 57 additions and 38 deletions

View File

@@ -11,10 +11,10 @@ export interface StashEntry {
export class Store<T extends { [key: string]: any }> { export class Store<T extends { [key: string]: any }> {
cache: DBCache<T> cache: DBCache<T>
sequence$: BehaviorSubject<number> sequence$: BehaviorSubject<number>
private watchedNodes: { [path: string]: ReplaySubject<any> } = {} private watchedNodes: { [path: string]: ReplaySubject<any> } = { }
private stash = new BTree<number, StashEntry>() private stash = new BTree<number, StashEntry>()
constructor( constructor (
private readonly http: Http<T>, private readonly http: Http<T>,
private readonly initialCache: DBCache<T>, private readonly initialCache: DBCache<T>,
) { ) {
@@ -22,14 +22,14 @@ export class Store<T extends { [key: string]: any }> {
this.sequence$ = new BehaviorSubject(initialCache.sequence) this.sequence$ = new BehaviorSubject(initialCache.sequence)
} }
watch$(): Observable<T> watch$ (): Observable<T>
watch$<P1 extends keyof T>(p1: P1): Observable<NonNullable<T[P1]>> watch$<P1 extends keyof T> (p1: P1): Observable<NonNullable<T[P1]>>
watch$<P1 extends keyof T, P2 extends keyof NonNullable<T[P1]>>(p1: P1, p2: P2): Observable<NonNullable<NonNullable<T[P1]>[P2]>> watch$<P1 extends keyof T, P2 extends keyof NonNullable<T[P1]>> (p1: P1, p2: P2): Observable<NonNullable<NonNullable<T[P1]>[P2]>>
watch$<P1 extends keyof T, P2 extends keyof NonNullable<T[P1]>, P3 extends keyof NonNullable<NonNullable<T[P1]>[P2]>>(p1: P1, p2: P2, p3: P3): Observable<NonNullable<NonNullable<NonNullable<T[P1]>[P2]>[P3]>> watch$<P1 extends keyof T, P2 extends keyof NonNullable<T[P1]>, P3 extends keyof NonNullable<NonNullable<T[P1]>[P2]>> (p1: P1, p2: P2, p3: P3): Observable<NonNullable<NonNullable<NonNullable<T[P1]>[P2]>[P3]>>
watch$<P1 extends keyof T, P2 extends keyof NonNullable<T[P1]>, P3 extends keyof NonNullable<NonNullable<T[P1]>[P2]>, P4 extends keyof NonNullable<NonNullable<NonNullable<T[P1]>[P2]>[P3]>>(p1: P1, p2: P2, p3: P3, p4: P4): Observable<NonNullable<NonNullable<NonNullable<NonNullable<T[P1]>[P2]>[P3]>[P4]>> watch$<P1 extends keyof T, P2 extends keyof NonNullable<T[P1]>, P3 extends keyof NonNullable<NonNullable<T[P1]>[P2]>, P4 extends keyof NonNullable<NonNullable<NonNullable<T[P1]>[P2]>[P3]>> (p1: P1, p2: P2, p3: P3, p4: P4): Observable<NonNullable<NonNullable<NonNullable<NonNullable<T[P1]>[P2]>[P3]>[P4]>>
watch$<P1 extends keyof T, P2 extends keyof NonNullable<T[P1]>, P3 extends keyof NonNullable<NonNullable<T[P1]>[P2]>, P4 extends keyof NonNullable<NonNullable<NonNullable<T[P1]>[P2]>[P3]>, P5 extends keyof NonNullable<NonNullable<NonNullable<NonNullable<T[P1]>[P2]>[P3]>[P4]>>(p1: P1, p2: P2, p3: P3, p4: P4, p5: P5): Observable<NonNullable<NonNullable<NonNullable<NonNullable<NonNullable<T[P1]>[P2]>[P3]>[P4]>[P5]>> watch$<P1 extends keyof T, P2 extends keyof NonNullable<T[P1]>, P3 extends keyof NonNullable<NonNullable<T[P1]>[P2]>, P4 extends keyof NonNullable<NonNullable<NonNullable<T[P1]>[P2]>[P3]>, P5 extends keyof NonNullable<NonNullable<NonNullable<NonNullable<T[P1]>[P2]>[P3]>[P4]>> (p1: P1, p2: P2, p3: P3, p4: P4, p5: P5): Observable<NonNullable<NonNullable<NonNullable<NonNullable<NonNullable<T[P1]>[P2]>[P3]>[P4]>[P5]>>
watch$<P1 extends keyof T, P2 extends keyof NonNullable<T[P1]>, P3 extends keyof NonNullable<NonNullable<T[P1]>[P2]>, P4 extends keyof NonNullable<NonNullable<NonNullable<T[P1]>[P2]>[P3]>, P5 extends keyof NonNullable<NonNullable<NonNullable<NonNullable<T[P1]>[P2]>[P3]>[P4]>, P6 extends keyof NonNullable<NonNullable<NonNullable<NonNullable<NonNullable<T[P1]>[P2]>[P3]>[P4]>[P5]>>(p1: P1, p2: P2, p3: P3, p4: P4, p5: P5, p6: P6): Observable<NonNullable<NonNullable<NonNullable<NonNullable<NonNullable<NonNullable<T[P1]>[P2]>[P3]>[P4]>[P5]>[P6]>> watch$<P1 extends keyof T, P2 extends keyof NonNullable<T[P1]>, P3 extends keyof NonNullable<NonNullable<T[P1]>[P2]>, P4 extends keyof NonNullable<NonNullable<NonNullable<T[P1]>[P2]>[P3]>, P5 extends keyof NonNullable<NonNullable<NonNullable<NonNullable<T[P1]>[P2]>[P3]>[P4]>, P6 extends keyof NonNullable<NonNullable<NonNullable<NonNullable<NonNullable<T[P1]>[P2]>[P3]>[P4]>[P5]>> (p1: P1, p2: P2, p3: P3, p4: P4, p5: P5, p6: P6): Observable<NonNullable<NonNullable<NonNullable<NonNullable<NonNullable<NonNullable<T[P1]>[P2]>[P3]>[P4]>[P5]>[P6]>>
watch$(...args: (string | number)[]): Observable<any> { watch$ (...args: (string | number)[]): Observable<any> {
const path = `/${args.join('/')}` const path = `/${args.join('/')}`
if (!this.watchedNodes[path]) { if (!this.watchedNodes[path]) {
this.watchedNodes[path] = new ReplaySubject(1) this.watchedNodes[path] = new ReplaySubject(1)
@@ -38,7 +38,7 @@ export class Store<T extends { [key: string]: any }> {
return this.watchedNodes[path].asObservable() return this.watchedNodes[path].asObservable()
} }
update(update: Update<T>): void { update (update: Update<T>): void {
if (this.isRevision(update)) { if (this.isRevision(update)) {
// if old or known, return // if old or known, return
if (update.id <= this.cache.sequence || this.stash.get(update.id)) return if (update.id <= this.cache.sequence || this.stash.get(update.id)) return
@@ -48,24 +48,24 @@ export class Store<T extends { [key: string]: any }> {
} }
} }
reset(): void { reset (): void {
Object.values(this.watchedNodes).forEach(node => node.complete()) Object.values(this.watchedNodes).forEach(node => node.complete())
this.watchedNodes = {} this.watchedNodes = { }
this.stash.clear() this.stash.clear()
this.sequence$.next(0) this.sequence$.next(0)
this.cache = { this.cache = {
sequence: 0, sequence: 0,
data: {} as any, data: { } as any,
} }
} }
private updateValue(path: string): void { private updateValue (path: string): void {
const value = getValueByPointer(this.cache.data, path) const value = getValueByPointer(this.cache.data, path)
this.watchedNodes[path].next(value) this.watchedNodes[path].next(value)
} }
private handleRevision(revision: Revision): void { private handleRevision (revision: Revision): void {
// stash the revision // stash the revision
this.stash.set(revision.id, { revision, undo: [] }) this.stash.set(revision.id, { revision, undo: [] })
@@ -77,7 +77,7 @@ export class Store<T extends { [key: string]: any }> {
this.processStashed(revision.id) this.processStashed(revision.id)
} }
private handleDump({ value, id }: Dump<T>): void { private handleDump ({ value, id }: Dump<T>): void {
this.cache.data = { ...value } this.cache.data = { ...value }
this.stash.deleteRange(this.cache.sequence, id, false) this.stash.deleteRange(this.cache.sequence, id, false)
this.updateWatchedNodes('') this.updateWatchedNodes('')
@@ -85,12 +85,12 @@ export class Store<T extends { [key: string]: any }> {
this.processStashed(id + 1) this.processStashed(id + 1)
} }
private processStashed(id: number): void { private processStashed (id: number): void {
this.undoRevisions(id) this.undoRevisions(id)
this.applyRevisions(id) this.applyRevisions(id)
} }
private undoRevisions(id: number): void { private undoRevisions (id: number): void {
let stashEntry = this.stash.get(this.stash.maxKey() as number) let stashEntry = this.stash.get(this.stash.maxKey() as number)
while (stashEntry && stashEntry.revision.id > id) { while (stashEntry && stashEntry.revision.id > id) {
@@ -99,7 +99,7 @@ export class Store<T extends { [key: string]: any }> {
} }
} }
private applyRevisions(id: number): void { private applyRevisions (id: number): void {
let revision = this.stash.get(id)?.revision let revision = this.stash.get(id)?.revision
while (revision) { while (revision) {
let undo: Operation<unknown>[] = [] let undo: Operation<unknown>[] = []
@@ -136,7 +136,7 @@ export class Store<T extends { [key: string]: any }> {
this.stash.deleteRange(0, this.cache.sequence, false) this.stash.deleteRange(0, this.cache.sequence, false)
} }
private updateWatchedNodes(revisionPath: string) { private updateWatchedNodes (revisionPath: string) {
const kill = (path: string) => { const kill = (path: string) => {
this.watchedNodes[path].complete() this.watchedNodes[path].complete()
delete this.watchedNodes[path] delete this.watchedNodes[path]
@@ -151,12 +151,12 @@ export class Store<T extends { [key: string]: any }> {
}) })
} }
private updateSequence(sequence: number): void { private updateSequence (sequence: number): void {
this.cache.sequence = sequence this.cache.sequence = sequence
this.sequence$.next(sequence) this.sequence$.next(sequence)
} }
private isRevision(update: Update<T>): update is Revision { private isRevision (update: Update<T>): update is Revision {
return 'patch' in update return 'patch' in update
} }
} }

View File

@@ -9,7 +9,6 @@
"version": "1.0.0", "version": "1.0.0",
"license": "MIT", "license": "MIT",
"dependencies": { "dependencies": {
"rxjs": "6.6.7",
"sorted-btree": "1.5.0", "sorted-btree": "1.5.0",
"uuid": "8.3.2" "uuid": "8.3.2"
}, },
@@ -19,6 +18,9 @@
"ts-node": "10.2.0", "ts-node": "10.2.0",
"tslint": "6.1.3", "tslint": "6.1.3",
"typescript": "4.3.5" "typescript": "4.3.5"
},
"peerDependencies": {
"rxjs": "^7.5.6"
} }
}, },
"node_modules/@babel/code-frame": { "node_modules/@babel/code-frame": {
@@ -438,16 +440,20 @@
} }
}, },
"node_modules/rxjs": { "node_modules/rxjs": {
"version": "6.6.7", "version": "7.5.6",
"resolved": "https://registry.npmjs.org/rxjs/-/rxjs-6.6.7.tgz", "resolved": "https://registry.npmjs.org/rxjs/-/rxjs-7.5.6.tgz",
"integrity": "sha512-hTdwr+7yYNIT5n4AMYp85KA6yw2Va0FLa3Rguvbpa4W3I5xynaBZo41cM3XM+4Q6fRMj3sBYIR1VAmZMXYJvRQ==", "integrity": "sha512-dnyv2/YsXhnm461G+R/Pe5bWP41Nm6LBXEYWI6eiFP4fiwx6WRI/CD0zbdVAudd9xwLEF2IDcKXLHit0FYjUzw==",
"peer": true,
"dependencies": { "dependencies": {
"tslib": "^1.9.0" "tslib": "^2.1.0"
},
"engines": {
"npm": ">=2.0.0"
} }
}, },
"node_modules/rxjs/node_modules/tslib": {
"version": "2.4.0",
"resolved": "https://registry.npmjs.org/tslib/-/tslib-2.4.0.tgz",
"integrity": "sha512-d6xOpEDfsi2CZVlPQzGeux8XMwLT9hssAsaPYExaQMuYskwb+x1x7J371tWlbBdWHroy99KnVB6qIkUbs5X3UQ==",
"peer": true
},
"node_modules/semver": { "node_modules/semver": {
"version": "5.7.1", "version": "5.7.1",
"resolved": "https://registry.npmjs.org/semver/-/semver-5.7.1.tgz", "resolved": "https://registry.npmjs.org/semver/-/semver-5.7.1.tgz",
@@ -527,7 +533,8 @@
"node_modules/tslib": { "node_modules/tslib": {
"version": "1.14.1", "version": "1.14.1",
"resolved": "https://registry.npmjs.org/tslib/-/tslib-1.14.1.tgz", "resolved": "https://registry.npmjs.org/tslib/-/tslib-1.14.1.tgz",
"integrity": "sha512-Xni35NKzjgMrwevysHTCArtLDpPvye8zV/0E4EyYn43P7/7qvQwPh9BGkHewbMulVntbigmcT7rdX3BNo9wRJg==" "integrity": "sha512-Xni35NKzjgMrwevysHTCArtLDpPvye8zV/0E4EyYn43P7/7qvQwPh9BGkHewbMulVntbigmcT7rdX3BNo9wRJg==",
"dev": true
}, },
"node_modules/tslint": { "node_modules/tslint": {
"version": "6.1.3", "version": "6.1.3",
@@ -951,11 +958,20 @@
} }
}, },
"rxjs": { "rxjs": {
"version": "6.6.7", "version": "7.5.6",
"resolved": "https://registry.npmjs.org/rxjs/-/rxjs-6.6.7.tgz", "resolved": "https://registry.npmjs.org/rxjs/-/rxjs-7.5.6.tgz",
"integrity": "sha512-hTdwr+7yYNIT5n4AMYp85KA6yw2Va0FLa3Rguvbpa4W3I5xynaBZo41cM3XM+4Q6fRMj3sBYIR1VAmZMXYJvRQ==", "integrity": "sha512-dnyv2/YsXhnm461G+R/Pe5bWP41Nm6LBXEYWI6eiFP4fiwx6WRI/CD0zbdVAudd9xwLEF2IDcKXLHit0FYjUzw==",
"peer": true,
"requires": { "requires": {
"tslib": "^1.9.0" "tslib": "^2.1.0"
},
"dependencies": {
"tslib": {
"version": "2.4.0",
"resolved": "https://registry.npmjs.org/tslib/-/tslib-2.4.0.tgz",
"integrity": "sha512-d6xOpEDfsi2CZVlPQzGeux8XMwLT9hssAsaPYExaQMuYskwb+x1x7J371tWlbBdWHroy99KnVB6qIkUbs5X3UQ==",
"peer": true
}
} }
}, },
"semver": { "semver": {
@@ -1007,7 +1023,8 @@
"tslib": { "tslib": {
"version": "1.14.1", "version": "1.14.1",
"resolved": "https://registry.npmjs.org/tslib/-/tslib-1.14.1.tgz", "resolved": "https://registry.npmjs.org/tslib/-/tslib-1.14.1.tgz",
"integrity": "sha512-Xni35NKzjgMrwevysHTCArtLDpPvye8zV/0E4EyYn43P7/7qvQwPh9BGkHewbMulVntbigmcT7rdX3BNo9wRJg==" "integrity": "sha512-Xni35NKzjgMrwevysHTCArtLDpPvye8zV/0E4EyYn43P7/7qvQwPh9BGkHewbMulVntbigmcT7rdX3BNo9wRJg==",
"dev": true
}, },
"tslint": { "tslint": {
"version": "6.1.3", "version": "6.1.3",

View File

@@ -10,10 +10,12 @@
"build": "tsc" "build": "tsc"
}, },
"dependencies": { "dependencies": {
"rxjs": "6.6.7",
"sorted-btree": "1.5.0", "sorted-btree": "1.5.0",
"uuid": "8.3.2" "uuid": "8.3.2"
}, },
"peerDependencies": {
"rxjs": ">=6.0.0"
},
"devDependencies": { "devDependencies": {
"@types/node": "16.4.13", "@types/node": "16.4.13",
"@types/uuid": "8.3.1", "@types/uuid": "8.3.1",