Use .last-update-check for update timestamps

Switch update-check storage from .dev to .last-update-check and add it to .gitignore. Rename timestamp helper functions and variables accordingly, and update logic to treat empty/invalid timestamp as null (instead of returning a far-future value). Improve dev-mode messaging and ensure --force-update always overrides dev mode; update the timestamp file after running the self-update check.

Author: tommusrhodus

.gitignore

@@ -1,5 +1,6 @@
 .dev
 .quiet
+.last-update-check
 
 # Created by https://www.toptal.com/developers/gitignore/api/macos,linux,windows,node,yarn,composer,phpstorm+all,visualstudiocode,sublimetext
 # Edit at https://www.toptal.com/developers/gitignore?templates=macos,linux,windows,node,yarn,composer,phpstorm+all,visualstudiocode,sublimetext

self-update.php

@@ -75,46 +75,45 @@ function team51_cli_self_update(): void {
 // region TIMESTAMP FUNCTIONS
 
 /**
- * Gets the timestamp from the .dev file.
+ * Gets the timestamp of the last update check.
  *
  * @return  int|null  The timestamp, or null if the file doesn't exist or doesn't contain a valid timestamp.
  */
-function team51_cli_get_dev_timestamp(): ?int {
-	$dev_file = TEAM51_CLI_ROOT_DIR . '/.dev';
+function team51_cli_get_last_update_check_timestamp(): ?int {
+	$timestamp_file = TEAM51_CLI_ROOT_DIR . '/.last-update-check';
 
-	if ( ! file_exists( $dev_file ) ) {
+	if ( ! file_exists( $timestamp_file ) ) {
 		return null;
 	}
 
-	$content = file_get_contents( $dev_file );
+	$content = file_get_contents( $timestamp_file );
 	if ( empty( $content ) || ! is_numeric( $content ) ) {
-		// If the file is empty or doesn't contain a valid timestamp, return a far future date (2035) to force dev mode.
-		return 2062837972;
+		return null;
 	}
 
 	return (int) $content;
 }
 
 /**
- * Writes the current timestamp to the .dev file.
+ * Records the current time as the last update check.
  *
  * @return  void
  */
-function team51_cli_update_dev_timestamp(): void {
-	$dev_file  = TEAM51_CLI_ROOT_DIR . '/.dev';
-	$timestamp = time();
+function team51_cli_update_last_update_check_timestamp(): void {
+	$timestamp_file = TEAM51_CLI_ROOT_DIR . '/.last-update-check';
 
-	file_put_contents( $dev_file, $timestamp );
+	file_put_contents( $timestamp_file, time() );
 }
 
 // endregion
 
 // region EXECUTION LOGIC
 
-$team51_cli_is_quiet     = file_exists( TEAM51_CLI_ROOT_DIR . '/.quiet' );
-$team51_cli_is_dev       = false; // Will be set based on .dev file timestamp or --dev flag
-$team51_is_autocomplete  = false;
-$team51_cli_force_update = false;
+$team51_cli_is_quiet        = file_exists( TEAM51_CLI_ROOT_DIR . '/.quiet' );
+$team51_cli_dev_file_exists = file_exists( TEAM51_CLI_ROOT_DIR . '/.dev' );
+$team51_cli_is_dev          = $team51_cli_dev_file_exists;
+$team51_is_autocomplete     = false;
+$team51_cli_force_update    = false;
 
 foreach ( $argv as $arg ) {
 	switch ( $arg ) {
@@ -138,22 +137,25 @@ function team51_cli_update_dev_timestamp(): void {
 // Print the ASCII art.
 team51_cli_print_message( file_get_contents( TEAM51_CLI_ROOT_DIR . '/.ascii' ) );
 
-// Check for updates.
-if ( $team51_cli_is_dev ) {
+// Check for updates. --force-update wins over dev mode so an explicit refresh always runs.
+if ( $team51_cli_is_dev && ! $team51_cli_force_update ) {
 	team51_cli_print_message( "\033[44mRunning in developer mode. Skipping update check.\033[0m" );
+	if ( $team51_cli_dev_file_exists ) {
+		team51_cli_print_message( "\033[33m.dev file detected — run `team51 --force-update <command>` periodically to pick up CLI changes, or `rm .dev` to re-enable the 7-day update buffer.\033[0m" );
+	}
 } else {
-	$dev_timestamp         = team51_cli_get_dev_timestamp();
+	$last_check_timestamp  = team51_cli_get_last_update_check_timestamp();
 	$seven_days_in_seconds = 7 * 24 * 60 * 60; // 7 days in seconds
 
 	$should_update = $team51_cli_force_update ||
-					null === $dev_timestamp ||
-					( time() - $dev_timestamp ) >= $seven_days_in_seconds;
+					null === $last_check_timestamp ||
+					( time() - $last_check_timestamp ) >= $seven_days_in_seconds;
 
 	if ( $should_update ) {
 		team51_cli_print_message( "\033[33mChecking for updates..\033[0m" );
 		team51_cli_self_update();
-		// Update the timestamp after checking for updates
-		team51_cli_update_dev_timestamp();
+		// Update the timestamp after checking for updates.
+		team51_cli_update_last_update_check_timestamp();
 	} else {
 		team51_cli_print_message( "\033[33mSkipping update check (less than 7 days since last check).\033[0m" );
 	}