update deps

[syuilo]

What

Why

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の差分

差分はこちら

Get diff files from Workflow Page

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 41.49%. Comparing base (4096dab) to head (aa55b59).
Report is 1 commits behind head on develop.

Additional details and impacted files

@@             Coverage Diff              @@
##           develop   #14057       +/-   ##
============================================
- Coverage    77.80%   41.49%   -36.31%     
============================================
  Files          184     1520     +1336     
  Lines        25438   191931   +166493     
  Branches       487     3527     +3040     
============================================
+ Hits         19791    79634    +59843     
- Misses        5640   111727   +106087     
- Partials         7      570      +563     

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

[syuilo]

めんどくさい :angry_ai: :angry_ai: :angry_ai:
https://eslint.org/docs/latest/use/configure/migration-guide

[syuilo]

tsconfigRootDir: __dirname の部分をなんとかする必要がある

[samunohito]

依存パッケージ側の何が変わって、どうすればいいのかはこれから確認しますが…
とりあえず、vitestと@vitest/coverage-v8のバージョンを0.34.6に戻せばテストは通るようになりました

[samunohito]

0.34.6→1.0.0（と1.0.4）の時点でもうダメな模様

[samunohito]

locales/index.jsの

const locales = languages.reduce((a, c) => (a[c] = yaml.load(clean(fs.readFileSync(new URL(`${c}.yml`, import.meta.url), 'utf-8'))) || {}, a), {});

を

const metaUrl = import.meta.url;
const locales = languages.reduce((a, c) => (a[c] = yaml.load(clean(fs.readFileSync(new URL(`${c}.yml`, metaUrl), 'utf-8'))) || {}, a), {});

こうしたらpassした…謎

[samunohito]

イマイチつかみきれてないが、この事例…？
vitest-dev/vitest#3988
vitest-dev/vitest#3935 (comment)
https://github.com/vitest-dev/vitest/pull/3491/files#diff-1fd54d15c1c99fad6c5e00531245ba4fabae341fa7a597475b7242487575f5f9

[samunohito]

https://vitest.dev/guide/environment

[samunohito]

夜まで触れないので忘れないうちにメモ

#14057 (comment)

↑
vitestを1.0以上にすることで、import.meta.urlの中にhttp:https://localhost:3000という文字列が入るようになる（というか、vitest側のプルリクを見るに、import.met.urlが置換されている）。一度変数におとすと動くのは、この置換処理を免れるからだと思う。
置換されなければ、従来通りファイルシステムから見たlocales/index.jsのパスが使われるので、ymlの読み込みに成功する。

（置換されてしまうと、http:https://localhost:3000/ja-JP.ymlみたいな感じになってymlの取得に失敗し…ということ）

[samunohito]

方針（いずれか）

• ローカル変数に落としてお茶を濁す（手っ取り早いが、パッと見でバグっぽい挙動を利用しているのでどうだろう感ある）
• locales/index.jsのURL解決方法を拡張し、http/fileの両プロトコルでいけるようにする（httpの場合、ymlが置かれてるパスは何処なのか調べる必要がある）
• vm？の上で動くのがまずそうなので、nodeで動くようにする？（出来るか分からない）

[samunohito]

locales/index.jsのURL解決方法を拡張し、http/fileの両プロトコルでいけるようにする（httpの場合、ymlが置かれてるパスは何処なのか調べる必要がある）

localhost:3000でサーバを作ってくれてないっぽい…？ので×

vm？の上で動くのがまずそうなので、nodeで動くようにする？（出来るか分からない）

Environmentを拡張してhappy-domを使うモードでtransformModeをssrにする方式↓にしたけど、別のエラーが出て×

// こんなかんじのファイルを作って
import type { Environment } from 'vitest'

import { builtinEnvironments } from 'vitest/environments'

export default <Environment>{
	...builtinEnvironments['happy-dom'],
	name: 'happy-dom-ssr',
        // 素の状態だとここがwebになってる
	transformMode: 'ssr',
}

// vite.config.tsの170行目あたりをこうした
		test: {
			environment: './vitest.env.happy-dom-ssr.ts',
			deps: {
				optimizer: {
					web: {
						include: [
							// XXX: misskey-dev/browser-image-resizer has no "type": "module"
							'browser-image-resizer',
						],
					},
				},
			},
			includeSource: ['src/**/*.ts'],
		},

[samunohito]

改めてvitestのissueを見直したけど、

ローカル変数に落としてお茶を濁す（手っ取り早いが、パッと見でバグっぽい挙動を利用するのでどう感じるだろう）

↑を提唱してたの、メンテナの人だった。
他に有効な手段を持った人がいない場合、もうこれでも良いかしら…

[samunohito]

ひとまず、import.meta.urlをローカル変数化する形で Test (frontend) / vitest (20.12.2) (pull_request) を通るようにしました。他にいい案がある場合はrevert願います。

ついでに最新のdevelopを取り込み済みです。

[samunohito]

（UI Testsも落ちてるけど、メッセージを見る限りインフラ起因ぽいので一旦見なかったことにしてる）

[samunohito]

こんどはe2eも落ちてるなあ

[zyoshoka]

なんか ip-cidr 側の問題な気がしています

[samunohito]

🤯

[zyoshoka]

とりあえず PR を書きました (https://redirect.github.com/ortexx/ip-cidr/pull/45) が、一先ずはエラー無視するか revert するかの二択だと思います

[zyoshoka]

あと Chromatic はおそらくですが play 内で sleep っぽいことをしているコンポーネント達が軒並み死んでいる感があります（何故かは分かってません）

[zyoshoka]

とりあえず PR を書きました (https://redirect.github.com/ortexx/ip-cidr/pull/45) が、一先ずはエラー無視するか revert するかの二択だと思います

思ってたより早くマージされましたので bump しました

[zyoshoka]

Chromatic の Infrastructure error は #14101 で直ると思います