7b:I[154500,[["$2",["$3","$4"],[14451,1389]],["$5",["$6","$7","$8","$9","$a","$b"],[40076,28925,10525,30632,7744,72054]]],"IconMark"] 78:T5ee,Fixes #197. I preserved an absent response body when fixture recording and replay reconstruct HTTP 204, 205, or 304. Previously, replay serialized null as a body and recording passed an empty string; both threw in the Response constructor before the connector could inspect the status. Regression coverage exercises all three statuses, null and empty fixture representations, record→replay, status/header preservation, and credential-header filtering. A 200 JSON-null response remains a payload. No fixture schema or transport ABI changes are needed. Validation (Deno 2.9.7): - Both no-content regressions fail before the fix. Fixture suite: 13 passed, 0 failed. - All 251 test modules imported into one temporary shared-process entry point: 1,301 passed, 0 failed, 221 gated live tests ignored. This is not a standard isolated-module `deno task test` run. - `deno task check`, `deno task lint`, changed-file `deno fmt --check`, `git diff --check`, and `deno task version:check --base upstream/main` pass. @ooctoo777, could you review and merge this fix? ## Summary by CodeRabbit * **Bug Fixes** * Recorded and replayed responses with status codes 204, 205, and 304 now correctly have no body, including when fixtures contain a null or empty body. * Responses with status 200 continue to preserve JSON `null` as a body. 79:T33c8,

No actionable comments were generated in the recent review.

Recent review info

Run configuration

