Feat: コントロールパネル内のファイル一覧でファイル名と説明で検索できるように

[tetsuya-ki]

What

コントロールパネルのファイル一覧において、ファイル名と説明で検索が可能になります。

Why

Fix #14789

Additional info (optional)

Checklist

• Read the contribution guide
• Test working in a local environment
• (If needed) Add story of storybook
• (If needed) Update CHANGELOG.md
• (If possible) Add tests

[github-actions]

このPRによるapi.jsonの差分

差分はこちら

--- base
+++ head
@@ -5469,6 +5469,18 @@
                     ],
                     "default": null,
                     "description": "The local host is represented with `null`."
+                  },
+                  "name": {
+                    "type": [
+                      "string",
+                      "null"
+                    ]
+                  },
+                  "comment": {
+                    "type": [
+                      "string",
+                      "null"
+                    ]
                   }
                 }
               }

Get diff files from Workflow Page

[codecov]

Codecov Report

Attention: Patch coverage is 5.55556% with 34 lines in your changes missing coverage. Please review.

Project coverage is 41.39%. Comparing base (58419e1) to head (2d8e6af).

Files with missing lines	Patch %	Lines
packages/frontend/src/pages/admin/files.vue	0.00%	24 Missing ⚠️
...kend/src/server/api/endpoints/admin/drive/files.ts	16.66%	10 Missing ⚠️

Additional details and impacted files

@@             Coverage Diff              @@
##           develop   #14790       +/-   ##
============================================
+ Coverage    19.35%   41.39%   +22.03%     
============================================
  Files          722     1559      +837     
  Lines       102432   201515    +99083     
  Branches       992     3655     +2663     
============================================
+ Hits         19825    83409    +63584     
- Misses       82056   117504    +35448     
- Partials       551      602       +51     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[kakkokari-gtyih]

• pnpm run build-misskey-js-with-types をやってください
• フォームが多くなってるので、よく使いそうな検索項目２～３個に絞って、それ以外はMkFolderとかMkFoldableSectionとかに入れたほうが良さそう

[tetsuya-ki]

指摘の2点について対応しました(GitHub Actionでも指摘があったのに漏れており申し訳ないです)。

misskeyサーバーを管理しているわけではないので、正直どれがよく使われているかわからなかったので、
先頭2項目以外をフォルダに格納しています。

[tetsuya-ki]

一つ自分でも気になっていることがあり、今回検索対象にしているdrive_fileの対象カラム(name,comment)ともにインデックスがないようです。
packages/backend/src/models/DriveFile.ts
ファイルの件数が少ないところは問題ないですが、多い場合はindexを追加した方が無難な可能性ありそうです。
ただ、その場合の後でインデックス更新する的なやり方(マイグレーション？)を自分は把握していません。

[syuilo]

インデックスあったとしてもパフォーマンス的に厳しそうだわね

[tetsuya-ki]

現状でもMIME typeの部分はLIKEによる前方一致検索がINDEXありの状態で使われているので、問題ないかもしれません。
巨大なサーバーではこれらの機能を使ってない可能性も高そうですが(照会かユーザーID一致検索あたりを利用している？)。。。

	@Column('varchar', {
		length: 256,
		comment: 'The file name of the DriveFile.',
	})
	public name: string;

	@Index()
	@Column('varchar', {
		length: 128,
		comment: 'The content type (MIME) of the DriveFile.',
	})
	public type: string;
	〜省略〜
	@Column('varchar', {
		length: 512, nullable: true,
		comment: 'The comment of the DriveFile.',
	})
	public comment: string | null;