remove unnecessary comments

carole-lavillonniere · carole-lavillonniere · commit 1f0cad842a0d · 2026-03-19T11:31:09.000+01:00
diff --git a/internal/container/logfilter.go b/internal/container/logfilter.go
@@ -6,7 +6,6 @@ import (
 	"github.com/localstack/lstk/internal/output"
 )
 
-// shouldFilter returns true if the log line is identified as noise and should not be shown.
 func shouldFilter(line string) bool {
 	if strings.Contains(line, "Docker not available") {
 		return true
@@ -23,17 +22,13 @@ func shouldFilter(line string) bool {
 	return false
 }
 
-// parseLogLine extracts the log level and logger name from a LocalStack log line.
 // Expected format: 2026-03-16T17:56:00.810  INFO --- [  MainThread] l.p.c.extensions.plugins   : message
-// Returns LogLevelUnknown and empty string if the line does not match the expected format.
 func parseLogLine(line string) (output.LogLevel, string) {
-	// Find the thread section separator
 	sepIdx := strings.Index(line, " --- [")
 	if sepIdx < 0 {
 		return output.LogLevelUnknown, ""
 	}
 
-	// Level is the last word before " --- ["
 	prefix := strings.TrimSpace(line[:sepIdx])
 	level := output.LogLevelUnknown
 	if spIdx := strings.LastIndex(prefix, " "); spIdx >= 0 {
@@ -49,7 +44,6 @@ func parseLogLine(line string) (output.LogLevel, string) {
 		}
 	}
 
-	// Logger is between "] " and "   : "
 	rest := line[sepIdx+6:] // skip " --- ["
 	bracketEnd := strings.Index(rest, "]")
 	if bracketEnd < 0 {
diff --git a/internal/output/events.go b/internal/output/events.go
@@ -141,9 +141,9 @@ const (
 )
 
 type LogLineEvent struct {
-	Source string   // use LogSource* constants
+	Source string
 	Line   string
-	Level  LogLevel // parsed from the log line; LogLevelUnknown if not parseable
+	Level  LogLevel
 }
 
 // Emit sends an event to the sink with compile-time type safety via generics.
diff --git a/internal/ui/logrender.go b/internal/ui/logrender.go
@@ -7,8 +7,6 @@ import (
 	"github.com/localstack/lstk/internal/ui/styles"
 )
 
-// renderLogLine renders a LocalStack log line with the metadata prefix (timestamp,
-// level, thread, logger) in grey and the message in a color matching the log level.
 func renderLogLine(line string, level output.LogLevel) string {
 	idx := strings.Index(line, " : ")
 	if idx < 0 {

Original file line number	Diff line number	Diff line change
`@@ -6,7 +6,6 @@ import (`
`6`	`6`	`"github.com/localstack/lstk/internal/output"`
`7`	`7`	`)`
`8`	`8`
`9`		`-// shouldFilter returns true if the log line is identified as noise and should not be shown.`
`10`	`9`	`func shouldFilter(line string) bool {`
`11`	`10`	`if strings.Contains(line, "Docker not available") {`
`12`	`11`	`return true`
`@@ -23,17 +22,13 @@ func shouldFilter(line string) bool {`
`23`	`22`	`return false`
`24`	`23`	`}`
`25`	`24`
`26`		`-// parseLogLine extracts the log level and logger name from a LocalStack log line.`
`27`	`25`	`// Expected format: 2026-03-16T17:56:00.810 INFO --- [ MainThread] l.p.c.extensions.plugins : message`
`28`		`-// Returns LogLevelUnknown and empty string if the line does not match the expected format.`
`29`	`26`	`func parseLogLine(line string) (output.LogLevel, string) {`
`30`		`- // Find the thread section separator`
`31`	`27`	`sepIdx := strings.Index(line, " --- [")`
`32`	`28`	`if sepIdx < 0 {`
`33`	`29`	`return output.LogLevelUnknown, ""`
`34`	`30`	`}`
`35`	`31`
`36`		`- // Level is the last word before " --- ["`
`37`	`32`	`prefix := strings.TrimSpace(line[:sepIdx])`
`38`	`33`	`level := output.LogLevelUnknown`
`39`	`34`	`if spIdx := strings.LastIndex(prefix, " "); spIdx >= 0 {`
`@@ -49,7 +44,6 @@ func parseLogLine(line string) (output.LogLevel, string) {`
`49`	`44`	`}`
`50`	`45`	`}`
`51`	`46`
`52`		`- // Logger is between "] " and " : "`
`53`	`47`	`rest := line[sepIdx+6:] // skip " --- ["`
`54`	`48`	`bracketEnd := strings.Index(rest, "]")`
`55`	`49`	`if bracketEnd < 0 {`
Original file line number	Diff line number	Diff line change
`@@ -141,9 +141,9 @@ const (`
`141`	`141`	`)`
`142`	`142`
`143`	`143`	`type LogLineEvent struct {`
`144`		`- Source string // use LogSource* constants`
	`144`	`+ Source string`
`145`	`145`	`Line string`
`146`		`- Level LogLevel // parsed from the log line; LogLevelUnknown if not parseable`
	`146`	`+ Level LogLevel`
`147`	`147`	`}`
`148`	`148`
`149`	`149`	`// Emit sends an event to the sink with compile-time type safety via generics.`
Original file line number	Diff line number	Diff line change
`@@ -7,8 +7,6 @@ import (`
`7`	`7`	`"github.com/localstack/lstk/internal/ui/styles"`
`8`	`8`	`)`
`9`	`9`
`10`		`-// renderLogLine renders a LocalStack log line with the metadata prefix (timestamp,`
`11`		`-// level, thread, logger) in grey and the message in a color matching the log level.`
`12`	`10`	`func renderLogLine(line string, level output.LogLevel) string {`
`13`	`11`	`idx := strings.Index(line, " : ")`
`14`	`12`	`if idx < 0 {`