Bug: Two (different) instances of fetchAndValidateSpec

[bh2smith]

Here is one:

make-agent/src/commands/dev.ts

Line 44 in 3f38010

async function fetchAndValidateSpec(url: string): Promise<ValidationResult> {

Here is the second:

make-agent/src/utils/openapi.ts

Lines 8 to 11 in 3f38010

	//parsing and validation done together to avoid fetching spec twice
	export async function validateAndParseOpenApiSpec(
	url: string | URL,
	): Promise<XMbSpec | undefined> {

To resolve this, we will write unit tests for both, determine which is preferred and delete the other.

[bh2smith]

So this isn't quite a bug, but we do have two different implementations, not of the parsing, but of the fetching. One of them has a few more [DEV] logs, but they are other wise identical.