Update dependency zod to v4 (#15)

* Update dependency zod to v4

* remove unused eslint-disable directives

* Update zod to v4 and remove zod-to-json-schema dependency

* Update DeploymentConfig and ModelParameters to accept additional parameters

---------

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Co-authored-by: Daniel Richter <github@crftartsc.de>

Author: renovate[bot]

pr-review/package-lock.json

@@ -27,8 +27,7 @@
     "mollitia": "0.2.0",
     "octokit": "5.0.3",
     "parse-diff": "0.11.1",
-    "zod": "3.25.76",
-    "zod-to-json-schema": "3.24.6"
+    "zod": "4.0.5"
   },
   "devDependencies": {
     "@eslint/js": "9.31.0",

pr-review/package.json

@@ -3,7 +3,6 @@ import { ChatMessage, OrchestrationClient, TokenUsage } from "@sap-ai-sdk/orches
 import axios from "axios"
 import { inspect } from "node:util"
 import { z } from "zod"
-import { zodToJsonSchema } from "zod-to-json-schema"
 import { config } from "./config.js"
 
 let modelName = config.model
@@ -63,7 +62,7 @@ export async function chatCompletion(messages: ChatMessage[]): Promise<string> {
  */
 export async function chatCompletionWithJsonSchema<T extends z.ZodTypeAny>(zodSchema: T, messages: ChatMessage[]): Promise<z.infer<T>> {
   process.env.AICORE_SERVICE_KEY = JSON.stringify(config.aicoreServiceKey)
-  const jsonSchema = zodToJsonSchema(zodSchema)
+  const jsonSchema = z.toJSONSchema(zodSchema)
 
   let responseJson
   try {
@@ -102,11 +101,11 @@ export async function chatCompletionWithJsonSchema<T extends z.ZodTypeAny>(zodSc
   core.info(responseJson)
   responseJson = responseJson.slice(responseJson.indexOf("{"), responseJson.lastIndexOf("}") + 1)
   try {
-    return zodSchema.parse(JSON.parse(responseJson)) // eslint-disable-line @typescript-eslint/no-unsafe-return
+    return zodSchema.parse(JSON.parse(responseJson))
   } catch (error) {
     if (error instanceof Error) core.warning(`Failed to parse JSON. Trying to replace newlines. Error was: ${error.message}`)
     const json = responseJson.replaceAll(/"(?:[^"\\]|\\.)*"/g, matched => matched.replaceAll("\n", String.raw`\n`))
-    return zodSchema.parse(JSON.parse(json)) // eslint-disable-line @typescript-eslint/no-unsafe-return
+    return zodSchema.parse(JSON.parse(json))
   }
 }
 

pr-review/src/ai-core-client.ts

@@ -27,7 +27,7 @@ if (process.env.NODE_ENV === "development") {
 function parseInput<T extends z.ZodTypeAny>(zodSchema: T, name: string): z.infer<T> {
   const value: string = core.getInput(name)
   try {
-    return zodSchema.parse(value) // eslint-disable-line @typescript-eslint/no-unsafe-return
+    return zodSchema.parse(value)
   } catch (error) {
     if (error instanceof Error) {
       core.error(`Failed to parse input "${name}": \`${value}\``)
@@ -40,7 +40,7 @@ function parseInput<T extends z.ZodTypeAny>(zodSchema: T, name: string): z.infer
 function parseInputAsJson<T extends z.ZodTypeAny>(zodSchema: T, name: string): z.infer<T> {
   const json: string = core.getInput(name)
   try {
-    return zodSchema.parse(JSON.parse(json)) // eslint-disable-line @typescript-eslint/no-unsafe-return
+    return zodSchema.parse(JSON.parse(json))
   } catch (error) {
     if (error instanceof Error) {
       core.error(`Failed to parse input "${name}": \`${json}\``)
@@ -56,7 +56,7 @@ function parseInputAsArray<T extends z.ZodTypeAny>(zodSchema: T, name: string):
     .split(/[\n,]/)
     .map(v => v.trim())
     .filter(Boolean)
-    .map(v => zodSchema.parse(v) as z.infer<T>) // eslint-disable-line @typescript-eslint/no-unsafe-return
+    .map(v => zodSchema.parse(v) as z.infer<T>)
 }
 
 function setSecret<T>(value: T): T {

pr-review/src/config.ts

@@ -32,9 +32,9 @@ export const ModelName: ZodType<ChatModel> = z.string()
 export type ModelName = z.infer<typeof ModelName>
 
 // DeploymentConfig
-export const DeploymentConfig: ZodType<ResourceGroupConfig> = z.record(z.any())
+export const DeploymentConfig: ZodType<ResourceGroupConfig> = z.record(z.any(), z.any())
 export type DeploymentConfig = z.infer<typeof DeploymentConfig>
 
 // ModelParameters
-export const ModelParameters: ZodType<LlmModelParams> = z.record(z.any())
+export const ModelParameters: ZodType<LlmModelParams> = z.record(z.any(), z.any())
 export type ModelParameters = z.infer<typeof ModelParameters>

pr-review/src/zod-schema.ts

@@ -26,7 +26,7 @@
     "mollitia": "0.2.0",
     "octokit": "5.0.3",
     "parse-diff": "0.11.1",
-    "zod": "3.25.76"
+    "zod": "4.0.5"
   },
   "devDependencies": {
     "@eslint/js": "9.31.0",

pr-summary/package-lock.json

@@ -27,7 +27,7 @@ if (process.env.NODE_ENV === "development") {
 function parseInput<T extends z.ZodTypeAny>(zodSchema: T, name: string): z.infer<T> {
   const value: string = core.getInput(name)
   try {
-    return zodSchema.parse(value) // eslint-disable-line @typescript-eslint/no-unsafe-return
+    return zodSchema.parse(value)
   } catch (error) {
     if (error instanceof Error) {
       core.error(`Failed to parse input "${name}": \`${value}\``)
@@ -40,7 +40,7 @@ function parseInput<T extends z.ZodTypeAny>(zodSchema: T, name: string): z.infer
 function parseInputAsJson<T extends z.ZodTypeAny>(zodSchema: T, name: string): z.infer<T> {
   const json: string = core.getInput(name)
   try {
-    return zodSchema.parse(JSON.parse(json)) // eslint-disable-line @typescript-eslint/no-unsafe-return
+    return zodSchema.parse(JSON.parse(json))
   } catch (error) {
     if (error instanceof Error) {
       core.error(`Failed to parse input "${name}": \`${json}\``)
@@ -56,7 +56,7 @@ function parseInputAsArray<T extends z.ZodTypeAny>(zodSchema: T, name: string):
     .split(/[\n,]/)
     .map(v => v.trim())
     .filter(Boolean)
-    .map(v => zodSchema.parse(v) as z.infer<T>) // eslint-disable-line @typescript-eslint/no-unsafe-return
+    .map(v => zodSchema.parse(v) as z.infer<T>)
 }
 
 function setSecret<T>(value: T): T {

pr-summary/package.json

@@ -32,9 +32,9 @@ export const ModelName: ZodType<ChatModel> = z.string()
 export type ModelName = z.infer<typeof ModelName>
 
 // DeploymentConfig
-export const DeploymentConfig: ZodType<ResourceGroupConfig> = z.record(z.any())
+export const DeploymentConfig: ZodType<ResourceGroupConfig> = z.record(z.any(), z.any())
 export type DeploymentConfig = z.infer<typeof DeploymentConfig>
 
 // ModelParameters
-export const ModelParameters: ZodType<LlmModelParams> = z.record(z.any())
+export const ModelParameters: ZodType<LlmModelParams> = z.record(z.any(), z.any())
 export type ModelParameters = z.infer<typeof ModelParameters>