Pull request overview

This PR updates the .NET Harness console UI rendering logic to better handle terminal wrapping and bottom-panel mode switches.

Changes:

• Adds ANSI-aware visible string length measurement.
• Updates queued/scroll text height calculations to account for wrapping.
• Forces bottom-panel repaint when switching modes to avoid stale content.

Reviewed changes

Copilot reviewed 4 out of 4 changed files in this pull request and generated 2 comments.

dotnet/samples/02-agents/Harness/ConsoleReactiveComponents/TextPanel.cs:58

• This loop compares the global currentRow against the current item's lineCount, so after rendering a one-line item currentRow is already 1 and subsequent one-line items are skipped entirely. It also advances only one row for a logical line that may wrap. Track rows rendered for the current item separately and advance currentRow by the physical rows occupied by each logical line/item.

            for (int j = 0; j < lines.Length && currentRow < lineCount; j++)
            {
                Console.Write(AnsiEscapes.MoveAndEraseLine(props.Y + currentRow));
                Console.Write(lines[j]);
                currentRow++;

Automated Code Review

Reviewers: 4 | Confidence: 89% | Result: All clear

Reviewed: Correctness, Security Reliability, Test Coverage, Design Approach

Automated review by westey-m's agents

Pull request overview

Copilot reviewed 6 out of 6 changed files in this pull request and generated 2 comments.

dotnet/samples/02-agents/Harness/ConsoleReactiveComponents/TextPanel.cs:58

• lineCount now includes wrapped terminal rows, but currentRow is still incremented only once per logical line. For a long line that wraps, the leftover-line cleanup starts too early and erases the wrapped continuation rows (and later items can be positioned over them); advance currentRow by the physical row count for each logical line instead.

            int lineCount = CountPhysicalLines(text, props.Width);

            for (int j = 0; j < lines.Length && currentRow < lineCount; j++)
            {
                Console.Write(AnsiEscapes.MoveAndEraseLine(props.Y + currentRow));
                Console.Write(lines[j]);
                currentRow++;