[BUG] Duplicate AsyncAPI specification loading in `fromTemplate.ts` command (unnecessary double parsing)

[sahillllllllll-bit]

Describe the bug.

File: src/apps/cli/commands/generate/fromTemplate.ts
The fromTemplate command loads the AsyncAPI file twice during execution.

First via:

const asyncapiInput = await this.loadAsyncAPIInput(asyncapi);

Then again via:

specification = await load(asyncapi);

This results in:

• Duplicate disk I/O
• Duplicate parsing
• Unnecessary memory usage
• Slower execution for large specifications

The first loaded result is not reused, making the second load redundant.

This is a performance inefficiency in CLI command flow.

Expected behavior

The AsyncAPI file should be loaded and parsed only once during command execution.

• The result of loadAsyncAPIInput(asyncapi) should be reused for generation.
• The CLI should not call load(asyncapi) again.
• There should be no duplicate disk reads or parsing steps.
• Large AsyncAPI files should not incur unnecessary performance overhead.
• Command execution should remain functionally identical but more efficient.

Screenshots

How to Reproduce

1. Create a large AsyncAPI file (e.g., with many channels and components).

2.Run:

time asyncapi generate fromTemplate large.yaml @asyncapi/html-template -o out

2. Observe execution time.

3. Add logging inside:

load(asyncapi)

You will see it is invoked after loadAsyncAPIInput() already processed the file.

🖥️ Device Information [optional]

• Operating System (OS): Windows(WSL)
• Browser:
• Browser Version:

👀 Have you checked for similar open issues?

• I checked and didn't find similar issue

🏢 Have you read the Contributing Guidelines?

• I have read the Contributing Guidelines

Are you willing to work on this issue ?

Yes I am willing to submit a PR!

[sahillllllllll-bit]

@Shurtu-gal @AayushSaini101 @Souvikns Waiting for approval :)

Proposed Fix :

Remove the redundant load(asyncapi) call and reuse the result from loadAsyncAPIInput(asyncapi).

Replace this block:

const asyncapiInput = await this.loadAsyncAPIInput(asyncapi);

this.specFile = asyncapiInput;
this.metricsMetadata.template = template;

let specification: Specification;
try {
  specification = await load(asyncapi);
} catch {
  return this.error(
    new ValidationError({
      type: 'invalid-file',
      filepath: asyncapi,
    }),
    { exit: 1 },
  );
}

With this:

let specification: Specification;
try {
  specification = await this.loadAsyncAPIInput(asyncapi);
} catch {
  return this.error(
    new ValidationError({
      type: 'invalid-file',
      filepath: asyncapi,
    }),
    { exit: 1 },
  );
}

this.specFile = specification;
this.metricsMetadata.template = template;

This ensures:

• The AsyncAPI file is loaded only once
• No duplicate disk I/O
• No duplicate parsing
• Improved performance for large specifications

No change in functional behavior

[1164148-7-eng]

/attempt #2017

I've claimed this issue. Working on it now! 🚀

[1164148-7-eng]

/attempt #2017

I've claimed this issue. Working on it now! 🚀