docs(RenderSystem): add @param em and @returns to flushStaleMeshBuffers JSDoc

[Copilot]

flushStaleMeshBuffers was missing @param em and @returns tags in its JSDoc block, leaving documentation generators with incomplete output and inconsistent API docs.

Changes

• src/core/ecs/systems/RenderSystem.ts — added @param em and @returns void tags to flushStaleMeshBuffers, positioned before the existing @example block:

/**
 * Releases GPU buffers for mesh components that are no longer attached to any
 * active entity. …
 *
 * @param em The entity manager whose active entities are checked.
 * @returns void
 * @example …
 */
flushStaleMeshBuffers(em: EntityManager): void {

• tests/ecs.test.ts — added two JSDoc contract tests (consistent with the repo's readFileSync + substring assertion pattern) asserting that the @param em and @returns tags are present in the JSDoc block immediately preceding the method signature.

[Copilot]

Pull request overview

This PR updates API documentation for RenderSystem.flushStaleMeshBuffers by completing its JSDoc contract and adding tests to ensure the tags remain present.

Changes:

• Added missing @param em and @returns tags to the flushStaleMeshBuffers JSDoc block.
• Added contract-style tests that assert the @param em and @returns tags exist in the JSDoc immediately preceding the method signature.

Reviewed changes

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

File	Description
src/core/ecs/systems/RenderSystem.ts	Completes the flushStaleMeshBuffers JSDoc by adding @param em and @returns tags.
tests/ecs.test.ts	Adds contract tests to prevent regression of the new JSDoc tags for flushStaleMeshBuffers.

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.