feat: create default shard info, update tests

packages/discovery/src/dns/constants.ts

@@ -3,14 +3,11 @@ import type { NodeCapabilityCount } from "@waku/interfaces";
 /**
  * The ENR tree for the different fleets.
  * SANDBOX and TEST fleets are for The Waku Network.
- * DEPRECATED_DEFAULT_PUBSUB is the fleet of nodes supporting the now deprecated DefaultPubsubTopic.
  */
 export const enrTree = {
   SANDBOX:
     "enrtree://AIRVQ5DDA4FFWLRBCHJWUWOO6X6S4ZTZ5B667LQ6AJU6PEYDLRD5O@sandbox.waku.nodes.status.im",
-  TEST: "enrtree://AOGYWMBYOUIMOENHXCHILPKY3ZRFEULMFI4DOM442QSZ73TT2A7VI@test.waku.nodes.status.im",
-  DEPRECATED_DEFAULT_PUBSUB:
-    "enrtree://ANEDLO25QVUGJOUTQFRYKWX6P4Z4GKVESBMHML7DZ6YK4LGS5FC5O@prod.wakuv2.nodes.status.im"
+  TEST: "enrtree://AOGYWMBYOUIMOENHXCHILPKY3ZRFEULMFI4DOM442QSZ73TT2A7VI@test.waku.nodes.status.im"
 };
 
 export const DEFAULT_BOOTSTRAP_TAG_NAME = "bootstrap";

packages/interfaces/src/constants.ts

@@ -1,9 +1,14 @@
-/**
- * DefaultPubsubTopic is the default gossipsub topic to use for Waku.
- */
-export const DefaultPubsubTopic = "/waku/2/rs/0/0";
+import { ShardInfo } from "./enr";
 
 /**
  * The default cluster ID for The Waku Network
  */
 export const DEFAULT_CLUSTER_ID = 1;
+
+/**
+ * DefaultShardInfo is default configuration for The Waku Network.
+ */
+export const DefaultShardInfo: ShardInfo = {
+  clusterId: DEFAULT_CLUSTER_ID,
+  shards: [0, 1, 2, 3, 4, 5, 6, 7, 8]
+};

packages/interfaces/src/protocols.ts

