Skip to content

[Remote envs] codebase indexing UI#10889

Merged
moirahuang merged 16 commits into
masterfrom
moira/codebase-indexing-ui
May 15, 2026
Merged

[Remote envs] codebase indexing UI#10889
moirahuang merged 16 commits into
masterfrom
moira/codebase-indexing-ui

Conversation

@moirahuang
Copy link
Copy Markdown
Contributor

Description

Add UI for remote codebase indexing repos

Testing

Locally tested + added unit tests

  • I have manually tested my changes locally with ./script/run

Screenshots / Videos

image (1)

Agent Mode

  • Warp Agent Mode - This PR was created via Warp's AI Agent Mode

@cla-bot cla-bot Bot added the cla-signed label May 14, 2026
@oz-for-oss
Copy link
Copy Markdown
Contributor

oz-for-oss Bot commented May 14, 2026

@moirahuang

I'm starting a first review of this pull request.

You can view the conversation on Warp.

I completed the review and no human review was requested for this pull request.

Comment /oz-review on this pull request to retrigger a review (up to 3 times on the same pull request).

Powered by Oz

@moirahuang moirahuang force-pushed the moira/codebase-indexing-ui branch from e6a461c to 12efd0b Compare May 14, 2026 03:18
@moirahuang moirahuang requested a review from kevinyang372 May 14, 2026 03:22
Copy link
Copy Markdown
Contributor

@oz-for-oss oz-for-oss Bot left a comment

Choose a reason for hiding this comment

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

Overview

This PR adds remote codebase indexing entries to the Code settings UI, model events for settings updates, host labels for remote hosts, and related tests. The attached screenshot satisfies the repo's visual-evidence requirement.

Concerns

  • After deleting a remote codebase index, the daemon returns a Disabled status, but the new UI hides the retry/resync action for that state. That leaves the row visible but only deletable again, so users cannot recreate the index from this settings surface.

Verdict

Found: 0 critical, 1 important, 0 suggestions

Request changes

Comment /oz-review on this pull request to retrigger a review (up to 3 times on the same pull request).

Powered by Oz

Comment thread app/src/settings_view/code_page.rs Outdated
@moirahuang moirahuang force-pushed the moira/codebase-indexing-ui branch from 1f6b3c8 to 452ac8a Compare May 14, 2026 16:45
@moirahuang moirahuang changed the base branch from master to moira/codebase-indexing-persistence May 14, 2026 16:45
@moirahuang moirahuang force-pushed the moira/codebase-indexing-ui branch from 452ac8a to 18cc005 Compare May 14, 2026 16:52
@moirahuang moirahuang force-pushed the moira/codebase-indexing-persistence branch from 8f2f718 to 91e5c0d Compare May 14, 2026 17:30
@moirahuang moirahuang force-pushed the moira/codebase-indexing-ui branch from 18cc005 to 65fa4ca Compare May 14, 2026 17:45
Copy link
Copy Markdown
Member

@kevinyang372 kevinyang372 left a comment

Choose a reason for hiding this comment

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

Approving to unblock

Comment thread app/src/remote_server/codebase_index_model.rs Outdated
Comment thread app/src/terminal/writeable_pty/remote_server_controller.rs Outdated
Comment thread crates/remote_server/src/manager.rs Outdated
Comment thread app/src/settings_view/code_page.rs Outdated
Comment thread app/src/settings_view/code_page.rs
Comment thread app/src/settings_view/code_page.rs
@moirahuang moirahuang force-pushed the moira/codebase-indexing-ui branch from 0182419 to 9970331 Compare May 15, 2026 00:25
@moirahuang moirahuang changed the base branch from moira/codebase-indexing-persistence to master May 15, 2026 00:25
@moirahuang moirahuang force-pushed the moira/codebase-indexing-ui branch from 9970331 to e21cf86 Compare May 15, 2026 00:26
@moirahuang moirahuang enabled auto-merge (squash) May 15, 2026 00:28
@moirahuang moirahuang force-pushed the moira/codebase-indexing-ui branch from e21cf86 to 64c23fe Compare May 15, 2026 00:34
@moirahuang
Copy link
Copy Markdown
Contributor Author

i'm going to wait to merge this because it's currently stacked on top of the persistence PR and i don't want to make that PR harder to review

@moirahuang moirahuang disabled auto-merge May 15, 2026 00:34
@moirahuang moirahuang changed the base branch from master to moira/codebase-indexing-persistence May 15, 2026 00:35
@moirahuang moirahuang force-pushed the moira/codebase-indexing-persistence branch from 2c36de0 to af4199e Compare May 15, 2026 18:17
Base automatically changed from moira/codebase-indexing-persistence to master May 15, 2026 19:13
@moirahuang moirahuang force-pushed the moira/codebase-indexing-ui branch 2 times, most recently from df96ad8 to 26473fc Compare May 15, 2026 21:57
@moirahuang moirahuang force-pushed the moira/codebase-indexing-ui branch from 26473fc to f55b80e Compare May 15, 2026 22:00
@moirahuang moirahuang merged commit aa9f908 into master May 15, 2026
25 checks passed
@moirahuang moirahuang deleted the moira/codebase-indexing-ui branch May 15, 2026 23:24
lawsmd pushed a commit to lawsmd/cortex that referenced this pull request May 22, 2026
## Description
Add UI for remote codebase indexing repos

## Testing
Locally tested + added unit tests
- [x] I have manually tested my changes locally with `./script/run`

### Screenshots / Videos
<img width="841" height="550" alt="image (1)"
src="https://github.com/user-attachments/assets/953b924a-7dc0-4b50-b799-fdde96df8889"
/>

## Agent Mode
- [x] Warp Agent Mode - This PR was created via Warp's AI Agent Mode

---------

Co-authored-by: Oz <oz-agent@warp.dev>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants