Skip to content

Fix editable installation and webhook failure isolation - #1

Open
Jemade wants to merge 1 commit into
mainfrom
fix-runtime-and-reliability
Open

Jemade wants to merge 1 commit into
mainfrom
fix-runtime-and-reliability

Conversation

@Jemade

@Jemade Jemade commented Oct 3, 2026

Copy link
Copy Markdown
Owner

Optional webhook dispatch failures previously entered the generation failure handler after a successful audit, risking a duplicate primary key and replacing a successful response. Restrict the provider exception handler to generation, isolate webhook failures, and roll back failed notification transactions. Preserve successful generation responses and original provider exceptions.

Fix setuptools package discovery for editable installs and declare SQLAlchemy's asyncio extra. Update README behavior and add regression tests for successful and failed generation with webhook errors.

Validation: editable install succeeds; 44 tests passed; Ruff lint and full format check passed; mypy passed (40 source files). Tests use SQLite and mocked upstream services. No deployment performed.

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