Add onNickConflict to Client.opts

README.md

@@ -20,3 +20,4 @@ You can access more detailed documentation for this module at [Read the Docs](ht
 The `node-irc` library isn't well maintained and there are a number of issues which are impacting development of the [Matrix-IRC application service](http://github.com/matrix-org/matrix-appservice-irc). We made the decision to fork the project in order to improve reliability of the application service. A summary of modifications from `[email protected]` are below:
  - https://github.com/matrix-org/node-irc/pull/1 - Manifested as [BOTS-80](https://matrix.org/jira/browse/BOTS-80)
  - https://github.com/matrix-org/node-irc/pull/4 - Manifested as [BOTS-73] (https://matrix.org/jira/browse/BOTS-73)
+ - Addition of `onNickConflict()` option which is called on `err_nicknameinuse`. This function should return the next nick to try. Defaults to suffixing monotonically increasing integers.

lib/irc.js

@@ -57,6 +57,12 @@ function Client(server, nick, opt) {
         channelPrefixes: '&#',
         messageSplit: 512,
         encoding: false,
+        onNickConflict: function() {
+            if (typeof (self.opt.nickMod) == 'undefined')
+                self.opt.nickMod = 0;
+            self.opt.nickMod++;
+            return self.opt.nick + self.opt.nickMod;
+        },
         webirc: {
           pass: '',
           ip: '',
@@ -228,11 +234,9 @@ function Client(server, nick, opt) {
                 // Random welcome crap, ignoring
                 break;
             case 'err_nicknameinuse':
-                if (typeof (self.opt.nickMod) == 'undefined')
-                    self.opt.nickMod = 0;
-                self.opt.nickMod++;
-                self.send('NICK', self.opt.nick + self.opt.nickMod);
-                self.nick = self.opt.nick + self.opt.nickMod;
+                var nextNick = self.opt.onNickConflict();
+                self.send('NICK', nextNick);
+                self.nick = nextNick;
                 self._updateMaxLineLength();
                 break;
             case 'PING':