Skip to content

Comments

Fix 8 bugs: NPE on missing EDN target, error map typo, unsafe read-string#174

Merged
yogthos merged 1 commit intomasterfrom
bug-fixes
Feb 18, 2026
Merged

Fix 8 bugs: NPE on missing EDN target, error map typo, unsafe read-string#174
yogthos merged 1 commit intomasterfrom
bug-fixes

Conversation

@yogthos
Copy link
Collaborator

@yogthos yogthos commented Feb 18, 2026

and silent git failures, dead code, and missing nil checks

  • Return data unchanged instead of NPE when EDN injection target path missing
  • Fix :path keyword literal to actual path value in read-files error map
  • Replace read-string with edn/read-string in git.clj, modules.clj, kit_clj.clj
  • Re-throw git sync exceptions as ex-info instead of silently printing
  • Use ex-info instead of Exception in edn-safe-merge
  • Remove duplicate :xtdb? condition and dead check-conflicts function
  • Add nil guard in read-config for missing resource files
  • Add test infrastructure for kit-core and new tests for injections

…ring, silent git failures, dead code, and missing nil checks

- Return data unchanged instead of NPE when EDN injection target path missing
- Fix :path keyword literal to actual path value in read-files error map
- Replace read-string with edn/read-string in git.clj, modules.clj, kit_clj.clj
- Re-throw git sync exceptions as ex-info instead of silently printing
- Use ex-info instead of Exception in edn-safe-merge
- Remove duplicate :xtdb? condition and dead check-conflicts function
- Add nil guard in read-config for missing resource files
- Add test infrastructure for kit-core and new tests for injections
@yogthos yogthos merged commit 1d40531 into master Feb 18, 2026
2 checks passed
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