Ihor Bratukh

Видимо, решетку надо взять в кавычки? a[href^="#"]

Фаервол

Определимся с элементом, у которого будем узнавать индекс - внутри кого он находится, что это за элемент, какой у него должен быть класс:

const parentSelector = 'div';
const childSelector = 'p';
const className = 'there';

const siblingsSelector = `${parentSelector} > ${childSelector}`;
const elementSelector = `${siblingsSelector}.${className}`;

Дальше есть варианты:

const index = Array.prototype.findIndex.call(
  document.querySelectorAll(siblingsSelector),
  n => n.classList.contains(className)
);

или

let index = -1;
for (
  let el = document.querySelector(elementSelector);
  el;
  index++, el = el.previousElementSibling
) ;

или

const el = document.querySelector(elementSelector);
const index = el ? [...el.parentNode.children].indexOf(el) : -1;

Если вдруг у элемента могут быть другие соседи, не соответствующие childSelector, и они не должны учитываться, то третий вариант не подходит, а во втором надо заменить index++ на index += el.matches(childSelector).

Просто убрать Reg Exp

function tag(input, key, value) {
    if (typeof key === 'string' && value !== undefined) {
        return input.replace('{' + key + '}', value);
    } else {
        for (let index in key) {
            input = input.replace('{' + index + '}', key[index]);
        }
    }

    return input;
}

Вот тесты:
Мой код 0.091064453125ms
Ваш 0.359130859375ms
Владлен Хеллсайт 0.243896484375ms

Засекать время через performance.now()

$.easing.bullshit = function(x, t, b, c, d) {
  return (t == d) ? b + c : c * (-Math.pow(2, -10 * t / d) + 1) + b;
};

$('.ach-number span').each(function() {
  $(this).prop('counter', 0).animate({
    counter: $(this).text(),
  }, {
    duration: 10000,
    easing: 'bullshit',
    step(val) {
      $(this).text(Math.ceil(val));
    },
  });
});

https://jsfiddle.net/ph4ku28v/

Ошибка в синтаксисе, jQuery надо сделать а не JQuery

const getCard = num => cards.find(n => n.id === num);
// ...
const card = getCard(cardRandom(1, 51));

Правда, не очень понятно, зачем такие сложности. Кроме того, непонятно, гарантируется ли существование объекта с указанным id. Может, лучше использовать случайное число как индекс? Типа так:

const card = cards[Math.random() * cards.length | 0];

Все уже давно реализовано и в лучшем виде.
_.get
_.set

const nestedVal = {
  get(path, root) {
    return path.split('.').reduce((p, c) => p[c], root);
  },
  set(path, root, val) {
    path = path.split('.');
    const key = path.pop();
    nestedVal.get(path.join('.'), root)[key] = val;
  }
};