b6790a4320
2 Commits
| Author | SHA1 | Message | Date | |
|---|---|---|---|---|
okayurisotto
|
4f876c9e8d
|
refactor(backend): core/activitypub/models (#11067)
* cleanup(`ApImageService.ts`)
* refactor(`ApImageService.ts`)
* cleanup(`check-https.ts`)
* cleanup(`ApMentionService.ts`)
* refactor(`ApMentionService.ts`)
* cleanup(`ApNoteService.ts`): unneeded `eslint-disable-next-line`
* cleanup(`ApNoteService.ts`)
* WIP(`ApImageService.ts`): `image.url`を`getApHrefNullable()`に通すかどうか悩んでいる
* refactor(`ApNoteService.ts`): function return type
* cleanup(`ApNoteService.ts`): deadcode
* refactor(`ApNoteService.ts`): `eslint-disable-next-line`
* refactor(`ApNoteService.ts`): non-null assertion
これまでは`getApId()`の方でエラーがスローされていた。
* cleanup(`ApNoteService.ts`): unneeded await
* refactor(`ApNoteService.ts`): note.attachment
- `toArray()`を使うように
- よくわからない条件式を整理
- `as`をなくすために`promiseLimit()`でジェネリクスを使うように
* cleanup(`ApNoteService.ts`)
* refactor(`ApNoteService.ts`): よりよい型定義
`res`が`null`でないことは確認されているようだったので`null`とのunionはなくした
* refactor(`ApNoteService.ts`): 不要な条件を削除
* cleanup(`ApNoteService.ts`)
* cleanup(`ApNoteService.ts`): 重要でない`as`を削除
* refactor(`ApNoteService.ts`): `eslint-disable-next-line`
* cleanup(`ApNoteService.ts`): deadcode
* cleanup(`ApNoteService.ts`): unneeded non-null assertion
* refactor(`ApNoteService.ts`): 不要な条件を削除
* WIP(`ApNoteService.ts`): `as`をなくす
エラーメッセージを考える
* cleanup(`ApNoteService.ts`): 不要な`as`を削除
* cleanup(`ApPersonService.ts`): `no-unused-vars`
* cleanup(`ApPersonService.ts`): deadcode
* refactor(`ApPersonService.ts`): function return type
* cleanup(`ApPersonService.ts`): deadcode
* cleanup(`ApPersonService.ts`): deadcode
* WIP(`ApPersonService.ts`): `as`を調整
`null`でないか確認する処理が続いていたので型アサーションは`null`とのunionにした。
より本質的な改善の余地があるように感じるのでひとまずWIPとしてコミット。
* refactor(`ApPersonService.ts`): `eslint-disable-next-line`
* WIP(`ApPersonService.ts`): `as any`をなくした
エラーをスローするようにせざるを得なかったのでエラーメッセージを考える必要がある。
* WIP(`ApNoteService.ts`): non-null assertion
non-nullアサーションを減らすために事前に存在確認をするようにした。
エラーをスローするようにしたのでメッセージを考えなければならない。
* refactor(`ApNoteService.ts`): non-null assertion -> optional chaining
* refactor(`ApPersonService.ts`): `eslint-disable-next-line`
* refactor(`ApPersonService.ts`): `eslint-disable-next-line`
* refactor(`ApPersonService.ts`): function return type
* refactor(`ApPersonService.ts`): type guardによるnon-null assertionの削除
* WIP(`ApPersonService.ts`): `analyzeAttachments`
- Field型を事前に定義しておくように
- `attachments`が`IObject`だった場合、返り値が`{ fields: [] }`になるようだが構わないのか？
- `toArray()`を通すべきでは？
* Revert "WIP(`ApImageService.ts`): `image.url`を`getApHrefNullable()`に通すかどうか悩んでいる"
This reverts commit aeefb843a8a688f8a356794e8981c58f8a2733af.
* cleanup(`ApImageService.ts`): `import`
* refactor(`ApImageService.ts`): 冗長だった部分を短く
* cleanup(`ApMentionService.ts`): `import`
* refactor(`ApImageService.ts`): `JSON.stringify()`でのindentationを追加
* cleanup(`ApNoteService.ts`): `import`
* cleanup(`ApNoteService.ts`)
* cleanup(`ApNoteService.ts`)
* cleanup(`ApNoteService.ts`)
* cleanup(`ApNoteService.ts`): `any`に対するnon-null assertion
* refactor(`ApNoteService.ts`): 添付ファイル
* cleanup(`ApPersonService.ts`): `import`
* refactor(`ApPersonService.ts`): より実情に即した`as`に
* cleanup(`ApPersonService.ts`)
* refactor(`ApPersonService.ts`): 冗長だった部分を修正
* cleanup(`ApPersonService.ts`): deadcode
* cleanup(`ApPersonService.ts`)
* cleanup(`ApQuestionService.ts`): `import`
* refactor(`ApQuestionService.ts`): `eslint-disable-next-line`
* refactor(`ApQuestionService.ts`): `eslint-disable-next-line`
* cleanup(`ApQuestionService.ts`)
* refactor(`ApQuestionService.ts`): non-null assertionを消した
* cleanup(`ApQuestionService.ts`)
* WIP(`ApQuestionService.ts`): non-null assertionを消す
エラーメッセージを考える必要がある。
* refactor(`ApQuestionService.ts`): `any`を消す
* refactor(`ApQuestionService.ts`): function return type
* WIP(`ApPersonService.ts`): 可読性の低い三項演算子を削除しつつnon-null assertionを回避
エラーメッセージを考える必要がある。
* cleanup(`ApPersonService.ts`): 不必要な三項演算子を削除
* cleanup(`ApPersonService.ts`): 不要な`as`
* cleanup(`ApPersonService.ts`)
* refactor(`ApPersonService.ts`)
* refactor(`ApPersonService.ts`): 可読性の低い三項演算子を削除
元の実装が悪いと判断し`null`かどうかの確認をより厳密に行うようにした。
* cleanup(`ApPersonService.ts`)
* cleanup(`ApPersonService.ts`)
* refactor(`ApPersonService.ts`): 返り値を`void`に統一
この返り値を参照しているコードは見当たらなかった。
また、普通に意味がない値であるように見受けられた。
* fixup! refactor(`ApPersonService.ts`): 返り値を`void`に統一
* refactor(`ApNoteService.ts`)
* refactor(`ApPersonService.ts`)
* cleanup(`ApPersonService.ts`)
* cleanup(`ApPersonService.ts`)
* refactor(`ApPersonService.ts`): 返り値の`void`統一と条件式の調整
この返り値を参照しているコードは見当たらなかった。
また、普通に意味がない値であるように見受けられた。
* cleanup(`ApQuestionService.ts`)
* refactor(`ApQuestionService.ts`)
* refactor(`ApQuestionService.ts`)
* refactor(`tag.ts`): function return type
* fixup! enhance: account migration (#10592)
* fixup! WIP(`ApPersonService.ts`): 可読性の低い三項演算子を削除しつつnon-null assertionを回避
* fixup! cleanup(`ApPersonService.ts`): 不要な`as`
* refactor: エラーメッセージを見繕った
* Revert "cleanup(`ApImageService.ts`): `import`"
This reverts commit 1454d04c377eaf46013b0f3c3ce664a4034fd53a.
* Revert "cleanup(`ApMentionService.ts`): `import`"
This reverts commit 244f6720c134a3434e33c1caf6e3e0c2c87b58f5.
* Revert "cleanup(`ApNoteService.ts`): `import`"
This reverts commit d8f0d769733c4cb0629821b04e557a0ae6f5ff5b.
* Revert "cleanup(`ApPersonService.ts`): `import`"
This reverts commit 5190ef954caf376da46c707f52e02208d53caafd.
# Conflicts:
# packages/backend/src/core/activitypub/models/ApPersonService.ts
* Revert "cleanup(`ApQuestionService.ts`): `import`"
This reverts commit 778585e2882477fec5f11fabf398b4b89cf26da2.
* processRemoteMoveはそのままにしてほしい
* Revert "fixup! refactor(`ApPersonService.ts`): 返り値を`void`に統一"
This reverts commit 083cd678abcd64325b9628895366c03b893e42ca.
* Revert "refactor(`ApPersonService.ts`): 返り値を`void`に統一"
This reverts commit bfa0fcd6f01a6e519ea0c68017358f9980d2ed96.
---------
Co-authored-by: tamaina <tamaina@hotmail.co.jp>
|
||
tamaina
|
09764b909b
|
enhance(dev): non-production環境でhttpサーバー間でもユーザー、ノートの連合が可能なように (#10717)
* enhance(dev): non-production環境でhttpサーバー間でもユーザー、ノートの連合が可能なように * refactor (use checkHttps) * MISSKEY_WEBFINGER_USE_HTTP * Environment Variable readme * NEVER USE IN PRODUCTION * fix punyHost |