@servicenow/sdk-build-plugins 4.5.0 → 4.6.0

package/src/table-plugin.ts

@@ -321,6 +321,20 @@ type SysDocumentationProperties = {
     help: string | undefined
 }
 
+type SysDictionaryOverrideProperties = {
+    name: string
+    element: string
+    base_table: string | undefined
+    default: string | undefined
+    calculation: string | undefined
+    reference_qual: string | undefined
+    read_only_option: string | undefined
+    dependent: string | undefined
+    mandatory: boolean | undefined
+    display: boolean | undefined
+    attributes: string | undefined
+}
+
 /**
  * The access levels are mapped to the following values on the platform
  *
@@ -390,7 +404,7 @@ export const TablePlugin = Plugin.create({
                     return { success: false }
                 }
 
-                const recordDefs = tableDefToRecordProperties(tableDef, config.tableDefaultLanguage)
+                const recordDefs = tableDefToRecordProperties(tableDef, config.defaultLanguage)
                 const records: Record[] = []
                 for (const [key, table] of [
                     ['sysDbObject', 'sys_db_object'],
@@ -398,6 +412,7 @@ export const TablePlugin = Plugin.create({
                     ['sysChoice', 'sys_choice'],
                     ['sysIndex', 'sys_index'],
                     ['sysDocumentation', 'sys_documentation'],
+                    ['sysDictionaryOverride', 'sys_dictionary_override'],
                 ] as const) {
                     for (const rec of [recordDefs[key]].flat()) {
                         records.push(
@@ -449,6 +464,10 @@ export const TablePlugin = Plugin.create({
                         },
                     },
                 },
+                sys_dictionary_override: {
+                    via: { name: 'name' },
+                    descendant: true,
+                },
                 ua_table_licensing_config: {
                     descendant: true,
                     via: { name: 'name' },
@@ -474,9 +493,12 @@ export const TablePlugin = Plugin.create({
                 const schema: { [key: string]: CallExpressionShape } = {}
                 let displayColumn: string | undefined
                 const columns = descendants.query('sys_dictionary')
+                const overrides = descendants.query('sys_dictionary_override')
                 let collectionRecord: Record
                 const choices = descendants.query('sys_choice')
                 const documentation = descendants.query('sys_documentation')
+
+                // Process regular columns
                 for (const column of columns) {
                     if (column.get('internal_type').getValue() === 'collection') {
                         // 'collection' sys_dictionary record only has table properties
@@ -489,13 +511,89 @@ export const TablePlugin = Plugin.create({
                         documentation: documentation.filter(
                             (d) => d.get('element').ifString()?.getValue() === columnName
                         ),
-                        tableDefaultLanguage: config.tableDefaultLanguage,
+                        tableDefaultLanguage: config.defaultLanguage,
                     })
                     if (column.get('display').ifDefined()?.toBoolean().getValue()) {
                         displayColumn = columnName
                     }
                 }
 
+                // Process dictionary overrides as OverrideColumn
+                for (const override of overrides) {
+                    const columnName = override.get('element').asString().getValue()
+                    schema[columnName] = new CallExpressionShape({
+                        source: override,
+                        callee: 'OverrideColumn',
+                        args: [
+                            override.transform(({ $ }) => ({
+                                baseTable: $.from('base_table'),
+                                default: $.from('default_value_override', 'default_value').map((flag, value) => {
+                                    return flag.toBoolean()?.getValue() ? value.ifString()?.getValue() : undefined
+                                }),
+                                calculation: $.from('calculation_override', 'calculation').map((flag, value) => {
+                                    return flag.toBoolean()?.getValue() ? value.ifString()?.getValue() : undefined
+                                }),
+                                referenceQualifier: $.from('reference_qual_override', 'reference_qual').map(
+                                    (flag, value) => {
+                                        return flag.toBoolean()?.getValue() ? value.ifString()?.getValue() : undefined
+                                    }
+                                ),
+                                readOnlyOption: $.from('read_only_option_override', 'read_only_option').map(
+                                    (flag, value) => {
+                                        return flag.toBoolean()?.getValue() ? value.ifString()?.getValue() : undefined
+                                    }
+                                ),
+                                dependent: $.from('dependent_override', 'dependent').map((flag, value) => {
+                                    return flag.toBoolean()?.getValue() ? value.ifString()?.getValue() : undefined
+                                }),
+                                mandatory: $.from('mandatory_override', 'mandatory').map((flag, value) => {
+                                    return flag.toBoolean()?.getValue() ? value.toBoolean()?.getValue() : undefined
+                                }),
+                                attributes: $.from('attributes_override', 'attributes').map((flag, attrs) => {
+                                    if (!flag.toBoolean()?.getValue() || !attrs.isString()) {
+                                        return undefined
+                                    }
+                                    const result: { [key: string]: string | number | boolean } = {}
+                                    attrs
+                                        .toString()
+                                        .getValue()
+                                        .split(',')
+                                        .forEach((attr) => {
+                                            if (attr === '') {
+                                                return
+                                            }
+                                            const [key, value] = attr.split('=').map((s) => s.trim())
+                                            if (!key || value === undefined) {
+                                                return
+                                            }
+                                            if (value === 'true') {
+                                                result[key] = true
+                                            } else if (value === 'false') {
+                                                result[key] = false
+                                            } else {
+                                                // Try to parse as number
+                                                const numValue = Number(value)
+                                                if (!isNaN(numValue) && value !== '') {
+                                                    result[key] = numValue
+                                                } else {
+                                                    result[key] = value
+                                                }
+                                            }
+                                        })
+                                    return result
+                                }),
+                                display: $.from('display_override').map((v) => {
+                                    if (!v.ifDefined()) {
+                                        return undefined
+                                    }
+                                    const boolValue = v.toBoolean().getValue()
+                                    return boolValue === true ? true : undefined
+                                }),
+                            })),
+                        ],
+                    })
+                }
+
                 const tableDocumentation = documentation.filter((d) => !d.get('element').getValue())
                 const licensing = descendants.query('ua_table_licensing_config')
                 const autoNumber = descendants.query('sys_number')
@@ -629,14 +727,14 @@ export const TablePlugin = Plugin.create({
                                     (label) =>
                                         label.ifString() ??
                                         (tableDocumentation.length &&
-                                        !isDefaultDocumentation('', tableDocumentation, config.tableDefaultLanguage)
+                                        !isDefaultDocumentation('', tableDocumentation, config.defaultLanguage)
                                             ? tableDocumentation.map((doc) =>
                                                   doc
                                                       .transform(({ $ }) => ({
                                                           label: $.def(''),
                                                           help: $.def(''),
                                                           hint: $.def(''),
-                                                          language: $.def(config.tableDefaultLanguage),
+                                                          language: $.def(config.defaultLanguage),
                                                           plural: $.def(''),
                                                           url: $.def(''),
                                                           urlTarget: $.from('url_target').def(''),
@@ -718,8 +816,9 @@ export const TablePlugin = Plugin.create({
                 const documentation = descendants.query('sys_documentation')
                 const licensing = descendants.query('ua_table_licensing_config')
                 const autoNumber = descendants.query('sys_number')
-                let collectionRecord: Record | undefined
+                const overrides = descendants.query('sys_dictionary_override')
                 let displayColumn: string | undefined
+                let collectionRecord: Record | undefined
                 const elements: XMLBuilder[] = []
                 for (const column of columns) {
                     const displayValue = column.get('display').ifBoolean()?.getValue()
@@ -889,7 +988,7 @@ export const TablePlugin = Plugin.create({
                             !isDefaultDocumentation(
                                 doc.get('element')?.toString().getValue(),
                                 [doc],
-                                config.tableDefaultLanguage
+                                config.defaultLanguage
                             )
                     ),
                     config,
@@ -903,6 +1002,7 @@ export const TablePlugin = Plugin.create({
                 )
 
                 const autoNumberFiles = await generateRecordXml(autoNumber, config, transform)
+                const overrideFiles = await generateRecordXml(overrides, config, transform)
 
                 return {
                     success: true,
@@ -916,6 +1016,7 @@ export const TablePlugin = Plugin.create({
                         ...documentationFiles,
                         ...licensingFiles,
                         ...autoNumberFiles,
+                        ...overrideFiles,
                     ],
                 }
             },
@@ -927,6 +1028,13 @@ export const TablePlugin = Plugin.create({
                 value: `sys_dictionary_${record.get('name').getValue()}_${record.get('element').getValue() || 'null'}`,
             }),
         },
+        sys_dictionary_override: {
+            coalesce: ['name', 'element'],
+            getUpdateName: (record) => ({
+                success: true,
+                value: `sys_dictionary_override_${record.get('name').getValue()}_${record.get('element').getValue() || 'null'}`,
+            }),
+        },
         sys_documentation: {
             coalesce: ['name', 'element', 'language'],
             getUpdateName: (record) => ({
@@ -1021,7 +1129,7 @@ export const TablePlugin = Plugin.create({
                     ignoreColumnNameCheck = true
                 }
 
-                // sys_dictionary
+                // sys_dictionary and sys_dictionary_override
                 const schema = table.get('schema').asObject()
                 const columnIdsMap = new Map<string, string>()
                 for (const [name, column] of schema.entries()) {
@@ -1031,37 +1139,116 @@ export const TablePlugin = Plugin.create({
                             'Column name must only contain lowercase letters, numbers, and underscores'
                         )
                     }
+
+                    // Check if this is an OverrideColumn
                     if (
-                        !ignoreColumnNameCheck &&
-                        !tableNameMatch &&
-                        !isSNScope(scopeName) &&
-                        scopeName !== 'global' &&
-                        !name.match(scopeRegex)
+                        column.is(CallExpressionShape) &&
+                        column.as(CallExpressionShape).getCallee() === 'OverrideColumn'
                     ) {
-                        // 'sn' and 'now' scoped apps ignore this validation
-                        diagnostics.error(
-                            column.getOriginalNode().getParentIfKind(ts.SyntaxKind.PropertyAssignment) ?? column,
-                            `Column name should be prefixed with scope '${scopeName}_' if table name does not contain prefix`
-                        )
-                    } else if (scopeName === 'global' && !globalTableNameMatch && !name.match(globalRegex)) {
-                        diagnostics.error(
-                            column.getOriginalNode().getParentIfKind(ts.SyntaxKind.PropertyAssignment) ?? column,
-                            `Column name should be prefixed with 'u_' custom prefix if table name does not contain this prefix, such as when adding columns to an existing global table`
-                        )
-                    }
-                    const display = table.get('display').ifString()?.getValue() === name
-                    const result = await transform.toRecord(
-                        addFieldsToColumn(
-                            { name, table: tableName.getValue(), display },
-                            column.as(CallExpressionShape)
+                        // This is an OverrideColumn() call - get the object from its first argument
+                        const columnObj = column.as(CallExpressionShape).getArgument(0).asObject()
+
+                        // Handle OverrideColumn - create sys_dictionary_override record
+                        // Validate that the table extends another table
+                        const extendsTable = table.get('extends').ifString()?.getValue()
+                        if (!extendsTable) {
+                            diagnostics.error(
+                                column,
+                                `Cannot use OverrideColumn in table '${tableName.getValue()}' because it does not extend another table`
+                            )
+                            return { success: false }
+                        }
+
+                        // Use baseTable if provided, otherwise default to extends
+                        const baseTableValue = columnObj.get('baseTable')
+                        let baseTable: string
+
+                        if (baseTableValue.ifString()) {
+                            baseTable = baseTableValue.asString().getValue()
+                        } else {
+                            // Default to extends if baseTable not provided
+                            baseTable = extendsTable
+                        }
+
+                        // Create sys_dictionary_override record
+                        const overrideRecord = await factory.createRecord({
+                            source: statement ?? callExpression,
+                            table: 'sys_dictionary_override',
+                            properties: columnObj.transform(({ $ }) => ({
+                                name: $.val(tableName),
+                                element: $.val(name),
+                                base_table: $.val(baseTable),
+                                default_value: $.from('default'),
+                                default_value_override: $.from('default').map((v) => !!v.ifDefined()),
+                                calculation: $.from('calculation'),
+                                calculation_override: $.from('calculation').map((v) => !!v.ifDefined()),
+                                reference_qual: $.from('referenceQualifier'),
+                                reference_qual_override: $.from('referenceQualifier').map((v) => !!v.ifDefined()),
+                                read_only_option: $.from('readOnlyOption'),
+                                read_only_option_override: $.from('readOnlyOption').map((v) => !!v.ifDefined()),
+                                read_only: $.from('readOnlyOption').map((readOnlyOption) => {
+                                    // read_only should be true if readOnlyOption has a value
+                                    return !!readOnlyOption.ifDefined()
+                                }),
+                                read_only_override: $.from('readOnlyOption').map((v) => !!v.ifDefined()),
+                                dependent: $.from('dependent'),
+                                dependent_override: $.from('dependent').map((v) => !!v.ifDefined()),
+                                mandatory: $.from('mandatory').toBoolean().def(false),
+                                mandatory_override: $.from('mandatory').map((v) => !!v.ifDefined()),
+                                attributes: $.from('attributes').map((attrs) => {
+                                    if (!attrs.isObject()) {
+                                        return
+                                    }
+                                    const attrsObj = attrs.asObject().getValue()
+                                    return Object.entries(attrsObj)
+                                        .map(([key, value]) => `${key}=${value}`)
+                                        .join(',')
+                                }),
+                                attributes_override: $.from('attributes').map((v) => !!v.ifDefined()),
+                                display_override: $.from('display').map((v) => {
+                                    const boolShape = v.ifBoolean()
+                                    if (!boolShape) {
+                                        return false
+                                    }
+                                    return boolShape.getValue() === true
+                                }),
+                            })),
+                        })
+                        relatedRecords.push(overrideRecord)
+                    } else {
+                        // Handle regular column - create sys_dictionary record
+                        if (
+                            !ignoreColumnNameCheck &&
+                            !tableNameMatch &&
+                            !isSNScope(scopeName) &&
+                            scopeName !== 'global' &&
+                            !name.match(scopeRegex)
+                        ) {
+                            // 'sn' and 'now' scoped apps ignore this validation
+                            diagnostics.error(
+                                column.getOriginalNode().getParentIfKind(ts.SyntaxKind.PropertyAssignment) ?? column,
+                                `Column name should be prefixed with scope '${scopeName}_' if table name does not contain prefix`
+                            )
+                        } else if (scopeName === 'global' && !globalTableNameMatch && !name.match(globalRegex)) {
+                            diagnostics.error(
+                                column.getOriginalNode().getParentIfKind(ts.SyntaxKind.PropertyAssignment) ?? column,
+                                `Column name should be prefixed with 'u_' custom prefix if table name does not contain this prefix, such as when adding columns to an existing global table`
+                            )
+                        }
+                        const display = table.get('display').ifString()?.getValue() === name
+                        const result = await transform.toRecord(
+                            addFieldsToColumn(
+                                { name, table: tableName.getValue(), display },
+                                column.as(CallExpressionShape)
+                            )
                         )
-                    )
-                    if (!result.success) {
-                        diagnostics.error(column, 'Invalid column in table schema')
-                        return { success: false }
+                        if (!result.success) {
+                            diagnostics.error(column, 'Invalid column in table schema')
+                            return { success: false }
+                        }
+                        relatedRecords.push(result.value)
+                        columnIdsMap.set(name, result.value.getId().getValue())
                     }
-                    relatedRecords.push(result.value)
-                    columnIdsMap.set(name, result.value.getId().getValue())
                 }
 
                 // sys_index
@@ -1193,7 +1380,7 @@ export const TablePlugin = Plugin.create({
                             properties: {
                                 name: tableName,
                                 element: undefined,
-                                language: config.tableDefaultLanguage,
+                                language: config.defaultLanguage,
                                 label: table.get('label').ifString() ?? tableName,
                                 plural: generatePlural(
                                     table.get('label')?.ifString()?.getValue() ?? tableName.getValue()
@@ -1292,7 +1479,7 @@ export const TablePlugin = Plugin.create({
                             .map((actions) => hasAction('create', actions))
                             .def(false),
                         is_extendable: $.from('extensible').toBoolean().def(false),
-                        label: $.map((label) => getLabelForDefaultLanguage(label, config.tableDefaultLanguage)),
+                        label: $.map((label) => getLabelForDefaultLanguage(label, config.defaultLanguage)),
                         live_feed_enabled: $.from('liveFeed').toBoolean().def(false),
                         name: $,
                         scriptable_table: $.from('scriptableTable').toBoolean().def(false),
@@ -1469,6 +1656,7 @@ function tableDefToRecordProperties(
     sysChoice: SysChoiceProperties[]
     sysIndex: SysIndexProperties[]
     sysDocumentation: SysDocumentationProperties[]
+    sysDictionaryOverride: SysDictionaryOverrideProperties[]
 } {
     const sysDbObject: SysDbObjectProperties = {
         name: tableDef.name,
@@ -1490,6 +1678,7 @@ function tableDefToRecordProperties(
     const sysDictionary: SysDictionaryProperties[] = []
     const sysChoice: SysChoiceProperties[] = []
     const sysDocumentation: SysDocumentationProperties[] = []
+    const sysDictionaryOverride: SysDictionaryOverrideProperties[] = []
 
     // table documentation
     if (tableDef.label && tableDef.plural) {
@@ -1647,6 +1836,7 @@ function tableDefToRecordProperties(
         sysChoice,
         sysIndex,
         sysDocumentation,
+        sysDictionaryOverride,
     }
 }
 

package/src/ui-action-plugin.ts

@@ -247,13 +247,6 @@ export const UiActionPlugin = Plugin.create({
                         ),
                     })),
                 })
-                const roles = arg.get('roles').ifArray()?.getElements() ?? []
-                if (!arg.get('condition').toString().getValue().trim() && roles.length === 0) {
-                    diagnostics.warn(
-                        arg.get('roles').ifDefined() ?? arg.get('condition').ifDefined() ?? arg,
-                        'UI Actions with an empty condition and no roles defined can be called by any logged-in users. Please restrict UI actions. The condition field should be specified to restrict execution of this UI Action to certain users. For example, current.canWrite() condition restricts the UI Action to the users who can modify the current record, gs.hasRole("admin") condition restricts the UI Action to the users with admin role.'
-                    )
-                }
 
                 if (arg.get('script').is(ModuleFunctionShape) && isClient.ifBoolean()?.getValue()) {
                     diagnostics.error(isClient, 'Module scripts (sys_module) cannot be used on client-side UI Actions')
@@ -309,7 +302,7 @@ export const UiActionPlugin = Plugin.create({
                         })
                     )
                 }
-
+                const roles = arg.get('roles').ifArray()?.getElements() ?? []
                 const roleRecords: Record[] = []
                 for (const role of roles) {
                     const roleReference = role.isString()

package/src/ui-page-plugin.ts

@@ -17,6 +17,7 @@ import {
     type Factory,
     type Transform,
 } from '@servicenow/sdk-build-core'
+import { parseDocument, DomUtils } from 'htmlparser2'
 import { XMLParser, XMLBuilder, type X2jOptions, type XmlBuilderOptions } from 'fast-xml-parser'
 import { create } from 'xmlbuilder2'
 import { NowIdShape } from './now-id-plugin'
@@ -51,6 +52,62 @@ const builderOptions: XmlBuilderOptions = {
     ],
 }
 
+const LT_PLACEHOLDER = '\0__SDK_LT__\0'
+const AMP_PLACEHOLDER = '\0__SDK_AMP__\0'
+const RAW_CONTENT_TAGS = ['script', 'style', 'textarea'] as const
+
+/**
+ * Uses htmlparser2 to find script/style/textarea content and replaces `<` and
+ * `&` with placeholders. This prevents fast-xml-parser from misinterpreting
+ * JavaScript comparison operators (e.g. `a < b`) as XML tag openers, and
+ * prevents the XMLBuilder's entity escaping from converting `&` to `$[AMP]`
+ * inside script content.
+ *
+ * htmlparser2 is used instead of regex because it correctly handles edge cases
+ * like `>` inside attribute values and script tags inside HTML comments.
+ */
+function escapeRawContent(html: string): string {
+    const doc = parseDocument(html, { withStartIndices: true, withEndIndices: true })
+
+    const regions: { start: number; end: number }[] = []
+    for (const tag of RAW_CONTENT_TAGS) {
+        for (const el of DomUtils.getElementsByTagName(tag, doc, true)) {
+            for (const child of el.children) {
+                if (child.type === 'text' && child.startIndex != null && child.endIndex != null) {
+                    const text = child.data
+                    if (text.includes('<') || text.includes('&')) {
+                        regions.push({ start: child.startIndex, end: child.endIndex + 1 })
+                    }
+                }
+            }
+        }
+    }
+
+    if (regions.length === 0) {
+        return html
+    }
+
+    // Sort by position descending so replacements don't shift earlier indices
+    regions.sort((a, b) => b.start - a.start)
+
+    let result = html
+    for (const { start, end } of regions) {
+        const content = result.slice(start, end)
+        const escaped = content.replace(/&/g, AMP_PLACEHOLDER).replace(/</g, LT_PLACEHOLDER)
+        result = result.slice(0, start) + escaped + result.slice(end)
+    }
+
+    return result
+}
+
+/**
+ * Restores `<` and `&` characters that were replaced by `escapeRawContent`
+ * after fast-xml-parser has finished processing.
+ */
+function restoreRawContent(html: string): string {
+    return html.replaceAll(LT_PLACEHOLDER, '<').replaceAll(AMP_PLACEHOLDER, '&')
+}
+
 const POLARIS_APPSHELL_THEME_ID = 'c86a62e2c7022010099a308dc7c26022'
 const BYOUI_ARTIFACT_NAME_SUFFIX = 'BYOUI Files'
 // Matches the prefix HtmlImportPlugin prepends when it resolves an `import x from '*.html'`
