7b:I[154500,[["$2",["$3","$4"],[14451,1389]],["$5",["$6","$7","$8","$9","$a","$b"],[40076,28925,10525,30632,7744,72054]]],"IconMark"] 78:T843,Fixes #104. ## What changes At settle, the engine picked the consolidated output with nullish coalescing: ```ts output = consolidated.output ?? raw; // engine/engine.ts:918 ``` `Consolidated.output` is `zJson.optional()` — `null` is valid JSON and a valid consolidated result, and the hook contract (shared/core/schema/hooks/usage-consolidate.ts) says "absent = unchanged". Nullish coalescing treats explicit `null` as absent, so a hook returning `{ credits: {...}, output: null }` had its result overwritten by the raw response — re-exposing exactly the vendor billing fields the hook stripped. The settle path now falls back to raw only when `consolidated.output` is `undefined`: ```ts output = consolidated.output === undefined ? raw : consolidated.output; ``` ## Verification No deno on this host, so: - HEAD-vs-fix semantics of the exact expression, Node 26: with `consolidated = { credits: {}, output: null }` and `raw = { results: [...], vendorTotal: 0 }`, HEAD produces `raw`, the fix produces `null`; with `output` omitted (undefined), both produce `raw` — the "absent = unchanged" case is preserved. - Grep over `engine/engine.ts`: line 918 was the only `?? raw` / consolidated-output fallback; the new `fromResponse` and `doc.output.schema` steps downstream are untouched and both already accept `null` (free-form outputs; `zJson` includes `null`). - Added a `Deno.test` regression in `engine/engine.test.ts` next to the empty-claim test, modeled on the same `usageUnit` harness: explicit `{ credits: {}, output: null }` settles `output === null`, and an omitted output still passes the raw payload through. I could not run `deno test` locally; CI is the authoritative run. ## Summary by CodeRabbit * **Bug Fixes** * Explicit `null` outputs from consolidation are now preserved, while omitted outputs continue to use the raw payload. * Usage is still settled when the consolidation claim is empty. 79:T31ce,

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**: `183b9f11-387c-410f-ab8b-2748d3dbebd4`

Commits

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

Files selected for processing: 2

* `engine/engine.test.ts` * `engine/engine.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

`LoadedEndpoint.settle` now uses the raw payload only when `usage.consolidate` returns `output: undefined`. Explicit `null` remains the settled output. Tests cover both cases and verify derived usage.

Priority: Low **Severity of issue fixed:** Low

Merge Risk: Minimal · up to efa08

The change implements the requested output behavior, with no identified issue blocking merge after normal checks.

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 the main change: preserving an explicit null consolidated output instead of falling back to the raw payload. | | 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 `#104` requires explicit `null` to remain the consolidated output and only `undefined` or an omitted `output` to preserve the raw payload. `engine/engine.ts` now uses an `undefined` check. \`engine… | | 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 pull request changes only the settle fallback and adds a regression test for issue `#104`. Both changes directly support the linked issue. No unrelated change is shown. |

Full details: Linked Issues check

**Explanation** Issue `#104` requires explicit `null` to remain the consolidated output and only `undefined` or an omitted `output` to preserve the raw payload. `engine/engine.ts` now uses an `undefined` check. `engine/engine.test.ts` covers both explicit `null` and omitted output.

