Apply suggestions from code review

Co-authored-by: stevensJourney <51082125+stevensJourney@users.noreply.github.com>

Author: benitav

packages/powersync-sdk-common/src/client/AbstractPowerSyncDatabase.ts

@@ -126,7 +126,7 @@ export abstract class AbstractPowerSyncDatabase extends BaseObserver<PowerSyncDB
   /**
    * The underlying database.
    * 
-   * For the most part, behavior is the same whether querying on the underlying database, or on {@link AbstractPowerSyncDatabase}. The main difference is in update notifications: the underlying database reports updates to the underlying tables, while AbstractPowerSyncDatabase reports updates to the higher-level views.
+   * For the most part, behavior is the same whether querying on the underlying database, or on {@link AbstractPowerSyncDatabase}.
    */
   protected get database() {
     return this.options.database;

packages/powersync-sdk-common/src/client/sync/bucket/CrudBatch.ts

@@ -1,7 +1,7 @@
 import { CrudEntry } from './CrudEntry';
 
 /**
- * TODO
+* A batch of client-side changes.
  */
 export class CrudBatch {
   constructor(

packages/powersync-sdk-common/src/client/sync/bucket/CrudTransaction.ts

@@ -15,7 +15,7 @@ export class CrudTransaction extends CrudBatch {
      */
     public complete: (checkpoint?: string) => Promise<void>,
     /**
-     * Unique transaction id.
+     * If null, this contains a list of changes recorded without an explicit transaction associated.
      */
     public transactionId?: number
   ) {

packages/powersync-sdk-common/src/db/crud/SyncStatus.ts

@@ -22,7 +22,8 @@ export class SyncStatus {
   }
 
   /**
-   * TODO
+   *  Time that a last sync has fully completed, if any.
+   *  Currently this is reset to null after a restart.
    */
   get lastSyncedAt() {
     return this.options.lastSyncedAt;