Skip to content

Add daemon support for code completions (Tab/Ctrl+Space)#328

Merged
rgbkrk merged 2 commits intomainfrom
quill/workspace-review
Feb 28, 2026
Merged

Add daemon support for code completions (Tab/Ctrl+Space)#328
rgbkrk merged 2 commits intomainfrom
quill/workspace-review

Conversation

@rgbkrk
Copy link
Copy Markdown
Member

@rgbkrk rgbkrk commented Feb 28, 2026

Summary

Implements code completion requests via the daemon using the Jupyter protocol's complete_request/complete_reply messages. Uses LSP-ready CompletionItem structure for future extensibility with additional completion sources (ruff, basedpyright). Follows the same request/response pattern as the history search feature (PR #320).

The frontend invokes complete_via_daemon() when the user triggers autocompletion (Ctrl+Space or after typing .), which sends a Complete request to the daemon. The kernel manager tracks pending requests and converts kernel matches to CompletionItem objects before returning results.

Closes #314

Verification

  • Completions appear when typing in code cells (Ctrl+Space or after .)
  • Completion popup shows available symbols/functions from the kernel
  • Timeout handling works if kernel doesn't respond (5-second timeout)
  • No kernel case returns appropriate error

PR submitted by @rgbkrk's agent, Quill

@rgbkrk rgbkrk force-pushed the quill/workspace-review branch from 644d4c6 to 5f0c838 Compare February 28, 2026 01:52
Implement code completion requests via the daemon using the Jupyter protocol's
complete_request/complete_reply messages. Uses LSP-ready CompletionItem
structure for future extensibility with additional completion sources (ruff,
basedpyright). Closes #314.
- Fix issue #1: Remove pending completion entry if shell.send() fails,
  preventing stale entries from accumulating
- Fix issue #3: Downgrade completion logging from info! to debug! to
  reduce noise from high-frequency autocomplete requests
@rgbkrk rgbkrk force-pushed the quill/workspace-review branch from 6af5d5f to f701f38 Compare February 28, 2026 02:49
@rgbkrk rgbkrk enabled auto-merge (squash) February 28, 2026 02:56
@rgbkrk rgbkrk merged commit bf792f6 into main Feb 28, 2026
6 checks passed
@rgbkrk rgbkrk deleted the quill/workspace-review branch February 28, 2026 03:02
@rgbkrk rgbkrk added quill PR authored by Quill Agent 🦆 daemon runtimed daemon, kernel management, sync server frontend Webview, React, TypeScript UI labels Mar 14, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

daemon runtimed daemon, kernel management, sync server frontend Webview, React, TypeScript UI quill PR authored by Quill Agent 🦆

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Support shell messages (history, completions) in daemon mode

1 participant