Skip to content

Conversation

siguenzaraul
Copy link
Contributor

@siguenzaraul siguenzaraul commented Sep 30, 2025

Description

I simplified the AIBlock code in the notes editor as much as possible.
Since the block’s content is no longer editable, there’s no need to configure an inner editor anymore.

Additionally, I moved the slash command logic into the aiBlock extension, introducing a new method in the extension that now handles the entire execution process.

Grabacion.de.pantalla.2025-10-01.a.las.17.04.25.mov

Copy link
Contributor

github-actions bot commented Sep 30, 2025

🔍 Visual review for your branch is published 🔍

Here are the links to:

Copy link
Contributor

github-actions bot commented Sep 30, 2025

size-limit report 📦

Path Size
JS: Stable 393.73 KB (-0.03% 🔽)
JS: Experimental 777.52 KB (-0.06% 🔽)
CSS 71.97 KB (0%)

@siguenzaraul siguenzaraul marked this pull request as ready for review October 1, 2025 14:38
@siguenzaraul siguenzaraul requested a review from a team as a code owner October 1, 2025 14:38
Copy link
Contributor

github-actions bot commented Oct 1, 2025

Coverage Report for packages/react

Status Category Percentage Covered / Total
🔵 Lines 20.21% 15560 / 76963
🔵 Statements 20.21% 15560 / 76963
🔵 Functions 42.3% 690 / 1631
🔵 Branches 69.68% 2138 / 3068
File Coverage
File Stmts Branches Functions Lines Uncovered Lines
Changed Files
packages/react/src/experimental/RichText/CoreEditor/Extensions/AIBlock/index.tsx 0% 0% 0% 0% 1-460
packages/react/src/experimental/RichText/CoreEditor/Extensions/SlashCommand/AvailableCommands.tsx 0% 100% 100% 0% 2-222
packages/react/src/experimental/RichText/NotesTextEditor/extensions.tsx 0% 0% 0% 0% 1-106
packages/react/src/experimental/RichText/NotesTextEditor/index.tsx 0% 0% 0% 0% 1-384
Generated in workflow #8297 for commit 3fa028b by the Vitest Coverage Report Action

@nlopin
Copy link
Contributor

nlopin commented Oct 1, 2025

a screenshot would be useful

@siguenzaraul
Copy link
Contributor Author

a screenshot would be useful

There is no visual change at all 😄

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