etc. centralize process handling logic on model methods #1485

New issue

Closed

opened 2026-03-01 14:49:59 +03:00 by kerem · 0 comments

kerem commented

2026-03-01 14:49:59 +03:00

Owner

📋 Pull Request Information

Original PR: https://github.com/ArchiveBox/ArchiveBox/pull/1734
Author: @pirate
Created: 12/31/2025
Status: ✅ Merged
Merged: 12/31/2025
Merged by: @pirate

Base: dev ← Head: claude/refactor-process-management-WcQyZ

📝 Commits (8)

f21fb55 Add comprehensive implementation plan for Process hierarchy tracking
4c4c065 Add Process.current() to implementation plan
554d743 Add robust PID reuse protection to Process.current() plan
e41ca37 Add detailed hook/run() changes to Process tracking plan
f3c91b4 Add detailed supervisord Process tracking to plan
caee376 Add Process.proc property for validated psutil access
3ae9410 Update TODO_process_tracking.md
dfe6841 Merge branch 'dev' into claude/refactor-process-management-WcQyZ

📊 Changes

1 file changed (+1723 additions, -0 deletions)

View changed files

➕ TODO_process_tracking.md (+1723 -0)

📄 Description

Summary

Changes these areas

Bugfixes
Feature behavior
Command line interface
Configuration options
Internal architecture
Snapshot data layout on disk

Summary by cubic

Added an implementation plan to centralize subprocess handling on the machine.Process model. It covers process hierarchy, Process.current(), safe lifecycle methods (launch/kill/wait), PID reuse protection, and phased changes across hooks, workers, CLI, migrations, and admin.

^{Written for commit dfe68412af. Summary will update on new commits.}

_{🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.}

## 📋 Pull Request Information **Original PR:** https://github.com/ArchiveBox/ArchiveBox/pull/1734 **Author:** [@pirate](https://github.com/pirate) **Created:** 12/31/2025 **Status:** ✅ Merged **Merged:** 12/31/2025 **Merged by:** [@pirate](https://github.com/pirate) **Base:** `dev` ← **Head:** `claude/refactor-process-management-WcQyZ` --- ### 📝 Commits (8) - [`f21fb55`](https://github.com/ArchiveBox/ArchiveBox/commit/f21fb55a2cffac8936f7d62481cff0fe74f89091) Add comprehensive implementation plan for Process hierarchy tracking - [`4c4c065`](https://github.com/ArchiveBox/ArchiveBox/commit/4c4c065697fb019784c029da9253bef8f2585bee) Add Process.current() to implementation plan - [`554d743`](https://github.com/ArchiveBox/ArchiveBox/commit/554d7437198a5e4705f1a5c4904f054b1b9d2163) Add robust PID reuse protection to Process.current() plan - [`e41ca37`](https://github.com/ArchiveBox/ArchiveBox/commit/e41ca3784850155dac8ccd5804269206b3ec560b) Add detailed hook/run() changes to Process tracking plan - [`f3c91b4`](https://github.com/ArchiveBox/ArchiveBox/commit/f3c91b4c4e82fcbacef51e38662e9bf394be003f) Add detailed supervisord Process tracking to plan - [`caee376`](https://github.com/ArchiveBox/ArchiveBox/commit/caee376749b60129e6a6e50eaa1e864dff69c904) Add Process.proc property for validated psutil access - [`3ae9410`](https://github.com/ArchiveBox/ArchiveBox/commit/3ae94101275360ed6060176e6ff4ad1a05d1411d) Update TODO_process_tracking.md - [`dfe6841`](https://github.com/ArchiveBox/ArchiveBox/commit/dfe68412af4e7a532c9ba4efe0053e75c47f0a25) Merge branch 'dev' into claude/refactor-process-management-WcQyZ ### 📊 Changes **1 file changed** (+1723 additions, -0 deletions) <details> <summary>View changed files</summary> ➕ `TODO_process_tracking.md` (+1723 -0) </details> ### 📄 Description  # Summary  # Related issues  # Changes these areas - [ ] Bugfixes - [ ] Feature behavior - [ ] Command line interface - [ ] Configuration options - [ ] Internal architecture - [ ] Snapshot data layout on disk  --- ## Summary by cubic Added an implementation plan to centralize subprocess handling on the machine.Process model. It covers process hierarchy, Process.current(), safe lifecycle methods (launch/kill/wait), PID reuse protection, and phased changes across hooks, workers, CLI, migrations, and admin. <sup>Written for commit dfe68412af4e7a532c9ba4efe0053e75c47f0a25. Summary will update on new commits.</sup>  --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>