Skip to content

fix(dispatch): avoid stale artifact false positives#37

Open
starwings2020 wants to merge 4 commits into
uluckyXH:mainfrom
starwings2020:fix/dispatch-artifact-timeout-guards
Open

fix(dispatch): avoid stale artifact false positives#37
starwings2020 wants to merge 4 commits into
uluckyXH:mainfrom
starwings2020:fix/dispatch-artifact-timeout-guards

Conversation

@starwings2020

Copy link
Copy Markdown

Summary

  • share one timeout budget across runner send/wait phases
  • require artifact fingerprint changes so pre-existing files do not count as fresh success
  • add minimal host workspace path mapper used by the helper server
  • cover helper and dispatch runner regressions with targeted tests

Test Plan

  • python -m pytest tests/test_dispatch_openclaw_runner.py tests/test_openclaw_agent_helper_server.py tests/test_workspace_path_mapper.py -q
  • python - <<'PY'
    import py_compile
    for path in [
    "app/services/dispatch_openclaw_runner.py",
    "scripts/openclaw_agent_helper_server.py",
    "app/services/workspace_path_mapper.py",
    ]:
    py_compile.compile(path, doraise=True)
    print("OK", path)
    PY

- share one timeout budget across run and wait phases\n- require artifact fingerprint changes instead of accepting pre-existing files\n- cover helper and dispatch runner regressions with targeted tests

(cherry picked from commit b265467fc1e02c4daff1774950c5afafc7ada348)
Archives ~16MB of run-time garbage for safe deletion from main:
- docs/novel-factory/artifacts/backups/ (6 x sqlite backups)
- docs/novel-factory/artifacts/openmoss-pdf-ocr-20260429/
- docs/novel-factory/artifacts/workspace-backfill-*.{json,md}
- docs/novel-factory/artifacts/workspace-field-coverage*.json
- input-archives/ (extracted zip + config snapshots)
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