7b:I[154500,[["$2",["$3","$4"],[14451,1389]],["$5",["$6","$7","$8","$9","$a","$b"],[40076,28925,10525,30632,7744,72054]]],"IconMark"] 78:T6ff,Fixes #196. I moved the common default-bearing schema outside the identifier alternatives for five DataForSEO endpoints. Omitting depth/limit now materializes the existing documented default before estimation and before sending the request, instead of failing with FN_CONTRACT. The original identifier union still enforces at least one selector. This restores the connector's existing D5 contract: optional depth/limit, with the vendor default applied on the wire. Tests compare omitted and explicit defaults, capture outgoing defaulted and overridden values, and retain the missing-identifier checks. Schema-inspection helpers now traverse the resulting allOf as well as anyOf. Validation (Deno 2.9.7): - New regression fails before the fix. DataForSEO suite: 22 passed, 0 failed, 2 gated live tests ignored. - All 251 test modules imported into one temporary shared-process entry point: 1,299 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** * Business review, domain technology, and shopping seller requests now consistently apply default values while still requiring the relevant search identifier. * When optional depth or limit values are omitted, charge estimates and submitted requests now include their defaults. Explicit values remain unchanged, and caller-provided input is preserved. 79:T39ad,

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**: `97fd1ec0-598f-42c4-b356-85023b63f4c2`

Commits

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

Files selected for processing: 6

* `connectors/dataforseo/endpoints/business/google-business-extended-reviews/endpoint.ts` * `connectors/dataforseo/endpoints/business/google-business-reviews/endpoint.ts` * `connectors/dataforseo/endpoints/business/tripadvisor-reviews/endpoint.ts` * `connectors/dataforseo/endpoints/domain/domain-domains-by-technology/endpoint.ts` * `connectors/dataforseo/endpoints/merchant/google-shopping-sellers/endpoint.ts` * `connectors/dataforseo/provider.test.ts`

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

Walkthrough

Five DataForSEO selector schemas now apply shared defaults outside their identifier unions. The identifier requirements remain. Provider tests cover default values in estimates and submitted request bodies, explicit values, unchanged caller input, and a terminal vendor error after one request. Schema\-property discovery and the identifier\-gate test now account for nested `allOf`.

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

Merge Risk: Minimal · up to fa025

No actionable issue remains from this review; the change is ready for normal merge 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: moving default materialization outside selector unions for DataForSEO endpoints. | | 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 `#196` requires that omitting the documented default for five DataForSEO selector endpoints no longer fails estimation with FN\_CONTRACT. The diff changes each of the five endpoint.ts body schemas … | | 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) | All changes map to `#196`. The five endpoint.ts files change only the body schema wrapper and its comment. provider.test.ts changes are the regression test and two helper updates. propsOf and the 'at le… |

Full details: Linked Issues check

**Explanation** Issue `#196` requires that omitting the documented default for five DataForSEO selector endpoints no longer fails estimation with FN\_CONTRACT. The diff changes each of the five endpoint.ts body schemas from z.union([...]) to zBody.and(z.union([...])). zBody is the local schema with the vendor default (depth 10 for google\-business/reviews and tripadvisor/reviews, depth 20 for google\-business/extended\-reviews, depth 10 for google\-shopping/sellers, limit 100 for domain/domains\-by\-technology). Defaults now apply before the identifier union is checked. I confirmed zBody and its default in google\-business/reviews/endpoint.ts and domain\-domains\-by\-technology/endpoint.ts at the head. The identifier union remains in the intersection, so missing\-identifier rejection is kept. provider.test.ts adds a regression test. It checks omitted versus explicit estimates, wire defaults for omitted values, explicit overrides, and that caller input is not mutated. Per the PR, the test failed before the fix and passes after it. I did not run the test suite. The zod intersection output merge behavior for defaults is supported by the reported test results, not by a run in this review.

Full details: Out of Scope Changes check

