Extract the index --pack writer from main into the packed module #5

New Issue

2026-06-01T01:05:44+02:00

Brummel commented

2026-06-01 01:05:44 +02:00

Architect drift finding at the close of the packed-versioned-index-bundle cycle (commits 43b1b6f, 79de4e1).

The alpha-id index --pack writer logic — completeness check, row collection in corpus order, header construction (dim/corpus_name/corpus_sha256/n_rows derivation), write — lives inline in main (src/bin/alpha_id.rs, the Index/pack branch), not in src/packed.rs. The spec's component table implies packing is a packed-module concern, and the inline header-construction code currently sits outside library test reach (covered only end-to-end via the pack CLI tests, not unit-tested).

Proposed direction (not committal): extract a packed::pack_from_store(...) -> Result<PackedHeader, AppError> (or similar) that the binary calls, so the header derivation is unit-testable without spawning the binary.

context: low-priority tidy debt; behaviour is correct and CLI-test-covered, this is about testability + module placement.
depends on: nothing (self-contained, behaviour-preserving refactor).

Architect drift finding at the close of the packed-versioned-index-bundle cycle (commits 43b1b6f, 79de4e1). The `alpha-id index --pack` writer logic — completeness check, row collection in corpus order, header construction (dim/corpus_name/corpus_sha256/n_rows derivation), write — lives inline in `main` (src/bin/alpha_id.rs, the Index/pack branch), not in `src/packed.rs`. The spec's component table implies packing is a `packed`-module concern, and the inline header-construction code currently sits outside library test reach (covered only end-to-end via the pack CLI tests, not unit-tested). Proposed direction (not committal): extract a `packed::pack_from_store(...) -> Result<PackedHeader, AppError>` (or similar) that the binary calls, so the header derivation is unit-testable without spawning the binary. context: low-priority tidy debt; behaviour is correct and CLI-test-covered, this is about testability + module placement. depends on: nothing (self-contained, behaviour-preserving refactor).

Brummel referenced this issue from a commit

2026-06-01 01:35:57 +02:00

audit: packed-versioned-bundle cycle close

Brummel referenced this issue from a commit

2026-06-01 01:35:57 +02:00

plan: extract the index --pack writer into the packed module (refs #5)

Brummel referenced this issue from a commit

2026-06-01 01:35:57 +02:00

refactor: move the index --pack writer into the packed module (closes #5)

Brummel closed this issue

2026-06-01 01:35:57 +02:00

Sign in to join this conversation.