zcb 0.3.4 → 0.3.6

package/package.json

@@ -1,24 +1,28 @@
 {
   "name": "zcb",
   "description": "Build configs with type safety from zod schema.",
-  "version": "0.3.4",
+  "version": "0.3.6",
   "author": "miami-man",
   "license": "MIT",
   "homepage": "https://github.com/badbatch/zod-config-builder",
-  "repository": "badbatch/zod-config-builder",
+  "repository": {
+    "type": "git",
+    "url": "badbatch/zod-config-builder"
+  },
   "bugs": "https://github.com/badbatch/zod-config-builder/issues",
   "type": "module",
   "bin": {
     "zcb": "./bin/zcb.js"
   },
-  "main": "./dist/cjs/index.cjs",
+  "main": "./dist/cjs/index.mjs",
   "module": "./dist/esm/index.mjs",
-  "types": "./dist/types/cjs/index.d.cts",
+  "types": "./dist/types/cjs/index.d.ts",
   "imports": {
     "#*": "./src/*"
   },
   "exports": {
     ".": {
+      "source": "./src/index.ts",
       "types": {
         "import": "./dist/types/esm/index.d.ts",
         "require": "./dist/types/cjs/index.d.cts"
@@ -27,6 +31,7 @@
       "require": "./dist/cjs/index.cjs"
     },
     "./cli": {
+      "source": "./src/cli.ts",
       "types": {
         "import": "./dist/types/esm/cli.d.ts",
         "require": "./dist/types/cjs/cli.d.cts"
@@ -42,56 +47,55 @@
     "@types/fs-extra": "^11.0.4",
     "@types/json-schema": "^7.0.15",
     "@types/lodash-es": "^4.17.12",
-    "@types/shelljs": "^0.8.17",
-    "@types/uuid": "^10.0.0",
-    "@types/yargs": "^17.0.33",
-    "core-js": "^3.45.0",
-    "fs-extra": "^11.3.1",
-    "handlebars": "^4.7.8",
+    "@types/shelljs": "^0.10.0",
+    "@types/uuid": "^11.0.0",
+    "@types/yargs": "^17.0.35",
+    "core-js": "^3.49.0",
+    "fs-extra": "^11.4.0",
+    "handlebars": "^4.7.9",
     "json-schema": "^0.4.0",
-    "lodash-es": "^4.17.21",
+    "lodash-es": "^4.18.1",
     "shelljs": "^0.10.0",
     "ts-node": "^10.9.2",
     "ts-toolbelt": "^9.6.0",
-    "type-fest": "^4.41.0",
-    "uuid": "^11.1.0",
-    "yargs": "^18.0.0",
-    "zod": "^3.25.76",
-    "zod-to-json-schema": "^3.24.6"
+    "type-fest": "^5.8.0",
+    "uuid": "^14.0.1",
+    "yargs": "^18.1.0",
+    "zod": "^4.4.3",
+    "zod-to-json-schema": "^3.25.2"
   },
   "devDependencies": {
-    "@commitlint/cli": "^19.8.1",
-    "@jest/globals": "^29.7.0",
-    "@repodog/cli": "^1.6.45",
-    "@repodog/commitlint-config": "^19.0.5",
-    "@repodog/eslint-config": "^9.0.18",
-    "@repodog/eslint-config-jest": "^9.0.6",
-    "@repodog/jest-config": "^29.0.5",
-    "@repodog/markdownlint-config": "^1.2.5",
-    "@repodog/prettier-config": "^3.0.3",
-    "@repodog/rollup-config": "^4.0.11",
-    "@repodog/swc-config": "^1.0.12",
-    "@repodog/syncpack-config": "^13.0.4",
-    "@repodog/ts-config": "^5.0.5",
-    "@rollup/plugin-swc": "^0.3.0",
-    "@swc/core": "^1.13.3",
+    "@commitlint/cli": "^21.2.1",
+    "@jest/globals": "^30.4.1",
+    "@repodog/cli": "^1.7.0",
+    "@repodog/commitlint-config": "^21.0.3",
+    "@repodog/eslint-config": "^9.0.30",
+    "@repodog/eslint-config-jest": "^9.0.10",
+    "@repodog/jest-config": "^30.0.4",
+    "@repodog/markdownlint-config": "^1.2.7",
+    "@repodog/prettier-config": "^3.0.5",
+    "@repodog/rollup-config": "^4.0.18",
+    "@repodog/swc-config": "^1.0.16",
+    "@repodog/syncpack-config": "^15.0.4",
+    "@repodog/ts-config": "^6.0.3",
+    "@rollup/plugin-swc": "^0.4.1",
+    "@swc/core": "^1.15.47",
     "@swc/jest": "^0.2.39",
-    "@types/jest": "^29.5.13",
-    "@types/node": "^24.2.1",
-    "cts-types": "^0.0.10",
-    "del-cli": "^6.0.0",
-    "eslint": "^9.33.0",
+    "@types/jest": "^30.0.0",
+    "@types/node": "^26.1.2",
+    "cts-types": "^0.0.11",
+    "del-cli": "^7.0.0",
+    "eslint": "^10.8.0",
     "generate-changelog": "^1.8.0",
     "husky": "^9.1.7",
-    "jest": "^29.7.0",
-    "jest-environment-jsdom": "^29.7.0",
-    "jest-environment-node": "^29.7.0",
-    "markdownlint-cli2": "^0.18.1",
-    "prettier": "^3.6.2",
-    "rollup": "^4.46.2",
+    "jest": "^30.4.2",
+    "jest-environment-node": "^30.4.1",
+    "markdownlint-cli2": "^0.23.2",
+    "prettier": "^3.9.6",
+    "rollup": "^4.62.3",
     "suppress-experimental-warnings": "^2.0.0",
-    "syncpack": "^13.0.4",
-    "typescript": "^5.9.2"
+    "syncpack": "^15.3.2",
+    "typescript": "^6.0.3"
   },
   "keywords": [
     "config",
@@ -113,7 +117,7 @@
     "lint:code": "eslint .",
     "lint:docs": "markdownlint-cli2 --config \".markdownlint.json\" \"**/*.md\" \"!**/node_modules/**\"",
     "repodog": "repodog",
-    "syncpack": "syncpack format && syncpack list-mismatches && syncpack lint-semver-ranges",
+    "syncpack": "syncpack format --check && syncpack lint",
     "test": "COMPILER=swc node --require=suppress-experimental-warnings --experimental-vm-modules node_modules/jest/bin/jest.js",
     "type-check": "tsc --noEmit",
     "validate": "pnpm run syncpack && pnpm run build && pnpm run lint && pnpm run type-check && pnpm run test",

package/pnpm-workspace.yaml

@@ -0,0 +1,26 @@
+allowBuilds:
+  '@swc/core': false
+  core-js: false
+  unrs-resolver: false
+
+minimumReleaseAgeExclude:
+  - '@repodog/cli-cut@1.4.30'
+  - '@repodog/cli-new@1.6.0'
+  - '@repodog/cli-postinstall@0.4.26'
+  - '@repodog/cli-publish@1.4.30'
+  - '@repodog/cli-setup@1.4.27'
+  - '@repodog/cli-utils@1.4.25'
+  - '@repodog/cli@1.7.0'
+  - '@repodog/commitlint-config@21.0.3'
+  - '@repodog/eslint-config-jest@9.0.10'
+  - '@repodog/eslint-config@9.0.30'
+  - '@repodog/rollup-config@4.0.18'
+  - '@repodog/swc-config@1.0.16'
+  - '@repodog/syncpack-config@15.0.4'
+
+overrides:
+  'body-parser@<1.20.6': '>=1.20.6'
+  'fast-uri@>=3.0.0 <=3.1.3': '>=3.1.4'
+  'js-yaml@^4.0.0': '^4.3.0'
+  'js-yaml@>=5.0.0 <=5.2.1': '>=5.2.2'
+  'markdown-it@<=14.1.1': '>=14.2.0'

package/src/__testUtils__/builtConfig.ts

@@ -1,4 +1,4 @@
-/* eslint-disable prettier/prettier, import-x/no-default-export, unicorn/numeric-separators-style, unicorn/no-null */
+/* eslint-disable import-x/no-default-export, unicorn/numeric-separators-style, unicorn/no-null */
 /* This file is autogenerated, do not edit directly, your changes will not perist. */
 
 export default {

package/src/__testUtils__/configBuilder.ts

@@ -17,6 +17,32 @@ const pageBuilder = createConfigBuilder<PageType>(pageSchema);
 const sectionBuilder = createConfigBuilder<SectionType>(sectionSchema);
 const subsectionBuilder = sectionBuilder.$fork();
 
+const contactDetailsPage = pageBuilder
+  .name('contactDetails')
+  .sections([
+    sectionBuilder.name('header').$flush(),
+    sectionBuilder
+      .name('body')
+      .sections([subsectionBuilder.name('main').$flush(), subsectionBuilder.name('sidebar').$flush()])
+      .$flush(),
+    sectionBuilder.name('footer').$flush(),
+  ])
+  .$flush();
+
+const personalDetailPage = pageBuilder
+  .name('personalDetails')
+  .sections([
+    sectionBuilder.name('header').$flush(),
+    sectionBuilder
+      .name('body')
+      .sections([subsectionBuilder.name('main').$flush(), subsectionBuilder.name('sidebar').$flush()])
+      .$flush(),
+    sectionBuilder.name('footer').$flush(),
+  ])
+  .$flush();
+
+// Required for how builder is executed
+// eslint-disable-next-line unicorn/no-top-level-side-effects
 configBuilder
   .countryCode('GB')
   // Want to test using null value
@@ -26,31 +52,11 @@ configBuilder
   .enabled(true)
   .languageCodes(['en'])
   .locales(({ countryCode, languageCodes }) =>
-    languageCodes?.length && countryCode ? languageCodes.map(code => `${code}_${countryCode}`) : [],
+    countryCode && languageCodes?.length ? languageCodes.map(code => `${code}_${countryCode}`) : [],
   )
   .pages({
-    contactDetails: pageBuilder
-      .name('contactDetails')
-      .sections([
-        sectionBuilder.name('header').$flush(),
-        sectionBuilder
-          .name('body')
-          .sections([subsectionBuilder.name('main').$flush(), subsectionBuilder.name('sidebar').$flush()])
-          .$flush(),
-        sectionBuilder.name('footer').$flush(),
-      ])
-      .$flush(),
-    personalDetails: pageBuilder
-      .name('personalDetails')
-      .sections([
-        sectionBuilder.name('header').$flush(),
-        sectionBuilder
-          .name('body')
-          .sections([subsectionBuilder.name('main').$flush(), subsectionBuilder.name('sidebar').$flush()])
-          .$flush(),
-        sectionBuilder.name('footer').$flush(),
-      ])
-      .$flush(),
+    contactDetails: contactDetailsPage,
+    personalDetails: personalDetailPage,
   })
   .routes([routeBuilder.page('personalDetails').$flush(), routeBuilder.page('contactDetails').$flush()])
   .timeouts({ apollo: 10_000 })

package/src/__testUtils__/schema.ts

@@ -48,9 +48,9 @@ export const configSchema = z.object({
   languageCodes: z.array(z.enum(languageCodes)).optional(),
   locales: z.array(z.string().regex(/[a-z]{2}_[A-Z]{2}/)).optional(),
   name: z.string().optional(),
-  pages: z.record(pageSchema).optional(),
+  pages: z.record(z.string(), pageSchema).optional(),
   routes: z.array(routeSchema).optional(),
-  timeouts: z.record(z.number()).optional(),
+  timeouts: z.record(z.string(), z.number()).optional(),
   timezone: z.enum(timezones).optional(),
 });
 

package/src/cli.ts

@@ -5,9 +5,15 @@ import yargs, { type Argv } from 'yargs';
 import { hideBin } from 'yargs/helpers';
 import { importValidateTransformWriteConfig } from './utils/importValidateTransformWriteConfig.ts';
 
-export type Commands = 'build' | 'watch';
+export type CliCommands = 'build' | 'watch';
 
-const generateArguments = (cmdYargs: Argv) =>
+export type CliArguments = Argv<{
+  'experiments-callback-file'?: string;
+  'input-file': string;
+  'output-file': string;
+}>;
+
+const generateArguments = (cmdYargs: Argv): CliArguments =>
   cmdYargs
     .positional('input-file', {
       demandOption: true,
@@ -24,7 +30,7 @@ const generateArguments = (cmdYargs: Argv) =>
       type: 'string',
     });
 
-export const cli = () => {
+export const cli = (): void => {
   // yargs does not provide a way to pass generic to type args.
   // eslint-disable-next-line @typescript-eslint/consistent-type-assertions
   const argv = yargs(hideBin(process.argv)) as Argv<{

package/src/createConfigBuilder.test.ts

@@ -1,5 +1,5 @@
 import { kebabCase } from 'lodash-es';
-import { ZodType, z } from 'zod';
+import { z } from 'zod';
 import {
   type ConfigType,
   type PageType,
@@ -112,13 +112,14 @@ describe('createConfigBuilder', () => {
           __experiment: 'FEAT_ALPHA@0.0.1',
           // eslint-disable-next-line @typescript-eslint/no-unsafe-assignment
           pages: expect.objectContaining({
-            // eslint-disable-next-line @typescript-eslint/no-unsafe-assignment
+            // eslint-disable-next-line @typescript-eslint/no-unsafe-assignment, unicorn/max-nested-calls
             contactDetails: expect.objectContaining({
               __experiment: 'FEAT_BRAVO@0.0.1',
             }),
           }),
           // eslint-disable-next-line @typescript-eslint/no-unsafe-assignment
           routes: expect.arrayContaining([
+            // eslint-disable-next-line unicorn/max-nested-calls
             expect.objectContaining({
               __experiment: 'FEAT_CHARLIE@0.0.1',
             }),
@@ -153,13 +154,14 @@ describe('createConfigBuilder', () => {
           __disabled: true,
           // eslint-disable-next-line @typescript-eslint/no-unsafe-assignment
           pages: expect.objectContaining({
-            // eslint-disable-next-line @typescript-eslint/no-unsafe-assignment
+            // eslint-disable-next-line @typescript-eslint/no-unsafe-assignment, unicorn/max-nested-calls
             contactDetails: expect.objectContaining({
               __disabled: true,
             }),
           }),
           // eslint-disable-next-line @typescript-eslint/no-unsafe-assignment
           routes: expect.arrayContaining([
+            // eslint-disable-next-line unicorn/max-nested-calls
             expect.objectContaining({
               __disabled: true,
             }),
@@ -223,7 +225,7 @@ describe('createConfigBuilder', () => {
           const { createConfigBuilder } = await import('./createConfigBuilder.ts');
 
           const extendedSchema = configSchema.extend({
-            flags: z.record(z.boolean()).optional().default({ alpha: true, bravo: false, charlie: false }),
+            flags: z.record(z.string(), z.boolean()).optional().default({ alpha: true, bravo: false, charlie: false }),
           });
 
           const config = createConfigBuilder<z.infer<typeof extendedSchema>>(extendedSchema);
@@ -329,7 +331,7 @@ describe('createConfigBuilder', () => {
           const { createConfigBuilder } = await import('./createConfigBuilder.ts');
 
           const extendedSchema = configSchema.extend({
-            flags: z.record(z.boolean().default(true)).optional(),
+            flags: z.record(z.string(), z.boolean().default(true)).optional(),
           });
 
           expect(() => createConfigBuilder<z.infer<typeof extendedSchema>>(extendedSchema)).toThrow(
@@ -587,7 +589,7 @@ describe('createConfigBuilder', () => {
           .languageCodes(['en'])
           .locales(({ countryCode, languageCodes }) =>
             // eslint-disable-next-line jest/no-conditional-in-test
-            languageCodes?.length && countryCode ? languageCodes.map(code => `${code}_${countryCode}` as const) : [],
+            countryCode && languageCodes?.length ? languageCodes.map(code => `${code}_${countryCode}` as const) : [],
           );
 
         expect(config.$values()).toEqual({
@@ -606,7 +608,7 @@ describe('createConfigBuilder', () => {
           .languageCodes(['en'])
           .locales(({ countryCode, languageCodes }) =>
             // eslint-disable-next-line jest/no-conditional-in-test
-            languageCodes?.length && countryCode ? languageCodes.map(code => `${code}_${countryCode}` as const) : [],
+            countryCode && languageCodes?.length ? languageCodes.map(code => `${code}_${countryCode}` as const) : [],
           )
           .languageCodes(['fr'], true);
 
@@ -626,7 +628,7 @@ describe('createConfigBuilder', () => {
           .languageCodes(['en'])
           .locales(({ countryCode, languageCodes }) =>
             // eslint-disable-next-line jest/no-conditional-in-test
-            languageCodes?.length && countryCode ? languageCodes.map(code => `${code}_${countryCode}`) : [],
+            countryCode && languageCodes?.length ? languageCodes.map(code => `${code}_${countryCode}`) : [],
           );
 
         expect(config.$validate()).toBe(true);
@@ -635,27 +637,29 @@ describe('createConfigBuilder', () => {
   });
 
   describe('when a user stringifies the config', () => {
-    describe('when a config value is a zod schema', () => {
-      it('should stringify the value to JSON schema correctly', async () => {
-        const { createConfigBuilder } = await import('./createConfigBuilder.ts');
-
-        const extendedSchema = configSchema.extend({
-          schema: z.instanceof(ZodType),
-        });
-
-        const config = createConfigBuilder<z.infer<typeof extendedSchema>>(extendedSchema);
+    it('should stringify the value to JSON correctly', async () => {
+      const { createConfigBuilder } = await import('./createConfigBuilder.ts');
+      const config = createConfigBuilder<ConfigType>(configSchema);
 
-        config.schema(
-          z
-            .object({
-              alpha: z.string(),
-              bravo: z.boolean(),
-            })
-            .optional(),
+      config
+        .countryCode('GB')
+        .languageCodes(['en'])
+        .locales(({ countryCode, languageCodes }) =>
+          // eslint-disable-next-line jest/no-conditional-in-test
+          countryCode && languageCodes?.length ? languageCodes.map(code => `${code}_${countryCode}`) : [],
         );
 
-        expect(config.$toJson()).toMatchSnapshot();
-      });
+      expect(config.$toJson()).toMatchInlineSnapshot(`
+       "{
+         "countryCode": "GB",
+         "languageCodes": [
+           "en"
+         ],
+         "locales": [
+           "en_GB"
+         ]
+       }"
+      `);
     });
   });
 });

package/src/createConfigBuilder.ts

@@ -1,8 +1,6 @@
-import { type JSONSchema7 } from 'json-schema';
 import { cloneDeep, merge } from 'lodash-es';
 import { v4 as uuidV4 } from 'uuid';
-import { type ZodError, type z } from 'zod';
-import { zodToJsonSchema } from 'zod-to-json-schema';
+import { ZodError, z } from 'zod';
 import { cloneNonEnumerableValues } from './transformers/cloneNonEnumerableValues.ts';
 import { collateDefaultValues } from './utils/collateDefaultValues.ts';
 import { isDerivedValueCallback } from './utils/isDerivedValueCallback.ts';
@@ -10,17 +8,16 @@ import { isInvalidPropertyOverride } from './utils/isInvalidPropertyOverride.ts'
 import { RESERVED_KEYWORDS, isPropertyReservedWord } from './utils/isPropertyReservedWord.ts';
 import { isSchemaValid } from './utils/isSchemaValid.ts';
 import { isValidValue } from './utils/isValidValue.ts';
-import { jsonStringifyReplacer } from './utils/jsonStringifyReplacer.ts';
 import { transformConfigSync } from './utils/transformConfig.ts';
 
 export type ConfigBuilder<ZodTypes> = {
   [Key in keyof Required<ZodTypes>]: (
     value: ZodTypes[Key] | ((c: ZodTypes) => ZodTypes[Key]),
-    override?: boolean,
+    isOverride?: boolean,
   ) => ConfigBuilder<ZodTypes>;
 } & {
   $disable: () => ConfigBuilder<ZodTypes>;
-  $errors: () => ZodError['errors'];
+  $errors: () => ZodError['issues'];
   $experiment: (key: string) => ConfigBuilder<ZodTypes>;
   $extend: (configBuilder: ConfigBuilder<ZodTypes>) => ConfigBuilder<ZodTypes>;
   $flush: () => ZodTypes;
@@ -38,7 +35,7 @@ export type CreateConfigBuilderOptions = {
 };
 
 export const createConfigBuilder = <ZodTypes>(
-  zodSchema: z.ZodSchema,
+  zodSchema: z.ZodType,
   options: CreateConfigBuilderOptions = {},
   derivedValueCallbacks: Partial<
     Record<
@@ -58,9 +55,7 @@ export const createConfigBuilder = <ZodTypes>(
 
   type DerivedValueCallback<K extends keyof Config = keyof Config> = (c: Config) => Config[K];
 
-  // Added to simplify type returned from zodToJsonSchema.
-  // eslint-disable-next-line @typescript-eslint/consistent-type-assertions
-  const jsonSchema = zodToJsonSchema(zodSchema) as JSONSchema7;
+  const jsonSchema = z.toJSONSchema(zodSchema);
 
   if (isSchemaValid(jsonSchema)) {
     throw new Error(`The root type of a config schema must be "object", but received "${String(jsonSchema.type)}"`);
@@ -68,29 +63,29 @@ export const createConfigBuilder = <ZodTypes>(
 
   const uuid = options.overrides?.uuid ?? uuidV4;
 
-  const addNonEnumeralBaseProperties = (conf: Config) => {
-    Object.defineProperty(conf, '__zcb', {
-      configurable: false,
-      enumerable: false,
-      value: true,
-    });
-
-    Object.defineProperty(conf, '__id', {
-      configurable: false,
-      enumerable: false,
-      value: uuid(),
-    });
-
-    if (options.type) {
-      Object.defineProperty(conf, '__type', {
+  const addNonEnumeralBaseProperties = (conf: Config): void => {
+    Object.defineProperties(conf, {
+      __id: {
         configurable: false,
         enumerable: false,
-        value: options.type,
-      });
-    }
+        value: uuid(),
+      },
+      __zcb: {
+        configurable: false,
+        enumerable: false,
+        value: true,
+      },
+      ...(options.type && {
+        __type: {
+          configurable: false,
+          enumerable: false,
+          value: options.type,
+        },
+      }),
+    });
   };
 
-  const createInitialConfig = () => {
+  const createInitialConfig = (): Config => {
     // Not casting here as loads of downstream impacts that make
     // subsequent typing really difficult.
     // eslint-disable-next-line @typescript-eslint/consistent-type-assertions
@@ -102,7 +97,7 @@ export const createConfigBuilder = <ZodTypes>(
   let config = createInitialConfig();
   let callbacks: Partial<Record<keyof Config, DerivedValueCallback>> = { ...derivedValueCallbacks };
 
-  // Aimed at making it easier for Typescript to derive type.
+  // Aimed at making it easier for TypeScript to derive type.
   // eslint-disable-next-line @typescript-eslint/consistent-type-assertions
   const configBuilder = {
     $disable: () => {
@@ -119,9 +114,7 @@ export const createConfigBuilder = <ZodTypes>(
         zodSchema.parse(configBuilder.$values());
         return [];
       } catch (error: unknown) {
-        // This will always be an instance of ZodError.
-        // eslint-disable-next-line @typescript-eslint/consistent-type-assertions
-        return (error as ZodError).errors;
+        return error instanceof ZodError ? error.issues : [];
       }
     },
     $experiment: (key: string) => {
@@ -136,7 +129,7 @@ export const createConfigBuilder = <ZodTypes>(
     $extend: (builder: ConfigBuilder<Config>) => {
       config = transformConfigSync<Config>(builder.$values(), [cloneNonEnumerableValues]);
       // @ts-expect-error private property
-      // As __callbacks is private property, Typescript doesn't know what type is.
+      // As __callbacks is private property, TypeScript doesn't know what type is.
       // eslint-disable-next-line @typescript-eslint/consistent-type-assertions
       callbacks = { ...builder.__callbacks } as Partial<Record<keyof Config, DerivedValueCallback>>;
     },
@@ -146,7 +139,7 @@ export const createConfigBuilder = <ZodTypes>(
       return values;
     },
     $fork: () => createConfigBuilder<Config>(zodSchema, options, derivedValueCallbacks, initialValues),
-    $toJson: () => JSON.stringify(configBuilder.$values(), jsonStringifyReplacer, 2),
+    $toJson: () => JSON.stringify(configBuilder.$values(), undefined, 2),
     $validate: () => {
       try {
         zodSchema.parse(configBuilder.$values());
@@ -157,13 +150,13 @@ export const createConfigBuilder = <ZodTypes>(
       }
     },
     $values: () => {
-      for (const property in callbacks) {
-        const callback = callbacks[property];
+      for (const propertyName in callbacks) {
+        // eslint-disable-next-line @typescript-eslint/consistent-type-assertions
+        const castPropertyName = propertyName as keyof Config;
+        const callback = callbacks[castPropertyName];
 
         if (callback) {
-          // Typescript not inferring property is key of Config.
-          // eslint-disable-next-line @typescript-eslint/consistent-type-assertions
-          config[property as keyof Config] = callback(config);
+          config[castPropertyName] = callback(config);
         }
       }
 
@@ -190,33 +183,34 @@ export const createConfigBuilder = <ZodTypes>(
     // eslint-disable-next-line @typescript-eslint/consistent-type-assertions
     const castPropertyName = propertyName as keyof Config;
 
-    if (!jsonSchema.required?.includes(propertyName) && !(propertyName in config)) {
+    if (!jsonSchema.required?.includes(propertyName) && !Object.hasOwn(config, propertyName)) {
       // @ts-expect-error Optional properties we want to exist in the config with undefined initial values.
       config[propertyName] = undefined;
     }
 
-    // Aimed at making it easier for Typescript to derive type.
+    // Aimed at making it easier for TypeScript to derive type.
     // eslint-disable-next-line @typescript-eslint/consistent-type-assertions
-    configBuilder[castPropertyName] = ((value: Config[keyof Config] | DerivedValueCallback, override?: boolean) => {
-      if (isInvalidPropertyOverride(config[castPropertyName], override)) {
+    configBuilder[castPropertyName] = ((value: Config[keyof Config] | DerivedValueCallback, toOverride?: boolean) => {
+      if (isInvalidPropertyOverride(config[castPropertyName], toOverride)) {
         throw new Error(
-          `A value already exists for "${String(
-            castPropertyName,
-          )}". You may be trying to add a new values before flushing the old one. If you intended to override the existing value, pass in true as the second argument.`,
+          // ESLint unable to derive value can also be symbol or number
+          // eslint-disable-next-line unicorn/no-useless-coercion
+          `A value already exists for "${String(castPropertyName)}". You may be trying to add a new values before flushing the old one. If you intended to override the existing value, pass in true as the second argument.`,
         );
       }
 
-      let propertyValue: Config[keyof Config];
       const MAX_DEPTH = 1;
 
       if (!isValidValue(value)) {
         throw new Error(
-          `"${String(
-            castPropertyName,
-          )}" value has a depth greater than ${String(MAX_DEPTH)}. To pass in objects with a depth greater than ${String(MAX_DEPTH)}, create a builder for that config slice.`,
+          // ESLint unable to derive value can also be symbol or number
+          // eslint-disable-next-line unicorn/no-useless-coercion
+          `"${String(castPropertyName)}" value has a depth greater than ${String(MAX_DEPTH)}. To pass in objects with a depth greater than ${String(MAX_DEPTH)}, create a builder for that config slice.`,
         );
       }
 
+      let propertyValue: Config[keyof Config];
+
       if (isDerivedValueCallback<DerivedValueCallback>(value)) {
         callbacks[castPropertyName] = value;
         propertyValue = value(config);

package/src/createConfigParser.ts

@@ -5,7 +5,7 @@ import { transformConfig } from './utils/transformConfig.ts';
 export const createConfigParser = async <Config extends object>(
   config: Config,
   { experimentsCallback }: ConfigParserOptions = {},
-) => {
+): Promise<Config> => {
   const handlers: TransformConfigHandler[] = [];
 
   if (experimentsCallback) {

package/src/createConfigReader.test.ts

@@ -20,8 +20,8 @@ describe('createConfigReader', () => {
   describe('when that value is a boolean', () => {
     it('should return the correct value', () => {
       const reader = createReader();
-      const value = reader.read('enabled');
-      expect(value).toBe(true);
+      const isEnabled = reader.read('enabled');
+      expect(isEnabled).toBe(true);
     });
   });