From b953d118d9aed5d6beeb29c47aafdd03d02478bb Mon Sep 17 00:00:00 2001
From: GeeeekExplorer <2651904866@qq.com>
Date: Fri, 20 Jun 2025 16:02:40 +0800
Subject: [PATCH] refactor: simplify timer.py

---
 slime/utils/timer.py | 36 +++++++++++++-----------------------
 1 file changed, 13 insertions(+), 23 deletions(-)

diff --git a/slime/utils/timer.py b/slime/utils/timer.py
index 2a8d49826c..6ada05cbbf 100644
--- a/slime/utils/timer.py
+++ b/slime/utils/timer.py
@@ -1,4 +1,6 @@
 from time import time
+from functools import wraps
+from contextlib import contextmanager
 
 
 __all__ = ["Timer", "timer"]
@@ -40,20 +42,13 @@ def add(self, name, elapsed_time):
     def log_dict(self):
         return self.timers
 
-
-class TimerContext:
-    def __init__(self, name):
-        self.name = name
-        self.timer_instance = Timer()
-
-    def __enter__(self):
-        self.start_time = time()
-        return self
-
-    def __exit__(self, exc_type, exc_val, exc_tb):
-        self.end_time = time()
-        self.timer_instance.add(self.name, self.end_time - self.start_time)
-        return False  # Don't suppress exceptions
+    @contextmanager
+    def context(self, name):
+        self.start(name)
+        try:
+            yield
+        finally:
+            self.end(name)
 
 
 def timer(name_or_func):
@@ -72,17 +67,12 @@ def func():
     # When used as a context manager
     if isinstance(name_or_func, str):
         name = name_or_func
-        return TimerContext(name)
+        return Timer().context(name)
 
     func = name_or_func
-    name = func.__name__
-
+    @wraps(func)
     def wrapper(*args, **kwargs):
-        timer_instance = Timer()
-        start_time = time()
-        result = func(*args, **kwargs)
-        end_time = time()
-        timer_instance.add(name, end_time - start_time)
-        return result
+        with Timer().context(func.__name__):
+            return func(*args, **kwargs)
 
     return wrapper