More accurate and complete comments.

Author: wu-hui

packages/firestore/index.bundle.ts

@@ -19,7 +19,7 @@ import { Firestore } from './src/api/database';
 import { loadBundle, namedQuery } from './src/api/bundle';
 
 /**
- * Registers the memory-only Firestore build with the components framework.
+ * Registers Firestore bundle loading features with the components framework.
  */
 export function registerBundle(instance: typeof Firestore): void {
   instance.prototype.loadBundle = function (
@@ -34,8 +34,6 @@ export function registerBundle(instance: typeof Firestore): void {
   ) {
     return namedQuery(this, queryName);
   };
-
-  //TODO(wuandy): add loadBundle and namedQuery to the firestore namespace
 }
 
 registerBundle(Firestore);

packages/firestore/src/core/bundle_types.ts

@@ -26,8 +26,15 @@ import { SnapshotVersion } from './snapshot_version';
  * Represents a Firestore bundle saved by the SDK in its local storage.
  */
 export interface Bundle {
+  /**
+   * Id of the bundle. It is used together with `createTime` to determine if a
+   * bundle has been loaded by the SDK.
+   */
   readonly id: string;
+
+  /** Schema version of the bundle. */
   readonly version: number;
+
   /**
    * Set to the snapshot version of the bundle if created by the Server SDKs.
    * Otherwise set to SnapshotVersion.MIN.
@@ -39,7 +46,9 @@ export interface Bundle {
  * Represents a Query saved by the SDK in its local storage.
  */
 export interface NamedQuery {
+  /** The name of the query. */
   readonly name: string;
+  /** The underlying query associated with `name`. */
   readonly query: Query;
   /** The time at which the results for this query were read. */
   readonly readTime: SnapshotVersion;