Math:gen_rngを削除

[syuilo]

AiScript実装から当該実装は削除し、必要に応じてホスト側で用意するようにする

• 仕様と実装が比較的複雑で、かつ外部ライブラリにも依存しているため、light-weightな言語という特徴には相容れず、言語機能として持っておくには負担が大きい
• 乱数生成には様々なニーズ(軽量で良いから精度は低くて良いケース、負荷が高くても良いから一様性を担保したいケースなど)があるため、それらをカバーするのはコストがかかる

[FineArchs]

Math:rngがデカくなったのは #731 からですね
私はこのPRを通した側ですが、正直当時aiscriptのアイデンティティが「バンドルサイズが軽いこと」だということについてよく考えてなかったですね
今は反対だった人の気持ちも分かるかも

[FineArchs]

（それとして当時の人は反対意見があるならちゃんと言葉にしてくれません？通した後にリアクションだけつけられても困ります）

[FineArchs]

#671
これを解決すれば、やることは今のMath:gen_rngを別リポジトリに移動して元々のMath:gen_rngを差し戻すだけで済みますね　どうします？

[takejohn]

#731 ではMath:rndに加えてMath:gen_rngも実装が巨大化していますね。
このissueのタイトルは「Math:gen_rngを削除」ですが、この2関数をそれぞれ削除するか、実装を変えるかなどの議論が必要そう

[syuilo]

意図としてはシードを受け取って乱数を生成できるような・生成アルゴリズムを指定して生成できるような機能は標準では不要ではないかという感じですね

引数なしで単に乱数を返す関数(=JSのMath.random()と同等)はあって良いと思います

[syuilo]

ん、Math:rng ってどれのことかしら(リファレンスを見ています)
Math:rnd？

[takejohn]

あ、自分のはtypoです……

[syuilo]

rndについては別にしました:
#1021

[FineArchs]

ん、Math:rng ってどれのことかしら(リファレンスを見ています) Math:rnd？

自分のもtypoです…（gen_rngのつもりでした）

[FineArchs]

それで、Math:gen_rngは元々

aiscript/src/interpreter/lib/std.ts

Lines 459 to 471 in ac1510e

	'Math:gen_rng': FN_NATIVE(([seed]) => {
	expectAny(seed);
	if (seed.type !== 'num' && seed.type !== 'str') return NULL;
	const rng = seedrandom(seed.value.toString());
	return FN_NATIVE(([min, max]) => {
	if (min && min.type === 'num' && max && max.type === 'num') {
	return NUM(Math.floor(rng() * (Math.floor(max.value) - Math.ceil(min.value) + 1) + Math.ceil(min.value)));
	}
	return NUM(rng());
	});
	}),

のような小さい実装でしたが、戻します？それとも消しちゃいます？
個人的には戻すでいいと思います

[FineArchs]

あーでもseedrandomに依存している分のバンドルサイズの肥大化がありますね
消しちゃいますか

[syuilo]

バンドルサイズもそうですが、ランタイムで別のパッケージに依存しているという事実自体が若干マイナス(昨今はサプライチェーン攻撃とかも多いですし、そうでなくてもセキュリティリスクは増えますし)なので、依存は少ないに越したことはないという感じです

[syuilo]

(無理してまで目指す必要もないですが)できればzero-dependencyを目指したい