test(desktop): remove redundant release note assertion

[t3-code]

the out-of-range numeric entity test invoked normalizeDesktopUpdateReleaseNotes twice with identical inputs. the first invocation only asserted that it did not throw, while the second asserted the exact returned release-note structure.

this removes the redundant not.toThrow() invocation. the remaining exact-value assertion necessarily fails if the same call throws and still verifies that the invalid entity is preserved, so behavior coverage is unchanged.

verification:

• before: ./node_modules/.bin/vp test run apps/desktop/src/updates/releaseNotes.test.ts (1 file passed, 6 tests passed)
• after: ./node_modules/.bin/vp test run apps/desktop/src/updates/releaseNotes.test.ts (1 file passed, 6 tests passed)
• ./node_modules/.bin/vp run --filter @t3tools/desktop typecheck (passed with 2 pre-existing effect suggestions)
• ./node_modules/.bin/vp lint --report-unused-disable-directives apps/desktop/src/updates/releaseNotes.test.ts (passed)
• ./node_modules/.bin/vp fmt --check apps/desktop/src/updates/releaseNotes.test.ts (passed)
• git diff --check (passed)

made by gpt-5.6-sol via hermes agent.

---

Note

Low Risk
Test-only cleanup with no production or security impact.

Overview
Removes a duplicate not.toThrow() call in the out-of-range HTML entity test for normalizeDesktopUpdateReleaseNotes. The remaining equality assertion still fails if the function throws and still checks that the invalid entity is kept as a literal.

^{Reviewed by Cursor Bugbot for commit fdccd40. Bugbot is set up for automated code reviews on this repo. Configure here.}

Note

Remove redundant not.toThrow assertion from normalizeDesktopUpdateReleaseNotes test

Removes the explicit expect(...).not.toThrow() assertion from the out-of-range numeric entities test in releaseNotes.test.ts. The test still verifies the returned notes content, which implicitly confirms no exception is thrown.

^{Macroscope summarized fdccd40.}

[macroscopeapp]

Approvability

Verdict: Approved at fdccd40

Macroscope's review found this PR approvable — Test-only change removing a redundant assertion. The subsequent line in the same test already validates the function doesn't throw by calling it and checking its return value. No runtime impact.

Notes:

• No code objects were reviewed. Approvability was decided on eligibility alone.

^{You can add or adjust custom eligibility rules. Learn more.}

[github-actions]

Thread transfer impact

✅ Thread transfer remains within every enforced ceiling.

Provider	Metric	Main baseline	This PR	Impact	PR ceiling
Codex	Total thread wire	13.4 KiB	13.4 KiB	+10 B (+0.1%)	15.1 KiB	✅
Codex	Thread snapshot wire	6.9 KiB	6.9 KiB	−4 B (−0.1%)	7.3 KiB	✅
Codex	Live turn WebSocket wire	6.5 KiB	6.6 KiB	+14 B (+0.2%)	7.8 KiB	✅
Codex	Live turn WebSocket decoded	55.0 KiB	55.0 KiB	0 B (0.0%)	66.4 KiB	✅
Codex	Live turn messages	16	16	0 (0.0%)	21	✅
Claude	Total thread wire	13.4 KiB	13.5 KiB	+17 B (+0.1%)	15.1 KiB	✅
Claude	Thread snapshot wire	6.9 KiB	6.9 KiB	−1 B (−0.0%)	7.3 KiB	✅
Claude	Live turn WebSocket wire	6.5 KiB	6.6 KiB	+18 B (+0.3%)	7.8 KiB	✅
Claude	Live turn WebSocket decoded	55.8 KiB	55.8 KiB	0 B (0.0%)	66.4 KiB	✅
Claude	Live turn messages	16	16	0 (0.0%)	21	✅

Baseline: 11f0513 · PR result: fdccd40 · Source CI: success

Scenario and decoded snapshot size

10 historical turns, 5 command tools per turn, 878.9 KiB retained MCP result per historical turn, and a 1.05 MiB retained result in the measured turn.

• Codex decoded thread snapshot: 109.4 KiB
• Claude decoded thread snapshot: 110.1 KiB

Updated in place by a trusted workflow. PR artifacts are strictly validated and never executed.