Some changes to the linter to support column number in error and scope option so we can eventually work out how we can run the linter on server files

Author: Monte Goulding

lib/main.coffee

@@ -21,7 +21,7 @@ module.exports =
   provideLinter: ->
     helpers = require('atom-linter')
     provider =
-      grammarScopes: ['source.livecodescript']
+      grammarScopes: ['source.livecodescript', 'source.iRev']
       scope: 'file'
       lintOnFly: true
       lint: (textEditor) =>
@@ -31,14 +31,16 @@ module.exports =
           parameters.push('-ui')
           stackfile = @linterPath
           parameters.push(stackfile)
+          scope = '-scope=' + textEditor.getRootScopeDescriptor()
+          parameters.push(scope)
           text = textEditor.getText()
           return helpers.exec(command, parameters, {stdin: text}).then (output) ->
-            regex = /(\d+),(.*)/g
+            regex = /(\d+),(\d+),(.*)/g
             messages = []
             while((match = regex.exec(output)) isnt null)
               messages.push
                 type: "Error"
                 filePath: filePath
-                range: helpers.rangeFromLineNumber(textEditor, match[1]-1)
-                text: match[2]
+                range: [[match[1]-1, match[2]-0],[match[1]-1, textEditor.getBuffer().lineLengthForRow(match[1]-1)]]
+                text: match[3]
             return messages

tools/Linter.livecodescript

@@ -1,39 +1,44 @@
 script "Linter"
 
+local sErrorsList
+
 on startup
   try
-    read from stdin until empty
-    put it into theScript
+    local tScope
+    repeat for each element theArgument in the commandArguments
+      if theArgument begins with "-scope" then
+         split theArgument with "="
+         put theArgument[2] into tScope
+      end if
+    end repeat
 
-    local isScriptOnly = false
-    if word 1 of theScript is "script" then
-      put true into isScriptOnly
-    end if
+    read from stdin until empty
+    put it into tScript
 
     put the filename of me into theFilename
     set the itemDelimiter to slash
     put "scriptErrors.txt" into item -1 of theFilename
-    put url ("file:"&theFilename) into theErrorsList
+    put url ("file:"&theFilename) into sErrorsList
+    split sErrorsList with return
 
     create script only stack "TestScript"
 
-    if isScriptOnly then
-      delete line 1 of theScript
-    end if
-
-    set the script of stack "TestScript" to theScript
+    if tScope is empty or tScope is ".source.livecodescript" then
+      local tLineOffset = 0
 
-    put the result into theErrors
-    set the itemDelimiter to comma
+      -- check for script only
+      if word 1 of tScript is "script" then
+        put 1 into tLineOffset
+      end if
 
-    repeat for each line theError in theErrors
-      if theError is not empty then
-        if isScriptOnly then
-          add 1 to item 2 of theError
-        end if
-        write item 2 of theError, line (item 1 of theError) of theErrorsList & linefeed to stdout
+      if tLineOffset is 1 then
+        delete line 1 of tScript
       end if
-    end repeat
+
+      LintScript tScript, tLineOffset
+    else if tScope is ".source.iRev" then
+
+    end if
 
     write linefeed to stdout
   catch e
@@ -42,3 +47,16 @@ on startup
 
   quit
 end startup
+
+command LintScript pScript, pLineOffset
+  set the script of stack "TestScript" to pScript
+
+  put the result into theErrors
+  set the itemDelimiter to comma
+
+  repeat for each line theError in theErrors
+    if theError is not empty then
+      write item 2 of theError + pLineOffset, item 3 of theError, sErrorsList[item 1 of theError] & linefeed to stdout
+    end if
+  end repeat
+end LintScript