afelix/setochka

Divide et impera

setochka's Issues

Запись результирующих файлов некорректно работает не из каталога с файлами

Не учитывается, что "имя файла" может содержать путь с директориямии и получается:

Error: EBADF, Bad file descriptor 'webkit.pages/index/_index.css'

Feedback for "setochka"

Пустая страница http://ru.bem.info/tools/optimizers/setochka/

setochka

https://github.com/afelix/setochka/tree/master/README.ru.md

Ошибка при селекторах состоящих только из псевдоселекторов

Очень возможно что это баг cssp, но вот такие стили не обрабатываются:

:link:hover, :visited:hover { color: #f00 !important }

Полноценный матчинг mark

Надо, чтобы Сеточка матчила mark (по сути, комментарий) так же полноценно (regexp, функции), как и остальное.

Алиасы

setochka -a "alias0, alias1, aliasN"
setochka --aliases "alias0, alias1, aliasN"
    перечисляет синонимы для использования

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

было бы удобно иметь стандартное поведение для перечислимых опций:

setochka -a alias0 -a alias1 -a aliasN

Учитывать at-rule

Сеточка не умеет матчить at-rule. Должна уметь.

Обновить npm-установку и конфиг

Обновить всё для последних node и npm.

Валится на определенных вхождениях

1. Валится на хаке для ие "//":

.class {//color:red;}

без ошибки.

1. Валится на фильтрах

.class {
filter: progid:DXImageTransform.Microsoft.AlphaImageLoader(src =/res/default/Images/prompt_yellow.png, sizingMethod = crop);
}

с ошибкой

Rule: filterp not found!
Trace
    at CSSTranslator.invoke (/usr/local/lib/node_modules/setochka/node_modules/cssp/node_modules/ometajs/lib/ometajs/core/grammar.js:159:15)
    at CSSTranslator.cache [as _cache] (/usr/local/lib/node_modules/setochka/node_modules/cssp/node_modules/ometajs/lib/ometajs/core/parser.js:123:16)
    at CSSTranslator._invoke (/usr/local/lib/node_modules/setochka/node_modules/cssp/node_modules/ometajs/lib/ometajs/core/grammar.js:168:17)
    at CSSTranslator.rule [as _rule] (/usr/local/lib/node_modules/setochka/node_modules/cssp/node_modules/ometajs/lib/ometajs/core/grammar.js:269:16)
    at Function.matchAll (/usr/local/lib/node_modules/setochka/node_modules/cssp/node_modules/ometajs/lib/ometajs/core/grammar.js:60:13)
    at Function.match (/usr/local/lib/node_modules/setochka/node_modules/cssp/node_modules/ometajs/lib/ometajs/core/grammar.js:36:15)
    at Object.exports.translate (/usr/local/lib/node_modules/setochka/node_modules/cssp/lib/csspapi.js:18:56)
    at translateBlock (/usr/local/lib/node_modules/setochka/lib/setochka.js:193:22)
    at extractFromBlock (/usr/local/lib/node_modules/setochka/lib/setochka.js:146:18)
    at extractFromTree (/usr/local/lib/node_modules/setochka/lib/setochka.js:79:28)
    at Object.<anonymous> (/usr/local/lib/node_modules/setochka/lib/setochka.js:33:9)

/usr/local/lib/node_modules/setochka/node_modules/cssp/node_modules/ometajs/lib/ometajs/core/grammar.js:74
  if (!errback) errback = function (err) { throw err; };
                                                 ^
Error: filterp rule failed at: [ [ 'property', [ 'ident', '_filter' ] ] ]:0
    at CSSTranslator.getError [as _getError] (/usr/local/lib/node_modules/setochka/node_modules/cssp/node_modules/ometajs/lib/ometajs/core/grammar.js:81:37)
    at Function.matchAll (/usr/local/lib/node_modules/setochka/node_modules/cssp/node_modules/ometajs/lib/ometajs/core/grammar.js:61:17)
    at Function.match (/usr/local/lib/node_modules/setochka/node_modules/cssp/node_modules/ometajs/lib/ometajs/core/grammar.js:36:15)
    at Object.exports.translate (/usr/local/lib/node_modules/setochka/node_modules/cssp/lib/csspapi.js:18:56)
    at translateBlock (/usr/local/lib/node_modules/setochka/lib/setochka.js:193:22)
    at extractFromBlock (/usr/local/lib/node_modules/setochka/lib/setochka.js:146:18)
    at extractFromTree (/usr/local/lib/node_modules/setochka/lib/setochka.js:79:28)
    at Object.<anonymous> (/usr/local/lib/node_modules/setochka/lib/setochka.js:33:9)
    at Module._compile (module.js:456:26)
    at Object.Module._extensions..js (module.js:474:10)
    at Module.load (module.js:356:32)

1. Валится на ошибках в стилях

.class{color:red:}
{display: none;}

с ошибкой

TypeError: Cannot read property 'length' of undefined
    at translateBlock (/usr/local/lib/node_modules/setochka/lib/setochka.js:186:30)
    at extractFromBlock (/usr/local/lib/node_modules/setochka/lib/setochka.js:146:18)
    at extractFromTree (/usr/local/lib/node_modules/setochka/lib/setochka.js:79:28)
    at Object.<anonymous> (/usr/local/lib/node_modules/setochka/lib/setochka.js:33:9)
    at Module._compile (module.js:456:26)
    at Object.Module._extensions..js (module.js:474:10)
    at Module.load (module.js:356:32)
    at Function.Module._load (module.js:312:12)
    at Function.Module.runMain (module.js:497:10)
    at startup (node.js:119:16)

Ладно что валится на ошибках в стилях, но нигде это не показывает, а в некоторых случаях валится без ошибки и вообще не понятно, что не все работает как надо, просто до проблемного места стили парсятся, а дальше нет.

Более кроссплатформенное использование bash

diff --git a/bin/setochka b/bin/setochka
index 3ef6375..52956c1 100755
--- a/bin/setochka
+++ b/bin/setochka
@@ -1,4 +1,4 @@
-#!/bin/bash
+#!/usr/bin/env bash
·
 SELF_PATH=$(cd -P -- "$(dirname -- "$0")" && pwd -P) && SELF_PATH="$SELF_PATH/$(basename -- "$0")"

В качестве шаблонов удобно было бы иметь и регекспы и функции

Тесты

Написать тесты.

Ошибка чтения конфига

Не понимаю, как такое могло получиться, но без вот такого патча конфиги вообще не обрабатывались:

diff --git a/lib/setochka.js b/lib/setochka.js
index 0b7c9e7..99fe1e1 100644
--- a/lib/setochka.js
+++ b/lib/setochka.js
@@ -18,7 +18,7 @@ if (opts['--version'] || opts['-v']) printFile('VERSION');
 else if (opts['--help'] || opts['-h']) printFile('USAGE');
 else {
     if (usrCfgFile = (opts['--replace-config'] || opts['-rc'])) {
-        config = require(usrCfgFile);
+        config = require(usrCfgFile).config;
     } else if (usrCfgFile = (opts['--merge-config'] || opts['-mc'])) {
         mergeConfig(config, require(usrCfgFile));
     }

Вычистить код от остатков прототипа

Вычистить код от остатков прототипа. Сейчас очень грязно.