feat: add UserComparator::name() for stable identity persistence#101

Merged

polaz merged 19 commits into

mainfrom

feat/#74-feat-add-usercomparatorname-for-stable-identity-pe

Mar 23, 2026

Commits on Mar 23, 2026

feat: add UserComparator::name() for stable identity persistence
polaz
committed
fix: add size bound for comparator_name manifest section
polaz
committed
test(manifest): add unit tests for comparator_name backward compat
polaz
committed
fix(manifest): add write-side name length assert, remove unwrap in tests
polaz
committed
test(comparator): add unit tests for DefaultUserComparator::name()
polaz
committed
docs: qualify intra-doc links for UserComparator::name
polaz
committed
docs(persist): clarify why assert is used for name length check
polaz
committed
fix(recover): validate manifest before recover_levels
polaz
committed
test(manifest): add invalid UTF-8 comparator name rejection test
polaz
committed
test(table): exercise ReverseComparator::name() for coverage
polaz
committed
refactor: unify MAX_COMPARATOR_NAME_BYTES, downgrade mismatch log level
polaz
committed
fix(comparator): use pub for module-private const, fix stale doc
polaz
committed
fix(persist): return Result instead of panicking on oversized name
polaz
committed
style(table): use method syntax for ReverseComparator::name() call
polaz
committed
test(comparator): add oversized name rejection test
polaz
committed
fix(persist): include actual name length in oversized error message
polaz
committed
test(comparator): assert InvalidInput error kind for oversized name
polaz
committed
refactor(manifest): use read_to_end for comparator_name section
polaz
committed
refactor(manifest): simplify sfa::Writer::finish in test helpers
polaz
committed

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

feat: add UserComparator::name() for stable identity persistence#101

feat: add UserComparator::name() for stable identity persistence#101
polaz merged 19 commits into
mainfrom
feat/#74-feat-add-usercomparatorname-for-stable-identity-pe

Commits on Mar 23, 2026

feat: add UserComparator::name() for stable identity persistence

fix: add size bound for comparator_name manifest section

test(manifest): add unit tests for comparator_name backward compat

fix(manifest): add write-side name length assert, remove unwrap in tests

test(comparator): add unit tests for DefaultUserComparator::name()

docs: qualify intra-doc links for UserComparator::name

docs(persist): clarify why assert is used for name length check

fix(recover): validate manifest before recover_levels

test(manifest): add invalid UTF-8 comparator name rejection test

test(table): exercise ReverseComparator::name() for coverage

refactor: unify MAX_COMPARATOR_NAME_BYTES, downgrade mismatch log level

fix(comparator): use pub for module-private const, fix stale doc

fix(persist): return Result instead of panicking on oversized name

style(table): use method syntax for ReverseComparator::name() call

test(comparator): add oversized name rejection test

fix(persist): include actual name length in oversized error message

test(comparator): assert InvalidInput error kind for oversized name

refactor(manifest): use read_to_end for comparator_name section

refactor(manifest): simplify sfa::Writer::finish in test helpers