[PR #1734] [MERGED] Process.launch()/kill()/.pidfile/.wait()/etc. centralize process handling logic on model methods #4499

Closed
opened 2026-03-15 01:47:58 +03:00 by kerem · 0 comments
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: devHead: 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

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.

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 <!-- IMPORTANT: Do not submit PRs with only formatting / PEP8 / line length changes. --> # Summary <!--e.g. This PR fixes ABC or adds the ability to do XYZ...--> # Related issues <!-- e.g. #123 or Roadmap goal # https://github.com/pirate/ArchiveBox/wiki/Roadmap --> # Changes these areas - [ ] Bugfixes - [ ] Feature behavior - [ ] Command line interface - [ ] Configuration options - [ ] Internal architecture - [ ] Snapshot data layout on disk <!-- This is an auto-generated description by cubic. --> --- ## 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> <!-- End of auto-generated description by cubic. --> --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
kerem 2026-03-15 01:47:58 +03:00
Sign in to join this conversation.
No milestone
No project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
starred/ArchiveBox#4499
No description provided.