Skip to content

feat: ktlint enforcement and short UUID prefix lookup fix#73

Merged
jpicklyk merged 3 commits intomainfrom
pr/ktlint-and-prefix-fix
Mar 9, 2026
Merged

feat: ktlint enforcement and short UUID prefix lookup fix#73
jpicklyk merged 3 commits intomainfrom
pr/ktlint-and-prefix-fix

Conversation

@jpicklyk
Copy link
Owner

@jpicklyk jpicklyk commented Mar 9, 2026

Summary

  • ktlint linting — Added Gradle ktlint plugin (v12.1.2 / ktlint 1.5.0), .editorconfig with project conventions, CI enforcement via ktlintCheck step in test workflow, and full codebase reformatting
  • Short UUID prefix fixCAST(blob AS VARCHAR) in SQLite produces raw bytes, not a formatted UUID string, so prefix lookups always returned empty. Switched to LOWER(HEX(id)) for SQLite and LOWER(RAWTOHEX(id)) for H2, both producing dashless lowercase hex for correct LIKE matching

Test plan

  • All tests pass (./gradlew :current:test)
  • CI ktlint check passes
  • Short UUID prefix lookup works against live SQLite (verified manually)

🤖 Generated with Claude Code

jpicklyk and others added 3 commits March 9, 2026 13:41
…CI enforcement

Add ktlint (v12.1.2, engine v1.5.0) via jlleitschuh/ktlint-gradle plugin for
automated Kotlin code style enforcement. Includes .editorconfig with wildcard
import allowlist, disabled trailing comma rules, and 140-char line limit.
CI workflow gets a ktlintCheck step before tests. Applied ktlint formatting
across ~108 source files (mechanical whitespace/blank line changes only).

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
CAST(blob AS VARCHAR) produces raw bytes in SQLite, not a formatted UUID
string, causing prefix lookups to always return empty results. Switch to
LOWER(HEX(id)) for SQLite and LOWER(RAWTOHEX(id)) for H2, which both
produce dashless lowercase hex strings suitable for LIKE prefix matching.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@jpicklyk jpicklyk merged commit dacc6e2 into main Mar 9, 2026
2 checks passed
@jpicklyk jpicklyk deleted the pr/ktlint-and-prefix-fix branch March 9, 2026 18:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant