Skip to content

console.time / console.timeEnd を実装#225

Open
tknkaa wants to merge 2 commits into
mainfrom
fix/console-time-timeend
Open

console.time / console.timeEnd を実装#225
tknkaa wants to merge 2 commits into
mainfrom
fix/console-time-timeend

Conversation

@tknkaa

@tknkaa tknkaa commented Jul 23, 2026

Copy link
Copy Markdown

#125 の console.time と console.timeEnd を実装し、テストを追加しました。console.log のフォーマット指定子については、現在のドキュメントでは使われていないため、いったんこの PR では実装していません。
また、テストについては、@testing-library/dom が @testing-library/react の peerDependency として要求されていたので、devDependencies に追加し、新しく追加したテストが通ることを確認しました。

console.time/timeEndは未実装のためワーカーグローバルの元のconsoleにフォールバックしており、
呼び出しても出力パネルに何も表示されなかった(#125)。
labelごとの開始時刻を記録し、timeEnd時に経過時間を出力するよう実装。

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
@cloudflare-workers-and-pages

cloudflare-workers-and-pages Bot commented Jul 23, 2026

Copy link
Copy Markdown

Deploying with  Cloudflare Workers  Cloudflare Workers

The latest updates on your project. Learn more about integrating Git with Workers.

Status Name Latest Commit Preview URL Updated (UTC)
✅ Deployment successful!
View logs
my-code c0488f8 Commit Preview URL

Branch Preview URL
Jul 23 2026, 01:58 PM

@tknkaa
tknkaa marked this pull request as draft July 23, 2026 13:52
@testing-library/react が要求するpeer dependencyが宣言されておらず、
vitestの依存解決(esbuild)が失敗しCIのtest-runtimeジョブが実行前に落ちていた。

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
@tknkaa
tknkaa marked this pull request as ready for review July 23, 2026 14:04
@tknkaa
tknkaa requested a review from na-trium-144 July 23, 2026 14:04
@na-trium-144

Copy link
Copy Markdown
Contributor
  • timeLog という関数があるらしいので、time,timeEndの実装時についでに実装してもいいかも (ドキュメントでは使ってないけど) https://developer.mozilla.org/en-US/docs/Web/API/console/timeLog_static
  • 1言語専用のテストをreplのテストのファイルに書くのは変な感じがする
    • これを機にconsoleの実装とテストを packages/jsEval/ に移してもいいかもしれない (別PRのタスクにしてもいいし、このPRでもいいかも)
  • Node.jsのtimeEndの挙動を再現してくれたみたいですが、Node.jsは1分を超えるとまた表記が変わるようですね (そこまで再現する必要はない) (1時間超えるとどうなるんだろう?)
    a: 1:13.059 (m:ss.mmm)
    
    • ちなみに気になったので他の環境も試してみたところchrome,safariは1秒超えてもms表記でした
    • bunはなんか違う
      [3.62s] default
      
  • runtimeテストは2日前まで動いてたんだけどな...

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.

2 participants