fix(sheets-drawing): sheet-drawing error depend on renderManagerService

packages/sheets-drawing/src/controllers/render-controllers/sheet-drawing.render-controller.ts → packages/sheets-drawing-ui/src/controllers/render-controllers/sheet-drawing.render-controller.ts

@@ -16,8 +16,8 @@
 
 import { Disposable, Inject, LifecycleService, LifecycleStages, UniverInstanceType } from '@univerjs/core';
 import type { IRenderContext, IRenderModule } from '@univerjs/engine-render';
+import { ISheetDrawingService } from '@univerjs/sheets-drawing';
 import { filter, first } from 'rxjs';
-import { ISheetDrawingService } from '../../services/sheet-drawing.service';
 
 export class SheetsDrawingRenderController extends Disposable implements IRenderModule {
     constructor(

packages/sheets-drawing-ui/src/plugin.ts

@@ -29,6 +29,7 @@ import { SheetCanvasFloatDomManagerService } from './services/canvas-float-dom-m
 import { SheetDrawingPrintingController } from './controllers/sheet-drawing-printing.controller';
 import { SheetDrawingPermissionController } from './controllers/sheet-drawing-permission.controller';
 import { SheetsDrawingCopyPasteController } from './controllers/sheet-drawing-copy-paste.controller';
+import { SheetsDrawingRenderController } from './controllers/render-controllers/sheet-drawing.render-controller';
 
 const PLUGIN_NAME = 'SHEET_IMAGE_UI_PLUGIN';
 
@@ -76,6 +77,7 @@ export class UniverSheetsDrawingUIPlugin extends Plugin {
         ([
             [SheetDrawingUpdateController],
             [SheetDrawingTransformAffectedController],
+            [SheetsDrawingRenderController],
         ] as Dependency[]).forEach((m) => {
             this.disposeWithMe(this._renderManagerService.registerRenderModule(UniverInstanceType.UNIVER_SHEET, m));
         });

packages/sheets-drawing/package.json

@@ -60,13 +60,11 @@
     "peerDependencies": {
         "@univerjs/core": "workspace:*",
         "@univerjs/drawing": "workspace:*",
-        "@univerjs/engine-render": "workspace:*",
         "rxjs": ">=7.0.0"
     },
     "devDependencies": {
         "@univerjs/core": "workspace:*",
         "@univerjs/drawing": "workspace:*",
-        "@univerjs/engine-render": "workspace:*",
         "@univerjs/shared": "workspace:*",
         "@univerjs/sheets": "workspace:*",
         "typescript": "^5.5.4",

packages/sheets-drawing/src/plugin.ts

@@ -16,19 +16,16 @@
 
 import { Inject, Injector, Plugin, UniverInstanceType } from '@univerjs/core';
 import type { Dependency } from '@univerjs/core';
-import { IRenderManagerService } from '@univerjs/engine-render';
 import { SHEET_DRAWING_PLUGIN, SheetsDrawingLoadController } from './controllers/sheet-drawing.controller';
 import { ISheetDrawingService, SheetDrawingService } from './services/sheet-drawing.service';
-import { SheetsDrawingRenderController } from './controllers/render-controllers/sheet-drawing.render-controller';
 
 export class UniverSheetsDrawingPlugin extends Plugin {
     static override pluginName = SHEET_DRAWING_PLUGIN;
     static override type = UniverInstanceType.UNIVER_SHEET;
 
     constructor(
         private readonly _config: unknown,
-        @Inject(Injector) override readonly _injector: Injector,
-        @IRenderManagerService private readonly _renderManagerService: IRenderManagerService
+        @Inject(Injector) override readonly _injector: Injector
     ) {
         super();
     }
@@ -39,10 +36,4 @@ export class UniverSheetsDrawingPlugin extends Plugin {
             [ISheetDrawingService, { useClass: SheetDrawingService }],
         ] as Dependency[]).forEach((dependency) => this._injector.add(dependency));
     }
-
-    override onRendered(): void {
-        ([[SheetsDrawingRenderController]] as Dependency[]).forEach((m) => {
-            this.disposeWithMe(this._renderManagerService.registerRenderModule(UniverInstanceType.UNIVER_SHEET, m));
-        });
-    }
 }