Feature/csb manifest

[vorobey]

No description provided.

[Copilot]

Pull request overview

This PR adds generation of a per-demo demo.manifest.json intended for CodeSandbox, capturing the npm packages (and peer dependencies) required by each bundled demo, and enhances import scanning/version resolution logic in the server-side bundling utilities.

Changes:

• Extend vendor-bundle.js to (a) avoid scanning generated bundle artifacts and (b) resolve imported package versions + required peer dependencies into manifests.
• Emit a per-demo demo.manifest.json during React/Vue and Angular demo bundling.
• Update demos .gitignore to exclude the generated manifest, and adjust minor workflow/comment formatting.

Reviewed changes

Copilot reviewed 6 out of 6 changed files in this pull request and generated 3 comments.

Show a summary per file

File	Description
apps/demos/utils/server/vendor-bundle.js	Adds robust specifier collection helpers, ignores generated artifacts, and resolves package versions/peers into vendor manifests (plus exports utilities for demo manifests).
apps/demos/utils/server/csp-bundle.js	Writes demo.manifest.json for each React/Vue demo after bundling.
apps/demos/utils/server/csp-bundle-angular.js	Writes demo.manifest.json for each Angular demo (including implicit packages list).
apps/demos/utils/server/build-vendor-bundles.js	Clarifies CLI header comment about Angular vendor bundle limitations.
apps/demos/.gitignore	Ignores generated demo.manifest.json in demo source folders.
.github/workflows/visual-tests-demos.yml	Removes an extraneous blank line in the workflow YAML.

---

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

[Copilot]

Pull request overview

Copilot reviewed 6 out of 6 changed files in this pull request and generated no new comments.

Suppressed comments (2)

apps/demos/utils/server/csp-bundle.js:20

• The script no longer reads BUNDLE_IN_PLACE, but CI still sets it (e.g. .github/workflows/visual-tests-demos.yml uses BUNDLE_IN_PLACE: '1'). This env var is now a no-op and may hide configuration mistakes. Consider updating callers to use CSP_BUNDLE_GENERATE_MANIFESTS (if the intent is to emit demo.manifest.json) and/or removing BUNDLE_IN_PLACE from workflows/docs.

const IS_GENERATE_MANIFESTS = process.env.CSP_BUNDLE_GENERATE_MANIFESTS === '1';

apps/demos/utils/server/csp-bundle-angular.js:16

• BUNDLE_IN_PLACE support appears to have been removed (replaced by CSP_BUNDLE_GENERATE_MANIFESTS), but some automation still sets BUNDLE_IN_PLACE. If this script is invoked directly (or via csp-bundle.js for Angular), that legacy env var will be ignored; update any callers/docs that still rely on it.

const IS_GENERATE_MANIFESTS = process.env.CSP_BUNDLE_GENERATE_MANIFESTS === '1';
const NODE_MODULES = path.join(DEMOS_APP_ROOT, 'node_modules');