var AllDialogTimer = {}
var s = top.frames[external.mainFrameIndex].document.documentElement.innerHTML;
 
function AllDialog(enabled)
{
    if(enabled)
    {
        AllDialogTimer[mylogin] = setInterval("AllDialogHandler()", 3000);
    }
    else
    {
        clearInterval(AllDialogTimer[mylogin]);        
    }
}
function AllDialogHandler()
{
if (external.SCity.Text == "emeraldscity")
{
var Say = 0;
var pattern = "drwfl\\(\"Берт|Ализей\"";
if(!top.frames[external.mainFrameIndex].document.all[0].innerHTML.match(pattern)) return;
 
if(s.match(/Что тебе еще нужно?/))
    {
    if (typeof(Say)  == "undefined") {var Say = 0;}
    if (Say == 0) {Say = Say + 1;
            if(s.match(/Ты же мне задания давал? Я по поводу них.../))
            external.ClickLink("Ты же мне задания давал? Я по поводу них...",2);}
    else if (Say == 1) {Say = Say + 1;
            if(s.match(/Ты похож на человека, у которого есть для меня задания./))
            external.ClickLink("Ты похож на человека, у которого есть для меня задания.",2);}
    else if (Say == 2) {Say = Say + 1;
            if(s.match(/Ты похож на человека, у которого есть для меня задания./))
            external.ClickLink("Ты похож на человека, у которого есть для меня задания.",2);}
    else if(Say == 3) {Say = Say + 1;
            if(s.match(/Да нет, уже ничего не надо./))
            external.ClickLink("Да нет, уже ничего не надо.",2);}
    else { Say = 0; } // обнуление при любом значение кроме 0,1,2,3
    }
}
 
function ExitAllDialogHandler()
{
    top.frames[external.mainFrameIndex].location = "http://" + external.SCity.Text + ".combats.com/" + dungName  + "?rnd=" + Math.random() +"&path=m3";
}

Упрощенно:
Дано: 1, 2, 3, 4, 5
Надо ткнуть на: 3, 1, 1 и else 5

что именно не пашет?
я игровых диалогов не знаю... давай так сделаем...
выложи под спойлером (что бы не захламлять ветку) дерево диалога в виде

if(s.match(/if(s.match(/Что тебе еще нужно?/))

Насколько понимаю случайно продублировалось, попробовал конструкцию плагин на других диалогах начал работать, до этого вообще колом вставал. Но именно на этой развязке стопорится и ничего делать не хочет. Попробовал внутри конструкции еще раз ввести поиск, нефига не помогло:

if(s.match(/Что тебе еще нужно?/))
    {
    if (typeof(Say)  == "undefined") {var Say = 0;}
    if (Say == 0) {Say = Say + 1;
            if(s.match(/Ты же мне задания давал? Я по поводу них.../))
            external.ClickLink("Ты же мне задания давал? Я по поводу них...",2);}
   else if (Say == 1) {Say = Say + 1;
            if(s.match(/Ты похож на человека, у которого есть для меня задания./))
            external.ClickLink("Ты похож на человека, у которого есть для меня задания.",2);}
    else if (Say == 2) {Say = Say + 1;
            if(s.match(/Ты похож на человека, у которого есть для меня задания./))
            external.ClickLink("Ты похож на человека, у которого есть для меня задания.",2);}
    else if(Say == 3) {Say = Say + 1;
            if(s.match(/Да нет, уже ничего не надо./))
            external.ClickLink("Да нет, уже ничего не надо.",2);}
    else { Say = 0; } // обнуление при любом значение кроме 0,1,2,3
    }

Ну суть понятно, подойти сдать задание, взять задание, если задание было отменено крестиком перезайти и взять, ну и на всякий непредвиденный случай, если не выкинет из разговора после взятия задания, то выйти. Может есть другой способ как это реализовать? Просто стандартными способом не представляю как заставить работать, ведь если ввести по поиску Взять задание и Сдать задание, то будет кликаться только то что в ХТМЛ первым стоит по порядку.

Хотя вот выше посмотрел, можно попробовать реализовать на примере обменника катовского. Но сколько его смотрел структуру пока в логику не въехал, как его переделывать 

if (typeof(Say)  == "undefined" || Say == 4) {var Say = 0;}

2. переменная "Say" и "say" это две разных переменных
Ты определяешь есть ли переменная Say и задаешь ее равной 0

if (typeof(Say)  == "undefined" || Say == 4) {var Say = 0;}

потом проверяешь значение переменной say

if (say == 0);{Say = Say + 1;

надо

if (Say == 0);{Say = Say + 1;

3. Лишняя ; после скобок у оператора if(), правильно так

if (say == 0){Say = Say + 1;
            external.ClickLink("Ты же мне задания давал? Я по поводу них...",2);}

+ еще один момент в данной конструкции у тебя по очереди выполняются все клики за один проход кода.
наверное стоит использовать конструкцию if (...) {...} else if (...) {...} else {...}

if(s.match(/if(s.match(/Что тебе еще нужно?/))
    {
    if (typeof(Say)  == "undefined") {var Say = 0;}
    if (Say == 0) {Say = Say + 1;
            external.ClickLink("Ты же мне задания давал? Я по поводу них...",2);}
   else if (Say == 1) {Say = Say + 1;
            external.ClickLink("Ты похож на человека, у которого есть для меня задания.",2);}
    else if (Say == 2) {Say = Say + 1;
            external.ClickLink("Ты похож на человека, у которого есть для меня задания.",2);}
    else if(Say == 3) {Say = Say + 1;
            external.ClickLink("Да нет, уже ничего не надо.",2);}
    else { Say = 0; } // обнуление при любом значение кроме 0,1,2,3
    }

в данном виде у тебя за один проход кода будет выполнятся один клик