fix: @PrePersist/@PreUpdate で LocalDateTime.now() をシステムクロックに依存している

[claude]

概要

Task.java および Tenant.java の @PrePersist / @PreUpdate コールバックで LocalDateTime.now() を直接使用しており、JVM のシステムクロックに依存している。

問題点

// Task.java / Tenant.java 共通
void onCreate() {
    LocalDateTime now = LocalDateTime.now(); // JVM のシステム TZ・クロックに依存
    ...
}

• 方針(docs: JST 全層統一(設計規約 §2.4 / コーディング規約 §12.1 反転 + infrastructure-plan に JVM/ECS TZ 明文化) #265 で確定): 全層 Asia/Tokyo 統一前提だが、LocalDateTime.now() 直接呼び出しは JVM デフォルト TZ・クロックに依存するため、TZ 強制が崩れた場合(単発テストで TZ が未設定で実行された等)に静かにズレる
• テスト時に時刻を固定できないため、createdAt/updatedAt の値を厳密にアサートするテストが書けない

対応案

Spring の Clock Bean をインジェクションして LocalDateTime.now(clock) を使用する。Clock は Clock.system(ZoneId.of("Asia/Tokyo")) で定義し、TZ 強制とテスト時の差し替え(Clock.fixed(...))を両立する。

@Configuration
class ClockConfig {
    @Bean
    Clock clock() {
        return Clock.system(ZoneId.of("Asia/Tokyo"));
    }
}

// Entity 側
@Autowired private Clock clock;

@PrePersist
void onCreate() {
    LocalDateTime now = LocalDateTime.now(clock);
    this.createdAt = now;
    this.updatedAt = now;
}

影響ファイル

• webapi/src/main/java/xyz/dgz48/tasks/webapi/task/Task.java
• webapi/src/main/java/xyz/dgz48/tasks/webapi/tenant/Tenant.java

参考

• PR db(flyway): 全 FK に ON DELETE を明示し、スキーマを設計書 v1.3 に揃える (#81) #130 セルフレビュー 指摘 2 より
• 関連: docs: JST 全層統一(設計規約 §2.4 / コーディング規約 §12.1 反転 + infrastructure-plan に JVM/ECS TZ 明文化) #265(JST 全層統一)

[claude-automation-impl]

実装方針（着手宣言）

方針サマリ

Issue の指摘通り LocalDateTime.now() をシステムクロック依存から脱却するため、Spring Data JPA Auditing + DateTimeProvider + Clock Bean の組み合わせで対応します。

採用アプローチ

@Autowired Clock clock を JPA エンティティに直接注入する案はエンティティが Spring Bean ではないため動作しません。代わりに Spring Data JPA の正式パターンを使用します。

1. ClockConfig.java を root パッケージに新規作成

   • @EnableJpaAuditing(dateTimeProviderRef = "clockDateTimeProvider")
   • @Bean Clock clock() → Clock.system(ZoneId.of("Asia/Tokyo"))
   • @Bean DateTimeProvider clockDateTimeProvider(Clock clock) → () -> Optional.of(LocalDateTime.now(clock))

2. Task.java / Tenant.java 修正

   • @EntityListeners(AuditingEntityListener.class) を追加
   • createdAt に @CreatedDate、updatedAt に @LastModifiedDate を追加
   • @PrePersist / @PreUpdate メソッドを削除

3. テスト追加

   • FixedClockConfiguration.java（@TestConfiguration）: @Primary Clock.fixed(...) で固定時刻を提供
   • TaskEntityTest：固定クロックで createdAt / updatedAt の厳密アサートを追加
   • TenantEntityTest.java：Tenant のタイムスタンプ検証テストを新規作成

[win2cot]

Action failed with error: SDK execution error: Error: Claude Code returned an error result: Reached maximum number of turns (40)

[win2cot]

実装完了 (PR #283)

実装内容

Issue の対応案に従い、Spring Data JPA Auditing パターンで実装しました。

主な変更

ファイル	内容
shared/infra/ClockConfig.java	@EnableJpaAuditing(dateTimeProviderRef = "clockDateTimeProvider") + Clock.system(Asia/Tokyo) + clockDateTimeProvider Bean
TaskJpaEntity / TenantJpaEntity	@PrePersist/@PreUpdate 削除 → @EntityListeners(AuditingEntityListener.class) + @CreatedDate/@LastModifiedDate
FixedClockConfiguration（テスト用）	固定 DateTimeProvider を提供する @TestConfiguration
TaskEntityTest / TenantEntityTest	固定時刻でのタイムスタンプ厳密アサートを追加

---

設計上の注意点：なぜ @ConditionalOnMissingBean が必要か

当初は Issue のコメントにある「@Primary Clock.fixed(...) で差し替える」案で実装しましたが、@SpringBootTest の Bean 登録順序の問題でテストが失敗しました。

問題の原因

@SpringBootTest の処理順:
1. テストクラスの @Import を処理 → FixedClockConfiguration の clockDateTimeProvider が登録される
2. @SpringBootApplication のコンポーネントスキャンを処理
   → ClockConfig が発見され、clockDateTimeProvider が後から登録されて上書きされる
3. 結果: production の clockDateTimeProvider（実時刻）が勝つ

コンポーネントスキャン（ClockConfig）は @SpringBootTest の @Import より後に処理されるため、@Primary による単純な override では production Bean が最終的に勝ってしまいます。

解決策：@ConditionalOnMissingBean

// ClockConfig.java
@Bean
@ConditionalOnMissingBean(name = "clockDateTimeProvider")
DateTimeProvider clockDateTimeProvider(Clock clock) {
    return () -> Optional.of(LocalDateTime.now(clock));
}

@ConditionalOnMissingBean を使うと、コンポーネントスキャン時点（ステップ2）で FixedClockConfiguration の Bean がすでに存在するため、ClockConfig は自身の Bean 登録をスキップします。これにより allow-bean-definition-overriding=true も不要になります。

テスト時の差し替え方法

@TestConfiguration
public class FixedClockConfiguration {
  public static final LocalDateTime FIXED_NOW = LocalDateTime.of(2026, 1, 15, 19, 0, 0);

  @Bean("clockDateTimeProvider")
  public DateTimeProvider clockDateTimeProvider() {
    return () -> Optional.of(FIXED_NOW);
  }
}

テストクラスで @Import(FixedClockConfiguration.class) するだけで、createdAt/updatedAt を固定時刻でアサートできます。