Skip to content

[codex] Add library integration smoke#23

Merged
dsafdsaf132 merged 6 commits into
mainfrom
codex/three-integration-smoke
Jun 10, 2026
Merged

[codex] Add library integration smoke#23
dsafdsaf132 merged 6 commits into
mainfrom
codex/three-integration-smoke

Conversation

@dsafdsaf132

@dsafdsaf132 dsafdsaf132 commented Jun 10, 2026

Copy link
Copy Markdown
Owner

Summary

  • add pinned three.js and twgl.js dev dependencies for integration testing
  • add a three.js WebGLRenderer smoke that renders a triangle, circle, and rotated box with multi-pixel sampling
  • add a twgl.js smoke covering programInfo, bufferInfo, uniform setters, indexed drawing, and multi-pixel sampling
  • keep library integration tests out of npm test and run them only from GitHub Actions runtime smoke steps

Validation

  • npm run build
  • npm test
  • npm run test:three
  • npm run test:twgl
  • git diff --check

@dsafdsaf132 dsafdsaf132 changed the title [codex] Add three.js integration smoke [codex] Add library integration smoke Jun 10, 2026

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 78119b78cd

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread test/twgl-smoke.js Outdated
@dsafdsaf132 dsafdsaf132 merged commit 8c75ee9 into main Jun 10, 2026
7 checks passed
@dsafdsaf132 dsafdsaf132 deleted the codex/three-integration-smoke branch June 10, 2026 08:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant