internal/core/errors.go (1)

40-40: Defensive format string handling applied.

The explicit format string "%s" prevents accidental interpretation of format specifiers if msg contains % characters. This is good defensive programming, though the risk is minimal since msg is typically a controlled string literal.

For consistency, consider auditing other direct WithMessage(string) calls in the codebase:

#!/bin/bash
# Find other potential WithMessage calls that might benefit from explicit formatting
rg -nP '\.WithMessage\([^"]*\)' --type=go -g '!*_test.go'

internal/core/controllers.go (1)

50-50: Optimize string formatting to avoid unnecessary concatenation.

The explicit format string is good for consistency, but the string concatenation before formatting creates an unnecessary allocation.

Apply this diff to use format arguments directly:

-					setStatus(NewPendingError().WithMessage("%s", "pending condition: "+str))
+					setStatus(NewPendingError().WithMessage("pending condition: %s", str))

This is more efficient and idiomatic as it performs concatenation directly through fmt.Sprintf rather than creating an intermediate string.

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

Disabled knowledge base sources:

• Linear integration is disabled by default for public repositories

You can enable these sources in your CodeRabbit configuration.