Refactor library

* Throw if attaching without valid parser
* Use descriptive name for plugin

Author: wooorm

index.js

@@ -1,19 +1,26 @@
 'use strict';
 
-var gemoji = require('gemoji').name;
+var byName = require('gemoji').name;
 
-module.exports = plugin;
+module.exports = gemoji;
 
 var colon = ':';
 var own = {}.hasOwnProperty;
 
-function plugin() {
-  var proto = this.Parser.prototype;
-  var methods = proto.inlineMethods;
+tokenize.locator = locate;
 
-  proto.inlineTokenizers.gemojiShortCode = tokenize;
+function gemoji() {
+  var parser = this.Parser;
+  var proto;
+
+  if (!isRemarkParser(parser)) {
+    throw new Error('Missing parser to attach `remark-gemoji` to');
+  }
+
+  proto = this.Parser.prototype;
 
-  methods.splice(methods.indexOf('strong'), 0, 'gemojiShortCode');
+  proto.inlineTokenizers.gemojiShortCode = tokenize;
+  proto.inlineMethods.splice(proto.inlineMethods.indexOf('strong'), 0, 'gemojiShortCode');
 }
 
 function tokenize(eat, value, silent) {
@@ -33,7 +40,7 @@ function tokenize(eat, value, silent) {
 
   subvalue = value.slice(1, pos);
 
-  if (!own.call(gemoji, subvalue)) {
+  if (!own.call(byName, subvalue)) {
     return;
   }
 
@@ -48,14 +55,13 @@ function tokenize(eat, value, silent) {
   /* Eat a text-node. */
   subvalue = colon + subvalue + colon;
 
-  return eat(subvalue)({
-    type: 'text',
-    value: subvalue
-  });
+  return eat(subvalue)({type: 'text', value: subvalue});
 }
 
-tokenize.locator = locate;
-
 function locate(value, fromIndex) {
   return value.indexOf(colon, fromIndex);
 }
+
+function isRemarkParser(parser) {
+  return Boolean(parser && parser.prototype && parser.prototype.inlineTokenizers);
+}

package.json

@@ -27,10 +27,11 @@
     "browserify": "^14.0.0",
     "esmangle": "^1.0.0",
     "nyc": "^11.0.0",
-    "remark": "^8.0.0",
     "remark-cli": "^4.0.0",
+    "remark-parse": "^4.0.0",
     "remark-preset-wooorm": "^3.0.0",
     "tape": "^4.0.0",
+    "unified": "^6.1.6",
     "xo": "^0.18.0"
   },
   "scripts": {

test.js

@@ -1,13 +1,22 @@
 'use strict';
 
 var test = require('tape');
-var remark = require('remark');
-var gemoji = require('./index.js');
+var unified = require('unified');
+var parse = require('remark-parse');
+var gemoji = require('./');
 
 test('remark-gemoji', function (t) {
+  t.throws(
+    function () {
+      unified().use(gemoji).freeze();
+    },
+    /^Error: Missing parser to attach `remark-gemoji` to/,
+    'should throw if without parser'
+  );
+
   t.deepEqual(
-    remark()
-      .data('settings', {pedantic: true, position: false})
+    unified()
+      .use(parse, {pedantic: true, position: false})
       .use(gemoji)
       .parse(':a_b_c:'),
     {
@@ -26,12 +35,13 @@ test('remark-gemoji', function (t) {
           {type: 'text', value: 'c:'}
         ]
       }]
-    }
+    },
+    'should ignore unknown gemoji'
   );
 
   t.deepEqual(
-    remark()
-      .data('settings', {pedantic: true, position: false})
+    unified()
+      .use(parse, {pedantic: true, position: false})
       .use(gemoji)
       .parse(':heavy_check_mark:'),
     {
@@ -45,7 +55,8 @@ test('remark-gemoji', function (t) {
           }
         ]
       }]
-    }
+    },
+    'should support gemoji'
   );
 
   t.end();