Skip to content

Mark Phase 5, 7 tasks complete and add full pipeline tests#13

Merged
coordt merged 3 commits intomainfrom
phase7
Apr 21, 2026
Merged

Mark Phase 5, 7 tasks complete and add full pipeline tests#13
coordt merged 3 commits intomainfrom
phase7

Conversation

@coordt
Copy link
Copy Markdown
Member

@coordt coordt commented Apr 21, 2026

Description

  • Marked tasks for Phase 5 and Phase 7 as complete in todo.md.
  • Added end-to-end integration tests for the full issue triage pipeline:
    • Covers poller event through router, dispatcher, executor, and into memory (SQLite DB).
    • Includes robust test cases for GitHub label/comment application, memory updates, and action logging.
  • Final test coverage achieved: 96% across 214 passing tests.

coordt and others added 3 commits April 20, 2026 06:45
Covers the complete path: poller event → router → dispatcher → executor →
memory (real SQLite DB). Mocks are limited to PyGithub and httpx boundaries.

Six tests across two classes:
- TestFullTriagePipeline: label+comment applied, memory updated, action logged
  before GitHub call, prior summary injected, close_issue blocked when
  allow_close=False
- TestPollerFeedsDispatcher: poller.poll_all callback routes and dispatches
  a polled issue end-to-end

214 tests passing.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
config.example.yaml already matches full schema and loads cleanly.
CHANGELOG.md already maintained by bump-my-version toolchain.
214 tests passing, 96% line coverage (target ≥85%), pre-commit clean.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Apr 21, 2026

PR Preview Action v1.8.1
Preview removed because the pull request was closed.
2026-04-21 14:20 UTC

@github-actions
Copy link
Copy Markdown
Contributor

Version hint: no-release
Current version: 0.2.4
New version (when merged): 0.2.4

Comment ID: Display version hint-auto-generated

@coordt coordt merged commit f438a3d into main Apr 21, 2026
7 checks passed
@coordt coordt deleted the phase7 branch April 21, 2026 14:20
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