Move DEFAULT_VERTEX_SOURCE / DEFAULT_FRAGMENT_SOURCE JSDoc from re-export to declaration site

[Copilot]

@security annotations and usage docs for DEFAULT_VERTEX_SOURCE and DEFAULT_FRAGMENT_SOURCE were attached only to the re-export lines in index.ts, so "Go to Definition" in any IDE landed on an undocumented constant.

Changes

• src/core/index.ts — Dropped the duplicate JSDoc blocks from the two separate re-exports; collapsed all three ./Material exports into one line:

  export { Material, DEFAULT_VERTEX_SOURCE, DEFAULT_FRAGMENT_SOURCE } from './Material';

• tests/core-index.test.ts — Replaced the four tests asserting JSDoc on re-export lines with two tests matching the new contract: both constants appear in a combined export, and no @security text is duplicated in the barrel.

The full JSDoc (including @security) already existed at the declaration site in Material.ts and remains validated by material.test.ts.

[Copilot]

Pull request overview

Moves JSDoc (including @security guidance) for DEFAULT_VERTEX_SOURCE / DEFAULT_FRAGMENT_SOURCE from the barrel re-export in src/core/index.ts to the actual declaration site so IDE “Go to Definition” lands on documented constants.

Changes:

• Collapses Material + default shader source exports into a single re-export line from ./Material.
• Updates barrel-export tests to reflect the new expectation: combined export with no duplicated @security docs in index.ts.

Reviewed changes

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

File	Description
src/core/index.ts	Replaces three separate ./Material exports (and duplicated JSDoc) with a single combined re-export.
tests/core-index.test.ts	Adjusts tests to check for combined re-export and absence of duplicated @security docs in the barrel.

---

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