test(fragment): add baseBMP ARX decode regression tests

[baanish]

Summary

• Adds two regression tests in tests/fragment.test.ts for decoding baseBMP-encoded ARX fragments via decodeFragmentAsync
• Tests both direct baseBMP payloads and percent-escaped BMP characters (URL-escaped non-ASCII)
• Covers the decode path for v1.arx.N.<baseBMP-payload> fragments that was previously only tested in arx-codec.test.ts

Test plan

• npx vitest run tests/fragment.test.ts — all 12 tests pass (10 existing + 2 new)

🤖 Generated with Claude Code

Summary by CodeRabbit

• Tests
  • Added test cases to verify ARX fragment decoding functionality
  • Validates successful parsing of serialized and compressed payloads
  • Tests include both standard hash formats and percent-encoded character handling
  • Ensures envelope structure parsing works correctly
  • Covers multiple encoding scenarios to verify cross-format compatibility

[coderabbitai]

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: 15ace89d-f142-4c1c-a6f8-17c7b497ec4b

📥 Commits

Reviewing files that changed from the base of the PR and between 07e0942 and 4288ffb.

📒 Files selected for processing (1)

• tests/fragment.test.ts

---

📝 Walkthrough

Walkthrough

The PR adds two async test cases to verify ARX fragment decoding functionality. Tests construct ARX-codec PayloadEnvelopes, serialize and compress them, embed results into hash fragments, and verify successful parsing with correct title content, including one test with percent-escaped non-ASCII characters.

Changes

Cohort / File(s)	Summary
ARX Fragment Decoding Tests tests/fragment.test.ts	Two new async test cases for decodeFragmentAsync API: one decodes raw ARX hash fragments, another tests percent-escaped non-ASCII character handling. Tests utilize packEnvelope, arxCompressBMP, and getActiveDictVersion() utilities to construct and encode test payloads.

Possibly related PRs

• feat: add arx codec with domain dictionary and multi-tier Unicode encoding #4: Introduces ARX async encode/decode APIs (decodeFragmentAsync, arxCompressBMP, getActiveDictVersion) that these tests directly exercise.
• Update example payload to showcase arx compression codec #12: Adds decodeFragmentAsync and ARX codec helper functions that this PR imports and tests.

Poem

🐰 Fragments decoded with flair so bright,
Async tests hopping left and right,
ARX compresses, escapes play,
Another piece of code today! ✨

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

🚥 Pre-merge checks | ✅ 3

✅ Passed checks (3 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	The title clearly and specifically describes the main change: adding regression tests for baseBMP ARX fragment decoding, which aligns with the changeset that adds two async test cases for ARX fragment decoding.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing Touches

📝 Generate docstrings

• Create stacked PR
• Commit on current branch

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Commit unit tests in branch claude/add-basebmp-arx-fragment-tests

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

[kilo-code-bot]

SUGGESTION: Misleading test name — says "sync path" but uses decodeFragmentAsync. The sync decodeFragment explicitly rejects arx codec (see src/lib/payload/fragment.ts:321).

Suggested change

	it("decodes a baseBMP-encoded arx fragment (sync path)", async () => {
	it("decodes a baseBMP-encoded arx fragment (async path)", async () => {

[kilo-code-bot]

Code Review Summary

Status: 1 Issue Found | Recommendation: Address before merge

Overview

Severity	Count
SUGGESTION	1

Issue Details (click to expand)

SUGGESTION

File	Line	Issue
tests/fragment.test.ts	185	Test name says "sync path" but uses decodeFragmentAsync — misleading name since sync decodeFragment explicitly rejects arx codec

Files Reviewed (1 files)

• tests/fragment.test.ts - 1 suggestion (misleading test name)

Notes

The two new tests look correct — they properly exercise the decodeFragmentAsync path for baseBMP-encoded arx fragments, including the percent-escaped variant. The imports and test structure follow existing conventions in the file. The only issue is the test name on line 185.

---

_{Reviewed by mimo-v2-pro-20260318 · 106,938 tokens}

[cloudflare-workers-and-pages]

Deploying agent-render with    Cloudflare Pages

Latest commit:	4288ffb
Status:	✅  Deploy successful!
Preview URL:	https://4e3e2162.agent-render.pages.dev
Branch Preview URL:	https://claude-add-basebmp-arx-fragm.agent-render.pages.dev

View logs