cal/major-domo-database
1
0
Fork 0
Code Issues 25 Pull Requests 25 Actions Packages Projects Releases Wiki Activity

fix: replace manual db.close() calls with middleware-based connection management (#71) #97

Open
Claude wants to merge 4 commits from issue/71-refactor-manual-db-close-calls-to-middleware-based into main
pull from: issue/71-refactor-manual-db-close-calls-to-middleware-based
merge into: cal:main
Conversation 1 Commits 4 Files Changed 27 +13 -199

4 Commits

Author SHA1 Message Date
Cal Corum
172a7a662e ci: retrigger build after transient Docker Hub push failure
All checks were successful
Build Docker Image / build (pull_request) Successful in 6m34s
Details
2026-03-30 18:32:10 -05:00
cal
c415fb8be7 Merge branch 'main' into issue/71-refactor-manual-db-close-calls-to-middleware-based
Some checks failed
Build Docker Image / build (pull_request) Failing after 4m14s
Details
2026-03-30 23:23:14 +00:00
Cal Corum
ac2c5920ce fix: remove empty finally clauses in custom_commands and help_commands
All checks were successful
Build Docker Image / build (pull_request) Successful in 2m17s
Details 
After removing db.close() calls, 22 finally: blocks were left empty
(12 in custom_commands.py, 10 in help_commands.py), causing
IndentationError at import time. Removed the finally: clause entirely
since connection lifecycle is now handled by the middleware.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
 2026-03-27 06:34:37 -05:00
Cal Corum
ab90dfc437 fix: replace manual db.close() calls with middleware-based connection management (#71)
All checks were successful
Build Docker Image / build (pull_request) Successful in 2m28s
Details 
Closes #71

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
 2026-03-27 06:04:56 -05:00