Skip to content

Prevent parser deadlocks and add Windows portability - #453

Open
PhillipsT-Ai2 wants to merge 1 commit into
douglasmonsky:mainfrom
PhillipsT-Ai2:fix/audit-reliability-windows
Open

PhillipsT-Ai2 wants to merge 1 commit into
douglasmonsky:mainfrom
PhillipsT-Ai2:fix/audit-reliability-windows

Conversation

@PhillipsT-Ai2

Copy link
Copy Markdown

Summary

  • remove the queue-capacity deadlock and make parser shutdown cancellable
  • add portable RSS, ownership, and fsync helpers
  • use platform-appropriate storage paths and security checks
  • add Windows CI coverage and stable line-ending rules

Verification

  • 145 focused portable-core tests passed
  • targeted mypy passed
  • Ruff and staged diff checks passed

Known limitation

The legacy Unix-specific one-shot benchmark harness still depends on facilities such as fork, wait4, POSIX signals, and symlink privileges, so that harness is not fully runnable on Windows. The portable production paths and new Windows CI scope are covered.

@PhillipsT-Ai2

Copy link
Copy Markdown
Author

Maintainer action requested: approve and start the fork pull-request workflows.

GitHub currently reports this PR as BLOCKED, and no pull-request workflow run or check suite has been created for the head SHA. The branch is conflict-free. Local validation completed successfully: 145 focused portable-core tests, targeted mypy, Ruff, and diff checks.

Could a maintainer please approve the Actions run and review the PR?

This branch has not been deployed

No deployments
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