#10530 - Refactor: Explicit/implicit Any type clean up from packages\ketcher-core\src\application\render\renderers\sequence\SequenceRenderer.ts file

[Copilot]

How the feature works? / How did you fix the issue?

SequenceRenderer.ts had implicit/unspecified any in method params and local variables, weakening type safety in sequence editing/rendering flows. This change replaces those with explicit domain-safe types and keeps runtime behavior unchanged.

• Typed API surface

  • Added explicit parameter/return types for previously untyped methods (addNewEmptyChainIfNeeded, shiftArrowSelectionInEditMode).

• Typed local state in critical paths

  • Replaced untyped locals with explicit unions where needed (number | undefined, SequenceNode | undefined, SequenceBondRenderer, BaseSequenceItemRenderer | undefined).

• Safer narrowing for renderer resolution

  • Kept getRendererByMonomer strict by narrowing with instanceof BaseSequenceItemRenderer before assignment.

• BBox invariant made explicit

  • Consolidated bbox guards into a single assertion before width/height calculation.

private static addNewEmptyChainIfNeeded(
  chainBeforeNewEmptyChainIndex?: number,
): SequenceViewModelChain | undefined { ... }

public static shiftArrowSelectionInEditMode(event: KeyboardEvent) { ... }

let nodeIndexInRow: number | undefined;
let previousNode: SequenceNode | undefined;
let bondRenderer: SequenceBondRenderer;

Check list

• unit-tests written
• e2e-tests written
• documentation updated
• PR name follows the pattern #1234 – issue name
• branch name doesn't contain '#'
• PR is linked with the issue
• base branch (master or release/xx) is correct
• task status changed to "Code review"
• reviewers are notified about the pull request