Skip to content

[codex] Fix default issue count#139

Merged
jeremymcs merged 1 commit into
mainfrom
codex/fix-issue-counts
May 24, 2026
Merged

[codex] Fix default issue count#139
jeremymcs merged 1 commit into
mainfrom
codex/fix-issue-counts

Conversation

@jeremymcs

Copy link
Copy Markdown
Owner

Summary

  • exclude closed worked issue history from the default issue list/count
  • keep open worked issues visible in the default issue list
  • add a regression test for stale closed worked rows inflating totals

Root cause

The runtime requested synced issues with includeWorked: true, which made the default dashboard include closed issues that had prior PatchDeck work. The storage filter also treated includeWorked: false as excluding open worked issues, so the runtime could not simply turn it off without hiding active worked issues.

Verification

  • node --import tsx --test server/appRuntime.test.ts --test-name-pattern "closed worked issues"
  • npm run check

Note: repo pre-commit guidance asks for /simplify when claude is available. I ran claude -p "/simplify server/appRuntime.ts server/memoryStorage.ts server/sqliteStorage.ts server/appRuntime.test.ts" --permission-mode bypassPermissions, but it produced no output and hung until stopped.

@github-actions

Copy link
Copy Markdown
Contributor

📄 Documentation Review Reminder

This PR changes source files but does not update any public documentation.

Changed source files:

server/appRuntime.test.ts
server/appRuntime.ts
server/memoryStorage.ts
server/sqliteStorage.ts

Please check if documentation in docs/public/ needs to be updated:

  • No documentation update needed
  • Documentation updated in this PR
  • Follow-up documentation issue created

Public docs are Markdown files in docs/public/. They are automatically
rendered to HTML and deployed to GitHub Pages when merged to main.

Available docs: getting-started.md, pr-babysitter.md, agent-dispatch.md, configuration.md, pr-questions.md

@jeremymcs jeremymcs marked this pull request as ready for review May 24, 2026 11:45
@jeremymcs jeremymcs merged commit 149c63e into main May 24, 2026
8 checks passed
@jeremymcs jeremymcs deleted the codex/fix-issue-counts branch May 24, 2026 11:45
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