Skip to content

refactor(loghelper): unify nil-safe logger initialization fallback#150

Merged
radimsem merged 1 commit into
devfrom
refactor/unify-nil-safe-logger-fallback
May 19, 2026
Merged

refactor(loghelper): unify nil-safe logger initialization fallback#150
radimsem merged 1 commit into
devfrom
refactor/unify-nil-safe-logger-fallback

Conversation

@radimsem

Copy link
Copy Markdown
Owner

Summary

Closes #143

Self-review

Touched

Pipeline

  • pkg/parser/ — fuzz target covers any new shape
  • pkg/transformer/ or pkg/diff/ — ID stability / hash inputs unchanged
  • pkg/emitter/ — exactly one snapshot per write
  • pkg/store/ + migrations/ — FTS5 triggers in sync; add-store-query followed
  • pkg/query/ or pkg/compiler/ — token budget honored

MCP surface

  • pkg/mcp/skills/remind (read) or skills/memoize (write) updated
  • pkg/temperature/ — both public skills reflect new values

Edges

  • cmd/remindb/ — README CLI section updated
  • internal/ignore/, internal/tempfile/, internal/bench/ — relevant fuzz / fixture / scenario added
  • plugins/<agent>/ — manifest version bumped if shipping

Process

  • Tested manually via CLI or local MCP plugin install
  • make test-all + make fuzz green

@radimsem radimsem self-assigned this May 19, 2026
@radimsem radimsem added mcp Affects the MCP server implementation refactor Code refactoring / architecture cleanup labels May 19, 2026
@radimsem radimsem merged commit 8f16510 into dev May 19, 2026
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

mcp Affects the MCP server implementation refactor Code refactoring / architecture cleanup

Projects

None yet

Development

Successfully merging this pull request may close these issues.

refactor(loghelper): unified nil-safe logger fallback (8 sites)

1 participant