Update to support new linter APIs

README.md

@@ -3,23 +3,19 @@ linter-gjslint
 
 This linter plugin for [Linter](https://github.com/AtomLinter/Linter) provides an interface to [gjslint](https://developers.google.com/closure/utilities/). It will be used with files that have the “JavaScript” syntax.
 
-## Installation
-Linter package must be installed in order to use this plugin. If Linter is not installed, please follow the instructions [here](https://github.com/AtomLinter/Linter).
 
 ### gjslint installation
 Before using this plugin, you must ensure that `gjslint` is installed on your system. To install `gjslint`, read [this](https://developers.google.com/closure/utilities/docs/linter_howto).
 
-### Plugin installation
-```
-$ apm install linter-gjslint
-```
+## Installation
+Run `amp install linter-gjslint` or search for `linter-gjslint` in Atom package manager.
 
 ## Settings
 You can configure linter-gjslint by editing ~/.atom/config.cson (choose Open Your Config in Atom menu):
 ```
 'linter-gjslint':
-  'gjslintExecutablePath': '' #gjslint path. run 'which gjslint' to find the path
-  'gjslintIgnoreList': [] #gjslint ignore codes from http://goo.gl/OhYHYl
+  'executablePath': '' #gjslint path. run 'which gjslint' to find the path
+  'gjslintIgnoreList': [] # Currently not implemented: gjslint ignore codes from http://goo.gl/OhYHYl
 ```
 
 ## Contributing

lib/init.coffee

@@ -1,13 +1,56 @@
+{CompositeDisposable} = require 'atom'
+{findFile, exec, tempFile} = helpers = require 'atom-linter'
+path = require 'path'
+
 module.exports =
   config:
-    gjslintExecutablePath:
+    executablePath:
       type: 'string'
-      default: ''
+      title: 'gjslint executable path'
+      default: 'gjslint'
     gjslintIgnoreList:
       type: 'array'
       default: []
       items:
         type: 'string'
 
   activate: ->
-    console.log 'activate linter-gjslint'
+    @subscriptions = new CompositeDisposable
+    @subscriptions.add atom.config.observe 'linter-gjslint.executablePath',
+      (executablePath) =>
+        @executablePath = executablePath
+  deactivate: ->
+    @subscriptions.dispose()
+  provideLinter: ->
+    helpers = require('atom-linter')
+    provider =
+      name: 'gjslint'
+      grammarScopes: ['source.js', 'source.js.jquery', 'text.html.basic']
+      scope: 'file'
+      lintOnFly: true
+      lint: (editor) =>
+        filePath = editor.getPath()
+        cwd = path.dirname(filePath)
+        tempFile path.basename(filePath), editor.getText(), (tmpFilePath) =>
+          params = [
+            '--nobeep',
+            '--quiet',
+            tmpFilePath
+          ]
+          return helpers.exec(@executablePath, params, {cwd}).then (stdout) ->
+            regex = /Line\s(\d+), E:(\d+):\s(.+)/
+            lines = stdout.split('\n').filter (line) ->
+              line.indexOf('Line') is 0
+
+            return [] unless lines.length
+            return lines.map (msg) ->
+              res = regex.exec(msg)
+              [all, line, code, text] = res
+              line = parseInt(line, 10)
+              text = 'E:' + code + ' - ' + text
+              return {
+                text: text,
+                type: 'error',
+                filePath: filePath,
+                range: helpers.rangeFromLineNumber(editor, line - 1)
+              }

package.json

@@ -1,13 +1,25 @@
 {
   "name": "linter-gjslint",
   "main": "./lib/init",
-  "linter-package": true,
   "version": "0.0.5",
   "description": "Linter plugin for JavaScript, using gjslint (Google Closure Linter)",
   "repository": "https://github.com/AtomLinter/linter-gjslint",
   "license": "MIT",
   "engines": {
     "atom": ">0.50.0"
   },
-  "dependencies": {}
+  "dependencies": {
+    "atom-linter": "^3.2.0",
+    "atom-package-deps": "^2.0.2"
+  },
+  "package-deps": [
+    "linter"
+  ],
+  "providedServices": {
+    "linter": {
+      "versions": {
+        "1.1.0": "provideLinter"
+      }
+    }
+  }
 }