@@ -388,9 +445,11 @@ export const UiPagePlugin = Plugin.create({
 
                 if (html) {
                     try {
+                        html = escapeRawContent(html)
                         const nodes = parser.parse(html)
                         const transformed = nodeTransformer(nodes)
                         html = new XMLBuilder(builderOptions).build(transformed)
+                        html = restoreRawContent(html)
                     } catch (error: unknown) {
                         if (error instanceof Error) {
                             diagnostics.error(arg.get('html'), error.message)
@@ -485,12 +544,14 @@ const getUIPageSourceFilePaths = (
 ): { files: string[]; assetNames: string[] } => {
     const empty = { files: [], assetNames: [] }
     try {
-        // Derive manifest path from HTML path
-        // The manifest is in the build output directory (staticContentDir), not the source directory
-        // e.g., src/client/index.html -> dist/static/index.ui-source-manifest.json
-        const htmlBasename = path.basename(htmlFilePath, '.html')
+        // Derive manifest path from HTML path by mirroring the directory structure
+        // from clientDir into staticContentDir and swapping the extension.
+        // e.g., src/client/index.html          -> dist/static/index.ui-source-manifest.json
+        //       src/client/admin/settings.html -> dist/static/admin/settings.ui-source-manifest.json
+        const clientAbsDir = path.join(rootDir, config.clientDir)
         const staticContentAbsDir = path.join(rootDir, config.staticContentDir)
-        const manifestPath = path.join(staticContentAbsDir, `${htmlBasename}.ui-source-manifest.json`)
+        const htmlRelPath = path.relative(clientAbsDir, htmlFilePath)
+        const manifestPath = path.join(staticContentAbsDir, htmlRelPath).replace(/\.html$/, '.ui-source-manifest.json')
 
         // Check if manifest file exists
         try {
@@ -510,24 +571,26 @@ const getUIPageSourceFilePaths = (
 
         // Derive the JS asset name from the manifest's entry field, matching
         // static-content-plugin's formula: path.join(scope, relativePath_without_ext).
-        // The manifest file is named after the HTML entry (e.g. index.ui-source-manifest.json)
-        // but the JS bundle is named after the JS entry (e.g. main.tsx -> main.jsdbx).
-        // Using manifest.entry's basename ensures the names align.
+        // The entry path is relative to the client directory and preserves subdirectories.
+        // e.g., src/client/main.tsx          -> scope/main
+        //       src/client/admin/settings.tsx -> scope/admin/settings
         if (!manifest.entry || typeof manifest.entry !== 'string') {
             logger.warn(`No entry field in manifest at ${manifestPath}`)
             return empty
         }
-        const entryBasename = path.basename(manifest.entry, path.extname(manifest.entry))
-        const entryAssetName = path.join(config.scope, entryBasename).replace(/\\/g, '/')
+        const entryRelativePath = path.relative(config.clientDir, manifest.entry).replace(/\\/g, '/')
+        const entryRelativeWithoutExt = entryRelativePath.replace(/\.[^.]+$/, '')
+        const entryAssetName = path.join(config.scope, entryRelativeWithoutExt).replace(/\\/g, '/')
 
         // Check if a source map bundle also exists in staticContentDir.
         // static-content-plugin names source map assets as: path.join(scope, relativePath.replace('dbx', ''))
-        // e.g. main.jsdbx.map -> main.js.map -> scope/main.js.map
+        // e.g. main.jsdbx.map -> scope/main.js.map
+        //      admin/settings.jsdbx.map -> scope/admin/settings.js.map
         const assetNames = [entryAssetName]
-        const sourceMapFilePath = path.join(staticContentAbsDir, `${entryBasename}.jsdbx.map`)
+        const sourceMapFilePath = path.join(staticContentAbsDir, `${entryRelativeWithoutExt}.jsdbx.map`)
         try {
             fs.accessSync(sourceMapFilePath)
-            const sourceMapAssetName = path.join(config.scope, `${entryBasename}.js.map`).replace(/\\/g, '/')
+            const sourceMapAssetName = path.join(config.scope, `${entryRelativeWithoutExt}.js.map`).replace(/\\/g, '/')
             assetNames.push(sourceMapAssetName)
         } catch {
             // no source map in this build output — skip

package/src/view-plugin.ts

@@ -1,4 +1,4 @@
-import { Plugin, Record, RecordId, Shape } from '@servicenow/sdk-build-core'
+import { Plugin, Record, RecordId, Shape, ts } from '@servicenow/sdk-build-core'
 
 export const ViewPlugin = Plugin.create({
     name: 'ViewPlugin',
@@ -11,7 +11,7 @@ export const ViewPlugin = Plugin.create({
     shapes: [
         {
             shape: Record,
-            inspect(record, { diagnostics }) {
+            inspect(record, { diagnostics, logger }) {
                 if (record.getTable() !== 'sys_ui_view') {
                     return
                 }
@@ -22,8 +22,14 @@ export const ViewPlugin = Plugin.create({
                 }
 
                 const viewName = record.get('name')
-                if (!viewName.isString() || !/^[a-zA-Z0-9_]+$/.test(viewName.getValue())) {
-                    diagnostics.error(viewName, `View name can only contain alphanumeric characters`)
+                if (!viewName.isString() || !/^[a-zA-Z0-9_,]+$/.test(viewName.getValue())) {
+                    if (ts.Node.isNode(viewName.getOriginalSource())) {
+                        diagnostics.error(viewName, `View name can only contain alphanumeric characters`)
+                    } else {
+                        logger.warn(
+                            `[ViewPlugin] View name '${viewName.isString() ? viewName.getValue() : ''}' in ${record.getOriginalFilePath()} can only contain alphanumeric characters`
+                        )
+                    }
                 }
             },
         },