fix(names): update chosen names logic using contract source

We were not passing the height around so the choseNames kept returning an empty array

src/names/contract-names-source.ts

@@ -32,18 +32,19 @@ export class ContractNamesSource implements ArnsNamesSource, ArnsNameList {
     return names;
   }
 
-  async getAllNames(): Promise<string[]> {
+  // we don't use height here, but it's required by the interface
+  async getAllNames(_height: number): Promise<string[]> {
     const names = await this.contract.getArNSRecords();
     const namesArray = Object.keys(names).sort();
     return namesArray;
   }
 
-  async getName(index: number): Promise<string> {
-    const names = await this.getAllNames();
+  async getName(height: number, index: number): Promise<string> {
+    const names = await this.getAllNames(height);
     return names[index];
   }
 
-  async getNamesCount(): Promise<number> {
-    return (await this.getAllNames()).length;
+  async getNamesCount(height: number): Promise<number> {
+    return (await this.getAllNames(height)).length;
   }
 }

src/system.ts

@@ -128,13 +128,6 @@ const namesSource = new ContractNamesSource({
   contract: networkContract,
 });
 
-const remoteCacheArnsNameList =
-  config.ARNS_NAMES.length > 0
-    ? new StaticArnsNameList({
-        names: config.ARNS_NAMES,
-      })
-    : namesSource;
-
 const chainEntropySource = new ChainEntropySource({
   arweaveBaseUrl: config.ARWEAVE_URL,
 });
@@ -150,8 +143,15 @@ const compositeEntropySource = new CompositeEntropySource({
   sources: [cachedEntropySource, chainEntropySource],
 });
 
+const nameListSource =
+  config.ARNS_NAMES.length > 0
+    ? new StaticArnsNameList({
+        names: config.ARNS_NAMES,
+      })
+    : namesSource; // use the contract source if nothing configured
+
 const chosenNamesSource = new RandomArnsNamesSource({
-  nameList: remoteCacheArnsNameList,
+  nameList: nameListSource,
   entropySource: compositeEntropySource,
   numNamesToSource: config.NUM_ARNS_NAMES_TO_OBSERVE_PER_GROUP,
 });