HelpMeeee

Вот по-быстрому накидал. Соответственно, у того элемента, у которого стоит класс active фон белый с обводкой красной, слева красные, справа серые. Если класса active нет вообще, считай, что путь пройден/завершен. Если задача в процессе, то хотя бы один элемент должен быть с классом active, а то иначе смысл тогда ...

Дело в приоритете операторов: у && он выше, чем у ||. Поэтому выполняется так:
pin.length == 4 || (pin.length == 6 && a == 1)
Лечится расстановкой скобок.

Кроме того желательно не сравнивать a с 1: т.к. метод test() возвращает Boolean true или false, можно прямо проверятьif (a && (pin.length === 4 || pin.length === 6)) {

Их нужно преобразовать в строку

Не нужно.

по идее должно выдать два совпадение (5,6)

Нет, в представленных массивах совпадают три элемента: 5, 6 и 8.
Впрочем, если необходимо чтоб сопадали ключ и значение, то тогда таких элементнов как раз будет два: 5 и 6

как реализовать задачу

$intersection = array_intersect_assoc($m, $m1);

PS. Если итоговый результат нужен все ж в виде строки, то можно преобразовать в нее выходной массив:
$intersectionString = implode(",", $intersection);