Skip to content

fix: align the directory tree with the sidebar - #70

Merged
naaiyy merged 2 commits into
mainfrom
agent/directory-tree-consistency
Aug 14, 2026
Merged

fix: align the directory tree with the sidebar#70
naaiyy merged 2 commits into
mainfrom
agent/directory-tree-consistency

Conversation

@naaiyy

@naaiyy naaiyy commented Aug 14, 2026

Copy link
Copy Markdown
Collaborator

Summary

  • match directory rows to sidebar row height, spacing, and indentation
  • keep opened directory entries selected and add file-type icons
  • preserve custom-block context menus without reloading the viewer
  • replace the local OpenEditor patch with published 0.0.48 packages

Validation

  • bun install --frozen-lockfile
  • bun test apps/web/features/libraries/library-file-icon.test.ts
  • bun --cwd apps/web check-types
  • bun --cwd apps/web build
  • browser regression check for right-click context menus

@vercel

vercel Bot commented Aug 14, 2026

Copy link
Copy Markdown
Contributor

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
base-blocks Ready Ready Preview Aug 14, 2026 8:54pm

Request Review

@github-actions

github-actions Bot commented Aug 14, 2026

Copy link
Copy Markdown

React Doctor found 1 new issue in 1 file · 1 warning · score 85 / 100 (Great) · 0 fixed · vs main

1 warning

features/libraries/library-file-icon.tsx

  • ⚠️ L31 Non-component export in component file only-export-components

Reviewed by React Doctor for commit 2189ee1. See inline comments for fixes.

@naaiyy
naaiyy force-pushed the agent/directory-tree-consistency branch from b36a0dd to 2189ee1 Compare August 14, 2026 20:53

const extensionPattern = /\.([^.]+)$/;

export function getLibraryFileIconKind(

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

React Doctor · react-doctor/only-export-components (warning)

This file exports non-components, so Fast Refresh can't safely preserve component state.

Fix → Move non-component exports out of component files so Fast Refresh can preserve component state instead of full-reloading.

Docs

@naaiyy
naaiyy merged commit 0d9a68f into main Aug 14, 2026
9 checks passed
@naaiyy
naaiyy deleted the agent/directory-tree-consistency branch August 14, 2026 20:56
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.

1 participant