- [ ] 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":208}],"data":{"repositoryId":"4088","detail":{"author":{"avatarUrl":"https://cdn.trylle.com/media/images/a6ff5eb12756bacea455c0ab4f75cdfd?key=Ad1tdHn%2FgFWR%2B1WnbDprk3t01rnDOo4EGMa8w%2Bl4RH7OtfiSYpqxN6GiZKvbjDopdhv95VV5BQDpTW%2F4Zbz%2FhlmvtMbO%2FirMo1AAtGY8OYTGJJZWeKUqrvVaMPcSFmcIHZG8116Ip4FqSbWBnOvQebWgv3I4oBo36cybnpf8gbYd1vfc%2B6kuVhTvaw%3D%3D","externalProvider":"github","login":"rahul05ranjan","type":"User","url":"https://github.com/rahul05ranjan"},"authorId":"11793455","baseRefName":"main","closedAt":null,"comments":2,"createdAt":"2026-10-27T11:31:42Z","headRefName":"rahul05ranjan/fix/consolidate-null-output","id":"4957564","isDraft":false,"labels":[],"number":208,"repository":{"name":"monid","owner":"monid-ai","provider":"forgejo"},"state":"open","title":"fix(engine): keep an explicit null consolidated output instead of raw","updatedAt":"2026-10-27T12:24:58Z","url":"https://trylle.com/monid-ai/monid/pulls/208","body":"$78","additions":65,"deletions":1,"changedFiles":2,"commits":1,"reviewComments":1,"headSha":"efa08da1e0319bfaac16882026888ead7ac77e20","baseSha":"09d54479142e36eff876383beacfbd37b33b5908","headRepoOwner":"monid-ai","headRepoName":"monid","isMerged":false,"mergeCommitSha":null,"mergedBy":null,"mergeable":true,"mergeableState":"clean","conflictingFiles":null,"requestedTeams":[],"requestedReviewers":[],"graphqlId":"issue:208"},"comments":[{"id":27021359,"graphqlId":"27021359","body":"$79","createdAt":"2026-10-27T11:31:57Z","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":"efa08da1e0319bfaac16882026888ead7ac77e20","message":"fix(engine): keep an explicit null consolidated output instead of raw\n\nThe settle path used nullish coalescing when choosing between the\nconsolidated output and the raw response, so a usage.consolidate hook\nreturning {credits: ..., output: null} had its null overwritten by the\nraw vendor payload, re-exposing the billing fields the hook stripped.\nOnly an absent output means the raw payload stays; null is valid JSON\nand now stands as the consolidated result.\n\nFixes #104\n\nCo-authored-by: rahul05ranjan <11793455+rahul05ranjan@users.noreply.github.com>\n","createdAt":"2026-10-27T17:00:59+05:30","author":{"avatarUrl":"","login":"","name":"Rahul Ranjan","type":"User","url":""},"verification":{"verified":false,"reason":"gpg.error.not_signed_commit"}}],"events":[],"reviews":[{"id":3636344,"state":"REQUEST_REVIEW","author":{"avatarUrl":"https://cdn.trylle.com/media/images/12b8c277369bb3562bbfe696be942cb5?key=AUBgRFSvC4XfDv%2Fm1ot99RR4%2F%2B6PmiQS8IlXiIddcy4NirDjiJs9uIzh2RejXXbXtISsabXvAENNopKyJZkuJIYFIbCxLlpbp3%2F0NHKyXKYBqIFBEFqC35hIn4Lc5GViwObIi89lLx5fET0dwu3ga01cWDPapSbzF3Faog0fnUP19zZcbVltdHMkpA%3D%3D","externalProvider":"github","login":"ZqyDaoDao","type":"User","url":"https://github.com/ZqyDaoDao"},"submittedAt":"2026-10-27T11:31:42Z","official":false}],"stackEvents":[],"commentPagination":{"totalCount":2,"perPage":30,"loadedPages":[1]},"eventPagination":{"loadedPages":[1],"hasMore":false},"headRefDeleted":false},"fetchedAt":1793524401744}],"children":"$L7a"}] 64:null 6b:[["$","title","0",{"children":"Commits · fix(engine): keep an explicit null consolidated output instead of raw · monid-ai/monid | Trylle"}],["$","meta","1",{"name":"description","content":"Commits on fix(engine): keep an explicit null consolidated output instead of raw (#208) 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(engine): keep an explicit null consolidated output instead of raw · 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/208/commits"}],["$","link","10",{"rel":"canonical","href":"https://trylle.com/monid-ai/monid/pulls/208/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(engine): keep an explicit null consolidated output instead of raw · monid-ai/monid | Trylle"}],["$","meta","16",{"property":"og:description","content":"Commits on fix(engine): keep an explicit null consolidated output instead of raw (#208) in monid-ai/monid."}],["$","meta","17",{"property":"og:url","content":"https://trylle.com/monid-ai/monid/pulls/208/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(engine): keep an explicit null consolidated output instead of raw · monid-ai/monid | Trylle"}],["$","meta","30",{"name":"twitter:description","content":"Commits on fix(engine): keep an explicit null consolidated output instead of raw (#208) 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 Tue, Oct 27, 2026