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

Change to not expose exec artifacts (#5700)

Browse Source 
Now that the artifact graph is built completely in Rust, these are an
implementation detail that shouldn't be exposed to TS.
This commit is contained in:
Jonathan Tran
2025-03-11 15:09:11 -04:00
committed by GitHub
parent df6b4f4c37
commit b5028f7aa8
3 changed files with 0 additions and 8 deletions
Download Patch File Download Diff File Expand all files Collapse all files

4
src/lang/wasm.ts
View File

@ -295,7 +295,6 @@ export const isPathToNodeNumber = (
export interface ExecState {
variables: { [key in string]?: KclValue }
operations: Operation[]
artifacts: { [key in ArtifactId]?: RustArtifact }
artifactCommands: ArtifactCommand[]
artifactGraph: ArtifactGraph
errors: CompilationError[]
@ -310,7 +309,6 @@ export function emptyExecState(): ExecState {
return {
variables: {},
operations: [],
artifacts: {},
artifactCommands: [],
artifactGraph: defaultArtifactGraph(),
errors: [],
@ -337,7 +335,6 @@ function execStateFromRust(
return {
variables: execOutcome.variables,
operations: execOutcome.operations,
artifacts: execOutcome.artifacts,
artifactCommands: execOutcome.artifactCommands,
artifactGraph,
errors: execOutcome.errors,
@ -349,7 +346,6 @@ function mockExecStateFromRust(execOutcome: RustExecOutcome): ExecState {
return {
variables: execOutcome.variables,
operations: execOutcome.operations,
artifacts: execOutcome.artifacts,
artifactCommands: execOutcome.artifactCommands,
artifactGraph: new Map<ArtifactId, Artifact>(),
errors: execOutcome.errors,