Add ability to add row in a loop to existing table (#190)

Co-authored-by: swissspidy <841956+swissspidy@users.noreply.github.com>
Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>

Author: Copilot

lib/cli/Table.php

@@ -82,6 +82,17 @@ public function resetTable()
 		return $this;
 	}
 
+	/**
+	 * Resets only the rows in the table, keeping headers, footers, and width information.
+	 *
+	 * @return $this
+	 */
+	public function resetRows()
+	{
+		$this->_rows = array();
+		return $this;
+	}
+
 	/**
 	 * Sets the renderer used by this table.
 	 *
@@ -127,6 +138,33 @@ public function display() {
 		}
 	}
 
+	/**
+	 * Display a single row without headers or top border.
+	 *
+	 * This method is useful for adding rows incrementally to an already-rendered table.
+	 * It will display the row with side borders and a bottom border (if using Ascii renderer).
+	 *
+	 * @param array $row The row data to display.
+	 */
+	public function displayRow(array $row) {
+		// Update widths if this row has wider content
+		$row = $this->checkRow($row);
+		
+		// Recalculate widths for the renderer
+		$this->_renderer->setWidths($this->_width, false);
+		
+		$rendered_row = $this->_renderer->row($row);
+		$row_lines = explode( PHP_EOL, $rendered_row );
+		foreach ( $row_lines as $line ) {
+			Streams::line( $line );
+		}
+		
+		$border = $this->_renderer->border();
+		if (isset($border)) {
+			Streams::line( $border );
+		}
+	}
+
 	/**
 	 * Get the table lines to output.
 	 *
@@ -154,7 +192,8 @@ public function getDisplayLines() {
 			$out = array_merge( $out, $row );
 		}
 
-		if (isset($border)) {
+		// Only add final border if there are rows
+		if (!empty($this->_rows) && isset($border)) {
 			$out[] = $border;
 		}
 

lib/cli/table/Ascii.php

@@ -83,6 +83,8 @@ public function setWidths(array $widths, $fallback = false) {
 		}
 
 		$this->_widths = $widths;
+		// Reset border cache when widths change
+		$this->_border = null;
 	}
 
 	/**

tests/Test_Table.php

@@ -289,4 +289,79 @@ public function test_null_values_are_handled() {
 		];
 		$this->assertSame( $expected, $out, 'Null values should be safely converted to empty strings in table output.' );
 	}
+
+	public function test_resetRows() {
+		$table = new cli\Table();
+		$table->setHeaders( array( 'Name', 'Age' ) );
+		$table->addRow( array( 'Alice', '30' ) );
+		$table->addRow( array( 'Bob', '25' ) );
+		
+		$this->assertEquals( 2, $table->countRows() );
+		
+		$table->resetRows();
+		
+		$this->assertEquals( 0, $table->countRows() );
+		
+		// Headers should still be intact
+		$out = $table->getDisplayLines();
+		$this->assertGreaterThan( 0, count( $out ) );
+	}
+
+	public function test_displayRow_ascii() {
+		$mockFile = tempnam( sys_get_temp_dir(), 'temp' );
+		$resource = fopen( $mockFile, 'wb' );
+
+		try {
+			\cli\Streams::setStream( 'out', $resource );
+			
+			$table    = new cli\Table();
+			$renderer = new cli\Table\Ascii();
+			$table->setRenderer( $renderer );
+			$table->setHeaders( array( 'Name', 'Age' ) );
+			
+			// Display a single row
+			$table->displayRow( array( 'Alice', '30' ) );
+			
+			$output = file_get_contents( $mockFile );
+			
+			// Should contain the row data
+			$this->assertStringContainsString( 'Alice', $output );
+			$this->assertStringContainsString( '30', $output );
+			
+			// Should contain borders
+			$this->assertStringContainsString( '|', $output );
+			$this->assertStringContainsString( '+', $output );
+		} finally {
+			if ( $mockFile && file_exists( $mockFile ) ) {
+				unlink( $mockFile );
+			}
+		}
+	}
+
+	public function test_displayRow_tabular() {
+		$mockFile = tempnam( sys_get_temp_dir(), 'temp' );
+		$resource = fopen( $mockFile, 'wb' );
+
+		try {
+			\cli\Streams::setStream( 'out', $resource );
+			
+			$table    = new cli\Table();
+			$renderer = new cli\Table\Tabular();
+			$table->setRenderer( $renderer );
+			$table->setHeaders( array( 'Name', 'Age' ) );
+			
+			// Display a single row
+			$table->displayRow( array( 'Alice', '30' ) );
+			
+			$output = file_get_contents( $mockFile );
+			
+			// Should contain the row data with tabs
+			$this->assertStringContainsString( 'Alice', $output );
+			$this->assertStringContainsString( '30', $output );
+		} finally {
+			if ( $mockFile && file_exists( $mockFile ) ) {
+				unlink( $mockFile );
+			}
+		}
+	}
 }

tests/Test_Table_Ascii.php

@@ -249,7 +249,6 @@ public function testDrawWithHeadersNoData() {
 +----------+----------+
 | header 1 | header 2 |
 +----------+----------+
-+----------+----------+
 
 OUT;
 		$this->assertInOutEquals(array($headers, $rows), $output);