Remove redundant functions.

Author: rkistner

packages/service-core-tests/src/tests/register-data-storage-parameter-tests.ts

@@ -1,11 +1,5 @@
 import { storage } from '@powersync/service-core';
-import {
-  mergeBucketParameterQueriers,
-  ParameterLookup,
-  PendingQueriers,
-  RequestParameters
-} from '@powersync/service-sync-rules';
-import { SqlBucketDescriptor } from '@powersync/service-sync-rules/src/SqlBucketDescriptor.js';
+import { ParameterLookup, RequestParameters } from '@powersync/service-sync-rules';
 import { expect, test } from 'vitest';
 import * as test_utils from '../test-utils/test-utils-index.js';
 import { TEST_TABLE } from './util.js';

packages/sync-rules/src/BucketSource.ts

@@ -56,8 +56,6 @@ export interface BucketParameterSourceDefinition {
   getSourceTables(): Set<TablePattern>;
   createParameterSource(params: CreateSourceParams): BucketParameterSource;
 
-  getSourceTables(): Set<TablePattern>;
-
   /** Whether the table possibly affects the buckets resolved by this source. */
   tableSyncsParameters(table: SourceTableInterface): boolean;
 }
@@ -110,11 +108,6 @@ export interface BucketParameterSource {
    * @param options Options, including parameters that may affect the buckets loaded by this source.
    */
   pushBucketParameterQueriers(result: PendingQueriers, options: GetQuerierOptions): void;
-
-  /**
-   * @deprecated Use `pushBucketParameterQueriers` instead and merge at the top-level.
-   */
-  getBucketParameterQuerier(options: GetQuerierOptions): BucketParameterQuerier;
 }
 
 export enum BucketSourceType {

packages/sync-rules/src/SqlParameterQuery.ts

@@ -344,16 +344,6 @@ export class SqlParameterQuery implements BucketParameterSourceDefinition {
       pushBucketParameterQueriers: (result: PendingQueriers, options: GetQuerierOptions) => {
         const q = this.getBucketParameterQuerier(options.globalParameters, ['default'], params.bucketIdTransformer);
         result.queriers.push(q);
-      },
-
-      /**
-       * @deprecated Use `pushBucketParameterQueriers` instead and merge at the top-level.
-       */
-      getBucketParameterQuerier(options: GetQuerierOptions): BucketParameterQuerier {
-        const queriers: BucketParameterQuerier[] = [];
-        this.pushBucketParameterQueriers({ queriers, errors: [] }, options);
-
-        return mergeBucketParameterQueriers(queriers);
       }
     };
   }

packages/sync-rules/src/StaticSqlParameterQuery.ts

@@ -218,16 +218,6 @@ export class StaticSqlParameterQuery implements BucketParameterSourceDefinition
           queryDynamicBucketDescriptions: async () => []
         } satisfies BucketParameterQuerier;
         result.queriers.push(staticQuerier);
-      },
-
-      /**
-       * @deprecated Use `pushBucketParameterQueriers` instead and merge at the top-level.
-       */
-      getBucketParameterQuerier(options: GetQuerierOptions): BucketParameterQuerier {
-        const queriers: BucketParameterQuerier[] = [];
-        this.pushBucketParameterQueriers({ queriers, errors: [] }, options);
-
-        return mergeBucketParameterQueriers(queriers);
       }
     };
   }

packages/sync-rules/src/TableValuedFunctionSqlParameterQuery.ts

@@ -262,16 +262,6 @@ export class TableValuedFunctionSqlParameterQuery implements BucketParameterSour
           queryDynamicBucketDescriptions: async () => []
         } satisfies BucketParameterQuerier;
         result.queriers.push(staticQuerier);
-      },
-
-      /**
-       * @deprecated Use `pushBucketParameterQueriers` instead and merge at the top-level.
-       */
-      getBucketParameterQuerier(options: GetQuerierOptions): BucketParameterQuerier {
-        const queriers: BucketParameterQuerier[] = [];
-        this.pushBucketParameterQueriers({ queriers, errors: [] }, options);
-
-        return mergeBucketParameterQueriers(queriers);
       }
     };
   }

packages/sync-rules/src/streams/stream.ts

@@ -118,16 +118,6 @@ export class SyncStream implements BucketDataSourceDefinition, BucketParameterSo
         }
 
         return result;
-      },
-
-      /**
-       * @deprecated Use `pushBucketParameterQueriers` instead and merge at the top-level.
-       */
-      getBucketParameterQuerier(options: GetQuerierOptions): BucketParameterQuerier {
-        const queriers: BucketParameterQuerier[] = [];
-        this.pushBucketParameterQueriers({ queriers, errors: [] }, options);
-
-        return mergeBucketParameterQueriers(queriers);
       }
     };
   }