Skip to content

Conversation

@sriramveeraghanta
Copy link
Member

@sriramveeraghanta sriramveeraghanta commented Feb 10, 2026

Description

  • remove ee folder from the web app

Type of Change

  • Code refactoring

Summary by CodeRabbit

  • Refactor

    • Updated internal module import paths for improved code organization.
    • Removed redundant re-export layers to streamline the codebase architecture.
    • Simplified path alias configuration for better maintainability.
  • Chores

    • Cleaned up unused module files as part of ongoing code structure optimization.

Copilot AI review requested due to automatic review settings February 10, 2026 14:02
@coderabbitai
Copy link
Contributor

coderabbitai bot commented Feb 10, 2026

📝 Walkthrough

Walkthrough

Large-scale refactoring removing re-export barrel files across the EE layer and consolidating import paths. Eliminates wildcard re-exports from EE components, stores, services, helpers, hooks, types, and layouts directories. Updates import paths to direct module sources instead of through re-export aliases.

Changes

Cohort / File(s) Summary
Import Path Updates
apps/web/core/components/issues/issue-modal/context/issue-modal-context.tsx, apps/web/core/components/settings/profile/.../preferences/default-list.tsx, apps/web/core/store/workspace/home.ts, apps/web/core/store/workspace/index.ts, apps/web/core/store/workspace/link.store.ts
Updated import sources to use absolute path aliases (e.g., @/plane-web/*, @/services/workspace.service) instead of relative or alternate paths.
EE Component Re-exports Removal
apps/web/ee/components/active-cycles/..., apps/web/ee/components/app-rail/..., apps/web/ee/components/comments/..., apps/web/ee/components/common/..., apps/web/ee/components/cycles/..., apps/web/ee/components/estimates/..., apps/web/ee/components/gantt-chart/..., apps/web/ee/components/global/..., apps/web/ee/components/home/..., apps/web/ee/components/inbox/..., apps/web/ee/components/instance/..., apps/web/ee/components/issues/issue-details/..., apps/web/ee/components/navigations/..., apps/web/ee/components/pages/..., apps/web/ee/components/projects/..., apps/web/ee/components/relations/..., apps/web/ee/components/sidebar/..., apps/web/ee/components/views/..., apps/web/ee/components/workflow/...
Deleted or emptied re-export barrel files and index files. Removed export * from "ce/components/..." statements and associated license headers.
EE Constants/Helpers Re-exports Removal
apps/web/ee/constants/..., apps/web/ee/helpers/...
Removed public re-exports from constants (sidebar-favorites) and helpers (command-palette, epic-analytics, instance.helper, issue-action-helper, issue-filter.helper, project-settings, workspace.helper). Deleted files and index re-export statements.
EE Hooks Re-exports Removal
apps/web/ee/hooks/...
Eliminated re-export files and index barrels for hooks (app-rail, store, use-additional-editor-mention, use-additional-favorite-item-details, use-editor-flagging, use-file-size, use-issue-embed, use-issue-properties, use-page-flag, use-timeline-chart).
EE Store Re-exports Removal
apps/web/ee/store/...
Removed re-export barrels and public exports from store modules including analytics.store, command-palette.store, cycle, estimates, issue helpers and details, member, project-inbox, root, state, timeline, and user permission stores.
EE Services/Types/Layouts Re-exports Removal
apps/web/ee/services/..., apps/web/ee/types/..., apps/web/ee/layouts/...
Deleted re-export index files and removed public re-exports from project estimate service, project-state service, type indices, and layout wrappers (project-wrapper, workspace-wrapper).
Configuration
apps/web/tsconfig.json
Removed path alias mappings for app/* and ce/* from TypeScript configuration. Retained @/plane-web/* mapping to ./ce/*.

Estimated code review effort

🎯 4 (Complex) | ⏱️ ~50 minutes

Poem

🐰 The warren grew tangled with re-export chains,
So we hopped through and trimmed all the refrain,
Direct paths now gleam where barrels once lay,
Simpler imports light up the way!
thump thump 🌟

🚥 Pre-merge checks | ✅ 2 | ❌ 1
❌ Failed checks (1 warning)
Check name Status Explanation Resolution
Description check ⚠️ Warning The PR description is largely incomplete. While it mentions removing the ee folder, it lacks detail about the scope, impact, and testing. The Type of Change section is present but no option is selected despite the work being primarily code refactoring. Expand the description to explain why the ee folder is being removed, document the scope of changes, and select 'Code refactoring' in the Type of Change section. Add test scenarios and any relevant references.
✅ Passed checks (2 passed)
Check name Status Explanation
Title check ✅ Passed The title 'fix: remove ee folder from web' clearly and concisely describes the primary change of removing the ee folder from the web application.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing touches
  • 📝 Generate docstrings
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch chore-merge-folders

No actionable comments were generated in the recent review. 🎉

🧹 Recent nitpick comments
apps/web/core/components/issues/issue-modal/context/issue-modal-context.tsx (1)

8-9: Stale comment: // ce imports no longer applies.

The TIssueFields import that previously came from ce/... has moved to the @/plane-web/... section. Line 9 imports from react-hook-form, so this section comment is now misleading.

Proposed fix
-// ce imports
-import type { UseFormReset, UseFormWatch } from "react-hook-form";
+// react-hook-form imports
+import type { UseFormReset, UseFormWatch } from "react-hook-form";

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.

Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Removes the (now-unused) apps/web/ee layer from the web app and updates a few remaining imports/TS path aliases to rely on the existing core and ce (“plane-web”) code paths.

Changes:

  • Delete the apps/web/ee/** shims (mostly re-export files) to eliminate the EE folder from apps/web.
  • Simplify apps/web/tsconfig.json path mappings by removing unused aliases.
  • Update a handful of core imports to use stable @/services/... / @/plane-web/... paths.

Reviewed changes

Copilot reviewed 95 out of 96 changed files in this pull request and generated no comments.

Show a summary per file
File Description
apps/web/tsconfig.json Removes unused TS path aliases and keeps @/plane-web/* mapping to ce.
apps/web/ee/types/projects/projects.ts Removes EE re-export shim.
apps/web/ee/types/projects/index.ts Removes EE re-export shim.
apps/web/ee/types/issue-types/issue-property-values.d.ts Removes EE re-export shim.
apps/web/ee/types/issue-types/index.ts Removes EE re-export shim.
apps/web/ee/types/index.ts Removes EE re-export shim.
apps/web/ee/store/user/permission.store.ts Removes EE re-export shim.
apps/web/ee/store/timeline/base-timeline.store.ts Removes EE re-export shim.
apps/web/ee/store/state.store.ts Removes EE re-export shim.
apps/web/ee/store/root.store.ts Removes EE re-export shim.
apps/web/ee/store/project-inbox.store.ts Removes EE re-export shim.
apps/web/ee/store/member/project-member.store.ts Removes EE re-export shim.
apps/web/ee/store/issue/team/index.ts Removes EE re-export shim.
apps/web/ee/store/issue/team-views/index.ts Removes EE re-export shim.
apps/web/ee/store/issue/issue-details/root.store.ts Removes EE re-export shim.
apps/web/ee/store/issue/issue-details/activity.store.ts Removes EE re-export shim.
apps/web/ee/store/issue/helpers/base-issue.store.ts Removes EE re-export shim.
apps/web/ee/store/estimates/estimate.ts Removes EE re-export shim.
apps/web/ee/store/cycle/index.ts Removes EE re-export shim.
apps/web/ee/store/command-palette.store.ts Removes EE re-export shim.
apps/web/ee/store/analytics.store.ts Removes EE-specific wrapper store.
apps/web/ee/services/project/project-state.service.ts Removes EE re-export shim.
apps/web/ee/services/project/index.ts Removes EE re-export shim.
apps/web/ee/services/project/estimate.service.ts Removes EE re-export shim.
apps/web/ee/services/index.ts Removes EE re-export shim.
apps/web/ee/layouts/workspace-wrapper.tsx Removes EE re-export shim.
apps/web/ee/layouts/project-wrapper.tsx Removes EE re-export shim.
apps/web/ee/hooks/use-timeline-chart.ts Removes EE re-export shim.
apps/web/ee/hooks/use-page-flag.ts Removes EE re-export shim.
apps/web/ee/hooks/use-issue-properties.tsx Removes EE re-export shim.
apps/web/ee/hooks/use-issue-embed.tsx Removes EE re-export shim.
apps/web/ee/hooks/use-file-size.ts Removes EE re-export shim.
apps/web/ee/hooks/use-editor-flagging.ts Removes EE re-export shim.
apps/web/ee/hooks/use-additional-favorite-item-details.ts Removes EE re-export shim.
apps/web/ee/hooks/use-additional-editor-mention.tsx Removes EE re-export shim.
apps/web/ee/hooks/store/index.ts Removes EE re-export shim.
apps/web/ee/hooks/app-rail/index.ts Removes EE re-export shim.
apps/web/ee/helpers/workspace.helper.ts Removes EE re-export shim.
apps/web/ee/helpers/project-settings.ts Removes EE re-export shim.
apps/web/ee/helpers/issue-filter.helper.ts Removes EE re-export shim.
apps/web/ee/helpers/issue-action-helper.ts Removes EE re-export shim.
apps/web/ee/helpers/instance.helper.ts Removes EE re-export shim.
apps/web/ee/helpers/epic-analytics.ts Removes EE re-export shim.
apps/web/ee/helpers/command-palette.ts Removes EE re-export shim.
apps/web/ee/constants/sidebar-favorites.ts Removes EE re-export shim.
apps/web/ee/constants/project/settings/index.ts Removes EE re-export shim.
apps/web/ee/constants/project/settings/features.tsx Removes EE re-export shim.
apps/web/ee/constants/project/index.ts Removes EE re-export shim.
apps/web/ee/constants/index.ts Removes EE re-export shim.
apps/web/ee/components/workflow/index.ts Removes EE re-export shim.
apps/web/ee/components/views/publish/index.ts Removes EE re-export shim.
apps/web/ee/components/views/filters/access-filter.tsx Removes EE re-export shim.
apps/web/ee/components/views/access-controller.tsx Removes EE re-export shim.
apps/web/ee/components/sidebar/index.ts Removes EE re-export shim.
apps/web/ee/components/relations/index.tsx Removes EE re-export shim.
apps/web/ee/components/projects/settings/useProjectColumns.tsx Removes EE re-export shim.
apps/web/ee/components/projects/settings/features-list.tsx Removes EE shim export (points to core component).
apps/web/ee/components/projects/page.tsx Removes EE re-export shim.
apps/web/ee/components/projects/mobile-header.tsx Removes EE re-export shim.
apps/web/ee/components/projects/header.tsx Removes EE re-export shim.
apps/web/ee/components/projects/create/root.tsx Removes EE re-export shim.
apps/web/ee/components/projects/create/attributes.tsx Removes EE re-export shim.
apps/web/ee/components/pages/index.ts Removes EE re-export shim.
apps/web/ee/components/navigations/index.ts Removes EE re-export shim.
apps/web/ee/components/issues/issue-details/sidebar/transfer-hop-info.tsx Removes EE-only file/header stub.
apps/web/ee/components/instance/maintenance-message.tsx Removes EE re-export shim.
apps/web/ee/components/instance/index.ts Removes EE re-export shim.
apps/web/ee/components/inbox/source-pill.tsx Removes EE re-export shim.
apps/web/ee/components/home/index.ts Removes EE re-export shim.
apps/web/ee/components/home/header.tsx Removes unused EE placeholder component.
apps/web/ee/components/global/index.ts Removes EE re-export shim.
apps/web/ee/components/gantt-chart/index.ts Removes EE re-export shim.
apps/web/ee/components/gantt-chart/blocks/blocks-list.tsx Removes EE re-export shim.
apps/web/ee/components/gantt-chart/blocks/block-row-list.tsx Removes EE re-export shim.
apps/web/ee/components/estimates/update/modal.tsx Removes EE re-export shim.
apps/web/ee/components/estimates/update/index.ts Removes EE re-export shim.
apps/web/ee/components/estimates/points/index.ts Removes EE re-export shim.
apps/web/ee/components/estimates/points/delete.tsx Removes EE re-export shim.
apps/web/ee/components/estimates/index.ts Removes EE re-export shim.
apps/web/ee/components/estimates/estimate-list-item-buttons.tsx Removes EE re-export shim.
apps/web/ee/components/cycles/index.ts Removes EE re-export shim.
apps/web/ee/components/cycles/end-cycle/index.ts Removes EE re-export shim.
apps/web/ee/components/cycles/analytics-sidebar/index.ts Removes EE re-export shim.
apps/web/ee/components/cycles/active-cycle/index.ts Removes EE re-export shim.
apps/web/ee/components/common/index.ts Removes EE re-export shim.
apps/web/ee/components/common/extended-app-header.tsx Removes EE re-export shim.
apps/web/ee/components/comments/index.ts Removes EE re-export shim.
apps/web/ee/components/app-rail/index.ts Removes EE re-export shim.
apps/web/ee/components/active-cycles/workspace-active-cycles-upgrade.tsx Removes EE re-export shim.
apps/web/ee/components/active-cycles/root.tsx Removes EE re-export shim.
apps/web/ee/components/active-cycles/index.ts Removes EE re-export shim.
apps/web/core/store/workspace/link.store.ts Switches WorkspaceService import to core services path.
apps/web/core/store/workspace/index.ts Switches WorkspaceService import to core services path.
apps/web/core/store/workspace/home.ts Switches WorkspaceService import to core services path.
apps/web/core/components/settings/profile/content/pages/preferences/default-list.tsx Updates ThemeSwitcher import to use @/plane-web/... instead of removed ce/... alias.
apps/web/core/components/issues/issue-modal/context/issue-modal-context.tsx Updates TIssueFields type import to use @/plane-web/... instead of removed ce/... alias.

@sriramveeraghanta sriramveeraghanta merged commit 06e4a16 into preview Feb 10, 2026
18 of 20 checks passed
@sriramveeraghanta sriramveeraghanta deleted the chore-merge-folders branch February 10, 2026 14:31
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