Skip to content

sync with original - #1

Closed
illyrius666 wants to merge 2 commits into
XodiumSoftware:mainfrom
zed-extensions:main
Closed

illyrius666 wants to merge 2 commits into
XodiumSoftware:mainfrom
zed-extensions:main

Conversation

@illyrius666

@illyrius666 illyrius666 commented Oct 11, 2026 •

Copy link
Copy Markdown
Member

Description

How Has This Been Tested?

  • Unit tests
  • Integration tests
  • Manual testing
  • Other (please describe)

Summary by CodeRabbit

  • New Features
    • Kotlin brackets and quotes can now autoclose before additional punctuation and closing characters, including semicolons, commas, and parentheses.
  • Chores
    • Updated the Kotlin extension version to 0.3.3.

Dofoerix and others added 2 commits September 28, 2026 11:40
Any other language extension has `autoclose_before = ";:.,=}])>"` line,
that fixes this issue

Examples:

```
println(|) // `|` is a cursor

println("|) // Typed double quote
println("|") // This is what expected

println(someList[|) // Same problem with other paired symbols
println(someList[|]) // Expected outcome
```
This PR bumps the version of this extension to v0.3.3

Co-authored-by: zed-zippy[bot] <234243425+zed-zippy[bot]@users.noreply.github.com>
@github-actions github-actions Bot changed the title sync with original main Oct 11, 2026
@github-actions

Copy link
Copy Markdown

🤖 I've automatically updated the PR title to match the branch name: main. This helps maintain consistency in our git history.

@coderabbitai

coderabbitai Bot commented Oct 11, 2026 •

Copy link
Copy Markdown

Review in Change Stack →

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration
  • Configuration used: Organization UI
  • Review profile: ASSERTIVE
  • Plan: Advanced
  • Run ID: 621465eb-6af3-4126-aad2-7e1bf8b009bd


📥 Commits

Reviewing files that changed from the base of the PR and between 88524b5 and 861761c.



⛔ Files ignored due to path filters (1)
  • Cargo.lock is excluded by !**/*.lock


📒 Files selected for processing (3)
  • Cargo.toml
  • extension.toml
  • languages/kotlin/config.toml


Included review availability: This review used your included allowance. Your plan provides up to 1 included review per hour; 0 remain after this review.




📝 Walkthrough
📝 Walkthrough

Walkthrough

The package and extension versions change from 0.3.2 to 0.3.3. The Kotlin language configuration adds autoclose_before with the value ;:.,=}])>.

Changes

Kotlin extension update

Layer / File(s) Summary
Version metadata and autoclose configuration
Cargo.toml, extension.toml, languages/kotlin/config.toml
The package and extension versions change to 0.3.3. The Kotlin configuration adds autoclose_before with the value ;:.,=}])>.

Priority: ⬇️ Low

Estimated code review effort: 1 (Trivial) | ~4 minutes

Change: Bug fix



Merge Risk: ⚪ Minimal · up to 86176

The described change is limited to Kotlin autoclose configuration and a matching version update; no actionable merge risk is apparent.

Pre-merge checks | Passed 3 | Failed 1 | Inconclusive 1

❌ Failed checks (1 warning, 1 inconclusive)

Check name Status Explanation Resolution
Description check Warning The description includes the required headings, but it contains no change summary and reports no test results. All test categories remain unchecked. Add a concise change summary and document the tests performed. Check the applicable test categories, or explain why no tests were run.
Title check Inconclusive The title relates to synchronizing the Kotlin extension, but it is vague and does not identify the version update or autoclosing change. Use a specific title that states the main change, such as "Sync Kotlin extension autoclosing behavior and version 0.3.3".
✅ Passed checks (3 passed)
Check name Status Explanation
Docstring Coverage Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 0…
Linked Issues check Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check Passed Check skipped because no linked issues were found for this pull request.

  • Fix all pre-merge checks with AI
✨ Finishing Touches 💡 1
⚔️ Resolve merge conflicts 💡
  • Resolve merge conflict in branch main


🧪 Generate unit tests (beta)
  • Create a new PR



  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

A rabbit checks the Kotlin settings with care.
New version numbers hop into place.
Semicolons and braces join the list.
A carrot waits beside the config.
The rabbit bounds off, pleased with the update.

Comment @coderabbitai help to get the list of available commands.

@illyrius666 illyrius666 changed the title main sync with original Oct 11, 2026
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