Conversation
- 【実バグ修正】work-log の日時表示が new Date().getHours() = ブラウザのローカル時刻を使い、 受け取っている timeZone prop を無視していた。入力側(workLogToFormInput)は timeZone を 尊重しているため入力と表示が食い違っていた。gridMetrics に timeZone 対応の formatWorkLogDateTime / formatWorkLogRange を追加して寄せ、自前フォーマッタを撤去。 JST 環境では出力不変、他 TZ ではアプリ設定基準の表示に変わる(意図した修正)。 JST/UTC/LA で出力が変わることをテストで固定。履歴行は縦に密に並ぶためゼロ詰めの桁揃えは維持 (既存 formatTime に寄せると TZ 以外の見た目変更が混ざるため) - GitHubPane 1395 → 341行。PaneSection / WorkQueueSection / ManualWorkLogForm / WorkLogHistory へ 分割(クラス名・props シグネチャ・JSX は無変更、CSS 分割はしない)。PaneSection は WorkQueueSection から使われるため独立ファイルに(同居させると相互 import になる) - 純関数化: collectRepoOwners / reposForOwner(workLogEntry)、splitRunningAndIdleQueue(workQueue)。 GitHubPane 側は3つの useMemo が1つに - SettingsModal の toLocaleString 直書きを mcpTokens.ts の mcpTokenCreatedLabel に切り出し (隣の mcpTokenLastUsedLabel と形を揃えた)。トークンの発行日時は壁時計情報ではなく監査メタ なので timeZone 対応はしない(理由をコメントに明記) web 871 tests, typecheck/build clean。 Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01HGESE2zBNzoW8MYvtEySBQ
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
1a: EventDetailCard 独立 + ツールチップ hook 化
4ファイル(MonthView/AllDayBar/OooRailLine/WorkingLocationRailBand)がポップオーバーのためだけに 1160行モジュールを経由していた状態を解消。EventBlock 1160 → 678行。移設は機械的コピーで JSX がバイト同一であることを diff で確認。ホバーツールチップの4重複(
HOVER_DELAY_MSの4重定義込み)を hook に集約。会議/場所行の判定を純関数化しテスト8件。1b: GitHubPane 分割 + 【実バグ修正】タイムゾーン
work-log の日時表示が
new Date().getHours()(ブラウザのローカル時刻)でtimeZoneprop を無視しており、入力側は timeZone を尊重しているため食い違っていた。timeZone 対応の純関数に寄せて修正(JST 環境は出力不変、他 TZ はアプリ設定基準に変わる = 意図した修正)。GitHubPane 1395 → 341行の分割、純関数化3件。web 871 tests / typecheck / build clean。
🤖 Generated with Claude Code
https://claude.ai/code/session_01HGESE2zBNzoW8MYvtEySBQ