fix backup status display, sync loop safety, and startup log noise

- paperclip-sync.py: write explicit status+message fields in sync_to_backup
and sync_from_backup (previously relied on compat shim which mapped
last_error→message, causing empty message on success and duplicate
"Last sync" lines in dashboard)
- paperclip-sync.py: change log format to [sync] %(message)s (drop verbose
timestamp/level prefix that cluttered startup restore output)
- start.sh: sanitize SYNC_INTERVAL (strip non-digits, clamp ≥60s) to
prevent sleep "" failure causing tight spin loop

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

paperclip-sync.py

@@ -32,7 +32,7 @@ huggingface_hub.utils.disable_progress_bars()
 # Logging: WARNING level keeps errors visible, suppresses routine INFO chatter
 logging.basicConfig(
     level=logging.WARNING,
-    format='%(asctime)s - %(levelname)s - %(message)s'
+    format='[sync] %(message)s'
 )
 logger = logging.getLogger(__name__)
 # Set our own logger to INFO so backup/restore start+finish lines still print
@@ -490,16 +490,16 @@ def sync_to_backup() -> bool:
         # Step 1: Backup database
         dump_file, success = backup_database()
         if not success or not dump_file:
-            status['last_error'] = 'Database backup failed'
-            status['db_status'] = 'error'
+            status.update({'last_error': 'Database backup failed', 'db_status': 'error',
+                           'status': 'error', 'message': 'Backup failed: pg_dump error'})
             write_status(status)
             return False
 
         # Step 2: Create tarball
         tarball_file, success = create_backup_tarball(dump_file)
         if not success or not tarball_file:
-            status['last_error'] = 'Tarball creation failed'
-            status['db_status'] = 'error'
+            status.update({'last_error': 'Tarball creation failed', 'db_status': 'error',
+                           'status': 'error', 'message': 'Backup failed: tarball too large or I/O error'})
             write_status(status)
             return False
 
@@ -507,10 +507,13 @@ def sync_to_backup() -> bool:
         success = sync_to_hf(tarball_file)
 
         # Update status
-        status['last_sync_time'] = datetime.now(timezone.utc).isoformat().replace('+00:00', 'Z')
+        ts = datetime.now(timezone.utc).isoformat().replace('+00:00', 'Z')
+        status['last_sync_time'] = ts
         status['db_status'] = 'connected' if success else 'error'
         status['last_error'] = None if success else 'Upload failed'
         status['sync_count'] = status.get('sync_count', 0) + 1
+        status['status'] = 'success' if success else 'error'
+        status['message'] = 'Backup uploaded to HF Dataset' if success else 'Upload to HF Dataset failed'
 
         write_status(status)
 
@@ -526,6 +529,8 @@ def sync_to_backup() -> bool:
         logger.error(f'Backup operation failed: {e}')
         status['last_error'] = str(e)
         status['db_status'] = 'error'
+        status['status'] = 'error'
+        status['message'] = f'Backup error: {e}'
         write_status(status)
         return False
 
@@ -542,18 +547,24 @@ def sync_from_backup() -> bool:
             # No backup exists yet (first boot) — not an error
             status['db_status'] = 'connected'
             status['last_error'] = None
+            status['status'] = 'configured'
+            status['message'] = 'Fresh instance — no prior backup'
             write_status(status)
             logger.info('No prior backup — fresh instance')
             return True
         elif success:
             status['db_status'] = 'connected'
             status['last_error'] = None
+            status['status'] = 'restored'
+            status['message'] = 'Restored from HF Dataset'
             write_status(status)
             logger.info('Restore OK')
             return True
         else:
             status['db_status'] = 'error'
             status['last_error'] = 'Restore failed'
+            status['status'] = 'error'
+            status['message'] = 'Restore from HF Dataset failed'
             write_status(status)
             logger.warning('Restore operation failed')
             return False
@@ -562,6 +573,8 @@ def sync_from_backup() -> bool:
         logger.error(f'Restore operation failed: {e}')
         status['last_error'] = str(e)
         status['db_status'] = 'error'
+        status['status'] = 'error'
+        status['message'] = f'Restore error: {e}'
         write_status(status)
         return False
 

start.sh

@@ -17,7 +17,10 @@ export PAPERCLIP_CONFIG="${PAPERCLIP_CONFIG:-${PAPERCLIP_HOME}/instances/default
 export PAPERCLIP_TELEMETRY_DISABLED="${PAPERCLIP_TELEMETRY_DISABLED:-1}"
 export DO_NOT_TRACK="${DO_NOT_TRACK:-1}"
 export OPENCODE_ALLOW_ALL_MODELS="${OPENCODE_ALLOW_ALL_MODELS:-true}"
-export SYNC_INTERVAL="${SYNC_INTERVAL:-3600}"
+# Sanitize: strip non-digits, clamp minimum to 60s to prevent spin loops.
+SYNC_INTERVAL=$(printf '%s' "${SYNC_INTERVAL:-3600}" | tr -dc '0-9')
+{ [ -z "${SYNC_INTERVAL}" ] || [ "${SYNC_INTERVAL}" -lt 60 ]; } && SYNC_INTERVAL=3600
+export SYNC_INTERVAL
 export SYNC_MAX_FILE_BYTES="${SYNC_MAX_FILE_BYTES:-52428800}"
 export BACKUP_DATASET_NAME="${BACKUP_DATASET_NAME:-huggingclip-backup}"