refactor: cleanup and test fixes - #13

Merged

frap129 merged 31 commits into

devfrom

desloppify/code-health

May 3, 2026

Commits on May 3, 2026

fix(tests): eliminate flaky concurrent-writes test by awaiting write promises
frap129
committed
54f984d
refactor: remove pass-through wrappers createSessionStore and extractUserPromptFromParts
frap129
committed
93d06e3
refactor: extract shared logWarning helper and remove redundant guard
frap129
committed
9e3be25
style: remove restating comments in rule-discovery, rule-metadata, and active-rules-state
frap129
committed
f004d2f
chore: ignore .desloppify directory and scorecard.png
frap129
committed
80fd6bf
refactor(runtime-context): move debugLog and projectDirectory into BuildFilterContextOptions
frap129
committed
bfbc145
refactor(active-rules-state): remove fire-and-forget catch so callers can handle errors
frap129
committed
6f1f6e3
refactor: standardize I/O boundary error returns to null instead of undefined
frap129
committed
bdcccfd
refactor(runtime): replace as any cast with typed OpenCodeClient interface
frap129
committed
58ba71c
refactor(runtime): add completion audit log for hook side-effects
frap129
committed
f3f0119
fix(message-context): strictly require role === 'user' in extractLatestUserPrompt
frap129
committed
728a1ad
fix(active-rules-state): throw on invalid sessionId to align with getStateFilePath
frap129
committed
0af29be
style(test-fixtures): rename duplicate section headers to be unique
frap129
committed
76fbeef
style: add .js extension to local imports in active-rules-state and rule-discovery
frap129
committed
8c8f7e0
style: add node: prefix to all Node built-in module imports
frap129
committed
fa6b160
style(active-rules-state): rename sessionId to sessionID to match convention
frap129
committed
34262df
style(tui): add .js extension to cross-package imports from src
frap129
committed
c61448d
docs(utils): clarify barrel file as intentional public API facade
frap129
committed
c3f58c0
refactor: extract shared hasConditions to rule-metadata and deduplicate callers
frap129
committed
ee0ffec
docs(utils): explicitly document internal modules not re-exported by facade
frap129
committed
88ecc75
refactor: extend utils.ts facade for TUI needs and update cross-package imports
frap129
committed
1be1050
fix(deps): remove duplicated @opentui/* and solid-js from devDependencies
frap129
committed
f8c67c4
fix(deps): add @opentui/* as optional peerDependencies
frap129
committed
b11642e
fix(deps): remove redundant @types/minimatch
frap129
committed
7f09415
fix(deps): align @opentui versions in manifest with lock file (0.1.93→0.1.97)
frap129
committed
797fb1e
refactor(rule-filter): extract repeated condition checks into evaluateConditionChecks helper
frap129
committed
ba14f11
refactor(tui): consolidate loadRules and deduplicate metadata display in SidebarContent
frap129
committed
387f1f9
fix: improve error consistency, fix cross-module imports, and update tests
frap129
committed
11b23f4
refactor: fix convention drift, naming, and AI debt across 10 files
frap129
committed
baa2d92
refactor(error-consistency): add await to write call, fix type duplication, document exception contracts
frap129
committed
d5bee5c
fix(bun-compat): achieve 100% pass rate by eliminating vi.mock and adding dependency injection
frap129
committed
d22c78c