diff options
| author | okayurisotto <okayurisotto@proton.me> | 2023-07-13 12:48:34 +0900 |
|---|---|---|
| committer | GitHub <noreply@github.com> | 2023-07-13 12:48:34 +0900 |
| commit | e35a370af072bbab967e7a654f133193936f3c76 (patch) | |
| tree | 65058481bc09bb7e28855de1447787d4321fdc0e /packages/backend/src/server/ActivityPubServerService.ts | |
| parent | fix(storybook): solve test failures (#11262) (diff) | |
| download | sharkey-e35a370af072bbab967e7a654f133193936f3c76.tar.gz sharkey-e35a370af072bbab967e7a654f133193936f3c76.tar.bz2 sharkey-e35a370af072bbab967e7a654f133193936f3c76.zip | |
refactor(backend): `core/activitypub` (#11247)
* eslint: `explicit-function-return-type`
* eslint: `no-unnecessary-condition`
* eslint: `eslint-disable-next-line`
* eslint: `no-unused-vars`
* eslint: `comma-dangle`
* eslint: `import/order`
* cleanup: unnecessary non-null assertion
* cleanup: `IActivity`に`actor`は常に存在するようなので
* cleanup: unnecessary `as`
* cleanup: unnecessary `Promise.resolve`
* cleanup
* refactor: `String.prototype.match()`である必要がない部分をよりシンプルな書き方に変更
* refactor: よりよい型定義
* refactor: よりよい型定義
- `LdSignature`の`normalize`メソッドでの使われ方から、
- `data`引数の型定義を`any`から`JsonLdDocument`へ修正
- `getLoader`メソッドの返り値の型定義の一部を`any`から`RemoteDocument`へ修正
- `contextUrl`が不正な値(`null`)となっていたことが判明したため`undefined`へ修正
- `document`の型と合わせるために`CONTEXTS`の型定義の一部を`unknown`から`JsonLd`へ修正
- とりあえず`satisfies`を使用
- `document`の型と合わせるために`fetchDocument`メソッドの返り値の型定義の一部を`unknown`から`JsonLd`へ修正
- どうしようもなく`as`を使用
* refactor: 型ガードを使うことでnon-null assertionをやめた
* refactor: non-null assertionをやめた
`.filter()`で行っている型ガードなどの文脈から、より適しているだろうと思われる書き方に変更した。
* refactor: 型ガードを使うことで`as`をやめた
* refactor: `as`をやめた
* refactor: よりよい型定義
- `id`は`null`とのunionになっていたが、`null`を渡している場面はなかった
- またおそらくこのメソッドは`IOrderedCollection`を返すため、そちらに合わせて`null`とのunionをやめた
- `IOrderedCollection`とはまだ型に相違がある
- `totalItems`をコメントや使われ方を元に`number`へ推論
* refactor: `for-of` -> `Array.prototype.map`
* refactor: `delete`演算子を使わない形に
Diffstat (limited to 'packages/backend/src/server/ActivityPubServerService.ts')
| -rw-r--r-- | packages/backend/src/server/ActivityPubServerService.ts | 21 |
1 files changed, 17 insertions, 4 deletions
diff --git a/packages/backend/src/server/ActivityPubServerService.ts b/packages/backend/src/server/ActivityPubServerService.ts index f53351d999..f751709345 100644 --- a/packages/backend/src/server/ActivityPubServerService.ts +++ b/packages/backend/src/server/ActivityPubServerService.ts @@ -189,7 +189,11 @@ export class ActivityPubServerService { return (this.apRendererService.addContext(rendered)); } else { // index page - const rendered = this.apRendererService.renderOrderedCollection(partOf, user.followersCount, `${partOf}?page=true`); + const rendered = this.apRendererService.renderOrderedCollection( + partOf, + user.followersCount, + `${partOf}?page=true`, + ); reply.header('Cache-Control', 'public, max-age=180'); this.setResponseType(request, reply); return (this.apRendererService.addContext(rendered)); @@ -277,7 +281,11 @@ export class ActivityPubServerService { return (this.apRendererService.addContext(rendered)); } else { // index page - const rendered = this.apRendererService.renderOrderedCollection(partOf, user.followingCount, `${partOf}?page=true`); + const rendered = this.apRendererService.renderOrderedCollection( + partOf, + user.followingCount, + `${partOf}?page=true`, + ); reply.header('Cache-Control', 'public, max-age=180'); this.setResponseType(request, reply); return (this.apRendererService.addContext(rendered)); @@ -310,7 +318,10 @@ export class ActivityPubServerService { const rendered = this.apRendererService.renderOrderedCollection( `${this.config.url}/users/${userId}/collections/featured`, - renderedNotes.length, undefined, undefined, renderedNotes, + renderedNotes.length, + undefined, + undefined, + renderedNotes, ); reply.header('Cache-Control', 'public, max-age=180'); @@ -395,7 +406,9 @@ export class ActivityPubServerService { return (this.apRendererService.addContext(rendered)); } else { // index page - const rendered = this.apRendererService.renderOrderedCollection(partOf, user.notesCount, + const rendered = this.apRendererService.renderOrderedCollection( + partOf, + user.notesCount, `${partOf}?page=true`, `${partOf}?page=true&since_id=000000000000000000000000`, ); |