fix: normalize night light state marker

Author: lucidust

Cargo.lock

@@ -1,6 +1,6 @@
 [package]
 name = "wnlctl"
-version = "0.1.1"
+version = "0.1.2"
 edition = "2024"
 license = "MIT"
 description = "Windows Night Light control CLI"

Cargo.toml

@@ -2,6 +2,15 @@
 
 Maintainer notes for publishing a new `wnlctl` release.
 
+## Codex Shortcut
+
+For the usual patch release flow, tell Codex:
+
+```text
+wnlctl 패치 릴리스 진행해줘. 버전 bump, 검증, 커밋, 태그, 푸시,
+scoop-bucket workflow 실행, 로컬 scoop 업데이트까지 해줘.
+```
+
 ## Manual Release
 
 1. Update the package version in `Cargo.toml`.

RELEASING.md

@@ -34,7 +34,7 @@ impl NightlightState {
         reader.read_marshaled_header()?;
 
         let mut is_enabled = false;
-        let mut initialized: i32 = 0;
+        let mut initialized: i32 = 1;
         let mut last_transition_filetime: u64 = 0;
 
         loop {
@@ -105,26 +105,30 @@ impl NightlightState {
             + u64::from(now.timestamp_subsec_nanos() / 100);
     }
 
-    /// Enables the nightlight and updates the timestamp.
-    /// Returns true if a change was made (i.e. the nightlight was previously disabled).
-    pub fn enable(&mut self) -> bool {
-        if self.is_enabled {
+    fn set_enabled(&mut self, is_enabled: bool) -> bool {
+        let state_changed = self.is_enabled != is_enabled;
+        let marker_changed = self.initialized != 1;
+
+        if !state_changed && !marker_changed {
             return false;
         }
-        self.is_enabled = true;
+
+        self.is_enabled = is_enabled;
+        self.initialized = 1;
         self.update_transition_timestamps();
         true
     }
 
+    /// Enables the nightlight and updates the timestamp.
+    /// Returns true if a change was made.
+    pub fn enable(&mut self) -> bool {
+        self.set_enabled(true)
+    }
+
     /// Disables the nightlight and updates the timestamp.
-    /// Returns true if a change was made (i.e. the nightlight was previously enabled).
+    /// Returns true if a change was made.
     pub fn disable(&mut self) -> bool {
-        if !self.is_enabled {
-            return false;
-        }
-        self.is_enabled = false;
-        self.update_transition_timestamps();
-        true
+        self.set_enabled(false)
     }
 }
 
@@ -194,4 +198,25 @@ mod tests {
         let state_deserialized = NightlightState::deserialize_from_bytes(&bytes).unwrap();
         assert_eq!(state_deserialized, state_enabled);
     }
+
+    #[test]
+    fn state_changes_normalize_initialized_marker() {
+        let mut state = NightlightState {
+            timestamp: 0,
+            is_enabled: false,
+            initialized: 0,
+            last_transition_filetime: 0,
+        };
+
+        assert!(state.enable());
+        assert_eq!(state.initialized, 1);
+
+        state.initialized = 0;
+        assert!(state.disable());
+        assert_eq!(state.initialized, 1);
+
+        state.initialized = 0;
+        assert!(state.disable());
+        assert_eq!(state.initialized, 1);
+    }
 }