Aiday Marlen Kyzy

This commit **reverts** a previous change that consolidated line break options into a computed editor options object, effectively **restoring the original mechanism** for handling these settings. It **reinstates direct parameter passing** for line break options to functions like `createLineBreaksComputer` and `setWrappingSettings`, rather than extracting them from a larger options object. This **maintenance** task primarily impacts the **editor's line breaking logic** across `domLineBreaksComputer`, `monospaceLineBreaksComputer`, and the **view model's line management** within `viewModelImpl` and `viewModelLines`. The revert ensures that the `ILineBreaksComputerFactory` interface and its implementations revert to their prior, more explicit handling of line break configurations.

This commit introduces a significant **refactoring** to the **editor's line breaking and view model logic**, centralizing the management of line break options. Instead of passing individual parameters, functions like `createLineBreaksComputer` and `ViewModelLinesFromProjectedModel` now consistently accept an `IComputedEditorOptions` object. This change improves the **maintainability and extensibility** of the editor's rendering pipeline by streamlining how wrapping and line breaking settings are propagated. The update affects core **editor components** responsible for text layout and includes corresponding adjustments to relevant test suites.

This commit **refactors** the **editor's view model** and **line break computation logic** to consistently use `IComputedEditorOptions` instead of direct option values. Specifically, functions like `createLineBreaksComputer` and classes such as `ViewModelLinesFromProjectedModel` now accept this computed options object, centralizing configuration access. This **maintenance** effort improves the robustness and consistency of how editor settings are managed across the `src/vs/editor` module. It streamlines how options are passed to components responsible for determining line breaks and text wrapping, ensuring a unified approach to editor configuration.

This commit introduces a significant **refactoring** of the **editor's line break computation mechanism**, transitioning from direct content passing to a context-based approach. A new `ILineBreaksComputerContext` is now utilized by `ILineBreaksComputer` implementations to retrieve line content and injected text using *post-edit line numbers*, centralizing data access. This **maintenance** change impacts the **editor core**, including the **view model**, **text model**, and various **line break computer factories** (e.g., `MonospaceLineBreaksComputer`, `domLineBreaksComputer`), as well as the **diff editor**. The refactoring simplifies event handling for `ModelRawLineChanged` and `ModelRawLinesInserted` and enhances the robustness and flexibility of line break calculations, particularly for scenarios involving **injected text**.

This commit performs a **maintenance refactoring** to **optimize custom line height management** within the **editor's view layout**. It introduces a `stagedIdMap` for more efficient tracking and removal of custom lines, primarily impacting the `_doRemoveCustomLineHeight` function in `src/vs/editor/common/viewLayout/lineHeights.ts`. This change enhances the performance of operations involving custom line heights, ensuring smoother rendering and interaction within the editor. Additionally, the `ArrayMap` utility gains a new `clear` method as part of this improvement.

This commit **refactors the native edit context update mechanism** within the editor's browser controller. Previously, the edit context was updated reactively to specific line insertion, deletion, or change events, which could lead to redundant processing. Now, the update logic in `nativeEditContext.ts` is triggered by a more general content change, and critically, it includes a check to only update the context when the text content has genuinely changed. This **optimization and correctness fix** ensures the **editor's native edit context** is updated more efficiently and accurately, preventing unnecessary state changes and improving overall responsiveness.

This commit **fixes** an issue where whitespace was not correctly rendered in the **editor overlay** for lines utilizing variable fonts. Previously, a condition within the `viewLine.ts` component prevented whitespace from being displayed on such lines. By removing the `hasVariableFonts` check, this change ensures that the **editor rendering** consistently displays whitespace characters, regardless of the font properties of the line. This improves the visual accuracy and debugging capabilities within the **editor's view lines** for users working with diverse font styles.

This commit introduces a **significant refactoring** to the **editor's line height management**, ensuring custom line heights are applied *after* all text edits have been fully processed. This **new capability** in `viewModelImpl.ts` defers the application of custom line heights, collecting ranges during edit processing and applying them once projections are stable. The change simplifies the `onLinesInserted` logic in `lineHeights.ts` and `linesLayout.ts` by decoupling line height calculations from the initial insertion, ultimately improving the **editor's visual consistency** and robustness when handling complex text modifications.

This is an isolated internal refactor within the editor's `viewModel` and `inlineDecorations` components. It improves the internal architecture of how inline decorations are computed without affecting external APIs or user-facing functionality.

