Contributor

Switched OBJ face parsing to use .at() for array access instead of bracket notation. Negative indices (like -1 for last vertex) now work per OBJ spec, positive indices still get converted from 1-based to 0-based.

Resolves #8493

Changes

• Also removed TODO about negative indices from src/webgl/loading.js
• Uses .at() for loadedVerts.v, loadedVerts.vt, loadedVerts.vn lookups to handle both negative and positive indexing
• Added test/unit/assets/cube-negative-indices.obj ,, test file with negative face indices
• Added two tests confirming negative index OBJs load and produce identical geometry to positive index versions

PR Checklist

• npm run lint passes
• Inline reference is included / updated
• Unit tests are included / updated

OBJ loader doesn't handle negative vertex indices #8493

skip FES checks on internal calls #8517

davepagurek approved these changes Feb 12, 2026

View reviewed changes

Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.