**Explanation** All changes map to `#196`. The five endpoint.ts files change only the body schema wrapper and its comment. provider.test.ts changes are the regression test and two helper updates. propsOf and the 'at least one of' gate now traverse allOf as well as anyOf, which the new body shape requires. No unrelated endpoints, version files, or runtime modules appear in the whole\-PR diff against the merge base.

- [ ] 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":198}],"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:30Z","headRefName":"unknown repository","id":"4957554","isDraft":false,"labels":[],"number":198,"repository":{"name":"monid","owner":"monid-ai","provider":"forgejo"},"state":"open","title":"fix(dataforseo): materialize defaults outside selector unions","updatedAt":"2026-10-25T02:14:27Z","url":"https://trylle.com/monid-ai/monid/pulls/198","body":"$78","additions":79,"deletions":16,"changedFiles":6,"commits":1,"reviewComments":1,"headSha":"fa0254982be84c996a1a9975da1265ca7d875ae4","baseSha":"09d54479142e36eff876383beacfbd37b33b5908","headRepoOwner":"monid-ai","headRepoName":"monid","isMerged":false,"mergeCommitSha":null,"mergedBy":null,"mergeable":true,"mergeableState":"clean","conflictingFiles":null,"requestedTeams":[],"requestedReviewers":[],"graphqlId":"issue:198"},"comments":[{"id":27021346,"graphqlId":"27021346","body":"$79","createdAt":"2026-10-25T02:10:45Z","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":"fa0254982be84c996a1a9975da1265ca7d875ae4","message":"fix(dataforseo): materialize defaults outside selector unions\n","createdAt":"2026-10-25T02:10:20Z","author":{"avatarUrl":"","login":"","name":"sdivyanshu90","type":"User","url":""},"verification":{"verified":false,"reason":"gpg.error.not_signed_commit"}}],"events":[],"reviews":[{"id":3636307,"state":"REQUEST_REVIEW","author":{"avatarUrl":"https://cdn.trylle.com/media/images/1ec5c00f9995b5dc5ca35af1a0870887?key=AaIaiXNENNoLHWYsNT9HhOMffcuSHcUM0aNJr7rpnJJaybk8kOBUO0z9tl87uDS%2BldW83peTE%2B%2BpbIdbg6scl4vuXQQlVZvucTGYTNh7Bh99jjJ%2FK6zS2g9Vwtc%2B02qF4HsZBMLgEqny6b%2B7nWvpfnkQrM5bdVZ3uRDt4yIY2KCKwuHYv5ypOCsoz6I%3D","externalProvider":"github","login":"ooctoo777","type":"User","url":"https://github.com/ooctoo777"},"submittedAt":"2026-10-25T02:10:30Z","official":false}],"stackEvents":[],"commentPagination":{"totalCount":2,"perPage":30,"loadedPages":[1]},"eventPagination":{"loadedPages":[1],"hasMore":false},"headRefDeleted":false},"fetchedAt":1793524410997}],"children":"$L7a"}] 64:null 6b:[["$","title","0",{"children":"Commits · fix(dataforseo): materialize defaults outside selector unions · monid-ai/monid | Trylle"}],["$","meta","1",{"name":"description","content":"Commits on fix(dataforseo): materialize defaults outside selector unions (#198) 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(dataforseo): materialize defaults outside selector unions · 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/198/commits"}],["$","link","10",{"rel":"canonical","href":"https://trylle.com/monid-ai/monid/pulls/198/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(dataforseo): materialize defaults outside selector unions · monid-ai/monid | Trylle"}],["$","meta","16",{"property":"og:description","content":"Commits on fix(dataforseo): materialize defaults outside selector unions (#198) in monid-ai/monid."}],["$","meta","17",{"property":"og:url","content":"https://trylle.com/monid-ai/monid/pulls/198/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(dataforseo): materialize defaults outside selector unions · monid-ai/monid | Trylle"}],["$","meta","30",{"name":"twitter:description","content":"Commits on fix(dataforseo): materialize defaults outside selector unions (#198) 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