- **Source mode**: Auto - **Repository**: [monid\-ai/monid](https://github.com/monid-ai/monid) - **Branch**: [main](https://github.com/monid-ai/monid/tree/main) - **File**: [.coderabbit.yaml](https://github.com/monid-ai/monid/blob/09d54479142e36eff876383beacfbd37b33b5908/.coderabbit.yaml) - **Commit**: [09d5447](https://github.com/monid-ai/monid/commit/09d54479142e36eff876383beacfbd37b33b5908) - **Layers**: Repository YAML - **Review profile**: CHILL - **Plan**: Advanced - **Run ID**: `ad100ee5-f3ff-4b3d-9f96-15153aaf7ce5`

Commits

Reviewing files that changed from the base of the PR and between 09d54479142e36eff876383beacfbd37b33b5908 and b4e9754ba34c1ff9f2ff8e590cb161ac2463c5f6.

Files selected for processing: 2

* `shared/testing/fixture-size.test.ts` * `shared/testing/fixtures.ts`

**Included review availability:** This review used your included allowance. Your plan provides up to 8 included reviews per hour; 6 remain after this review.

Walkthrough

`replayFetch` and `recordingFetch` now construct responses with null bodies for statuses 204, 205, and 304. Tests verify status and header handling, empty response bodies, and that a JSON `null` payload remains a body for status 200.

Priority: Normal **Severity of issue fixed:** Medium

Merge Risk: Minimal · up to b4e97

The fixture change appears ready to merge after normal checks; no actionable remaining risk was identified.

Pre-merge checks: 4

| Check name | Status | Explanation | |:-------------------------- |:----------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------:|:------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------ | | Description Check | ![Passed](https://cdn.trylle.com/media/images/9b9f7fc0af96c9889ff30dd8dc8752bd?key=Aec7YlhhL%2FWojK8gn54mkt9KD7Uvp%2B14YdVeU%2F7AOnbnO8x76NKhQn886i%2FCQT5dcj7%2FiiBjzzzUypazN2QvWoTYfhbCW723SX%2B0ZlbFieBXi1YY7J7gIBXHpKWTAj2Q4%2BCa4dgdBnn8d3HEdUGlUW2Xi%2BXlSDbHrrQTK5zYO2kQgyBp3%2BIaYZB2v57Hd07YU35qEgQV377f%2FXo0E72gVN4H6btm4nsvMaw%3D) | Check skipped \- CodeRabbit’s high\-level summary is enabled. | | Title check | ![Passed](https://cdn.trylle.com/media/images/9b9f7fc0af96c9889ff30dd8dc8752bd?key=Aec7YlhhL%2FWojK8gn54mkt9KD7Uvp%2B14YdVeU%2F7AOnbnO8x76NKhQn886i%2FCQT5dcj7%2FiiBjzzzUypazN2QvWoTYfhbCW723SX%2B0ZlbFieBXi1YY7J7gIBXHpKWTAj2Q4%2BCa4dgdBnn8d3HEdUGlUW2Xi%2BXlSDbHrrQTK5zYO2kQgyBp3%2BIaYZB2v57Hd07YU35qEgQV377f%2FXo0E72gVN4H6btm4nsvMaw%3D) | The title clearly and concisely describes preserving no\-content responses in the fixture helpers, which is the main change. | | Linked Issues check | ![Passed](https://cdn.trylle.com/media/images/9b9f7fc0af96c9889ff30dd8dc8752bd?key=Aec7YlhhL%2FWojK8gn54mkt9KD7Uvp%2B14YdVeU%2F7AOnbnO8x76NKhQn886i%2FCQT5dcj7%2FiiBjzzzUypazN2QvWoTYfhbCW723SX%2B0ZlbFieBXi1YY7J7gIBXHpKWTAj2Q4%2BCa4dgdBnn8d3HEdUGlUW2Xi%2BXlSDbHrrQTK5zYO2kQgyBp3%2BIaYZB2v57Hd07YU35qEgQV377f%2FXo0E72gVN4H6btm4nsvMaw%3D) | Issue `#197` requires bodyless reconstruction for statuses 204, 205, and 304. `replayFetch` now passes `null` for those statuses, including fixtures with `null` and empty\-string bodies. `recordingFetch`… | | Out of Scope Changes check | ![Passed](https://cdn.trylle.com/media/images/9b9f7fc0af96c9889ff30dd8dc8752bd?key=Aec7YlhhL%2FWojK8gn54mkt9KD7Uvp%2B14YdVeU%2F7AOnbnO8x76NKhQn886i%2FCQT5dcj7%2FiiBjzzzUypazN2QvWoTYfhbCW723SX%2B0ZlbFieBXi1YY7J7gIBXHpKWTAj2Q4%2BCa4dgdBnn8d3HEdUGlUW2Xi%2BXlSDbHrrQTK5zYO2kQgyBp3%2BIaYZB2v57Hd07YU35qEgQV377f%2FXo0E72gVN4H6btm4nsvMaw%3D) | The diff changes only fixture reconstruction and its regression tests. The changes preserve existing headers and ordinary response\-body behavior. No binary or multipart transport work from `#170` is inc… |

Full details: Linked Issues check

**Explanation** Issue `#197` requires bodyless reconstruction for statuses 204, 205, and 304. `replayFetch` now passes `null` for those statuses, including fixtures with `null` and empty\-string bodies. `recordingFetch` also passes `null` for those statuses. The tests cover all three statuses, record\-to\-replay, allowlisted headers, credential\-header filtering, and a status\-200 JSON `null` payload.

Full details: Out of Scope Changes check

**Explanation** The diff changes only fixture reconstruction and its regression tests. The changes preserve existing headers and ordinary response\-body behavior. No binary or multipart transport work from `#170` is included.

- [ ] Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts _{Comment `@coderabbitai help` to get the list of available commands.} 62:["$","$L75",null,{"entries":[{"key":["platform",{"userId":"anon"},"pulls","page",{"owner":"monid-ai","repo":"monid","pullNumber":199}],"data":{"repositoryId":"4088","detail":{"author":{"avatarUrl":"https://cdn.trylle.com/media/images/1bc83579e339c3c72f090fe78c25dd78?key=Abjs%2F6AbU9R%2FuuoD5B9F0nUSTbaZOboxvZZ8KUlf9sIRNedcr9I01AG7woYuwGIYVA9WWyU28%2B1LtQDAtm2kiDcU%2B6RfueNob%2FO7xPcwmlI%2B%2BhnGBUSCPdf9L3uIOAitz%2FZPiayIOiR7PtjVf4TSCQenAAKvD74HDvwLFRaJI9axpfeDOiDwadsZOQ%3D%3D","externalProvider":"github","login":"sdivyanshu90","type":"User","url":"https://github.com/sdivyanshu90"},"authorId":"97019230","baseRefName":"main","closedAt":null,"comments":2,"createdAt":"2026-10-25T02:10:38Z","headRefName":"unknown repository","id":"4957555","isDraft":false,"labels":[],"number":199,"repository":{"name":"monid","owner":"monid-ai","provider":"forgejo"},"state":"open","title":"fix(testing): preserve no-content responses in fixture helpers","updatedAt":"2026-10-25T02:13:07Z","url":"https://trylle.com/monid-ai/monid/pulls/199","body":"$78","additions":87,"deletions":13,"changedFiles":2,"commits":1,"reviewComments":1,"headSha":"b4e9754ba34c1ff9f2ff8e590cb161ac2463c5f6","baseSha":"09d54479142e36eff876383beacfbd37b33b5908","headRepoOwner":"monid-ai","headRepoName":"monid","isMerged":false,"mergeCommitSha":null,"mergedBy":null,"mergeable":true,"mergeableState":"clean","conflictingFiles":null,"requestedTeams":[],"requestedReviewers":[],"graphqlId":"issue:199"},"comments":[{"id":27021347,"graphqlId":"27021347","body":"$79","createdAt":"2026-10-25T02:10:53Z","author":{"avatarUrl":"https://cdn.trylle.com/media/images/d39e0f17743c08d7bf625ba6f80c545a?key=Ac3dCIMfeEeTRToXUBeRdRfe7IGnH7FaknGcutnXETLNIDauVDMEeYfL%2BNJ1UW6PltLPYQ8AAbXVYTqKXZ2sbxmdR%2FJeAINpLPz%2Fgbhi3if42haJXKHU1rlVIjj4bCQzj6%2FWYhkNT360oU7wzsBjYZsV7FyaaYuuIlrb3O4TqxAm%2FsfmH46nFvfFacg%3D","externalProvider":"github","login":"coderabbitai[bot]","type":"Bot","url":"https://github.com/apps/coderabbitai"}}],"commits":[{"sha":"b4e9754ba34c1ff9f2ff8e590cb161ac2463c5f6","message":"fix(testing): preserve no-content response bodies in fixtures\n","createdAt":"2026-10-25T02:10:21Z","author":{"avatarUrl":"","login":"","name":"sdivyanshu90","type":"User","url":""},"verification":{"verified":false,"reason":"gpg.error.not_signed_commit"}}],"events":[],"reviews":[{"id":3636308,"state":"REQUEST_REVIEW","author":{"avatarUrl":"https://cdn.trylle.com/media/images/2cd867bf4a6c51c4b46d0a1c4c6b5793?key=AWgbe6xZjf4Epnp6zGHN9zRuDhM%2FVA59uk4iuqOyekcAo%2BhxFlwbfBmcuxiWoQafcBuEl6LS5SUy4FexgeYp%2FusZv8EiOdVjG5LaLAFlUhn4H1V%2BHPeA9zDgg0vhWtTOgGYIxVx7gmZ5g%2BQIVWyPYh2oYvrr5jLSU%2FwWWghstlr6qYn4bCtPugOGhQ%3D%3D","externalProvider":"github","login":"FeiyouG","type":"User","url":"https://github.com/FeiyouG"},"submittedAt":"2026-10-25T02:10:38Z","official":false}],"stackEvents":[],"commentPagination":{"totalCount":2,"perPage":30,"loadedPages":[1]},"eventPagination":{"loadedPages":[1],"hasMore":false},"headRefDeleted":false},"fetchedAt":1793524372174}],"children":"$L7a"}] 64:null 6b:[["$","title","0",{"children":"Commits · fix(testing): preserve no-content responses in fixture helpers · monid-ai/monid | Trylle"}],["$","meta","1",{"name":"description","content":"Commits on fix(testing): preserve no-content responses in fixture helpers (#199) in monid-ai/monid."}],["$","meta","2",{"name":"application-name","content":"Trylle"}],["$","link","3",{"rel":"manifest","href":"/manifest.json","crossOrigin":"$undefined"}],["$","meta","4",{"name":"keywords","content":"Commits · fix(testing): preserve no-content responses in fixture helpers · monid-ai/monid, monid-ai/monid, monid-ai/monid repository, monid, TypeScript repository, source code, trylle, agent-native git platform, git platform for agents and humans, git platform for ai agents, coding agent pull requests, ai code review, repository automation, stacked pull requests, two-way github sync, github-compatible actions, try cli"}],["$","meta","5",{"name":"robots","content":"index, follow, max-image-preview:large, max-snippet:-1, max-video-preview:-1"}],["$","meta","6",{"name":"mobile-web-app-capable","content":"yes"}],["$","meta","7",{"name":"googlebot","content":"index, follow, max-image-preview:large, max-snippet:-1, max-video-preview:-1"}],["$","meta","8",{"name":"twitter:domain","content":"trylle.com"}],["$","meta","9",{"name":"twitter:url","content":"https://trylle.com/monid-ai/monid/pulls/199/commits"}],["$","link","10",{"rel":"canonical","href":"https://trylle.com/monid-ai/monid/pulls/199/commits"}],["$","meta","11",{"name":"format-detection","content":"telephone=no"}],["$","meta","12",{"name":"mobile-web-app-capable","content":"yes"}],["$","meta","13",{"name":"apple-mobile-web-app-title","content":"Trylle"}],["$","meta","14",{"name":"apple-mobile-web-app-status-bar-style","content":"black-translucent"}],["$","meta","15",{"property":"og:title","content":"Commits · fix(testing): preserve no-content responses in fixture helpers · monid-ai/monid | Trylle"}],["$","meta","16",{"property":"og:description","content":"Commits on fix(testing): preserve no-content responses in fixture helpers (#199) in monid-ai/monid."}],["$","meta","17",{"property":"og:url","content":"https://trylle.com/monid-ai/monid/pulls/199/commits"}],["$","meta","18",{"property":"og:site_name","content":"Trylle"}],["$","meta","19",{"property":"og:locale","content":"en_US"}],["$","meta","20",{"property":"og:image","content":"https://trylle.com/api/og.png?kind=repo&owner=monid-ai&repo=monid"}],["$","meta","21",{"property":"og:image:secure_url","content":"https://trylle.com/api/og.png?kind=repo&owner=monid-ai&repo=monid"}],["$","meta","22",{"property":"og:image:type","content":"image/png"}],["$","meta","23",{"property":"og:image:width","content":"1200"}],["$","meta","24",{"property":"og:image:height","content":"630"}],["$","meta","25",{"property":"og:image:alt","content":"monid-ai/monid repository preview on Trylle"}],["$","meta","26",{"property":"og:type","content":"website"}],["$","meta","27",{"name":"twitter:card","content":"summary_large_image"}],["$","meta","28",{"name":"twitter:site","content":"@trylle"}],["$","meta","29",{"name":"twitter:title","content":"Commits · fix(testing): preserve no-content responses in fixture helpers · monid-ai/monid | Trylle"}],["$","meta","30",{"name":"twitter:description","content":"Commits on fix(testing): preserve no-content responses in fixture helpers (#199) in monid-ai/monid."}],["$","meta","31",{"name":"twitter:image","content":"https://trylle.com/api/og.png?kind=repo&owner=monid-ai&repo=monid"}],["$","meta","32",{"name":"twitter:image:alt","content":"monid-ai/monid repository preview on Trylle"}],["$","link","33",{"rel":"icon","href":"/favicon.png","type":"image/png"}],["$","link","34",{"rel":"icon","href":"/favicon-white.png","type":"image/png","media":"(prefers-color-scheme: dark)"}],["$","link","35",{"rel":"apple-touch-icon","href":"/logo192.png"}],["$","$L7b","36",{}]]

Commits on Sun, Oct 25, 2026