Skip to content

Fix #66: ignore closed issues and prune them from the state#68

Merged
max-tet merged 1 commit intomainfrom
clayde/issue-66-prune-closed-issues
Apr 8, 2026
Merged

Fix #66: ignore closed issues and prune them from the state#68
max-tet merged 1 commit intomainfrom
clayde/issue-66-prune-closed-issues

Conversation

@ClaydeCode
Copy link
Copy Markdown
Owner

Closes #66

Added _prune_closed_issues() helper in orchestrator.py that checks each state.json entry against the GitHub API and removes entries for closed issues. It is called once per tick at the start of main(), before the main processing loop, ensuring closed issues are pruned and never acted on. API errors are handled gracefully (logged and skipped). Added 6 new tests and updated 4 existing tests that needed an extra load_state() side effect.

Add _prune_closed_issues() helper that checks each state.json entry against
the GitHub API and removes any whose issue is closed. Called once per tick
before the main processing loop, ensuring closed issues never accumulate in
state and are never acted on.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@ClaydeCode ClaydeCode requested a review from max-tet April 8, 2026 09:17
@max-tet max-tet merged commit 0e1e95a into main Apr 8, 2026
2 checks passed
@max-tet max-tet deleted the clayde/issue-66-prune-closed-issues branch April 8, 2026 09:44
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.

ignore closed issues and prune them from the state

2 participants