@adminide-stack/extension-api 9.1.1-alpha.52 → 9.1.1-alpha.62

package/lib/connections/jsonrpc2/connection.js

@@ -1,4 +1,4 @@
-import*as abortableRx from'../../utils/abortable-rx.js';import {Subject,Observable,isObservable,from}from'rxjs';import {isPromise}from'../../utils/util.js';import'rxjs-compat/observable/fromArray.js';import'rxjs-compat/OuterSubscriber.js';import'rxjs-compat/scheduler/asap.js';import'rxjs-compat/util/subscribeToResult.js';import'@vscode-alt/monaco-editor/esm/vs/base/common/path.js';import {Emitter}from'./events.js';import {LinkedMap}from'./linkedMap.js';import AbortController from'abort-controller';import {isNotificationMessage,isRequestMessage,ErrorCodes,ResponseError,isResponseMessage}from'./messages.js';import {noopTracer}from'./trace.js';const { toPromise } = abortableRx; // to support vite esm
+import*as abortableRx from'../../utils/abortable-rx.js';import {Subject,Observable,isObservable,from}from'rxjs';import {isPromise}from'../../utils/util.js';import'rxjs-compat/observable/fromArray.js';import'rxjs-compat/scheduler/asap.js';import'@vscode-alt/monaco-editor/esm/vs/base/common/path.js';import {Emitter}from'./events.js';import {LinkedMap}from'./linkedMap.js';import AbortController from'abort-controller';import {isNotificationMessage,isRequestMessage,ErrorCodes,ResponseError,isResponseMessage}from'./messages.js';import {noopTracer}from'./trace.js';const { toPromise } = abortableRx; // to support vite esm
 const NullLogger = Object.freeze({
     error: () => {
         /* noop */
@@ -155,7 +155,7 @@ function _createConnection(transports, logger) {
         }
     }
     const callback = (message) => {
-        console.log("[callback.ON_MESSAGE]: ", { transports, message });
+        console.log('[callback.ON_MESSAGE]: ', { transports, message });
         try {
             // We have received an abort signal. Check if the message is still in the queue and abort it if allowed
             // to do so.
@@ -237,7 +237,8 @@ function _createConnection(transports, logger) {
             transports.writer.write(message);
         }
         tracer.requestReceived(requestMessage);
-        const element = requestHandlers[requestMessage.method] || requestHandlers[matchedStartActor(Object.keys(requestHandlers), requestMessage.method)];
+        const element = requestHandlers[requestMessage.method] ||
+            requestHandlers[matchedStartActor(Object.keys(requestHandlers), requestMessage.method)];
         const requestHandler = element && element.handler;
         if (requestHandler || starRequestHandler) {
             const abortController = new AbortController();
@@ -256,7 +257,7 @@ function _createConnection(transports, logger) {
                     const onComplete = () => {
                         delete requestAbortControllers[signalKey];
                     };
-                    from(handlerResult).subscribe(value => reply(value, false), error => {
+                    from(handlerResult).subscribe((value) => reply(value, false), (error) => {
                         onComplete();
                         if (error instanceof ResponseError) {
                             replyError(error);
@@ -470,15 +471,14 @@ function _createConnection(transports, logger) {
         catch (e) {
             responseObserver.observer.error(new ResponseError(ErrorCodes.MessageWriteError, e.message ? e.message : 'Unknown reason'));
         }
-        return new Observable(observer => {
-            subject.subscribe(observer).add(() => {
-                if (!isUnsubscribed() &&
-                    responseObserver &&
-                    !responseObserver.complete &&
-                    !responseObserver.observer.closed) {
+        return new Observable((observer) => {
+            const subscription = subject.subscribe(observer);
+            subscription.add(() => {
+                if (!isUnsubscribed() && responseObserver && !responseObserver.complete && !subscription.closed) {
                     sendNotification(ABORT_REQUEST_METHOD, [id]);
                 }
             });
+            return subscription;
         });
     };
     const connection = {
@@ -557,7 +557,7 @@ function setImmediateCompat(f) {
 }
 function matchedStartActor(requestHandlersKeys, method) {
     try {
-        return requestHandlersKeys.filter(name => {
+        return requestHandlersKeys.filter((name) => {
             let methodName = method.split('/')[0];
             return `${methodName}/*` === name;
         })[0];

package/lib/connections/remote-rpc/browser-remote-rpc.js

@@ -1,4 +1,4 @@
-import {QueueingSubject}from'queueing-subject';import {VSBuffer}from'@vscode-alt/monaco-editor/esm/vs/base/common/buffer.js';import {Emitter}from'@vscode-alt/monaco-editor/esm/vs/base/common/event.js';import makeWebSocketObservable,{normalClosureMessage}from'./rxjs-websockets.js';import {switchMap,share}from'rxjs/operators/index.js';class BrowserRemoteRPC {
+import {QueueingSubject}from'queueing-subject';import {VSBuffer}from'@vscode-alt/monaco-editor/esm/vs/base/common/buffer.js';import {Emitter}from'@vscode-alt/monaco-editor/esm/vs/base/common/event.js';import makeWebSocketObservable,{normalClosureMessage}from'./rxjs-websockets.js';import {switchMap,share}from'rxjs';class BrowserRemoteRPC {
     // this subject queues as necessary to ensure every message is delivered
     input$;
     _localHostId;

package/lib/react/components/view-component/ViewComponent.js

@@ -1,4 +1,4 @@
-import React__default,{useState,useEffect}from'react';import {Subscription}from'rxjs';import {map}from'rxjs/operators/index.js';import {useExtensionController}from'../../useExtensionController.js';import {ContributableViewContainer}from'../../../interfaces/contributions/contribution.js';const ViewComponent = (props) => {
+import React__default,{useState,useEffect}from'react';import {Subscription,map}from'rxjs';import {useExtensionController}from'../../useExtensionController.js';import {ContributableViewContainer}from'../../../interfaces/contributions/contribution.js';const ViewComponent = (props) => {
     const [panelView, setPanelView] = useState(null);
     const controller = useExtensionController();
     const subscriptions = new Subscription();

package/lib/utils/abortable-rx.js

@@ -1,4 +1,4 @@
-import {Observable,Subscription,from}from'rxjs';import {switchMap as switchMap$1,concatMap as concatMap$1,mergeMap as mergeMap$1}from'rxjs/operators/index.js';/**
+import {Observable,Subscription,from,switchMap as switchMap$1,concatMap as concatMap$1,mergeMap as mergeMap$1}from'rxjs';/**
  * Copied from https://github.com/felixfbecker/abortable-rx/blob/master/src/index.ts
  * To support esm
  * @returns
@@ -12,7 +12,7 @@ const createAbortError = () => {
 /**
  * Creates an Observable just like RxJS `create`, but exposes an AbortSignal in addition to the subscriber
  */
-const create = (subscribe) => new Observable(subscriber => {
+const create = (subscribe) => new Observable((subscriber) => {
     const abortController = new AbortController();
     const subscription = new Subscription();
     const teardown = subscribe && subscribe(subscriber, abortController.signal);
@@ -45,9 +45,9 @@ const toPromise = (observable, signal) => new Promise((resolve, reject) => {
         }
     };
     let value;
-    subscription = observable.subscribe(val => {
+    subscription = observable.subscribe((val) => {
         value = val;
-    }, err => {
+    }, (err) => {
         cleanup();
         reject(err);
     }, () => {
@@ -77,7 +77,7 @@ const forEach = (source, next, signal) => new Promise((resolve, reject) => {
             signal.removeEventListener('abort', listener);
         }
     };
-    subscription = source.subscribe(value => {
+    subscription = source.subscribe((value) => {
         try {
             next(value);
         }
@@ -87,7 +87,7 @@ const forEach = (source, next, signal) => new Promise((resolve, reject) => {
                 subscription.unsubscribe();
             }
         }
-    }, err => {
+    }, (err) => {
         cleanup();
         reject(err);
     }, () => {
@@ -101,12 +101,12 @@ const forEach = (source, next, signal) => new Promise((resolve, reject) => {
 /**
  * Like RxJS `switchMap`, but passes an AbortSignal that is aborted when the source emits another element.
  */
-const switchMap = (project) => source => source.pipe(switchMap$1((value, index) => defer(abortSignal => project(value, index, abortSignal))));
+const switchMap = (project) => (source) => source.pipe(switchMap$1((value, index) => defer((abortSignal) => project(value, index, abortSignal))));
 /**
  * Like RxJS `concatMap`, but passes an AbortSignal that is aborted when the returned Observable is unsubscribed from.
  */
-const concatMap = (project) => source => source.pipe(concatMap$1((value, index) => defer(abortSignal => project(value, index, abortSignal))));
+const concatMap = (project) => (source) => source.pipe(concatMap$1((value, index) => defer((abortSignal) => project(value, index, abortSignal))));
 /**
  * Like RxJS `mergeMap`, but passes an AbortSignal that is aborted when the returned Observable is unsubscribed from.
  */
-const mergeMap = (project) => source => source.pipe(mergeMap$1((value, index) => defer(abortSignal => project(value, index, abortSignal))));export{concatMap,create,defer,forEach,mergeMap,switchMap,toPromise};
+const mergeMap = (project) => (source) => source.pipe(mergeMap$1((value, index) => defer((abortSignal) => project(value, index, abortSignal))));export{concatMap,create,defer,forEach,mergeMap,switchMap,toPromise};

package/lib/utils/rxjs/combineLatestOrDefault.js

@@ -1,4 +1,4 @@
-import {zip,of}from'rxjs';import {fromArray}from'rxjs-compat/observable/fromArray.js';import {OuterSubscriber}from'rxjs-compat/OuterSubscriber.js';import {asap}from'rxjs-compat/scheduler/asap.js';import {subscribeToResult}from'rxjs-compat/util/subscribeToResult.js';// tslint:disable no-use-before-declare
+import {zip,of,Subscriber,Subscription}from'rxjs';import {fromArray}from'rxjs-compat/observable/fromArray.js';import {asap}from'rxjs-compat/scheduler/asap.js';// tslint:disable no-use-before-declare
 /**
  * Like {@link combineLatest}, except that it does not wait for all Observables to emit before emitting an initial
  * value. It emits whenever any of the source Observables emit.
@@ -46,15 +46,19 @@ class CombineLatestOperator {
         return source.subscribe(new CombineLatestSubscriber(subscriber, this.defaultValue));
     }
 }
-class CombineLatestSubscriber extends OuterSubscriber {
+class CombineLatestSubscriber extends Subscriber {
     defaultValue;
     activeObservables = 0;
     values = [];
     observables = [];
     scheduled = false;
-    constructor(observer, defaultValue) {
-        super(observer);
+    subscriptions = new Subscription();
+    constructor(destination, defaultValue) {
+        super(destination);
         this.defaultValue = defaultValue;
+        if (!destination) {
+            throw new Error('Destination is required');
+        }
     }
     _next(observable) {
         if (this.defaultValue !== undefined) {
@@ -64,35 +68,34 @@ class CombineLatestSubscriber extends OuterSubscriber {
     }
     _complete() {
         this.activeObservables = this.observables.length;
+        // Use Observable's subscribe method directly
         for (let i = 0; i < this.observables.length; i++) {
-            this.add(subscribeToResult(this, this.observables[i], this.observables[i], i));
+            const observable = this.observables[i];
+            const subscription = observable.subscribe({
+                next: (value) => this.notifyNext(value, i),
+                complete: () => this.notifyComplete(),
+            });
+            this.subscriptions.add(subscription);
         }
     }
     notifyComplete() {
         this.activeObservables--;
-        if (this.activeObservables === 0 && this.destination.complete) {
+        if (this.activeObservables === 0 && this.destination) {
             this.destination.complete();
         }
     }
-    notifyNext(_outerValue, innerValue, outerIndex) {
-        const values = this.values;
-        values[outerIndex] = innerValue;
+    notifyNext(value, index) {
+        this.values[index] = value;
         if (this.activeObservables === 1) {
-            // Only 1 observable is active, so no need to buffer.
-            //
-            // This makes it possible to use RxJS's `of` in tests without specifying an explicit scheduler.
-            if (this.destination.next) {
+            if (this.destination && this.destination.next) {
                 this.destination.next(this.values.slice());
             }
             return;
         }
-        // Buffer all next values that are emitted at the same time into one emission.
-        //
-        // This makes tests (using expectObservable) easier to write.
         if (!this.scheduled) {
             this.scheduled = true;
             asap.schedule(() => {
-                if (this.scheduled && this.destination.next) {
+                if (this.scheduled && this.destination && this.destination.next) {
                     this.destination.next(this.values.slice());
                 }
                 this.scheduled = false;

package/package.json

@@ -1,6 +1,6 @@
 {
     "name": "@adminide-stack/extension-api",
-    "version": "9.1.1-alpha.52",
+    "version": "9.1.1-alpha.62",
     "description": "Workbench core for higher packages to depend on",
     "license": "ISC",
     "author": "CDMBase LLC",
@@ -52,7 +52,7 @@
     },
     "peerDependencies": {
         "@vscode-alt/monaco-editor": ">=0.17.1",
-        "rxjs": "^6.3.3"
+        "rxjs": "^7.8.1"
     },
     "publishConfig": {
         "access": "public"
@@ -60,5 +60,5 @@
     "typescript": {
         "definition": "lib/index.d.ts"
     },
-    "gitHead": "52c2c05beffb0f957159789f3e7aa8c491c4dc1d"
+    "gitHead": "009822bc9c2a3879520e8d8c4137ec5b9b3abaac"
 }