refactor(web): remove legacy editor

[andrewmelchor]

Summary

This PR removes the legacy .http code editor from the HTTP route in packages/web, so the Request Workspace (Params / Headers / Body) is now the only authoring surface for HTTP requests in the left pane.

Why

Prevents dual-editing surfaces (workspace tabs + raw editor) that can drift and confuse save/execute behavior.
Aligns the UI with the phased request-workspace rollout where tab-based request editing is the primary flow.

[greptile-apps]

Greptile Summary

This PR removes the legacy HttpEditor (raw .http code editor) from the HTTP route in EditorWithExecution.tsx, making RequestWorkspaceTabs the sole authoring surface for HTTP requests in the left pane. The render block and import are cleanly removed, and the container div's layout classes are updated (flex flex-col → overflow-auto) to accommodate the workspace-only layout.

Key findings:

• The HttpEditor component and its supporting files (HttpEditor.tsx, the index.ts re-export, and the lazy.ts lazy wrapper) are not deleted and remain as dead code. The component's heavy dependencies (codemirror, all @codemirror/* plugins) will continue to be included in the bundle unless these are removed.
• The layout change on the left-pane container (flex h-full min-h-0 flex-col → h-full min-h-0 overflow-auto) is intentional and consistent with the removal of the flex-sibling HttpEditor block, but should be verified visually since it changes the layout model for RequestWorkspaceTabs.

Confidence Score: 3/5

• Safe to merge for functionality, but the incomplete cleanup leaves dead code and prevents full bundle savings.
• The active code path change is straightforward and low-risk — the HttpEditor render block is simply removed. However, the refactor is incomplete: HttpEditor.tsx, its export in index.ts, and its lazy wrapper in lazy.ts are all left behind, meaning the stated goal of removing the legacy editor is only half-done and the associated codemirror dependencies remain in the bundle.
• packages/web/src/components/editor/HttpEditor.tsx, packages/web/src/components/editor/index.ts, and packages/web/src/components/editor/lazy.ts — all retain references to the now-unused HttpEditor and should be cleaned up.

Important Files Changed

Filename	Overview
packages/web/src/components/editor/EditorWithExecution.tsx	Removes the HttpEditor import and its render block; container div loses flex flex-col in favour of overflow-auto. The HttpEditor usage is successfully removed, but HttpEditor.tsx, its re-export in index.ts, and its lazy wrapper in lazy.ts remain as dead code.

Flowchart

%%{init: {'theme': 'neutral'}}%%
flowchart TD
    A[EditorWithExecution] --> B{fileType?}
    B -- http --> C[RequestSelectorBar]
    C --> D[ResizableSplitPane]
    D --> E["Left Pane\n(div.h-full.overflow-auto)"]
    D --> F[Right Pane — ExecutionDetail]
    E --> G[RequestWorkspaceTabs\nParams / Headers / Body]
    G -.->|previously also rendered below| H["❌ HttpEditor (removed)"]
    B -- script/test --> I[ResizableSplitPane]
    I --> J[CodeEditor]
    I --> K[ScriptPanel]

    style H stroke-dasharray: 5 5,fill:#ffcccc,color:#333

Loading

_{Last reviewed commit: d89de71}

[greptile-apps]

_{1 file reviewed, 1 comment}

_{Edit Code Review Agent Settings | Greptile}

[greptile-apps]

Dead code not fully removed

The PR's stated goal is to remove the legacy HTTP editor, but HttpEditor.tsx itself was never deleted, and two other files still reference it:

• packages/web/src/components/editor/index.ts (line 4) still re-exports HttpEditor:

  export { HttpEditor } from './HttpEditor';

• packages/web/src/components/editor/lazy.ts (line 4) still exports a lazy-loaded version:

  export const LazyHttpEditor = lazy(() => import('./HttpEditor'));

Since HttpEditor is no longer consumed anywhere (confirmed via search — only these two export sites reference it), these exports are now dead code that keeps the bundler from tree-shaking the entire HttpEditor module and its dependencies (codemirror, @codemirror/*). The HttpEditor.tsx source file should also be deleted as part of this refactor.

[chatgpt-codex-connector]

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: d89de71509

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

• Open a pull request for review
• Mark a draft as ready
• Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".