No issue to link – this was a pain point when trying to bring the Coder generated types into Backstage as an SDK (similar to how the VS Code extension is doing things).

Changes

• Updates type-generation logic on the Go side to make all TypeScript arrays be readonly (see notes)
• Updates all UI code in TypeScript to remove any mutable array types, and remove accidental runtime mutations

Notes

Before, we had types like this:

interfaceValue{readonlyworkspaces:Workspace[];}

With this, typeValue'sworkspaces property can't be reassigned, but the underlying array is still fully mutable. TypeScript is perfectly happy with this:

functiondoStuff(value:Value){constnewWorkspace=makeWorkspace();value.workspaces.push(newWorkspace);}

With the update, the type definitions now look like this:

interfaceValue{readonlyworkspaces:readonlyWorkspace[];}

Both the property on the object, and the array the property points to are readonly

functiondoStuff(value:Value){constnewWorkspace=makeWorkspace();// TypeScript compiler complainsvalue.workspaces.push(newWorkspace);// Perfectly fineconstworkspacesCopy=[...value.workspaces];workspacesCopy.push(newWorkspace);}

Especially since both React and React Query are built around the assumption that 99% of their data is fully immutable, accidental mutations have a risk of introducing bugs that would be incredibly hard to track down