wip

Author: jasongrout

packages/services/src/kernel/kernel.ts

@@ -28,6 +28,15 @@ export namespace Kernel {
    * a websocket connection internally, and will auto-restart if the websocket
    * temporarily loses connection.  Restarting creates a new Kernel process on
    * the server, but preserves the Kernel id.
+   *
+   * The IKernelConnection is notably missing the full IKernel signals. This
+   * interface is for situations where a kernel may change, but we want a user
+   * to not have to worry about disconnecting and reconnecting signals when a
+   * kernel is swapped. The object that maintains an IKernel, but only provides
+   * a user with an IKernelConnection should proxy the appropriate IKernel
+   * signals for the user with its own signals. The advantage is that when the
+   * kernel is changed, the object itself can take care of disconnecting and
+   * reconnecting listeners.
    */
   export interface IKernelConnection extends IDisposable {
     /**
@@ -63,6 +72,11 @@ export namespace Kernel {
      */
     readonly status: Kernel.Status;
 
+    /**
+     * The current connection status of the kernel.
+     */
+    readonly connectionStatus: Kernel.ConnectionStatus;
+
     /**
      * The cached kernel info.
      *

packages/services/src/session/manager.ts

@@ -15,6 +15,17 @@ import { ServerConnection } from '../serverconnection';
 
 import { Session } from './session';
 
+/**
+ * We have a session manager that maintains a list of models from the server.
+ * Separately, we have a list of running sessions maintained by the DefaultSessions
+ *
+ * Perhaps we have *one* list of models, with a separate possible session connection for each model.
+ *
+ * Also, we should be able to modify the session information without a session connection - that's just a server request, and doesn't require a kernel connection.
+ *
+ *
+ */
+
 /**
  * An implementation of a session manager.
  */

packages/services/src/session/session.ts

@@ -21,6 +21,8 @@ import { DefaultSession } from './default';
 export namespace Session {
   /**
    * Interface of a session object.
+   *
+   * A session object represents a live connection to a session kernel.
    */
   export interface ISession extends IDisposable {
     /**