perf(ui): split right panel and secondary viewer chunks#1
Open
pascalandr wants to merge 1 commit intodevfrom
Open
perf(ui): split right panel and secondary viewer chunks#1pascalandr wants to merge 1 commit intodevfrom
pascalandr wants to merge 1 commit intodevfrom
Conversation
pascalandr
force-pushed
the
ready/ui-secondary-chunks
branch
2 times, most recently
from
d52b4cc to
e71f103
Compare
pascalandr
force-pushed
the
ready/ui-markdown-diff-perf
branch
from
16ee70f to
f1696aa
Compare
pascalandr
force-pushed
the
ready/ui-secondary-chunks
branch
from
e71f103 to
1cabeca
Compare
Author
|
Restacked this on top of the rebuilt ready/ui-markdown-diff-perf branch and marked it ready for review. The branch now only contains the secondary chunk and right panel follow-up changes on top of the refreshed base. |
Author
|
Restacked this again on top of the rebuilt ready/ui-markdown-diff-perf branch. This follow-up now sits on the refreshed 215 base and is ready for review. |
pascalandr
force-pushed
the
ready/ui-secondary-chunks
branch
from
1cabeca to
80bc741
Compare
pascalandr
force-pushed
the
ready/ui-secondary-chunks
branch
from
80bc741 to
d707166
Compare
Author
|
I reshuffled this branch so it no longer carries the Monaco-specific chunking. That part now lives in PR 215. This branch is now focused on the remaining right panel, picker, and tool-call secondary chunking work, and it now targets dev directly instead of stacking on PR 215. |
pascalandr
pushed a commit
that referenced
this pull request
Mar 22, 2026
…#235) ## What Fix slash command execution so `[pasted #N]` placeholders are resolved before calling `session.command`, matching normal prompt send behavior. ## Why When pasting long text into a slash command (e.g. `/some-command [pasted #1]`), the UI previously bypassed `resolvePastedPlaceholders(...)` for known slash commands and sent the literal placeholder text as command arguments. ## Changes - Resolve pasted placeholders (and other prompt placeholders handled by `resolvePastedPlaceholders`) in slash-command arguments before `executeCustomCommand(...)`. - Remove *consumed* pasted-text attachments (those referenced by placeholders in the slash-command args) so they don’t linger for the next prompt. Fixes NeuralNomadsAI#234. ## Notes - I attempted `npm run typecheck --workspace @codenomad/ui` locally but the workspace dependencies aren’t installed in this bot environment, so it fails with missing-module errors. CI should validate with a full install. -- Yours, [CodeNomadBot](https://github.com/NeuralNomadsAI/CodeNomad) Co-authored-by: Shantur Rathore <i@shantur.com>
shantur
pushed a commit
to NeuralNomadsAI/CodeNomad
that referenced
this pull request
Mar 22, 2026
## Summary - lazy-load the markdown and diff render paths so they stop inflating initial UI startup work - move shared text rendering helpers out of the markdown path and keep diff rendering on the deferred path - defer the Monaco secondary viewers so the markdown and diff path no longer keeps that work in the main bundle ## Follow-ups - related fork follow-up: Pagecran#1 - that follow-up is now independent on dev and only keeps the remaining right panel, picker, and tool-call secondary chunking work ## Testing - npm run typecheck --workspace @codenomad/ui - npm run build --workspace @codenomad/ui
pascalandr
force-pushed
the
dev
branch
2 times, most recently
from
577f30b to
c9c1cf2
Compare
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
1 participant
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.
Summary
Testing
Notes