Work on checkline logic.

tools/apiview/emitters/typespec-apiview/main.tsp

@@ -2,14 +2,9 @@
 namespace Azure.Test {
 
   op GetFoo is ResourceRead<
-    {
-      @doc("The name")
-      name: string;
-    },
     {
       parameters:
         {
-          @doc("The collection id.")
           fooId: string;
         };
     }
@@ -28,8 +23,7 @@ namespace Azure.Test {
 
   // op GetFoo is ResourceRead<string, string>;
 
-  op ResourceRead<TResource, TParams>(
-    resource: TResource,
-    params: TParams
-  ): TResource;
+  op ResourceRead<T>(
+    params: T
+  ): void;
 }

tools/apiview/emitters/typespec-apiview/src/apiview.ts

@@ -241,17 +241,24 @@ export class ApiView {
     }
   }
 
-  private getLastNonBlankLine(): ReviewLine | undefined {
-    if (this.currentLine.Tokens.length > 0) {
-      return this.currentLine;
+  private getLastLine(line: ReviewLine | undefined): ReviewLine | undefined {
+    if (!line) {
+      return undefined;
+    }
+    const lastLine = line.Children[line.Children.length - 1];
+    if (lastLine.Children.length > 0) {
+      return this.getLastLine(lastLine);
     }
-    // FIXME: Need to recursively search
-    return this.currentParent?.Children[this.currentParent.Children.length - 1];
+    return lastLine
   }
 
-  private removeLastNonBlankLine() {
-    // FIXME: Need to recursively search
-    this.currentParent?.Children.pop();
+  private removeLastLine(line: ReviewLine) {
+    const lastLine = line.Children[line.Children.length - 1];
+    if (lastLine.Children.length > 0) {
+      this.removeLastLine(lastLine);
+    } else {
+      line.Children.pop();
+    }
   }
 
   /** Chomps whitespace to any of the provided characters. If the first non-whitespace
@@ -260,7 +267,7 @@ export class ApiView {
   private trimTo(characters: string) {
     const allowed = new Set(characters.split(""));
     const trimCurrent = (this.currentLine.Tokens.length !== 0);
-    const checkLine = trimCurrent ? this.currentLine : this.getLastNonBlankLine();
+    const checkLine = trimCurrent ? this.currentLine : this.getLastLine(this.currentParent);
     if (!checkLine) {
       return;
     }
@@ -284,8 +291,8 @@ export class ApiView {
           this.currentLine = checkLine;
           this.currentLine.Tokens = trimmedTokens;
           // get rid of the current last child since it's now currentLine
-          if (!trimCurrent) {
-            this.removeLastNonBlankLine();
+          if (!trimCurrent && this.currentParent) {
+            this.removeLastLine(this.currentParent);
           }
         }
         return;