Skip to content

Add AC_wait_for_process (complete the file/port/process wait family)#211

Merged
JE-Chen merged 1 commit into
devfrom
feat/wait-for-process
Jun 18, 2026
Merged

Add AC_wait_for_process (complete the file/port/process wait family)#211
JE-Chen merged 1 commit into
devfrom
feat/wait-for-process

Conversation

@JE-Chen

@JE-Chen JE-Chen commented Jun 18, 2026

Copy link
Copy Markdown
Member

Summary

Completes the smart-wait family — file / port / process — by adding
wait_until_process / AC_wait_for_process. It polls a psutil-backed
process list until a process whose name contains the target exists
(present=True) or is gone (present=False), the natural companion to
launch_process / kill_process and the existing assert_process.

Wired through the full stack like the other waits:

  • headless wait_until_process (smart_waits, injectable lister)
  • facade re-export
  • AC_wait_for_process executor command
  • ac_wait_for_process MCP tool
  • visual Script Builder entry
  • docs: v5 reference page (EN + Traditional Chinese) and all three READMEs

Notes

  • Returns a WaitOutcome and honours a hard timeout_s cap (cannot hang).
  • Uses the existing psutil-based _running_process_names helper; psutil
    is already the optional backend for assert_process.
  • Tests inject the lister, so they need neither psutil nor real processes.
  • ruff / radon / bandit clean; import je_auto_control stays Qt-free.

Completes the wait family (file / port / process). Polls a psutil-backed
process list until a process whose name contains the target exists
(present=True) or is gone (present=False) — the companion to
launch_process / kill_process. Follows the smart-waits pattern (injectable
lister, WaitOutcome, hard timeout) and is wired through the full stack:
wait_until_process, facade, AC_wait_for_process, the ac_wait_for_process
MCP tool, a Script Builder entry, and the v5 / README docs.
@codacy-production

Copy link
Copy Markdown

Up to standards ✅

🟢 Issues 0 issues

Results:
0 new issues

View in Codacy

🟢 Metrics 19 complexity · 0 duplication

Metric Results
Complexity 19
Duplication 0

View in Codacy

NEW Get contextual insights on your PRs based on Codacy's metrics, along with PR and Jira context, without leaving GitHub. Enable AI reviewer
TIP This summary will be updated as you push new changes.

@sonarqubecloud

Copy link
Copy Markdown

@JE-Chen JE-Chen merged commit a855cb0 into dev Jun 18, 2026
16 checks passed
@JE-Chen JE-Chen deleted the feat/wait-for-process branch June 18, 2026 17:37
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.

1 participant