Use an object type to represent Position

actions-languageservice/src/utils/find-token.ts

@@ -141,13 +141,16 @@ function posInToken(pos: Position, token: TemplateToken): boolean {
   const tokenChar = pos.character + 1;
 
   // Check lines
-  if (r.start[0] > tokenLine || tokenLine > r.end[0]) {
+  if (r.start.line > tokenLine || tokenLine > r.end.line) {
     return false;
   }
 
   // Position is within the token lines. Check character/column if pos line matches
   // start or end
-  if ((r.start[0] === tokenLine && tokenChar < r.start[1]) || (r.end[0] === tokenLine && tokenChar > r.end[1])) {
+  if (
+    (r.start.line === tokenLine && tokenChar < r.start.column) ||
+    (r.end.line === tokenLine && tokenChar > r.end.column)
+  ) {
     return false;
   }
 
@@ -163,14 +166,14 @@ function onSameLine(pos: Position, key: TemplateToken, value: TemplateToken): bo
     return false;
   }
 
-  if (value.range.start[0] !== value.range.end[0]) {
+  if (value.range.start.line !== value.range.end.line) {
     // Token occupies multiple lines, can't be an empty node
     return false;
   }
 
   // TokenRange is one-based, Position is zero-based
   const posLine = pos.line + 1;
-  if (posLine != value.range.start[0]) {
+  if (posLine != value.range.start.line) {
     return false;
   }
 

actions-languageservice/src/utils/range.ts

@@ -23,7 +23,7 @@ export function mapRange(range: TokenRange | undefined): Range {
 
 export function mapPosition(position: TokenPosition): Position {
   return {
-    line: position[0] - 1,
-    character: position[1] - 1
+    line: position.line - 1,
+    character: position.column - 1
   };
 }

actions-languageservice/src/validate.ts

@@ -168,8 +168,8 @@ function getProviderContext(
 ): WorkflowContext {
   const {parent, path} = findToken(
     {
-      line: token.range!.start[0] - 1,
-      character: token.range!.start[1] - 1
+      line: token.range!.start.line - 1,
+      character: token.range!.start.column - 1
     },
     root
   );

actions-workflow-parser/src/templates/template-reader.ts

@@ -500,11 +500,11 @@ class TemplateReader {
         );
 
         let tr = token.range!;
-        if (tr.start[0] === tr.end[0]) {
+        if (tr.start.line === tr.end.line) {
           // If it's a single line expression, adjust the range to only cover the sub-expression
           tr = {
-            start: [tr.start[0], tr.start[1] + startExpression],
-            end: [tr.end[0], tr.start[1] + endExpression + 1]
+            start: {line: tr.start.line, column: tr.start.column + startExpression},
+            end: {line: tr.end.line, column: tr.start.column + endExpression + 1}
           };
         } else {
           // Adjust the range to only cover the expression for multi-line strings
@@ -515,8 +515,8 @@ class TemplateReader {
           const adjustedEnd = endExpression - beginningOfLine + 1;
 
           tr = {
-            start: [tr.start[0] + adjustedStartLine, adjustedStart],
-            end: [tr.start[0] + adjustedStartLine, adjustedEnd]
+            start: {line: tr.start.line + adjustedStartLine, column: adjustedStart},
+            end: {line: tr.start.line + adjustedStartLine, column: adjustedEnd}
           };
         }
 

actions-workflow-parser/src/templates/tokens/template-token.ts

@@ -42,11 +42,11 @@ export abstract class TemplateToken {
   }
 
   public get line(): number | undefined {
-    return this.range?.start[0];
+    return this.range?.start.line;
   }
 
   public get col(): number | undefined {
-    return this.range?.start[1];
+    return this.range?.start.column;
   }
 
   public get definition(): Definition | undefined {

actions-workflow-parser/src/templates/tokens/token-range.ts

@@ -1,4 +1,10 @@
-export type Position = [line: number, character: number];
+/** Represents the position within a template object */
+export type Position = {
+  /** The one-based line value */
+  line: number;
+  /** The one-based column value */
+  column: number;
+};
 
 export type TokenRange = {
   start: Position;

actions-workflow-parser/src/workflows/yaml-object-reader.ts

@@ -89,8 +89,8 @@ export class YamlObjectReader implements ObjectReader {
       const elp = this.lineCounter.linePos(endPos);
 
       return {
-        start: [slp.line, slp.col],
-        end: [elp.line, elp.col]
+        start: {line: slp.line, column: slp.col},
+        end: {line: elp.line, column: elp.col}
       };
     }
 
@@ -216,7 +216,7 @@ function rangeFromLinePos(linePos: [LinePos] | [LinePos, LinePos] | undefined):
     return;
   }
   // TokenRange and linePos are both 1-based
-  const start: Position = [linePos[0].line, linePos[0].col];
-  const end: Position = linePos.length == 2 ? [linePos[1].line, linePos[1].col] : start;
+  const start: Position = {line: linePos[0].line, column: linePos[0].col};
+  const end: Position = linePos.length == 2 ? {line: linePos[1].line, column: linePos[1].col} : start;
   return {start, end};
 }