Skip to content

2026-08-31 vxData bundle test tautology review

Reviewed PR #357 tests for independent behavioral oracles and implementation-mirroring checks.

Work Done

Reviewed mono PR #357 (feat: Allow the creation of bundles in vxdata), its new test_bundle.py, the bundle implementation, manifest schemas, and the shared in-memory S3 backend. Classified tests by whether they use an independent oracle or only restate private implementation state.

Lessons Learned: Pitfalls

The mandatory knowledge checkout exists in two locations. ~/.virdx/knowledge had diverged from upstream and could not fast-forward, while the required ~/.agents/knowledge checkout was current. The duplicate checkout instruction adds avoidable failure noise.

Lessons Learned: Improvements

A concise vxData test-review note could define independent oracles: parse produced archives with a standard reader, recompute hashes from stored bytes, and assert externally visible cleanup. Tests that seed private counters and then exercise the adjacent guard should be treated as implementation mirrors.

Navigation

Type to search…

↑↓ navigate↵ selectEsc close