v2.4.0: Phase 2 — meal time, past records, goals, CSV import, crash report
- Meal time dialog (right-click lunch/dinner button to enter actual times) - Calendar right-click context: add/edit/delete past records - Monthly goal settings + progress widget (overtime cap, avg daily) - CSV import (our standard format) with conflict policy - Global crash handler with Gitea Issues auto-report Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
This commit is contained in:
co-authored by
Claude Opus 4.7
parent
d3a4efc173
commit
9ebf4ad961
@@ -0,0 +1,173 @@
|
||||
"""
|
||||
전역 예외 후킹 + Gitea Issues 자동 보고.
|
||||
|
||||
sys.excepthook을 등록해 처리되지 않은 예외를 가로채:
|
||||
1. crash_log 테이블에 저장
|
||||
2. 사용자에게 다이얼로그로 알림 + "Gitea에 보고" / "복사" 옵션
|
||||
"""
|
||||
from __future__ import annotations
|
||||
import json
|
||||
import sys
|
||||
import traceback
|
||||
import urllib.request
|
||||
import urllib.error
|
||||
from datetime import datetime
|
||||
from typing import Optional
|
||||
|
||||
# 자체 호스팅 Gitea (updater_client와 동일)
|
||||
GITEA_API = 'https://kindnick-git.duckdns.org/api/v1/repos/kindnick/Clock_out_Time_Calculator'
|
||||
USER_AGENT = 'Mozilla/5.0 (Windows NT 10.0; Win64; x64) ClockOutCalculator/2.4'
|
||||
|
||||
|
||||
def install_global_handler(db, app_version: str = 'unknown') -> None:
|
||||
"""sys.excepthook 등록. db는 crash_log 저장용."""
|
||||
original = sys.excepthook
|
||||
|
||||
def hook(exc_type, exc_value, exc_tb):
|
||||
if exc_type is KeyboardInterrupt:
|
||||
original(exc_type, exc_value, exc_tb)
|
||||
return
|
||||
try:
|
||||
tb_str = ''.join(traceback.format_exception(exc_type, exc_value, exc_tb))
|
||||
_log_crash(db, exc_type.__name__, str(exc_value), tb_str, app_version)
|
||||
_show_dialog(db, exc_type.__name__, str(exc_value), tb_str, app_version)
|
||||
except Exception:
|
||||
pass # 후킹 자체 실패는 silent
|
||||
original(exc_type, exc_value, exc_tb)
|
||||
|
||||
sys.excepthook = hook
|
||||
|
||||
|
||||
def _log_crash(db, exc_type: str, message: str, tb: str, version: str) -> None:
|
||||
"""crash_log 테이블에 기록."""
|
||||
try:
|
||||
conn = db.get_connection()
|
||||
cursor = conn.cursor()
|
||||
# 테이블 자동 생성 (마이그레이션 누락 대비)
|
||||
cursor.execute('''
|
||||
CREATE TABLE IF NOT EXISTS crash_log (
|
||||
id INTEGER PRIMARY KEY AUTOINCREMENT,
|
||||
timestamp TIMESTAMP DEFAULT CURRENT_TIMESTAMP,
|
||||
exception_type TEXT,
|
||||
message TEXT,
|
||||
traceback TEXT,
|
||||
app_version TEXT,
|
||||
reported BOOLEAN DEFAULT 0
|
||||
)
|
||||
''')
|
||||
cursor.execute('''
|
||||
INSERT INTO crash_log (exception_type, message, traceback, app_version)
|
||||
VALUES (?, ?, ?, ?)
|
||||
''', (exc_type, message, tb, version))
|
||||
conn.commit()
|
||||
conn.close()
|
||||
except Exception:
|
||||
pass
|
||||
|
||||
|
||||
def _show_dialog(db, exc_type: str, message: str, tb: str, version: str) -> None:
|
||||
"""크래시 알림 + Gitea 보고/복사 버튼."""
|
||||
try:
|
||||
from PyQt5.QtWidgets import (QApplication, QMessageBox, QDialog,
|
||||
QVBoxLayout, QLabel, QTextEdit,
|
||||
QHBoxLayout, QPushButton, QLineEdit)
|
||||
except ImportError:
|
||||
return
|
||||
|
||||
app = QApplication.instance()
|
||||
if app is None:
|
||||
return
|
||||
|
||||
dlg = QDialog()
|
||||
dlg.setWindowTitle("⚠️ 오류 발생")
|
||||
dlg.setMinimumSize(560, 420)
|
||||
layout = QVBoxLayout()
|
||||
|
||||
title = QLabel(f"<b>{exc_type}</b>: {message[:200]}")
|
||||
title.setWordWrap(True)
|
||||
layout.addWidget(title)
|
||||
|
||||
layout.addWidget(QLabel("무엇을 하다가 발생했나요? (선택)"))
|
||||
user_note = QLineEdit()
|
||||
user_note.setPlaceholderText("예: 출근 버튼 누른 직후")
|
||||
layout.addWidget(user_note)
|
||||
|
||||
layout.addWidget(QLabel("기술 정보 (자동 보고에 포함):"))
|
||||
tb_view = QTextEdit()
|
||||
tb_view.setReadOnly(True)
|
||||
tb_view.setFont(__import__('PyQt5.QtGui', fromlist=['QFont']).QFont('Consolas', 9))
|
||||
tb_view.setText(tb[-3000:]) # 너무 긴 traceback 제한
|
||||
layout.addWidget(tb_view, 1)
|
||||
|
||||
btn_row = QHBoxLayout()
|
||||
cancel_btn = QPushButton("닫기")
|
||||
copy_btn = QPushButton("📋 복사")
|
||||
report_btn = QPushButton("📤 Gitea에 보고")
|
||||
|
||||
has_token = bool(db.get_setting('gitea_feedback_token', '') or '')
|
||||
enabled = db.get_setting('gitea_feedback_enabled', 'false').lower() == 'true'
|
||||
if not (has_token and enabled):
|
||||
report_btn.setEnabled(False)
|
||||
report_btn.setToolTip("설정 → 데이터 관리 → Gitea 피드백 토큰 입력 후 활성화 필요")
|
||||
|
||||
def do_copy():
|
||||
clipboard = QApplication.clipboard()
|
||||
text = (
|
||||
f"## {exc_type}\n\n{message}\n\n"
|
||||
f"**Version**: {version}\n**Note**: {user_note.text()}\n\n"
|
||||
f"```\n{tb}\n```"
|
||||
)
|
||||
clipboard.setText(text)
|
||||
copy_btn.setText("✓ 복사됨")
|
||||
|
||||
def do_report():
|
||||
token = db.get_setting('gitea_feedback_token', '') or ''
|
||||
if not token:
|
||||
QMessageBox.warning(dlg, "토큰 없음", "Gitea PAT를 설정에서 먼저 입력하세요.")
|
||||
return
|
||||
title_str = f"[Auto] {exc_type}: {message[:80]}"
|
||||
body = (
|
||||
f"**Version**: `{version}`\n"
|
||||
f"**Time**: {datetime.now().isoformat(timespec='seconds')}\n"
|
||||
f"**User note**: {user_note.text() or '(none)'}\n\n"
|
||||
f"### Traceback\n```\n{tb[-3000:]}\n```"
|
||||
)
|
||||
ok = _send_to_gitea(token, title_str, body)
|
||||
if ok:
|
||||
QMessageBox.information(dlg, "보고 완료", "Gitea Issues에 등록되었습니다.")
|
||||
report_btn.setText("✓ 보고됨")
|
||||
report_btn.setEnabled(False)
|
||||
else:
|
||||
QMessageBox.warning(dlg, "보고 실패", "네트워크 또는 토큰 권한 문제일 수 있습니다.")
|
||||
|
||||
cancel_btn.clicked.connect(dlg.reject)
|
||||
copy_btn.clicked.connect(do_copy)
|
||||
report_btn.clicked.connect(do_report)
|
||||
btn_row.addWidget(cancel_btn)
|
||||
btn_row.addStretch()
|
||||
btn_row.addWidget(copy_btn)
|
||||
btn_row.addWidget(report_btn)
|
||||
layout.addLayout(btn_row)
|
||||
|
||||
dlg.setLayout(layout)
|
||||
dlg.exec_()
|
||||
|
||||
|
||||
def _send_to_gitea(token: str, title: str, body: str) -> bool:
|
||||
"""Gitea Issues API로 issue 생성."""
|
||||
payload = json.dumps({'title': title, 'body': body}).encode('utf-8')
|
||||
req = urllib.request.Request(
|
||||
f"{GITEA_API}/issues",
|
||||
data=payload,
|
||||
headers={
|
||||
'Authorization': f'token {token}',
|
||||
'Content-Type': 'application/json',
|
||||
'User-Agent': USER_AGENT,
|
||||
},
|
||||
method='POST',
|
||||
)
|
||||
try:
|
||||
with urllib.request.urlopen(req, timeout=10) as resp:
|
||||
return 200 <= resp.status < 300
|
||||
except (urllib.error.URLError, urllib.error.HTTPError, OSError, TimeoutError):
|
||||
return False
|
||||
@@ -0,0 +1,157 @@
|
||||
"""
|
||||
CSV 가져오기 — 우리 표준 포맷.
|
||||
|
||||
표준 포맷:
|
||||
date,clock_in,clock_out,lunch_minutes,memo
|
||||
2026-04-01,09:00:00,18:30:00,60,"메모"
|
||||
|
||||
- 헤더 첫 줄 필수
|
||||
- date: YYYY-MM-DD
|
||||
- clock_in/out: HH:MM:SS 또는 HH:MM
|
||||
- lunch_minutes: 정수 (0이면 점심 미포함)
|
||||
- memo: 선택 (따옴표 가능)
|
||||
|
||||
기존 일자와 충돌 시 import 호출자가 'overwrite'/'skip' 정책 결정.
|
||||
"""
|
||||
from __future__ import annotations
|
||||
import csv
|
||||
from datetime import datetime
|
||||
from pathlib import Path
|
||||
from typing import List, Dict, Iterator, Tuple
|
||||
|
||||
|
||||
def parse_csv(path: str) -> List[Dict]:
|
||||
"""CSV 파일을 dict 리스트로 파싱. 검증 실패 시 ValueError."""
|
||||
rows = []
|
||||
p = Path(path)
|
||||
if not p.exists():
|
||||
raise FileNotFoundError(f"파일 없음: {path}")
|
||||
|
||||
with open(p, 'r', encoding='utf-8-sig', newline='') as f:
|
||||
reader = csv.DictReader(f)
|
||||
required = {'date', 'clock_in'}
|
||||
if not required.issubset(reader.fieldnames or []):
|
||||
raise ValueError(
|
||||
f"헤더에 필수 필드 누락: {required - set(reader.fieldnames or [])}\n"
|
||||
f"필수 헤더: date,clock_in,clock_out,lunch_minutes,memo"
|
||||
)
|
||||
|
||||
for i, row in enumerate(reader, start=2): # 데이터 시작 줄 번호 (1=헤더)
|
||||
try:
|
||||
clean = _normalize_row(row)
|
||||
rows.append(clean)
|
||||
except ValueError as e:
|
||||
raise ValueError(f"줄 {i}: {e}")
|
||||
return rows
|
||||
|
||||
|
||||
def _normalize_row(row: Dict) -> Dict:
|
||||
"""단일 행 검증 + 정규화."""
|
||||
date_str = (row.get('date') or '').strip()
|
||||
if not date_str:
|
||||
raise ValueError("date 비어있음")
|
||||
try:
|
||||
datetime.strptime(date_str, '%Y-%m-%d')
|
||||
except ValueError:
|
||||
raise ValueError(f"date 형식 오류: '{date_str}' (YYYY-MM-DD 필요)")
|
||||
|
||||
ci = _normalize_time(row.get('clock_in', '').strip(), 'clock_in')
|
||||
co_raw = (row.get('clock_out') or '').strip()
|
||||
co = _normalize_time(co_raw, 'clock_out') if co_raw else None
|
||||
|
||||
lunch = 0
|
||||
lm = (row.get('lunch_minutes') or '').strip()
|
||||
if lm:
|
||||
try:
|
||||
lunch = int(lm)
|
||||
if lunch < 0:
|
||||
raise ValueError
|
||||
except ValueError:
|
||||
raise ValueError(f"lunch_minutes는 0 이상 정수여야 함: '{lm}'")
|
||||
|
||||
memo = (row.get('memo') or '').strip()
|
||||
return {
|
||||
'date': date_str,
|
||||
'clock_in': ci,
|
||||
'clock_out': co,
|
||||
'lunch_minutes': lunch,
|
||||
'memo': memo,
|
||||
}
|
||||
|
||||
|
||||
def _normalize_time(s: str, field_name: str) -> str:
|
||||
"""'HH:MM' 또는 'HH:MM:SS' → 'HH:MM:SS'."""
|
||||
if not s:
|
||||
raise ValueError(f"{field_name} 비어있음")
|
||||
parts = s.split(':')
|
||||
if len(parts) == 2:
|
||||
s = f"{s}:00"
|
||||
elif len(parts) != 3:
|
||||
raise ValueError(f"{field_name} 형식 오류: '{s}' (HH:MM[:SS] 필요)")
|
||||
try:
|
||||
datetime.strptime(s, '%H:%M:%S')
|
||||
except ValueError:
|
||||
raise ValueError(f"{field_name} 시간 파싱 실패: '{s}'")
|
||||
return s
|
||||
|
||||
|
||||
def import_records(db, rows: List[Dict], on_conflict: str = 'skip') -> Tuple[int, int, int]:
|
||||
"""파싱된 rows를 DB에 일괄 입력.
|
||||
|
||||
Args:
|
||||
db: Database 인스턴스
|
||||
rows: parse_csv 결과
|
||||
on_conflict: 'skip' | 'overwrite'
|
||||
|
||||
Returns:
|
||||
(added, updated, skipped) 수
|
||||
"""
|
||||
if on_conflict not in ('skip', 'overwrite'):
|
||||
raise ValueError("on_conflict는 'skip' 또는 'overwrite'")
|
||||
|
||||
added = updated = skipped = 0
|
||||
|
||||
from core.time_calculator import TimeCalculator
|
||||
work_minutes = db.get_work_minutes()
|
||||
lunch_default = db.get_setting_int('lunch_duration_minutes', 60)
|
||||
|
||||
for row in rows:
|
||||
existing = db.get_work_record(row['date'])
|
||||
|
||||
if existing and on_conflict == 'skip':
|
||||
skipped += 1
|
||||
continue
|
||||
|
||||
if existing and on_conflict == 'overwrite':
|
||||
# 기존 레코드 삭제 후 재추가 (단순화)
|
||||
conn = db.get_connection()
|
||||
cursor = conn.cursor()
|
||||
try:
|
||||
cursor.execute("DELETE FROM overtime_bank WHERE date = ?", (row['date'],))
|
||||
cursor.execute("DELETE FROM break_records WHERE date = ?", (row['date'],))
|
||||
cursor.execute("DELETE FROM work_records WHERE date = ?", (row['date'],))
|
||||
conn.commit()
|
||||
finally:
|
||||
conn.close()
|
||||
updated += 1
|
||||
else:
|
||||
added += 1
|
||||
|
||||
wid = db.add_work_record(row['date'], row['clock_in'], is_manual=True)
|
||||
if row.get('clock_out'):
|
||||
ci_dt = datetime.strptime(f"{row['date']} {row['clock_in']}", '%Y-%m-%d %H:%M:%S')
|
||||
co_dt = datetime.strptime(f"{row['date']} {row['clock_out']}", '%Y-%m-%d %H:%M:%S')
|
||||
calc = TimeCalculator(work_minutes=work_minutes,
|
||||
lunch_duration_minutes=row.get('lunch_minutes') or lunch_default)
|
||||
include_lunch = (row.get('lunch_minutes') or 0) > 0
|
||||
total = (co_dt - ci_dt).total_seconds() / 3600
|
||||
ot_actual, ot_earned = calc.calculate_overtime(
|
||||
ci_dt, co_dt, include_lunch=include_lunch
|
||||
)
|
||||
db.update_clock_out(row['date'], row['clock_out'], total, ot_actual, ot_earned)
|
||||
if include_lunch:
|
||||
db.update_lunch_break(row['date'], True)
|
||||
if ot_earned > 0:
|
||||
db.add_overtime_earned(wid, ot_earned, row['date'])
|
||||
|
||||
return added, updated, skipped
|
||||
Reference in New Issue
Block a user