Skip to content
Permalink

Comparing changes

Choose two branches to see what’s changed or to start a new pull request. If you need to, you can also or learn more about diff comparisons.

Open a pull request

Create a new pull request by comparing changes across two branches. If you need to, you can also . Learn more about diff comparisons here.
base repository: chrisguidry/docket
Failed to load repositories. Confirm that selected base ref is valid, then try again.
Loading
base: 0.17.2
Choose a base ref
...
head repository: chrisguidry/docket
Failed to load repositories. Confirm that selected head ref is valid, then try again.
Loading
compare: 0.17.3
Choose a head ref
  • 1 commit
  • 1 file changed
  • 2 contributors

Commits on Jan 27, 2026

  1. Fix memory leak in _tasks_by_key cleanup (#309) 

    The worker was storing tasks in `_tasks_by_key[execution.key]` but then
trying to remove them with `_tasks_by_key.pop(task.get_name())`. Since
task names are formatted as `"docket-name - task:key"` while the dict
key is just `"key"`, the pop never found a match and entries accumulated
forever.

Each leaked entry held a reference to the asyncio.Task and its
associated Execution, so memory grew linearly with task count.

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-authored-by: Claude Opus 4.5 <noreply@anthropic.com>
    @chrisguidry @claude
    chrisguidry and claude authored Jan 27, 2026
    Configuration menu
    Copy the full SHA
    c5a2b6d View commit details
    Browse the repository at this point in the history
Loading