Skip to content

Fix clone 403, wrapped line redraw, mobile sidebar; add editor and file browser#3

Merged
pguyot merged 1 commit intomainfrom
claude/improve-git-client-Den5J
Mar 4, 2026
Merged

Fix clone 403, wrapped line redraw, mobile sidebar; add editor and file browser#3
pguyot merged 1 commit intomainfrom
claude/improve-git-client-Den5J

Conversation

@pguyot
Copy link
Owner

@pguyot pguyot commented Mar 4, 2026

Summary by CodeRabbit

Release Notes

  • New Features

    • Added an in-app file browser in the sidebar for viewing and navigating files
    • Integrated file editor overlay with save and close functionality
    • Introduced edit and browse commands for quick file and repository access
  • Improvements

    • Enhanced input cursor positioning for multi-line text handling
    • Optimized responsive layout for mobile and small screens
    • Improved data refresh when credentials or repository settings change

…le browser

Bug fixes:
- Fix double CORS proxy: http handler no longer prepends proxy URL since
  isomorphic-git's corsProxy option already does this (caused 403 errors)
- Fix redrawInput to handle line wrapping: uses \x1b[J (clear to end of
  screen) and tracks cursor line position instead of single-line \x1b[2K
- Fix left/right arrow keys to use redrawInput for correct wrapping
- Fix mobile sidebar: use !important to override base display:flex rule
- Remove terminal prints from UI button handlers (refresh, cred save)
  to prevent interference with command line input

New features:
- File browser in sidebar: shows current directory files, click folders
  to navigate, click files to edit
- File editor modal: edit command (also vi/vim/nano aliases) opens a
  textarea editor with save support and tab key insertion
- browse command: shortcut to open GitHub repo browser

https://claude.ai/code/session_018MffwgAzpWYMNRHB6FB41F
@pguyot pguyot merged commit 1018454 into main Mar 4, 2026
1 check was pending
@coderabbitai
Copy link

coderabbitai bot commented Mar 4, 2026

Caution

Review failed

The pull request is closed.

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: a1a91ff7-7317-47be-a81e-4d12d8818e45

📥 Commits

Reviewing files that changed from the base of the PR and between 0cc3765 and fbb0557.

📒 Files selected for processing (2)
  • app.js
  • index.html

📝 Walkthrough

Walkthrough

This pull request introduces in-app file editing and browsing functionality. New shell commands (edit/nano/vi/vim/browse) are wired to editor and repository browser UI overlays. The HTML adds a file list panel and editor modal; JavaScript implements file tracking, cursor positioning for multi-line input, editor state management, and command dispatch handlers.

Changes

Cohort / File(s) Summary
Editor and File Browser Integration
app.js
Added ANSI-stripping helper stripAnsi(), multi-line cursor tracking with inputCursorLine, and editor UI functions (updateFileList(), openEditor(), setupEditorUI()). Implemented file state tracking via currentEditFile and integrated editor initialization into DOM setup. Extended command dispatcher with edit/nano/vi/vim and browse cases; modified updateSidebar() to refresh file list. Cache invalidation added for credential/repo state changes.
HTML UI Components
index.html
Added Files section in sidebar with file list panel (#file-list); new File Editor Modal overlay with filename label, resizable textarea, and Save/Close actions. Enhanced responsive layout with mobile/small-screen adjustments (sidebar visibility, header scaling, modal widths, element wrapping).

Sequence Diagram

sequenceDiagram
    actor User
    participant CommandProcessor as Command Processor
    participant Editor as Editor UI
    participant FileSystem as File System

    User->>CommandProcessor: Enter "edit filename"
    CommandProcessor->>Editor: openEditor(filePath)
    Editor->>FileSystem: Read file contents
    FileSystem-->>Editor: File data
    Editor->>Editor: Display editor overlay with content
    User->>Editor: Edit text + Save
    Editor->>FileSystem: Write updated content
    FileSystem-->>Editor: Confirmation
    Editor->>Editor: Close overlay / Update sidebar
Loading

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~25 minutes

Poem

🐰 A whisker-twitch of joy—new commands take flight!
Edit files inline, browse repos with delight.
Cursor lines dancing, modals overlay bright,
The warren grows richer with features this night! ✨

✨ Finishing Touches
  • 📝 Generate docstrings (stacked PR)
  • 📝 Generate docstrings (commit on current branch)
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch claude/improve-git-client-Den5J

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

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