mirror of
https://github.com/renovatebot/renovate.git
synced 2026-10-31 17:51:49 +00:00
Sebastian Poxhofer
97327c3d28
* refactor(linters): add prefer-coerce-array rule Refs #45217 `x ?? []` and `coerceArray(x)` say the same thing, but only the second one is searchable and consistent. The rule flags a `??` whose default is a bare `[]` literal; a non-empty default or an asserted `([] as Foo[])` still passes. Applying it across `lib/` surfaced three spots the helper cannot cover, which keep `?? []` with a disable comment or a narrower fix: - `package-files.ts` destructures a tuple, which `coerceArray()` widens - `generate.spec.ts` defaults `matchAll()`, an iterator rather than an array - `mise/upgradeable-tooling.ts` declares a `Record` return, so its `?? []` fallback was wrong to begin with and becomes `coerceObject()` * refactor(platform/github): hoist the GHE version header lookup `Object.entries(...).find()` returns a tuple, and defaulting it with `?? []` only to destructure the second element reads as an array default when it is not one. Name the found entry instead, which drops the `?? []` rather than routing a tuple through `coerceArray()`.
31 lines
966 B
TypeScript
31 lines
966 B
TypeScript
import { isNonEmptyString } from '@sindresorhus/is';
|
|
import { z } from 'zod/v4';
|
|
import { coerceArray } from '../../../util/array.ts';
|
|
import { Toml } from '../../../util/schema-utils/index.ts';
|
|
import { depTypes, pep508ToPackageDependency } from '../pep621/utils.ts';
|
|
import type { PackageFileContent } from '../types.ts';
|
|
|
|
const Pep723Dep = z
|
|
.string()
|
|
.transform((dep) => pep508ToPackageDependency(depTypes.dependencies, dep));
|
|
|
|
export const Pep723 = Toml.pipe(
|
|
z
|
|
.object({
|
|
'requires-python': z.string().optional(),
|
|
dependencies: z
|
|
.array(Pep723Dep)
|
|
.transform((deps) => deps.filter((dep) => !!dep))
|
|
.optional(),
|
|
})
|
|
.transform(({ 'requires-python': requiresPython, dependencies }) => {
|
|
const res: PackageFileContent = { deps: coerceArray(dependencies) };
|
|
|
|
if (isNonEmptyString(requiresPython)) {
|
|
res.extractedConstraints = { python: requiresPython };
|
|
}
|
|
|
|
return res;
|
|
}),
|
|
);
|