Fix #3080

[JiuqingSong]

There is a type from my previous check in (#3046), we should check entity instead of selection marker

[Copilot]

Pull Request Overview

This PR corrects the enter-key handling logic by detecting Entity segments instead of SelectionMarker and updates related tests.

• Rename the helper from hasEnterForEntity to handleEnterForEntity and switch its segment check to Entity
• Update keyboardEnter to call the new predicate
• Rename and add tests in keyboardEnterTest.ts to cover entity vs. selection‐marker cases

Reviewed Changes

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

File	Description
packages/roosterjs-content-model-plugins/lib/edit/keyboardEnter.ts	Renamed helper and updated its logic to check for Entity segments
packages/roosterjs-content-model-plugins/test/edit/keyboardEnterTest.ts	Renamed tests and added a new case for only‐selection‐marker handling

Comments suppressed due to low confidence (4)

packages/roosterjs-content-model-plugins/lib/edit/keyboardEnter.ts:67

• [nitpick] Consider renaming this predicate to shouldHandleEnterForEntity to clearly convey that it returns a boolean decision rather than performing an action.

function handleEnterForEntity(paragraph: ReadonlyContentModelParagraph | undefined) {

packages/roosterjs-content-model-plugins/lib/edit/keyboardEnter.ts:67

• Add a brief JSDoc comment explaining the purpose of this helper (why we treat implicit or entity‐containing paragraphs specially) to improve maintainability.

function handleEnterForEntity(paragraph: ReadonlyContentModelParagraph | undefined) {

packages/roosterjs-content-model-plugins/test/edit/keyboardEnterTest.ts:1369

• [nitpick] Test names should follow a consistent should … pattern (e.g. should handle enter when paragraph contains an entity) for readability and consistency.

    it('Handle enter when there is entity', () => {

packages/roosterjs-content-model-plugins/test/edit/keyboardEnterTest.ts:1408

• This test relies on the spy having two total calls (including a previous test). Consider resetting or reinitializing spies at the start of each test to make them isolated and less fragile.

    it('Do not handle enter when there is only selection marker', () => {