Skip to content

feat(vscode): support multilingual notebook autocomplete - #11631

Merged
markijbema merged 3 commits into
mainfrom
mark/multilingual-notebook-autocomplete
Jul 8, 2026
Merged

feat(vscode): support multilingual notebook autocomplete#11631
markijbema merged 3 commits into
mainfrom
mark/multilingual-notebook-autocomplete

Conversation

@markijbema

Copy link
Copy Markdown
Contributor

Summary

  • Enable notebook autocomplete for every language represented in the existing autocomplete language catalog rather than limiting it to Python.
  • Build whole-notebook context in the active cell's language, commenting Markdown and foreign-language cells where syntax permits and preserving cursor offsets when content must be omitted.
  • Carry the active VS Code language ID through prompt construction so .ipynb paths no longer force Python-specific prompting and filtering.

This follows #11597 and keeps mixed-language notebooks from injecting foreign code as executable context.

Co-authored-by: kiloconnect[bot] <240665456+kiloconnect[bot]@users.noreply.github.com>
@kilo-code-bot

kilo-code-bot Bot commented Jun 24, 2026

Copy link
Copy Markdown
Contributor

Code Review Summary

Status: No Issues Found | Recommendation: Merge

Files Reviewed (2 files)
  • .changeset/bright-notebooks-complete.md
  • .changeset/multilingual-notebook-autocomplete.md
Previous Review Summaries (2 snapshots, latest commit 31ee9a6)

Current summary above is authoritative. Previous snapshots are kept for context only.

Previous review (commit 31ee9a6)

Status: No Issues Found | Recommendation: Merge

Files Reviewed (2 files)
  • packages/kilo-vscode/src/services/autocomplete/continuedev/core/autocomplete/notebook.ts
  • packages/kilo-vscode/tests/unit/notebook-context.test.ts

Previous review (commit 08577b2)

Status: 1 Issues Found | Recommendation: Address before merge

Overview

Severity Count
CRITICAL 0
WARNING 1
SUGGESTION 0

Fix these issues in Kilo Cloud

Issue Details (click to expand)

WARNING

File Line Issue
packages/kilo-vscode/src/services/autocomplete/continuedev/core/autocomplete/notebook.ts 97 Sibling JSON/JSONC cells are still concatenated verbatim, which can produce invalid single-document notebook context.
Files Reviewed (9 files)
  • .changeset/bright-notebooks-complete.md - 0 issues
  • packages/kilo-vscode/src/services/autocomplete/continuedev/core/autocomplete/constants/AutocompleteLanguageInfo.test.ts - 0 issues
  • packages/kilo-vscode/src/services/autocomplete/continuedev/core/autocomplete/constants/AutocompleteLanguageInfo.ts - 0 issues
  • packages/kilo-vscode/src/services/autocomplete/continuedev/core/autocomplete/notebook.ts - 1 issue
  • packages/kilo-vscode/src/services/autocomplete/continuedev/core/autocomplete/templating/constructPrefixSuffix.ts - 0 issues
  • packages/kilo-vscode/src/services/autocomplete/continuedev/core/autocomplete/util/HelperVars.ts - 0 issues
  • packages/kilo-vscode/src/services/autocomplete/continuedev/core/autocomplete/util/types.ts - 0 issues
  • packages/kilo-vscode/src/services/autocomplete/types.ts - 0 issues
  • packages/kilo-vscode/tests/unit/notebook-context.test.ts - 0 issues

Reviewed by claude-sonnet-5-20260630 · Input: 30 · Output: 8.6K · Cached: 635.9K

Review guidance: REVIEW.md from base branch main

markijbema and others added 2 commits June 24, 2026 10:53
Co-authored-by: kiloconnect[bot] <240665456+kiloconnect[bot]@users.noreply.github.com>
@markijbema
markijbema enabled auto-merge July 8, 2026 11:05
@markijbema
markijbema merged commit 86498df into main Jul 8, 2026
21 checks passed
@markijbema
markijbema deleted the mark/multilingual-notebook-autocomplete branch July 8, 2026 12:06
t7tran pushed a commit to t7tran/kilocode that referenced this pull request Aug 14, 2026
…ebook-autocomplete

feat(vscode): support multilingual notebook autocomplete
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.

2 participants