This commit **reverts** the recent introduction of **inline completions** within the **chat input** component of the workbench. It **undoes** the integration of the `InlineCompletionsController` and associated logic in `src/vs/workbench/contrib/chat/browser/widget/input/chatInputPart.ts`, effectively removing the capability for inline suggestions. This **rollback** is classified as `waste`, indicating that the previously added feature was deemed undesirable or problematic and is being removed to restore the chat input's prior behavior.

This commit introduces **inline completions** directly into the **Chat Input** field, enhancing the user experience for the **Workbench Chat** feature. It integrates the `InlineCompletionsController` into the editor contributions for `chatInputPart.ts`, enabling this **new capability**. By providing real-time suggestions as users type, this change streamlines the chat interaction. This is a significant **grow feature** that improves the overall usability and efficiency of the chat interface.

This commit performs a **maintenance update** to the project's **distribution configuration** by modifying the `package.json` file. Classified as a **chore**, this change specifically updates the "distro" information, which typically governs build targets, packaging, or deployment settings. This ensures the project's **build and release processes** remain current and compatible with target environments. The update helps maintain the integrity of the **project's build system** and prevents potential issues during package generation or deployment.

This commit introduces a **new unit test** for the **editor's view layout** component, specifically targeting the `onLinesInserted` method. The test, located in `src/vs/editor/test/common/viewLayout/lineHeights.test.ts`, verifies the correct handling of custom line height data when lines are inserted. This **maintenance** work improves **test coverage** for the editor's line height calculations, ensuring the robustness and accuracy of rendering after content modifications.

This commit introduces a robust mechanism to propagate and manage **custom line height information** during line insertions within the editor's text model and view layout. It enhances the **editor's core model events**, **view layout management**, and **view model integration** by adding a `lineHeightsAdded` array to relevant event structures and method calls, such as `onLinesInserted`. This **feature enhancement** refactors `ICustomLineHeightData` into a dedicated `CustomLineHeightData` class and updates various APIs to ensure accurate tracking of line height changes. The work is critical for maintaining **rendering accuracy** and correct scroll behavior in the editor, especially for features that dynamically adjust line heights, by ensuring the view layout correctly accounts for these custom dimensions when new lines are inserted.

This commit performs **UI maintenance** by updating the icons used for controlling **editor hover verbosity**. Specifically, the `increaseHoverVerbosityIcon` and `decreaseHoverVerbosityIcon` in `src/vs/editor/contrib/hover/browser/markdownHoverParticipant.ts` have been changed to use the more semantically appropriate `Codicon.addSmall` and `Codicon.removeSmall` respectively. This change refines the visual presentation of these controls, providing a clearer and more consistent user experience within the **editor's hover functionality**. Users will observe a subtle but improved visual cue when interacting with hover verbosity options.

This commit provides a **bug fix** to the **editor hover** functionality, addressing an issue where the hover could appear simultaneously with a text selection. It introduces a `PADDING` constant within `src/vs/editor/contrib/hover/browser/hoverUtils.ts` and applies it to the `isMousePositionWithinElement` function, creating a small buffer zone around elements. This modification prevents the hover from being triggered when a selection is actively being made, thereby resolving a visual conflict and enhancing the user experience. Accompanying tests in `src/vs/editor/contrib/hover/test/browser/hoverUtils.test.ts` have been updated to validate this new padding logic.

This commit introduces a **performance optimization** for the **Sticky Scroll feature** within the editor. It specifically **prevents unnecessary re-rendering** of the sticky scroll UI when only the cursor's column position changes, rather than its line number. By ensuring updates only occur on significant line number changes, this **fix** reduces wasted computation, thereby improving the **editor's responsiveness** and overall efficiency.

This commit performs a **maintenance refactoring** within the **editor's Sticky Scroll feature**, specifically in the `stickyScrollWidget.ts` file. It changes the assignment of the folding icon from a local variable to a class member `this.foldingIcon` within the `_renderLineNumberAndFoldingIcon` method of the `RenderedStickyLine` class. This internal code improvement enhances consistency in managing UI elements for sticky lines, with no direct user-facing impact.

This commit introduces a **fix** for a specific boundary condition within the **editor's annotation model**. The `getAnnotationsIntersecting` method in `src/vs/editor/common/model/tokens/annotations.ts` is now updated to correctly include the subsequent annotation when its start boundary precisely matches the query's end boundary. This ensures more accurate and robust **annotation retrieval**, preventing potential omissions in features that rely on these ranges. A new test case in `annotations.test.ts` has been added to validate this improved behavior.

Affects the core editor's annotation system; this fix ensures that deleting annotations functions correctly even when they intersect, improving the reliability of editor features that depend on annotation management.