feat: add location to error messages

src/go-slang/error.ts

@@ -1,4 +1,5 @@
 import { RuntimeSourceError } from '../errors/runtimeSourceError'
+import { NodeLocation } from './types'
 
 export class UnknownInstructionError extends RuntimeSourceError {
   private inst_type: string
@@ -20,9 +21,12 @@ export class UnknownInstructionError extends RuntimeSourceError {
 export class UndefinedError extends RuntimeSourceError {
   private identifier: string
 
-  constructor(identifier: string) {
+  public location: NodeLocation
+
+  constructor(identifier: string, location: NodeLocation) {
     super()
     this.identifier = identifier
+    this.location = location
   }
 
   public explain() {
@@ -35,11 +39,14 @@ export class FuncArityError extends RuntimeSourceError {
   private n_actual: number
   private n_expected: number
 
-  constructor(func_name: string, n_actual: number, n_expected: number) {
+  public location: NodeLocation
+
+  constructor(func_name: string, n_actual: number, n_expected: number, location: NodeLocation) {
     super()
     this.func_name = func_name
     this.n_actual = n_actual
     this.n_expected = n_expected
+    this.location = location
   }
 
   public explain() {
@@ -54,9 +61,12 @@ export class FuncArityError extends RuntimeSourceError {
 export class GoExprMustBeFunctionError extends RuntimeSourceError {
   private expr: string
 
-  constructor(expr: string) {
+  public location: NodeLocation
+
+  constructor(expr: string, location: NodeLocation) {
     super()
     this.expr = expr
+    this.location = location
   }
 
   public explain() {

src/go-slang/goroutine.ts

@@ -175,9 +175,9 @@ const Interpreter: {
     C.pushR(...H.allocM(right), ...H.allocM(asgmts.reverse()))
   },
 
-  GoStatement: ({ call }: GoStatement, { C, H, A }) => {
+  GoStatement: ({ call, loc }: GoStatement, { C, H, A }) => {
     if (call.type !== NodeType.CallExpression) {
-      return new GoExprMustBeFunctionError(call.type)
+      return new GoExprMustBeFunctionError(call.type, loc!)
     }
 
     const { callee, args } = call as CallExpression
@@ -198,9 +198,9 @@ const Interpreter: {
 
   Literal: (inst: Literal, { S, H }) => S.push(H.alloc(inst.value)),
 
-  Identifier: ({ name }: Identifier, { S, E, H }) => {
+  Identifier: ({ name, loc }: Identifier, { S, E, H }) => {
     const value = E.lookup(name)
-    return value === null ? new UndefinedError(name) : S.push(H.alloc(value))
+    return value === null ? new UndefinedError(name, loc!) : S.push(H.alloc(value))
   },
 
   UnaryExpression: ({ argument, operator: op }: UnaryExpression, { C, H, A }) =>
@@ -227,8 +227,8 @@ const Interpreter: {
   },
 
   AssignOp: ({ idNodeUid }: AssignOp, { S, E, H, A }) => {
-    const name = A.get<Identifier>(idNodeUid).name
-    !E.assign(name, H.resolve(S.pop())) ? new UndefinedError(name) : void {}
+    const id = A.get<Identifier>(idNodeUid)
+    !E.assign(id.name, H.resolve(S.pop())) ? new UndefinedError(id.name, id.loc!) : void {}
   },
 
   UnaryOp: ({ opNodeId }: UnaryOp, { S, H, A }) => {
@@ -257,7 +257,7 @@ const Interpreter: {
 
     if (paramNames.length !== values.length) {
       const calleeId = A.get<Identifier>(calleeNodeId)
-      return new FuncArityError(calleeId.name, values.length, params.length)
+      return new FuncArityError(calleeId.name, values.length, params.length, calleeId.loc!)
     }
 
     C.pushR(...H.allocM([body, RetMarker, { type: CommandType.EnvOp, envId: E.id() }]))
@@ -277,7 +277,7 @@ const Interpreter: {
 
     if (paramNames.length !== values.length) {
       const calleeId = A.get<Identifier>(calleeNodeId)
-      return new FuncArityError(calleeId.name, values.length, params.length)
+      return new FuncArityError(calleeId.name, values.length, params.length, calleeId.loc!)
     }
 
     const _C: Control = new Stack()

src/go-slang/types.ts

@@ -45,7 +45,7 @@ interface Position {
   column: number
   offset: number
 }
-interface NodeLocation {
+export interface NodeLocation {
   start: Position
   end: Position
   offset: number