api-extractor replacement test

[hsubox76]

No description provided.

[changeset-bot]

⚠️ No Changeset found

Latest commit: 8eab1fc

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

[wiz-9635d3485b]

Wiz Scan Summary

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

View scan details in Wiz

To detect these findings earlier in the dev lifecycle, try the Wiz Code extension for VS Code, JetBrains, or Visual Studio.

[gemini-code-assist]

Code Review

This pull request migrates the project from a custom api-extractor-me package to the official @microsoft/api-extractor package, updating configuration files and build scripts for the messaging package accordingly. It also updates the documentation generator script to support copying multiple API JSON files. Feedback on these changes includes correcting an invalid and temporarily hardcoded package path in the docgen script, as well as renaming a loop variable to avoid shadowing the imported path module.

[gemini-code-assist]

The hardcoded path 'firebase/messaging' seems to be incorrect as the package is located in 'packages/messaging'. Additionally, hardcoding a single package breaks the documentation generation for all other packages. This temporary change should be reverted before merging.

      await spawn('yarn', ['api-report'], {

[gemini-code-assist]

The loop variable path shadows the imported path module (from import * as path from 'path'). Rename the variable to filePath or apiPath to avoid shadowing and potential confusion.

Suggested change

	for (const path of paths) {
	const fileName = path.split('/').pop();
	fs.copyFileSync(path, `${tmpDir}/${fileName}`);
	}
	for (const apiPath of paths) {
	const fileName = apiPath.split('/').pop();
	fs.copyFileSync(apiPath, `${tmpDir}/${fileName}`);
	}