test(kiloclaw): add edge-case coverage for rolloutBucket

[St0rmz1]

Cover empty keys, non-ASCII keys, and key-ordering sensitivity so concatenated salt+subject keys are verified not to collide.

Summary

Verification

• [ ]

Visual Changes

Before	After

Reviewer Notes

[kilo-code-bot]

Code Review Summary

Status: No Issues Found | Recommendation: Merge

Executive Summary

The new tests (empty key, non-ASCII key, key-ordering sensitivity, salt-change sensitivity) are consistent with rolloutBucket's SHA-256-based implementation and correctly exercise the described edge cases with no logic or assertion errors.

Files Reviewed (1 files)

• services/kiloclaw/src/lib/rollout-bucket.test.ts

Previous Review Summaries (2 snapshots)

Current summary above is authoritative. Previous snapshots are kept for context only.

Previous review

Status: No Issues Found | Recommendation: Merge

Executive Summary

The new tests (empty key, non-ASCII key, key-ordering sensitivity, salt-change sensitivity) are consistent with rolloutBucket's SHA-256-based implementation and correctly exercise the described edge cases with no logic or assertion errors.

Files Reviewed (1 files)

• services/kiloclaw/src/lib/rollout-bucket.test.ts

Previous review

Status: No Issues Found | Recommendation: Merge

Executive Summary

The new tests (empty key, non-ASCII key, key-ordering sensitivity, salt-change sensitivity) are consistent with rolloutBucket's SHA-256-based implementation and correctly exercise the described edge cases with no logic or assertion errors.

Files Reviewed (1 files)

• services/kiloclaw/src/lib/rollout-bucket.test.ts

---

_{Reviewed by claude-sonnet-5 · Input: 6 · Output: 9.7K · Cached: 101.7K}

_{Review guidance: REVIEW.md from base branch main}