@@ -73,31 +73,35 @@ export type ProtocolUseOptions = {
 export type ProtocolCreateOptions = {
   /**
    * @deprecated
-   * Waku will stop supporting named sharding. Only static sharding and autosharding will be supported moving forward.
-   */
-  pubsubTopics?: PubsubTopic[];
-  /**
-   * Waku supports usage of multiple pubsub topics. This is achieved through static sharding for now, and auto-sharding in the future.
-   * The format to specify a shard is:
-   * clusterId: number, shards: number[]
-   * To learn more about the sharding specifications implemented, see [Relay Sharding](https://rfc.vac.dev/spec/51/).
-   * The Pubsub Topic to use. Defaults to {@link @waku/core!DefaultPubsubTopic }.
+   * Should be used ONLY if some other than The Waku Network is in use.
+   *
+   * See [Waku v2 Topic Usage Recommendations](https://github.com/vacp2p/rfc-index/blob/main/waku/informational/23/topics.md#pubsub-topics) for details.
    *
-   * If no pubsub topic is specified, the default pubsub topic is used.
-   * The set of pubsub topics that are used to initialize the Waku node, will need to be used by the protocols as well
-   * You cannot currently add or remove pubsub topics after initialization.
    * This is used by:
    * - WakuRelay to receive, route and send messages,
    * - WakuLightPush to send messages,
    * - WakuStore to retrieve messages.
-   * See [Waku v2 Topic Usage Recommendations](https://github.com/vacp2p/rfc-index/blob/main/waku/informational/23/topics.md) for details.
    *
+   * If no pubsub topic is specified, the default pubsub topic will be determined from DefaultShardInfo.
+   *
+   * You cannot add or remove pubsub topics after initialization of the node.
+   */
+  pubsubTopics?: PubsubTopic[];
+  /**
+   * ShardInfo is used to determine which network is in use.
+   * Defaults to {@link @waku/interfaces!DefaultShardInfo}.
+   * Default value is configured for The Waku Network
+   *
+   * The format to specify a shard is:
+   * clusterId: number, shards: number[]
+   * To learn more about the sharding specification, see [Relay Sharding](https://rfc.vac.dev/spec/51/).
    */
   shardInfo?: Partial<ShardingParams>;
   /**
-   * Content topics are used to determine pubsubTopics
-   * If not provided pubsubTopics will be determined based on shardInfo
-   * See [Waku v2 Topic Usage Recommendations](https://github.com/vacp2p/rfc-index/blob/main/waku/informational/23/topics.md) for details.
+   * Content topics are used to determine network in use.
+   * See [Waku v2 Topic Usage Recommendations](https://github.com/vacp2p/rfc-index/blob/main/waku/informational/23/topics.md#content-topics) for details.
+   *
+   * You cannot add or remove content topics after initialization of the node.
    */
   contentTopics?: string[];
   /**

packages/message-encryption/src/symmetric.ts

@@ -1,7 +1,6 @@
 import { Decoder as DecoderV0 } from "@waku/core/lib/message/version_0";
 import {
   type EncoderOptions as BaseEncoderOptions,
-  DefaultPubsubTopic,
   type IDecoder,
   type IEncoder,
   type IMessage,
@@ -101,7 +100,7 @@ export interface EncoderOptions extends BaseEncoderOptions {
  * in [26/WAKU2-PAYLOAD](https://rfc.vac.dev/spec/26/).
  */
 export function createEncoder({
-  pubsubTopic = DefaultPubsubTopic,
+  pubsubTopic,
   pubsubTopicShardInfo,
   contentTopic,
   symKey,
@@ -198,7 +197,7 @@ class Decoder extends DecoderV0 implements IDecoder<DecodedMessage> {
 export function createDecoder(
   contentTopic: string,
   symKey: Uint8Array,
-  pubsubTopicShardInfo: SingleShardInfo | PubsubTopic = DefaultPubsubTopic
+  pubsubTopicShardInfo?: SingleShardInfo | PubsubTopic
 ): Decoder {
   return new Decoder(
     determinePubsubTopic(contentTopic, pubsubTopicShardInfo),

packages/relay/src/index.ts

@@ -11,7 +11,6 @@ import { sha256 } from "@noble/hashes/sha256";
 import {
   ActiveSubscriptions,
   Callback,
-  DefaultPubsubTopic,
   IAsyncIterator,
   IDecodedMessage,
   IDecoder,
@@ -75,9 +74,8 @@ class Relay implements IRelay {
 
     this.observers = new Map();
 
-    // Default PubsubTopic decoder
     // TODO: User might want to decide what decoder should be used (e.g. for RLN)
-    this.defaultDecoder = new TopicOnlyDecoder();
+    this.defaultDecoder = new TopicOnlyDecoder(pubsubTopics[0]);
   }
 
   /**
@@ -204,8 +202,9 @@ class Relay implements IRelay {
     return map;
   }
 
-  public getMeshPeers(topic: TopicStr = DefaultPubsubTopic): PeerIdStr[] {
-    return this.gossipSub.getMeshPeers(topic);
+  public getMeshPeers(topic?: TopicStr): PeerIdStr[] {
+    // if no TopicStr is provided - returns empty array
+    return this.gossipSub.getMeshPeers(topic || "");
   }
 
   private subscribeToAllTopics(): void {

packages/relay/src/topic_only_message.ts

@@ -1,8 +1,8 @@
-import { DefaultPubsubTopic } from "@waku/interfaces";
 import type {
   IDecodedMessage,
   IDecoder,
-  IProtoMessage
+  IProtoMessage,
+  PubsubTopic
 } from "@waku/interfaces";
 import { TopicOnlyMessage as ProtoTopicOnlyMessage } from "@waku/proto";
 
@@ -23,10 +23,13 @@ export class TopicOnlyMessage implements IDecodedMessage {
   }
 }
 
+// This decoder is used only for reading `contentTopic` from the WakuMessage
 export class TopicOnlyDecoder implements IDecoder<TopicOnlyMessage> {
-  public pubsubTopic = DefaultPubsubTopic;
   public contentTopic = "";
 
+  // pubsubTopic is ignored
+  public constructor(public pubsubTopic: PubsubTopic) {}
+
   public fromWireToProtoObj(
     bytes: Uint8Array
   ): Promise<IProtoMessage | undefined> {

packages/sdk/src/create/discovery.ts

@@ -5,11 +5,7 @@ import {
   wakuLocalPeerCacheDiscovery,
   wakuPeerExchangeDiscovery
 } from "@waku/discovery";
-import {
-  DefaultPubsubTopic,
-  type Libp2pComponents,
-  PubsubTopic
-} from "@waku/interfaces";
+import { type Libp2pComponents, PubsubTopic } from "@waku/interfaces";
 
 const DEFAULT_NODE_REQUIREMENTS = {
   lightPush: 1,
@@ -20,10 +16,7 @@ const DEFAULT_NODE_REQUIREMENTS = {
 export function defaultPeerDiscoveries(
   pubsubTopics: PubsubTopic[]
 ): ((components: Libp2pComponents) => PeerDiscovery)[] {
-  const isDefaultPubsubTopic = pubsubTopics.includes(DefaultPubsubTopic);
-  const dnsEnrTrees = isDefaultPubsubTopic
-    ? [enrTree["DEPRECATED_DEFAULT_PUBSUB"]]
-    : [enrTree["SANDBOX"], enrTree["TEST"]];
+  const dnsEnrTrees = [enrTree["SANDBOX"], enrTree["TEST"]];
 
   const discoveries = [
     wakuDnsDiscovery(dnsEnrTrees, DEFAULT_NODE_REQUIREMENTS),

packages/sdk/src/create/libp2p.ts

@@ -9,7 +9,7 @@
 import { wakuMetadata } from "@waku/core";
 import {
   type CreateLibp2pOptions,
-  DefaultPubsubTopic,
+  DefaultShardInfo,
   type IMetadata,
   type Libp2p,
   type Libp2pComponents,
@@ -86,7 +86,7 @@
       ...pubsubService,
       ...options?.services
     }
   }) as any as Libp2p; // TODO: make libp2p include it;
 }
 
 export async function createLibp2pAndUpdateOptions(
@@ -138,12 +138,15 @@
     options.shardInfo = { contentTopics: options.contentTopics };
   }
 
+  if (!options.shardInfo) {
+    options.shardInfo = DefaultShardInfo;
+  }
+
   const shardInfo = options.shardInfo
     ? ensureShardingConfigured(options.shardInfo)
     : undefined;
 
-  options.pubsubTopics = shardInfo?.pubsubTopics ??
-    options.pubsubTopics ?? [DefaultPubsubTopic];
+  options.pubsubTopics = options.pubsubTopics ?? shardInfo?.pubsubTopics;
 
   return shardInfo?.shardInfo;
 }

packages/tests/src/lib/index.ts

@@ -1,12 +1,9 @@
 import { DecodedMessage } from "@waku/core";
-import {
-  DefaultPubsubTopic,
-  PubsubTopic,
-  ShardingParams
-} from "@waku/interfaces";
+import { PubsubTopic, ShardingParams } from "@waku/interfaces";
 import { ensureShardingConfigured, Logger } from "@waku/utils";
 import { expect } from "chai";
 
+import { DefaultTestPubsubTopic } from "../constants";
 import { Args, MessageRpcQuery, MessageRpcResponse } from "../types";
 import { delay, makeLogFileName } from "../utils/index.js";
 
@@ -105,7 +102,7 @@ export class ServiceNodesFleet {
 
   public async sendRelayMessage(
     message: MessageRpcQuery,
-    pubsubTopic: string = DefaultPubsubTopic
+    pubsubTopic: string = DefaultTestPubsubTopic
   ): Promise<boolean> {
     const relayMessagePromises: Promise<boolean>[] = this.nodes.map((node) =>
       node.sendMessage(message, pubsubTopic)
@@ -221,7 +218,7 @@ class MultipleNodesMessageCollector {
     }
   ): Promise<boolean> {
     const startTime = Date.now();
-    const pubsubTopic = options?.pubsubTopic || DefaultPubsubTopic;
+    const pubsubTopic = options?.pubsubTopic || DefaultTestPubsubTopic;
     const timeoutDuration = options?.timeoutDuration || 400;
     const exact = options?.exact || false;
 

packages/tests/src/lib/message_collector.ts

@@ -1,10 +1,10 @@
 import { DecodedMessage } from "@waku/core";
-import { DefaultPubsubTopic } from "@waku/interfaces";
 import { Logger } from "@waku/utils";
 import { bytesToUtf8, utf8ToBytes } from "@waku/utils/bytes";
 import { AssertionError, expect } from "chai";
 import { equals } from "uint8arrays/equals";
 
+import { DefaultTestPubsubTopic } from "../constants.js";
 import { MessageRpcResponse } from "../types.js";
 import { base64ToUtf8 } from "../utils/base64_utf8.js";
 import { delay } from "../utils/delay.js";
@@ -269,6 +269,8 @@ export class MessageCollector {
   }
 
   private getPubsubTopicToUse(pubsubTopic: string | undefined): string {
-    return pubsubTopic || this.nwaku?.pubsubTopics?.[0] || DefaultPubsubTopic;
+    return (
+      pubsubTopic || this.nwaku?.pubsubTopics?.[0] || DefaultTestPubsubTopic
+    );
   }
 }

packages/tests/src/lib/service_node.ts

@@ -1,12 +1,12 @@
 import type { PeerId } from "@libp2p/interface";
 import { peerIdFromString } from "@libp2p/peer-id";
 import { Multiaddr, multiaddr } from "@multiformats/multiaddr";
-import { DefaultPubsubTopic } from "@waku/interfaces";
 import { isDefined } from "@waku/utils";
 import { Logger } from "@waku/utils";
 import pRetry from "p-retry";
 import portfinder from "portfinder";
 
+import { DefaultTestPubsubTopic } from "../constants.js";
 import {
   Args,
   LogLevel,
@@ -245,7 +245,7 @@
   }
 
   public async ensureSubscriptions(
-    pubsubTopics: string[] = [DefaultPubsubTopic]
+    pubsubTopics: string[] = [DefaultTestPubsubTopic]
   ): Promise<boolean> {
     return this.restCall<boolean>(
       "/relay/v1/subscriptions",
@@ -257,7 +257,7 @@
 
   public async messages(pubsubTopic?: string): Promise<MessageRpcResponse[]> {
     return this.restCall<MessageRpcResponse[]>(
-      `/relay/v1/messages/${encodeURIComponent(pubsubTopic || this?.args?.pubsubTopic?.[0] || DefaultPubsubTopic)}`,
+      `/relay/v1/messages/${encodeURIComponent(pubsubTopic || this?.args?.pubsubTopic?.[0] || DefaultTestPubsubTopic)}`,
       "GET",
       null,
       async (response) => {
@@ -291,7 +291,7 @@
     }
 
     return this.restCall<boolean>(
-      `/relay/v1/messages/${encodeURIComponent(pubsubTopic || this.args?.pubsubTopic?.[0] || DefaultPubsubTopic)}`,
+      `/relay/v1/messages/${encodeURIComponent(pubsubTopic || this.args?.pubsubTopic?.[0] || DefaultTestPubsubTopic)}`,
       "POST",
       message,
       async (response) => response.status === 200
@@ -375,7 +375,7 @@
   public async restCall<T>(
     endpoint: string,
     method: "GET" | "POST",
     body: any = null,
     processResponse: (response: Response) => Promise<T>
   ): Promise<T> {
     this.checkProcess();
@@ -411,7 +411,8 @@
     rest: true,
     restAdmin: true,
     websocketSupport: true,
-    logLevel: LogLevel.Trace
+    logLevel: LogLevel.Trace,
+    pubsubTopic: ["/waku/2/rs/0/0"]
   };
 }
 

packages/tests/tests/connection-mananger/connection_state.spec.ts

@@ -90,10 +90,12 @@ describe("Connection state", function () {
 
   it("`waku:online` between 2 js-waku relay nodes", async function () {
     const waku1 = await createRelayNode({
-      staticNoiseKey: NOISE_KEY_1
+      staticNoiseKey: NOISE_KEY_1,
+      shardInfo: DefaultTestShardInfo
     });
     const waku2 = await createRelayNode({
-      libp2p: { addresses: { listen: ["/ip4/0.0.0.0/tcp/0/ws"] } }
+      libp2p: { addresses: { listen: ["/ip4/0.0.0.0/tcp/0/ws"] } },
+      shardInfo: DefaultTestShardInfo
     });
 
     let eventCount1 = 0;

packages/tests/tests/enr.node.spec.ts

@@ -13,6 +13,7 @@ import {
   ServiceNode,
   tearDownNodes
 } from "../src/index.js";
+import { DefaultTestShardInfo } from "../src/index.js";
 
 describe("ENR Interop: ServiceNode", function () {
   let waku: RelayNode;
@@ -35,7 +36,8 @@ describe("ENR Interop: ServiceNode", function () {
     const multiAddrWithId = await nwaku.getMultiaddrWithId();
 
     waku = await createRelayNode({
-      staticNoiseKey: NOISE_KEY_1
+      staticNoiseKey: NOISE_KEY_1,
+      shardInfo: DefaultTestShardInfo
     });
     await waku.start();
     await waku.dial(multiAddrWithId);
@@ -68,7 +70,8 @@ describe("ENR Interop: ServiceNode", function () {
     const multiAddrWithId = await nwaku.getMultiaddrWithId();
 
     waku = await createRelayNode({
-      staticNoiseKey: NOISE_KEY_1
+      staticNoiseKey: NOISE_KEY_1,
+      shardInfo: DefaultTestShardInfo
     });
     await waku.start();
     await waku.dial(multiAddrWithId);
@@ -102,7 +105,8 @@ describe("ENR Interop: ServiceNode", function () {
     const multiAddrWithId = await nwaku.getMultiaddrWithId();
 
     waku = await createRelayNode({
-      staticNoiseKey: NOISE_KEY_1
+      staticNoiseKey: NOISE_KEY_1,
+      shardInfo: DefaultTestShardInfo
     });
     await waku.start();
     await waku.dial(multiAddrWithId);