Skip to content

inspect: add Temporal support - #63154

Draft
Renegade334 wants to merge 1 commit into
nodejs:mainfrom
Renegade334:inspect-temporal
Draft

inspect: add Temporal support#63154
Renegade334 wants to merge 1 commit into
nodejs:mainfrom
Renegade334:inspect-temporal

Conversation

@Renegade334

@Renegade334 Renegade334 commented May 6, 2026 •
edited
Loading

Copy link
Copy Markdown
Member

Example output:

> Temporal.Now.instant()
Temporal.Instant 2026-05-06T16:19:21.226404053Z

> class XYZ extends Temporal.Duration {}
> new XYZ(1, 2, 3, 4, 5, 6, 7, 8, 9)
XYZ [Temporal.Duration] P1Y2M3W4DT5H6M7.008009S

> Object.assign(Temporal.Now.zonedDateTimeISO('Pacific/Honolulu'), { foo: 'bar' })
Temporal.ZonedDateTime 2026-05-06T06:24:22.920191895-10:00[Pacific/Honolulu] {
  foo: 'bar'
}

@nodejs-github-bot nodejs-github-bot added needs-ci PRs that need a full CI run. util Issues and PRs related to the built-in util module. labels May 6, 2026
@ljharb

ljharb commented May 6, 2026

Copy link
Copy Markdown
Member

i'm confused, why would it need to be lazily accessed instead of using primordials?

ljharb
ljharb reviewed May 6, 2026
View reviewed changes
Comment thread lib/internal/util/inspect.js Outdated
aduh95
aduh95 reviewed May 8, 2026
View reviewed changes
Comment thread lib/internal/util/inspect.js Outdated
@legendecas

legendecas commented May 11, 2026

Copy link
Copy Markdown
Member

the subsystem should be "util".

@Renegade334

Renegade334 commented May 11, 2026

Copy link
Copy Markdown
Member Author

Alternative approach to the "how to Temporal" problem. This hooks into the pre-execution initializers and creates some typechecking functions and saves the constructor and prototype properties, primordial-ish, if Temporal exists.

We don't use this approach for any other ES builtins, only Node.js APIs, so does need consideration as to whether this is kosher. However, it does also provide for if we want to create or manipulate Temporal objects elsewhere in the API.

@Renegade334 Renegade334 mentioned this pull request May 11, 2026
Open
@Renegade334
util: add Temporal support to inspect()
3f3868a 
Signed-off-by: Renegade334 <contact.9a5d6388@renegade334.me.uk>
@Renegade334 Renegade334 mentioned this pull request May 14, 2026
Open
@ethanresnick ethanresnick mentioned this pull request May 16, 2026
Open
@robobun robobun mentioned this pull request Aug 6, 2026
Open
@github-actions

github-actions Bot commented Aug 13, 2026

Copy link
Copy Markdown
Contributor

This pull request has been marked as stale due to 90 days of inactivity.
It will be automatically closed in 30 days if no further activity occurs. If this is still relevant, please leave a comment or update it to keep it open.

@github-actions github-actions Bot added the stale label Aug 13, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@ljharb ljharb ljharb left review comments
@aduh95 aduh95 aduh95 left review comments

At least 1 approving review is required to merge this pull request.

Assignees

No one assigned

Labels

needs-ci PRs that need a full CI run. stale util Issues and PRs related to the built-in util module.

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

5 participants

@Renegade334 @ljharb @legendecas @aduh95 @nodejs-github-bot