fix: fix type error

packages/docs/src/commands/commands/delete.command.ts

@@ -55,7 +55,7 @@ export const DeleteLeftCommand: ICommand = {
  const docSkeletonManagerService = getCommandSkeleton(accessor, unitId);
  const activeRange = textSelectionManagerService.getActiveRange();
  const ranges = textSelectionManagerService.getSelections();
- const skeleton = docSkeletonManagerService?.getSkeleton()?.skeleton;
+ const skeleton = docSkeletonManagerService?.getSkeleton();
  if (activeRange == null || skeleton == null || ranges == null) {
  return false;
  }
@@ -202,7 +202,7 @@ export const DeleteRightCommand: ICommand = {
 
  const activeRange = textSelectionManagerService.getActiveRange();
  const ranges = textSelectionManagerService.getSelections();
- const skeleton = docSkeletonManagerService?.getSkeleton()?.skeleton;
+ const skeleton = docSkeletonManagerService?.getSkeleton();
  if (activeRange == null || skeleton == null || ranges == null) {
  return false;
  }

packages/docs/src/controllers/move-cursor.controller.ts

@@ -99,7 +99,7 @@ export class MoveCursorController extends Disposable {
  }
 
  const skeleton = this._renderManagerService.getRenderById(docDataModel.getUnitId())
- ?.with(DocSkeletonManagerService).getSkeleton()?.skeleton;
+ ?.with(DocSkeletonManagerService).getSkeleton();
  const docObject = this._getDocObject();
 
  if (activeRange == null || skeleton == null || docObject == null) {
@@ -203,7 +203,7 @@ export class MoveCursorController extends Disposable {
  }
 
  const skeleton = this._renderManagerService.getRenderById(docDataModel.getUnitId())
- ?.with(DocSkeletonManagerService).getSkeleton()?.skeleton;
+ ?.with(DocSkeletonManagerService).getSkeleton();
  const docObject = this._getDocObject();
 
  if (activeRange == null || skeleton == null || docObject == null || allRanges == null) {

packages/docs/src/controllers/normal-input.controller.ts

@@ -65,7 +65,7 @@ export class NormalInputController extends Disposable {
  const e = event as InputEvent;
 
  const skeleton = this._renderManagerService.getRenderById(documentModel.getUnitId())
- ?.with(DocSkeletonManagerService).getSkeleton()?.skeleton;
+ ?.with(DocSkeletonManagerService).getSkeleton();
 
  if (e.data == null || skeleton == null) {
  return;

packages/engine-render/src/render-manager/render-manager.service.ts

@@ -50,7 +50,7 @@ export interface IRenderManagerService extends IDisposable {
  /** @deprecated */
  createRender$: Observable<string>;
  /** @deprecated this design is very very weird! Remove it. */
- create(unitId: Nullable<string>): void;
+ create(unitId: string): void;
 
  /** @deprecated There will be multi units to render at the same time, so there is no *current*. */
  getCurrent(): Nullable<IRender>;
@@ -127,7 +127,7 @@ export class RenderManagerService extends Disposable implements IRenderManagerSe
  return Array.from(this._renderControllers.get(type) ?? []);
  }
 
- create(unitId: Nullable<string>) {
+ create(unitId: string) {
  this._createRender$.next(unitId);
  }
 

packages/sheets-ui/src/controllers/editor/__tests__/end-edit.controller.spec.ts

@@ -20,7 +20,7 @@ import { LexerTreeBuilder } from '@univerjs/engine-formula';
 import { SpreadsheetSkeleton } from '@univerjs/engine-render';
 import type { Injector } from '@wendellhu/redi';
 import { afterEach, beforeEach, describe, expect, it } from 'vitest';
-import { getCellDataByInput } from '../end-edit.controller';
+import { getCellDataByInput } from '../editing.render-controller';
 import { createTestBed } from './create-test-bed';
 
 const richTextDemo: IDocumentData = {

packages/sheets-zen-editor/src/controllers/zen-editor.controller.ts

@@ -126,7 +126,7 @@ export class ZenEditorController extends RxDisposable {
 
  const { engine } = editorObject;
 
- const skeleton = this._docSkeletonManagerService.getSkeletonByUnitId(DOCS_ZEN_EDITOR_UNIT_ID_KEY)?.skeleton;
+ const skeleton = this._renderManagerService.getRenderById(DOCS_ZEN_EDITOR_UNIT_ID_KEY)?.with(DocSkeletonManagerService).getSkeleton();
 
  // Update page size when container resized.
  // zenEditorDataModel.updateDocumentDataPageSize(width);
@@ -220,7 +220,7 @@ export class ZenEditorController extends RxDisposable {
  DOCS_FORMULA_BAR_EDITOR_UNIT_ID_KEY,
  ];
 
- const skeleton = this._renderManagerService.getRenderById(unitId)?.with(DocSkeletonManagerService);
+ const skeleton = this._renderManagerService.getRenderById(unitId)?.with(DocSkeletonManagerService).getSkeleton();
  const docDataModel = this._univerInstanceService.getUniverDocInstance(unitId);
  const docViewModel = this._docViewModelManagerService.getViewModel(unitId);
 
@@ -245,7 +245,6 @@ export class ZenEditorController extends RxDisposable {
  docViewModel.reset(docDataModel);
 
  const currentRender = this._getDocObject();
-
  if (currentRender == null) {
  return;
  }