Skip to content

Conversation

wenzhengjiang
Copy link
Collaborator

@wenzhengjiang wenzhengjiang commented Aug 31, 2025

In in non-agent mode, the composer code block can be either

```xml
CODE BLOCK
```

or
```
CODE BLOCK
```

We need to support both

Description by Korbit AI

What change is being made?

Improve the regex for matching composer codeblocks, enforce consistent logging via logger.ts, and update versioning to reflect accurate package information.

Why are these changes being made?

The regex update allows for more flexible XML codeblock matching in ChatSingleMessage.tsx, the use of logger.ts standardizes error logging in ComposerTools.ts improving maintainability, and the versioning adjustments correct a previous oversight in manifest.json, package.json, and versions.json to ensure consistency and accuracy in version tracking.

Is this description stale? Ask me to generate a new description by commenting /korbit-generate-pr-description

@wenzhengjiang wenzhengjiang changed the title Improve the regex for composer codeblock FIX: Improve the regex for composer codeblock Aug 31, 2025
Copy link

@korbit-ai korbit-ai bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I've completed my review and didn't find any issues... but I did find this owl.

 ,_,
(.,.)
(   )
-"-"---
Files scanned
File Path Reviewed
src/tools/ComposerTools.ts
src/components/chat-components/ChatSingleMessage.tsx

Explore our documentation to understand the languages and file types we support and the files we ignore.

Check out our docs on how you can make Korbit work best for you and your team.

Loving Korbit!? Share us on LinkedIn Reddit and X

@logancyang logancyang merged commit 9239dab into master Sep 3, 2025
3 checks passed
@logancyang logancyang deleted the fix_composer_tag branch September 3, 2025 20:51
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