ByoungSooPark/modeling-app
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Use typed KclValue instead of any in sketchFromKclValue (#7144)

Browse Source 
* use KclValue instead of any

* no need for any in sketchFromKclValueOptional

* simplify error reason for sketchFromKclValueOptional
This commit is contained in:
Andrew Varga
2025-05-22 11:15:31 +02:00
committed by GitHub
parent 0a0e6abd3f
commit ae569b61db
1 changed files with 7 additions and 14 deletions
Download Patch File Download Diff File Expand all files Collapse all files

21
src/lang/wasm.ts
View File

@ -361,31 +361,24 @@ function rustArtifactGraphToMap(
return map
}
// TODO: In the future, make the parameter be a KclValue.
export function sketchFromKclValueOptional(
obj: any,
obj: KclValue | undefined,
varName: string | null
): Sketch | Reason {
if (obj?.value?.type === 'Sketch') return obj.value
if (obj?.value?.type === 'Solid') return obj.value.sketch
if (obj?.type === 'Sketch') return obj.value
if (obj?.type === 'Solid') return obj.value.sketch
if (!varName) {
varName = 'a KCL value'
}
const actualType = obj?.value?.type ?? obj?.type
if (actualType) {
return new Reason(
`Expected ${varName} to be a sketch or solid, but it was ${actualType} instead.`
)
} else {
return new Reason(`Expected ${varName} to be a sketch, but it wasn't.`)
}
const actualType = obj?.type ?? 'unknown'
return new Reason(
`Expected ${varName} to be a sketch or solid, but it was ${actualType} instead.`
)
}
// TODO: In the future, make the parameter be a KclValue.
export function sketchFromKclValue(
obj: any,
obj: KclValue | undefined,
varName: string | null
): Sketch | Error {
const result = sketchFromKclValueOptional(obj, varName)