[refactor?] 連合OFFなノートが作成された後に一部の連合向け機能が無駄打ちされている

[samunohito]

Summary

※後続処理で防がれているため連合そのものは発生しません

misskey/packages/backend/src/core/NoteCreateService.ts

Lines 680 to 685 in aa48a0e

	if (this.userEntityService.isLocalUser(user)) {
	(async () => {
	const noteActivity = await this.renderNoteOrRenoteActivity(data, note);
	const dm = this.apDeliverManagerService.createDeliverManager(user, noteActivity);
	// メンションされたリモートユーザーに配送

上記if文に対し、!data.localOnlyの条件を追加して後続処理の無駄打ちを防ぎます。

this.renderNoteOrRenoteActivity()にlocalOnlyなノートを渡すとnullが返ってくるのですが、このnullは最終的に↓に流れ着いて何も起こらずに処理が終わります（故に配送されない）

misskey/packages/backend/src/core/QueueService.ts

Line 144 in 7b9c884

if (content == null) return null;

Purpose

• 連合OFFノート作成時のパフォーマンスがちょっとだけ良くなります
  （ローカルでベンチマーク的なことをしてCPU使用率に数％ほど差があるのを確認しました）
• 無駄なasync/await、setImmediateの発行を抑止
• 無駄なDBアクセスを抑止

Do you want to implement this feature yourself?

• Yes, I will implement this by myself and send a pull request