enhance(backend): bundle backend using Rolldown#17068
enhance(backend): bundle backend using Rolldown#17068syuilo merged 36 commits intomisskey-dev:developfrom
Conversation
|
このPRによるapi.jsonの差分 |
|
コード54.4MBになった（が、起動には成功してない） |
|
謎 |
|
通常ならその直後に |
|
どっちかというと EndpointsModule というよりは EndpointsModule に依存している ServerModule かも |
|
関係あるかわからないけど… クラス名は維持しておかないとDI周りが死ぬので、バンドルの過程でクラス名などが変わっているのであれば維持するようにしてみるといいかも…？ |
いやさらに ServerModule に依存している CoreModule かも |
|
いかのどれかに問題があることは絞り込めた |
|
|
NoteEntityService に問題があることが判った |
|
NoteEntityServiceは循環参照があり、読み込み順的には |
Codecov Report❌ Patch coverage is Additional details and impacted files@@ Coverage Diff @@
## develop #17068 +/- ##
===========================================
- Coverage 63.65% 62.42% -1.23%
===========================================
Files 1161 1162 +1
Lines 116313 116557 +244
Branches 8407 9080 +673
===========================================
- Hits 74042 72764 -1278
- Misses 40063 41601 +1538
+ Partials 2208 2192 -16 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
いや嘘かも |
|
InstanceEntityServiceだけ読み込まれてないかも |
|
謎ということが判った |
|
とはいえ循環参照周りが原因な気はする |
|
バンドルでesmoduleそのままだと発生しない循環参照問題が発生するなら minify だけにしたい気がしてます。今後devなら問題ないのにbuildすると謎フリーズが今後増えるのはあまりうれしくないと思います |
|
ApAudienceService も正常に読み込まれていないserviceのひとつだけど、以下のように極限にシンプルな実装に書き換えると読み込まれるようになる before import { Injectable } from '@nestjs/common';
import promiseLimit from 'promise-limit';
import type { MiRemoteUser, MiUser } from '@/models/User.js';
import { concat, unique } from '@/misc/prelude/array.js';
import { bindThis } from '@/decorators.js';
import { getApIds } from './type.js';
import { ApPersonService } from './models/ApPersonService.js';
import type { ApObject } from './type.js';
import type { Resolver } from './ApResolverService.js';
type Visibility = 'public' | 'home' | 'followers' | 'specified';
type AudienceInfo = {
visibility: Visibility,
mentionedUsers: MiUser[],
visibleUsers: MiUser[],
};
type GroupedAudience = Record<'public' | 'followers' | 'other', string[]>;
@Injectable()
export class ApAudienceService {
constructor(
private apPersonService: ApPersonService,
) {
}
@bindThis
public async parseAudience(actor: MiRemoteUser, to?: ApObject, cc?: ApObject, resolver?: Resolver): Promise<AudienceInfo> {
const toGroups = this.groupingAudience(getApIds(to), actor);
const ccGroups = this.groupingAudience(getApIds(cc), actor);
const others = unique(concat([toGroups.other, ccGroups.other]));
const limit = promiseLimit<MiUser | null>(2);
const mentionedUsers = (await Promise.all(
others.map(id => limit(() => this.apPersonService.resolvePerson(id, resolver).catch(() => null))),
)).filter(x => x != null);
if (toGroups.public.length > 0) {
return {
visibility: 'public',
mentionedUsers,
visibleUsers: [],
};
}
if (ccGroups.public.length > 0) {
return {
visibility: 'home',
mentionedUsers,
visibleUsers: [],
};
}
if (toGroups.followers.length > 0 || ccGroups.followers.length > 0) {
return {
visibility: 'followers',
mentionedUsers,
visibleUsers: [],
};
}
return {
visibility: 'specified',
mentionedUsers,
visibleUsers: mentionedUsers,
};
}
@bindThis
private groupingAudience(ids: string[], actor: MiRemoteUser): GroupedAudience {
const groups: GroupedAudience = {
public: [],
followers: [],
other: [],
};
for (const id of ids) {
if (this.isPublic(id)) {
groups.public.push(id);
} else if (this.isFollowers(id, actor)) {
groups.followers.push(id);
} else {
groups.other.push(id);
}
}
groups.other = unique(groups.other);
return groups;
}
@bindThis
private isPublic(id: string): boolean {
return [
'https://www.w3.org/ns/activitystreams#Public',
'as:Public',
'Public',
].includes(id);
}
@bindThis
private isFollowers(id: string, actor: MiRemoteUser): boolean {
return id === (actor.followersUri ?? `${actor.uri}/followers`);
}
}after import { Injectable } from '@nestjs/common';
import type { MiRemoteUser } from '@/models/User.js';
import { bindThis } from '@/decorators.js';
import type { ApObject } from './type.js';
import type { Resolver } from './ApResolverService.js';
@Injectable()
export class ApAudienceService {
constructor(
) {
console.log('ApAudienceService initialized');
}
@bindThis
public async parseAudience(actor: MiRemoteUser, to?: ApObject, cc?: ApObject, resolver?: Resolver) {
return {
visibility: 'specified',
};
}
} |
コードに出てないわけでなさそう（ビルド成果物を検索したらヒットした） |
そもそもnestjsはバンドルされた環境で動かすことは想定されてるのかしら |
Webpackを使用する場合の設定方法が書いてあったからされてそう 
|
|
backendの開発サーバーをrolldown制御に変更した |
dosubot
Bot
added
the
size:XL
kakkokari-gtyih
changed the title
|
👍🏻👍🏻 |
|
pnpm devでエラー出るかも |
|
Nodeがv24で起動してそうだけどv22にしても発生するかしら |
|
ちなみにこちらの環境だと正常に起動している |
|
Windows? |
|
windows native環境でだけ発生する問題化も？(wsl linux macosじゃ発生しないかも |
|
rolldown.config.ts の |
|
変わらなかった |
|
なにがだめかわかったかも |
* enhance(backend): bundle backend using rolldown * fix * fix [ci skip] * remove unused build script * fix * enhance: 起動からlistenまでかかる時間を減らす (MisskeyIO#1410) * ✌️ * fix * update rolldown * fix(backend): extract static error classes to avoid rolldown design:paramtypes omission * update rolldown * Revert "fix(backend): extract static error classes to avoid rolldown design:paramtypes omission" This reverts commit e2243c9. * fix * perf: avoid generating sourcemap in production * fix * fix * fix * fix paths * fix * fix * fix * fix * fix * enhance: バックエンドの開発サーバー制御をrolldown側で行うように * remove nodemon * Update Changelog * tweak config * fix * fix * fix * clean up --------- Co-authored-by: あわわわとーにゅ <[email protected]> Co-authored-by: bab <[email protected]>
6 participants
What
Why
Fix #16852
Additional info (optional)
Checklist