refactor(store): split monolithic sqlite.go into domain-specific files

[appleboy]

Summary

• Split internal/store/sqlite.go (1,139 lines, 65+ methods) into 10 domain-specific files, each mapping to a core.Store interface group
• Pure method relocation — no logic changes, no interface or struct changes
• sqlite.go reduced from 1,139 → 290 lines (Store struct, New(), seedData(), infrastructure methods)

New files

File	Interface	Methods
user.go	UserReader + UserWriter	9
client.go	ClientReader + ClientWriter	10
token.go	TokenReader + TokenWriter	14
audit_log.go	AuditStore	5
user_authorization.go	UserAuthorizationStore	7
oauth_connection.go	OAuthConnectionStore	6
device_code.go	DeviceCodeStore	5
cleanup.go	CleanupStore + MetricsStore	5
authorization_code.go	AuthorizationCodeStore	3

Test plan

• make generate — templates compile
• make build — full build passes
• make test — all tests pass
• make lint — 0 issues

🤖 Generated with Claude Code

[codecov]

Codecov Report

❌ Patch coverage is 56.27706% with 202 lines in your changes missing coverage. Please review.

Files with missing lines	Patch %	Lines
internal/store/client.go	33.33%	48 Missing and 4 partials ⚠️
internal/store/audit_log.go	48.14%	24 Missing and 18 partials ⚠️
internal/store/token.go	51.31%	35 Missing and 2 partials ⚠️
internal/store/user_authorization.go	47.61%	31 Missing and 2 partials ⚠️
internal/store/user.go	77.21%	10 Missing and 8 partials ⚠️
internal/store/device_code.go	31.25%	10 Missing and 1 partial ⚠️
internal/store/authorization_code.go	58.82%	6 Missing and 1 partial ⚠️
internal/store/oauth_connection.go	92.30%	1 Missing and 1 partial ⚠️

📢 Thoughts on this report? Let us know!

[Copilot]

Pull request overview

Refactors the internal/store package by splitting the previously monolithic sqlite.go into smaller, domain-focused files aligned with core.Store interface groupings, while keeping behavior and public APIs unchanged.

Changes:

• Moved user, client, token, device code, OAuth connection, audit log, authorization code, and consent (user authorization) methods into dedicated files.
• Reduced internal/store/sqlite.go to store construction, seeding, and infrastructure/transaction helpers.
• Kept method receivers and signatures consistent so *store.Store still satisfies core.Store.

Reviewed changes

Copilot reviewed 10 out of 10 changed files in this pull request and generated no comments.

Show a summary per file

File	Description
internal/store/sqlite.go	Removed relocated domain methods; retains Store struct, New/seed, health/close, and transaction helpers.
internal/store/user.go	User read/write and external-user upsert operations moved here.
internal/store/client.go	OAuth client read/write and listing/counting operations moved here.
internal/store/token.go	Access token read/write, status updates, and revocation operations moved here.
internal/store/device_code.go	Device code CRUD operations moved here.
internal/store/oauth_connection.go	OAuth connection CRUD operations moved here.
internal/store/audit_log.go	Audit log create/query/stats operations moved here.
internal/store/cleanup.go	Expired-data cleanup and metrics counting operations moved here.
internal/store/authorization_code.go	Authorization code create/get/mark-used operations moved here.
internal/store/user_authorization.go	Consent (user authorization) read/write/revoke/list operations moved here.

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.