corysanin/morning-report
1
0
Fork 0
Code Issues Pull Requests Actions Packages Releases 3 Activity

replace Math.random() with crypto.randomInt()

Browse Source
This commit is contained in:
Cory Sanin
2025-10-15 00:24:59 -05:00
parent 6ecc03a670
commit 0d53a6ccde
2 changed files with 3 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files

3
src/sequencer.ts
View File

@@ -2,6 +2,7 @@ import { OpenWeatherAPI, type DailyWeather } from 'openweather-api-node';
import { voiceLines } from './voice.js';
import type { Config } from './index.js';
import type { Voice } from './voice.js';
import crypto from 'crypto';
type SegmentName = string;
type SequenceName = string;
@@ -16,7 +17,7 @@ type Sequences = { [sequence: SequenceName]: Sequence };
let config: Config = null;
function selectOne<T>(arr: T[]): T {
return arr[Math.floor(Math.random() * arr.length)];
return arr[crypto.randomInt(0, arr.length)];
}
function resolveSide(side: string, currentWeather: DailyWeather) {

3
test/sequencer.test.ts
View File

@@ -1,7 +1,6 @@
import { describe, expect, it, vi } from 'vitest';
import { type CurrentWeather, type Options } from 'openweather-api-node';
import { type Options } from 'openweather-api-node';
import { Sequencer } from '../src/sequencer.js';
import type { Config } from '../src/index.js';
const dummyWeather: Options = { key: 'dummy' };