veai ← cleanup#26577
Merged
bryantgillespie merged 6 commits intobry/ai-visual-editor-contextfrom Feb 4, 2026
Hidden character warning
The head ref may contain hidden characters: "veai-\u2190-cleanup"
Merged
Conversation
formfcw
requested review from
bryantgillespie
and removed request for
AlexGaillard
Contributor
Code reviewNo issues found. Checked for bugs and CLAUDE.md compliance. |
AlexGaillard
pushed a commit
that referenced
this pull request
Feb 4, 2026
…tant Context (#26512) * wip * cleanup * fix: Hide button when AI_ENABLED = false * wip: add chat to visual editor module * fix: menu tweaks * cleanup more * fix: only refresh if mutation * merge menu components * use new menu component * cleanup more * add changeset * cleanup more * add some tests * fix: make resize handles visible again * extract format context to separate util * fix: items tool prompt causing issues * fix visual editing notifications route * refactor out visual elements tool in favor of context plus items tool * refactor to use visual editor config to handle highligthing * wip: context for prompts * Update format-context.ts Co-authored-by: Copilot <[email protected]> * Update use-search-filter.ts Co-authored-by: Copilot <[email protected]> * Update ai-prompt-variables-modal.vue Co-authored-by: Copilot <[email protected]> * remove logging * more fixes * formatter * add some lightweight xml prompt injection protection * failed to add to context notification * remove unneeded prop * appease ai overlords again * fix tests * actually fix tests * changeset past tense * part of claude code review * no proxy stores * more of claudes fixes * fix claudes composable * more of claudes fixes * fix focus * restore staged context if send message throws * simplify because nested field updates weren't working * add v-textoverflow * move display value to core responsiblity instead of being passed from ve package * fix test * show errors on submit * fix escaping * Update app/src/lang/translations/en-US.yaml Co-authored-by: Florian C. Wachmann <[email protected]> * Update app/src/ai/components/ai-context-menu.vue Co-authored-by: Florian C. Wachmann <[email protected]> * Update app/src/modules/content/routes/item.vue Co-authored-by: Florian C. Wachmann <[email protected]> * latest round of feedback * change id to key to match conventions and fix tool calling issues * make codebot happy * change it to test * fix test * fix translation * one more * veai ← cleanup (#26577) * refactor: replace useVisualEditorAi with useAiSidebar for clarity * fix: add type annotation for ComputedRef in useAiSidebar function * fix: add type annotations for sidebarSize and sidebarCollapsed emit events * add comment * remove comment (from html) * changeset: add usage notice for visual editing library --------- Co-authored-by: Copilot <[email protected]> Co-authored-by: Florian C. Wachmann <[email protected]>
AlexGaillard
pushed a commit
that referenced
this pull request
Feb 18, 2026
…tant Context (#26512) * wip * cleanup * fix: Hide button when AI_ENABLED = false * wip: add chat to visual editor module * fix: menu tweaks * cleanup more * fix: only refresh if mutation * merge menu components * use new menu component * cleanup more * add changeset * cleanup more * add some tests * fix: make resize handles visible again * extract format context to separate util * fix: items tool prompt causing issues * fix visual editing notifications route * refactor out visual elements tool in favor of context plus items tool * refactor to use visual editor config to handle highligthing * wip: context for prompts * Update format-context.ts Co-authored-by: Copilot <[email protected]> * Update use-search-filter.ts Co-authored-by: Copilot <[email protected]> * Update ai-prompt-variables-modal.vue Co-authored-by: Copilot <[email protected]> * remove logging * more fixes * formatter * add some lightweight xml prompt injection protection * failed to add to context notification * remove unneeded prop * appease ai overlords again * fix tests * actually fix tests * changeset past tense * part of claude code review * no proxy stores * more of claudes fixes * fix claudes composable * more of claudes fixes * fix focus * restore staged context if send message throws * simplify because nested field updates weren't working * add v-textoverflow * move display value to core responsiblity instead of being passed from ve package * fix test * show errors on submit * fix escaping * Update app/src/lang/translations/en-US.yaml Co-authored-by: Florian C. Wachmann <[email protected]> * Update app/src/ai/components/ai-context-menu.vue Co-authored-by: Florian C. Wachmann <[email protected]> * Update app/src/modules/content/routes/item.vue Co-authored-by: Florian C. Wachmann <[email protected]> * latest round of feedback * change id to key to match conventions and fix tool calling issues * make codebot happy * change it to test * fix test * fix translation * one more * veai ← cleanup (#26577) * refactor: replace useVisualEditorAi with useAiSidebar for clarity * fix: add type annotation for ComputedRef in useAiSidebar function * fix: add type annotations for sidebarSize and sidebarCollapsed emit events * add comment * remove comment (from html) * changeset: add usage notice for visual editing library --------- Co-authored-by: Copilot <[email protected]> Co-authored-by: Florian C. Wachmann <[email protected]>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Small cleanup PR. See the commits.