Skip to content

fix(react-compiler): treat zero-argument new Date as impure - #26894

Merged
graphite-app[bot] merged 1 commit into
mainfrom
codex/react-compiler-new-date-purity
Sep 26, 2026
Merged

graphite-app[bot] merged 1 commit into
mainfrom
codex/react-compiler-new-date-purity

Conversation

@Boshen

@Boshen Boshen commented Sep 21, 2026 •

Copy link
Copy Markdown
Member

Ports React's Date purity check exactly, including the upstream fixtures and equivalent lint coverage. Zero-argument Date calls are impure; construction with a timestamp remains allowed.

AI disclosure: Codex was used to port, review, and test this change.

@chatgpt-codex-connector

chatgpt-codex-connector Bot commented Sep 21, 2026 •

Copy link
Copy Markdown

Codex Review Summary

This comment shows the latest Codex review activity on this pull request.

Review Status Commit Review trigger
📝 Code Review ✅ Completed 2026-09-26T15:23:01.566963Z b288747 New commits
ℹ️ 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" or "@codex security review".

Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings.

@Boshen
Boshen added this pull request to stack #26897 September 21, 2026 12:39
@github-actions github-actions Bot added the A-react-compiler Area - React Compiler label Sep 21, 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: c3a962c356

ℹ️ 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".

@codspeed

codspeed Bot commented Sep 21, 2026 •

Copy link
Copy Markdown
Contributor

Merging this PR will not alter performance

✅ 5 untouched benchmarks
⏩ 91 skipped benchmarks1


Comparing codex/react-compiler-new-date-purity (59bf43b) with main (571cfa3)

Open in CodSpeed

Footnotes

  1. 91 benchmarks were skipped, so the baseline results were used instead. If they were deleted from the codebase, click here and archive them to remove them from the performance reports. ↩

@Boshen
Boshen force-pushed the codex/react-compiler-captured-update-expressions branch from 7fbc119 to 23d1549 Compare September 22, 2026 02:01
@Boshen
Boshen force-pushed the codex/react-compiler-new-date-purity branch from c3a962c to 18b369c Compare September 22, 2026 02:01

@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: 18b369cf5a

ℹ️ 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".

@Boshen
Boshen force-pushed the codex/react-compiler-captured-update-expressions branch from 23d1549 to 9fef25b Compare September 22, 2026 02:17
@Boshen
Boshen force-pushed the codex/react-compiler-new-date-purity branch from 18b369c to 1d72426 Compare September 22, 2026 02:17

@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: 1d72426e1b

ℹ️ 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".

@Boshen
Boshen force-pushed the codex/react-compiler-captured-update-expressions branch from 9fef25b to 08f2918 Compare September 22, 2026 02:45
@Boshen
Boshen force-pushed the codex/react-compiler-new-date-purity branch from 1d72426 to 7854084 Compare September 22, 2026 02:45
@Boshen
Boshen force-pushed the codex/react-compiler-captured-update-expressions branch from 08f2918 to 16789ae Compare September 22, 2026 03:16
@Boshen
Boshen force-pushed the codex/react-compiler-new-date-purity branch from 7854084 to c1cee93 Compare September 22, 2026 03:16
@Boshen
Boshen force-pushed the codex/react-compiler-captured-update-expressions branch from 16789ae to 595e303 Compare September 22, 2026 09:56
@Boshen
Boshen force-pushed the codex/react-compiler-new-date-purity branch from c1cee93 to 4951e4a Compare September 22, 2026 09:56
@Boshen
Boshen force-pushed the codex/react-compiler-captured-update-expressions branch from 595e303 to a300ae0 Compare September 22, 2026 10:21
@Boshen
Boshen force-pushed the codex/react-compiler-new-date-purity branch from 4951e4a to bd96f51 Compare September 22, 2026 10:21
@Boshen
Boshen force-pushed the codex/react-compiler-captured-update-expressions branch from a300ae0 to 7d1846a Compare September 22, 2026 10:30
@Boshen
Boshen force-pushed the codex/react-compiler-new-date-purity branch from bd96f51 to ae04e74 Compare September 22, 2026 10:31
@Boshen
Boshen force-pushed the codex/react-compiler-captured-update-expressions branch from 7d1846a to ac845d8 Compare September 22, 2026 11:05
@Boshen
Boshen force-pushed the codex/react-compiler-new-date-purity branch from ae04e74 to 739fee5 Compare September 22, 2026 11:06
@Boshen
Boshen force-pushed the codex/react-compiler-captured-update-expressions branch from ac845d8 to 120deb3 Compare September 22, 2026 11:16
@Boshen
Boshen force-pushed the codex/react-compiler-new-date-purity branch from 739fee5 to 70649c8 Compare September 22, 2026 11:16
@Boshen
Boshen force-pushed the codex/react-compiler-captured-update-expressions branch from 120deb3 to 31b099e Compare September 22, 2026 11:27
@Boshen
Boshen force-pushed the codex/react-compiler-new-date-purity branch from 70649c8 to 2e90459 Compare September 22, 2026 11:27
@Boshen
Boshen force-pushed the codex/react-compiler-captured-update-expressions branch from 31b099e to 9611628 Compare September 22, 2026 11:45
@Boshen
Boshen force-pushed the codex/react-compiler-new-date-purity branch from 2e90459 to ec672ec Compare September 22, 2026 11:45
@Boshen
Boshen force-pushed the codex/react-compiler-captured-update-expressions branch from 9611628 to 3c49d6e Compare September 22, 2026 12:01
@Boshen
Boshen force-pushed the codex/react-compiler-new-date-purity branch from 68e99ba to 7b6e4c3 Compare September 24, 2026 05:02

@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: 7b6e4c3932

ℹ️ 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".

@Boshen
Boshen force-pushed the codex/react-compiler-new-date-purity branch from 7b6e4c3 to 2647bfa Compare September 24, 2026 05:25

@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: 2647bfafe2

ℹ️ 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".

@Boshen
Boshen force-pushed the codex/react-compiler-new-date-purity branch from 2647bfa to bdcf8d9 Compare September 24, 2026 05:43

@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: bdcf8d94b9

ℹ️ 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".

@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: 2786ca485b

ℹ️ 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".

@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: 7448a75025

ℹ️ 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".

@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: a9e1282826

ℹ️ 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".

@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: 9eb5897b0b

ℹ️ 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".

@Boshen
Boshen force-pushed the codex/react-compiler-new-date-purity branch from 9eb5897 to 5966d38 Compare September 24, 2026 07:07
@Boshen
Boshen removed this pull request from stack #26897 September 24, 2026 07:14
@Boshen
Boshen added this pull request to stack #27005 September 24, 2026 07:14

@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: 5966d38478

ℹ️ 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".

@Boshen
Boshen removed this pull request from stack #27005 September 25, 2026 08:03

@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: 0882d498a9

ℹ️ 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".

@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: d6daf6305a

ℹ️ 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".

@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: 27fb4f96e1

ℹ️ 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".

@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: 01fce1d730

ℹ️ 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".

Boshen commented Sep 26, 2026 •

Copy link
Copy Markdown
Member Author

Merge activity

Ports [React's Date purity check](react/react@ff8f88f) exactly, including the upstream fixtures and equivalent lint coverage. Zero-argument Date calls are impure; construction with a timestamp remains allowed.

AI disclosure: Codex was used to port, review, and test this change.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

A-linter Area - Linter A-react-compiler Area - React Compiler

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant