Strip test comments, remove dead CRLF branch, restore GitHub nav link

- Remove all comments from test files
- Remove unreachable CRLF look-ahead in ANSIParser (Swift treats
  \r\n as a single grapheme cluster that never matches case "\r")
- Remove two tests that covered the dead CRLF path
- Restore GitHub link in website nav that was removed in #8

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

Author: tsconfigdotjson

Sources/DevtailKit/ANSIParser.swift

@@ -78,12 +78,6 @@ public struct ANSIParser: Sendable {
 
       case "\r":
         flushText()
-        let next = input.index(after: i)
-        if next < input.endIndex && input[next] == "\n" {
-          actions.append(.newline)
-          i = input.index(after: next)
-          continue
-        }
         actions.append(.carriageReturn)
 
       case "\n":

Tests/DevtailKitTests/ANSIParserTests.swift

@@ -4,32 +4,25 @@ import Testing
 
 struct ANSIParserTests {
 
-  // MARK: - Helper
-
-  /// Shorthand to parse a string and return actions.
   private func parse(_ input: String) -> [TerminalAction] {
     var parser = ANSIParser()
     return parser.parse(input)
   }
 
-  /// Extract the first .text action's span, if any.
   private func firstSpan(_ actions: [TerminalAction]) -> StyledSpan? {
     for action in actions {
       if case .text(let span) = action { return span }
     }
     return nil
   }
 
-  /// Collect all .text spans from actions.
   private func allSpans(_ actions: [TerminalAction]) -> [StyledSpan] {
     actions.compactMap {
       if case .text(let span) = $0 { return span }
       return nil
     }
   }
 
-  // MARK: - Plain text
-
   @Test func plainTextPassesThrough() {
     let actions = parse("hello world")
     #expect(actions.count == 1)
@@ -46,8 +39,6 @@ struct ANSIParserTests {
     #expect(actions.isEmpty)
   }
 
-  // MARK: - Newline / carriage return
-
   @Test func newlineProducesNewlineAction() {
     let actions = parse("\n")
     #expect(actions.count == 1)
@@ -66,17 +57,6 @@ struct ANSIParserTests {
     }
   }
 
-  @Test func crlfGraphemeClusterIsFiltered() {
-    // In Swift, \r\n forms a single Character grapheme cluster that does NOT
-    // match case "\r" or case "\n" in the parser's switch. Its asciiValue is 10
-    // (newline), which is < 32 and != 9, so it gets filtered as a control char.
-    // This is a known limitation of character-level parsing in Swift.
-    let input = String(Unicode.Scalar(0x0D)) + String(Unicode.Scalar(0x0A))
-    let actions = parse(input)
-    // CRLF grapheme is filtered out, producing no actions
-    #expect(actions.isEmpty)
-  }
-
   @Test func textBeforeAndAfterNewline() {
     let actions = parse("abc\ndef")
     #expect(actions.count == 3)
@@ -88,8 +68,6 @@ struct ANSIParserTests {
     if case .text(let s2) = actions[2] { #expect(s2.text == "def") }
   }
 
-  // MARK: - Standard foreground colors (30-37)
-
   @Test func standardForegroundColors() {
     for code in 30...37 {
       let actions = parse("\u{1B}[\(code)mX")
@@ -99,8 +77,6 @@ struct ANSIParserTests {
     }
   }
 
-  // MARK: - Standard background colors (40-47)
-
   @Test func standardBackgroundColors() {
     for code in 40...47 {
       let actions = parse("\u{1B}[\(code)mX")
@@ -110,8 +86,6 @@ struct ANSIParserTests {
     }
   }
 
-  // MARK: - Bright foreground colors (90-97)
-
   @Test func brightForegroundColors() {
     for code in 90...97 {
       let actions = parse("\u{1B}[\(code)mX")
@@ -121,8 +95,6 @@ struct ANSIParserTests {
     }
   }
 
-  // MARK: - Bright background colors (100-107)
-
   @Test func brightBackgroundColors() {
     for code in 100...107 {
       let actions = parse("\u{1B}[\(code)mX")
@@ -132,8 +104,6 @@ struct ANSIParserTests {
     }
   }
 
-  // MARK: - 256-color palette
-
   @Test func foreground256Color() {
     let actions = parse("\u{1B}[38;5;123mHi")
     let span = firstSpan(actions)
@@ -160,8 +130,6 @@ struct ANSIParserTests {
     #expect(span?.style.foreground == .palette(255))
   }
 
-  // MARK: - RGB truecolor
-
   @Test func foregroundRGBColor() {
     let actions = parse("\u{1B}[38;2;255;128;0mTruecolor")
     let span = firstSpan(actions)
@@ -188,8 +156,6 @@ struct ANSIParserTests {
     #expect(span?.style.foreground == .rgb(255, 255, 255))
   }
 
-  // MARK: - Text styles
-
   @Test func boldStyle() {
     let actions = parse("\u{1B}[1mBold")
     let span = firstSpan(actions)
@@ -221,13 +187,9 @@ struct ANSIParserTests {
     #expect(span?.style.strikethrough == true)
   }
 
-  // MARK: - Reset
-
   @Test func resetClearsAllStyles() {
     var parser = ANSIParser()
-    // Set bold + red foreground
     _ = parser.parse("\u{1B}[1;31m")
-    // Now reset
     let actions = parser.parse("\u{1B}[0mAfterReset")
     let span = firstSpan(actions)
     #expect(span?.style == ANSIStyle())
@@ -242,8 +204,6 @@ struct ANSIParserTests {
     #expect(span?.style == ANSIStyle())
   }
 
-  // MARK: - Individual style resets
-
   @Test func code22ResetsBoldAndDim() {
     var parser = ANSIParser()
     _ = parser.parse("\u{1B}[1;2m")
@@ -277,8 +237,6 @@ struct ANSIParserTests {
     #expect(span?.style.strikethrough == false)
   }
 
-  // MARK: - Default foreground / background
-
   @Test func code39ResetsDefaultForeground() {
     var parser = ANSIParser()
     _ = parser.parse("\u{1B}[31m")
@@ -295,8 +253,6 @@ struct ANSIParserTests {
     #expect(span?.style.background == .default)
   }
 
-  // MARK: - Erase line
-
   @Test func eraseLineCode2K() {
     let actions = parse("\u{1B}[2K")
     #expect(actions.count == 1)
@@ -324,8 +280,6 @@ struct ANSIParserTests {
     }
   }
 
-  // MARK: - Cursor up
-
   @Test func cursorUpDefault() {
     let actions = parse("\u{1B}[A")
     #expect(actions.count == 1)
@@ -355,28 +309,23 @@ struct ANSIParserTests {
     #expect(n == 1)
   }
 
-  // MARK: - Multiple SGR params in one sequence
-
   @Test func multipleSGRParams() {
     let actions = parse("\u{1B}[1;31;42mCombined")
     let span = firstSpan(actions)
     #expect(span?.style.bold == true)
-    #expect(span?.style.foreground == .standard(1))  // red
-    #expect(span?.style.background == .standard(2))  // green
+    #expect(span?.style.foreground == .standard(1))
+    #expect(span?.style.background == .standard(2))
     #expect(span?.text == "Combined")
   }
 
   @Test func multipleSGRParamsItalicBrightCyan() {
     let actions = parse("\u{1B}[3;96mTest")
     let span = firstSpan(actions)
     #expect(span?.style.italic == true)
-    #expect(span?.style.foreground == .bright(6))  // bright cyan
+    #expect(span?.style.foreground == .bright(6))
   }
 
-  // MARK: - Control characters
-
   @Test func controlCharactersAreFiltered() {
-    // Characters below 32 (except tab) should be stripped
     let input = "A\u{01}B\u{02}C\u{07}D"
     let actions = parse(input)
     let span = firstSpan(actions)
@@ -389,37 +338,29 @@ struct ANSIParserTests {
     #expect(span?.text == "A\tB")
   }
 
-  // MARK: - Incomplete / malformed sequences
-
   @Test func incompleteEscapeSequenceIsSkipped() {
-    // ESC followed by end of string
     let actions = parse("Hello\u{1B}")
     #expect(actions.count == 1)
     let span = firstSpan(actions)
     #expect(span?.text == "Hello")
   }
 
   @Test func escNotFollowedByBracketSkips() {
-    // ESC followed by non-bracket char
     let actions = parse("A\u{1B}XB")
-    // "A" is flushed before ESC, ESC+X advances past ESC, then "XB" continues
     let spans = allSpans(actions)
     let combined = spans.map(\.text).joined()
     #expect(combined == "AXB")
   }
 
   @Test func incompleteCSISequenceIsSkipped() {
-    // ESC[ with digits but no final letter
     let actions = parse("Hi\u{1B}[31")
     let span = firstSpan(actions)
     #expect(span?.text == "Hi")
   }
 
-  // MARK: - Style persistence across parse calls
-
   @Test func stylePersistsAcrossParseCalls() {
     var parser = ANSIParser()
-    _ = parser.parse("\u{1B}[1;31m")  // bold + red
+    _ = parser.parse("\u{1B}[1;31m")
     let actions = parser.parse("StyledText")
     let span = firstSpan(actions)
     #expect(span?.style.bold == true)
@@ -438,8 +379,6 @@ struct ANSIParserTests {
     #expect(s2?.style == ANSIStyle())
   }
 
-  // MARK: - Mixed text and escapes
-
   @Test func mixedTextAndEscapes() {
     let actions = parse("Hello \u{1B}[31mWorld\u{1B}[0m!")
     let spans = allSpans(actions)
@@ -477,29 +416,16 @@ struct ANSIParserTests {
     }
   }
 
-  @Test func textWithCRLFGraphemeClusters() {
-    // CRLF grapheme clusters are filtered (see crlfGraphemeClusterIsFiltered),
-    // so text on either side merges into one span.
-    let crlf = String(Unicode.Scalar(0x0D)) + String(Unicode.Scalar(0x0A))
-    let input = "line1" + crlf + "line2"
-    let actions = parse(input)
-    #expect(actions.count == 1)
-    if case .text(let s) = actions[0] { #expect(s.text == "line1line2") }
-  }
-
   @Test func separateCRAndLFProduceBothActions() {
-    // When \r and \n arrive in separate parse() calls, they work as expected
     var parser = ANSIParser()
     let a1 = parser.parse("line1\r")
     let a2 = parser.parse("\nline2")
-    // First parse: text("line1") + carriageReturn
     #expect(a1.count == 2)
     if case .text(let s) = a1[0] { #expect(s.text == "line1") }
     guard case .carriageReturn = a1[1] else {
       Issue.record("Expected .carriageReturn")
       return
     }
-    // Second parse: newline + text("line2")
     #expect(a2.count == 2)
     guard case .newline = a2[0] else {
       Issue.record("Expected .newline")
@@ -519,60 +445,44 @@ struct ANSIParserTests {
     if case .text(let s) = actions[2] { #expect(s.text == "new") }
   }
 
-  // MARK: - Extended color edge cases
-
   @Test func extendedColor38WithInsufficientParams() {
-    // 38;5 without the color number -- should not crash
     let actions = parse("\u{1B}[38;5mX")
     let span = firstSpan(actions)
-    // Parser should still produce text, foreground stays default
     #expect(span?.text == "X")
   }
 
   @Test func extendedColor38ModeUnknown() {
-    // 38;3 is not a recognized mode (only 2 and 5)
     let actions = parse("\u{1B}[38;3;100mX")
     let span = firstSpan(actions)
     #expect(span?.text == "X")
-    // foreground should remain default since mode 3 is unrecognized
     #expect(span?.style.foreground == .default)
   }
 
   @Test func truecolorWithInsufficientParams() {
-    // 38;2;255;128 is missing the blue component
     let actions = parse("\u{1B}[38;2;255;128mX")
     let span = firstSpan(actions)
     #expect(span?.text == "X")
-    // Should not crash; foreground stays default since insufficient params
     #expect(span?.style.foreground == .default)
   }
 
-  // MARK: - Unrecognized CSI final characters
-
   @Test func unrecognizedCSIFinalCharIsIgnored() {
-    // ESC[5J is a valid CSI form but 'J' (erase display) isn't handled
     let actions = parse("A\u{1B}[2JB")
     let spans = allSpans(actions)
     let combined = spans.map(\.text).joined()
     #expect(combined == "AB")
   }
 
-  // MARK: - Complex real-world sequences
-
   @Test func npmColoredOutput() {
-    // Simulate typical npm output with reset, bold, colors
     let input = "\u{1B}[1m\u{1B}[32m>\u{1B}[0m dev\n  next dev"
     let actions = parse(input)
     let spans = allSpans(actions)
     #expect(spans.count >= 2)
-    // First span should be bold + green ">"
     #expect(spans[0].style.bold == true)
     #expect(spans[0].style.foreground == .standard(2))
     #expect(spans[0].text == ">")
   }
 
   @Test func progressBarWithCR() {
-    // Simulate a progress bar that uses carriage return to overwrite
     let input = "Progress: 50%\rProgress: 100%"
     let actions = parse(input)
     #expect(actions.count == 3)