Skip to content

Commit 9371afa

Browse files
committed
Fix additional properties for app.toml
1 parent 508bfae commit 9371afa

1 file changed

Lines changed: 7 additions & 0 deletions

File tree

packages/app/src/cli/services/generate/fetch-extension-specifications.ts

Lines changed: 7 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -90,6 +90,8 @@ async function mergeLocalAndRemoteSpecs(
9090
merged.uidStrategy = merged.options.uidStrategy ?? localSpec.uidStrategy ?? 'single'
9191

9292
// If configuration is inside an app.toml -- i.e. single UID mode -- we must be able to parse a partial slice.
93+
// DEPRECATED: not all single specs are config specs.
94+
// Should be removed once we can get the experience from the API.
9395
let handleInvalidAdditionalProperties: HandleInvalidAdditionalProperties
9496
switch (merged.uidStrategy) {
9597
case 'uuid':
@@ -103,6 +105,11 @@ async function mergeLocalAndRemoteSpecs(
103105
break
104106
}
105107

108+
// If the experience is 'configuration', force strip.
109+
if (merged.experience === 'configuration') {
110+
handleInvalidAdditionalProperties = 'strip'
111+
}
112+
106113
const parseConfigurationObject = await unifiedConfigurationParserFactory(merged, handleInvalidAdditionalProperties)
107114

108115
return {

0 commit comments

Comments
 (0)