Revisar los if(debug) time timeEnd

[asanzo]

Hay que pegarle una revisada a los logs de profiling como estos:

wollok-ts-cli/src/commands/run.ts

Lines 30 to 32 in 47b6bed

	if(debug) time(successDescription('Run finalized successfully'))
	interpret(environment, natives).run(programFQN)
	if(debug) timeEnd(successDescription('Run finalized successfully'))

wollok-ts-cli/src/utils.ts

Lines 15 to 29 in 47b6bed

	export async function buildEnvironmentForProject(cwd: string): Promise<Environment> {
	const paths = await globby('**/*.@(wlk|wtest|wpgm)', { cwd })
	const debug = logger.getLevel() <= logger.levels.DEBUG
	if(debug) time('Reading project files')
	const files = await Promise.all(paths.map(async name =>
	({ name, content: await readFile(join(cwd, name), 'utf8') })
	))
	if(debug) timeEnd('Reading project files')
	if(debug) time('Building environment')
	try { return buildEnvironment(files) }
	finally { if(debug) timeEnd('Building environment' ) }
	}

Quizás necesitemos otra herramienta de logging que me permita hacer esos profilings en lugar de usarla como un flag y nada más.

[PalumboN]

@fdodino esto se cierra con el logger que tenemos, no?

[fdodino]

@PalumboN , maso, podríamos

1. eliminar los if (debug) pero además
2. hay que ver qué estrategia adoptamos con el logger. En Wollok LSP IDE guardamos el tiempo de buildeo que excede los xxx milisegundos (por defecto 100). Pero en wollok-ts-cli no estamos logueando lo que tarda, con lo cual cuando hacemos un run o levantamos un REPL esa info no está.

[PalumboN]

Qué significa eliminar los if (debug)?

[fdodino]

Y... ya no necesitaríamos tener un time + timeEnd con el debug, habría que cambiar la estrategia para guardar el tiempo solamente y llamar al logger por otro.