Fix Angular

[falahat]

Description

The local builds code was hard-expecting this field to exist. However, that's only the case for nextjs apps (and only standalone ones). This PR makes the code gracefully handle missing values for this.

Scenarios Tested

Successfully deployed an angular app

[gemini-code-assist]

Code Review

This pull request updates the local build process to support frameworks like Angular that do not provide an outputFiles block in their bundle.yaml. It modifies the parsing logic to default to an empty array instead of throwing an error and updates the deployment preparation to handle cases with zero output files. The review identified a type safety issue regarding the use of 'any' in the test suite and highlighted a regression where multiple output files are now silently ignored instead of triggering an error.

[gemini-code-assist]

Avoid using any as an escape hatch, as per the repository style guide. Since the stubbed function is only expected to handle string paths in this test context, you should use a more specific type like string.

Suggested change

	sinon.stub(fsExtra, "readFileSync").callsFake((pathStr: any) => {
	sinon.stub(fsExtra, "readFileSync").callsFake((pathStr: string) => {

References

1. Never use any or unknown as an escape hatch. Define proper interfaces/types or use type guards. ^(link)

[wiz-9635d3485b]

Wiz Scan Summary

Scanner	Findings
Vulnerabilities	-
Sensitive Data	-
Secrets	-
IaC Misconfigurations	-
SAST Findings	2
Software Management Findings	-
Total	2

View scan details in Wiz

To detect these findings earlier in the dev lifecycle, try using Wiz Code VS Code Extension.