Skip to content

Log: add source field to identify which script deleted each item#28

Closed
DaLuSt wants to merge 1 commit intomainfrom
claude/log-add-source-field-7Wp79
Closed

Log: add source field to identify which script deleted each item#28
DaLuSt wants to merge 1 commit intomainfrom
claude/log-add-source-field-7Wp79

Conversation

@DaLuSt
Copy link
Copy Markdown
Owner

@DaLuSt DaLuSt commented Feb 21, 2026

What changed

When logs accumulate across manual CLI runs, weekly CI runs, and web-app sessions it is impossible to tell what deleted what. Adds a source tag as the last column in both log files.

Source value Set by
cli-mode-1 commentCleaner.py — "Delete old comments"
cli-mode-2 commentCleaner.py — "Remove comments with negative karma"
cli-mode-3 commentCleaner.py — "Remove 1-karma no-reply comments"
cli PostCleaner.py
ci weekly_cleanup.py (GitHub Actions)
web web/app.py

Files changed

commentCleaner.py (modes 1/2/3), PostCleaner.py, weekly_cleanup.py, web/app.py

https://claude.ai/code/session_014HLhrFtCVRFnEyfRexiy3d

@DaLuSt DaLuSt force-pushed the claude/log-add-source-field-7Wp79 branch from 2f2b42b to b0f5877 Compare February 21, 2026 22:12
Records which script and mode produced each log entry:
  cli-mode-1 / cli-mode-2 / cli-mode-3 — commentCleaner.py (per mode)
  cli                                    — PostCleaner.py
  ci                                     — weekly_cleanup.py
  web                                    — web/app.py

https://claude.ai/code/session_014HLhrFtCVRFnEyfRexiy3d
@DaLuSt DaLuSt force-pushed the claude/log-add-source-field-7Wp79 branch from b0f5877 to f853812 Compare February 27, 2026 08:59
@DaLuSt
Copy link
Copy Markdown
Owner Author

DaLuSt commented Feb 27, 2026

"Superseded by #33"

@DaLuSt DaLuSt closed this Feb 27, 2026
@DaLuSt DaLuSt deleted the claude/log-add-source-field-7Wp79 branch February 27, 2026 16:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants