Skip to content

test(runtime-host): validate Windows artifact recovery - #2488

Merged
Astro-Han merged 1 commit into
apache:mainfrom
liugddx:test/windows-atomic-publication
Aug 8, 2026
Merged

test(runtime-host): validate Windows artifact recovery#2488
Astro-Han merged 1 commit into
apache:mainfrom
liugddx:test/windows-atomic-publication

Conversation

@liugddx

@liugddx liugddx commented Aug 8, 2026

Copy link
Copy Markdown
Member

Summary

  • enable the production Artifact publication and owner-death recovery gate on Windows
  • close fixture-owned Artifact and execution stores before releasing the root lease
  • use the explicit Runtime Host fixture shutdown protocol for portable graceful teardown
  • allow a larger Windows cold-start budget for recovery of the seeded Artifact catalog
  • refresh the Windows skip inventory

Validation

  • npm --workspace @maka/runtime run build
  • npm --workspace @maka/runtime-host run build
  • npm --workspace @maka/runtime-host run typecheck
  • node --test packages/runtime-host/dist/__tests__/artifact-two-client-uds.test.js (passed twice before rebase and once after rebase on Windows)
  • npm run windows:inventory
  • git diff --check

The prerequisite Runtime Host fixture shutdown and deterministic execution-store cleanup changes are available on main through #2492. This PR now contains only the Artifact recovery gate and inventory update.

Part of #2142

@liugddx
liugddx marked this pull request as draft August 8, 2026 07:30
@liugddx
liugddx marked this pull request as ready for review August 8, 2026 08:23
@liugddx
liugddx force-pushed the test/windows-atomic-publication branch from 0a66815 to c63788b Compare August 8, 2026 08:28
@liugddx

liugddx commented Aug 8, 2026

Copy link
Copy Markdown
Member Author

@jackwener @Astro-Han

@Astro-Han Astro-Han left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The recovery test exercises the real Runtime Host, IPC, SQLite, filesystem residue, and owner-death boundary, and I found no production correctness issue. The remaining Windows CI-lane coverage gap is non-blocking.

@Astro-Han
Astro-Han merged commit 4e65134 into apache:main Aug 8, 2026
11 checks passed
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.

2 participants