Exclude admin/tsconfig.json from strict JSON validation and add base directory JSON validation (#700)

Co-authored-by: copilot-swe-agent[bot] <[email protected]>
Co-authored-by: Apollon77 <[email protected]>
Co-authored-by: Ingo Fischer <[email protected]>

Author: Copilot

CHANGELOG.md

@@ -4,6 +4,10 @@
 	PLACEHOLDER for the next version:
 	## **WORK IN PROGRESS**
 -->
+## **WORK IN PROGRESS**
+* (@Apollon77/@copilot) Exclude admin/tsconfig.json from strict JSON validation as it may contain JSON5 syntax
+* (@Apollon77/@copilot) Add JSON validation for package.json and io-package.json in base directory
+
 ## 5.2.1 (2025-11-08)
 * (@Apollon77/@copilot) Add validation for JSON files in admin/ and admin/i18n/ directories
 * (@Apollon77/@copilot) Logs npm and installation errors to console for easier debugging

build/tests/packageFiles/index.js

@@ -291,12 +291,26 @@ function validatePackageFiles(adapterDir) {
             });
         });
         describe(`Validate JSON files`, () => {
+            // Validate base directory JSON files
+            describe(`Base directory JSON files`, () => {
+                for (const filename of ['package.json', 'io-package.json']) {
+                    const filePath = path.join(adapterDir, filename);
+                    if (fs.existsSync(filePath)) {
+                        it(`${filename} contains valid JSON`, () => {
+                            (0, chai_1.expect)(() => {
+                                JSON.parse(fs.readFileSync(filePath, 'utf8'));
+                            }, `${filename} contains invalid JSON!`).not.to.throw();
+                        });
+                    }
+                }
+            });
             // Find all JSON and JSON5 files in admin/ directory (recursively)
             const adminDir = path.join(adapterDir, 'admin');
             const allAdminJsonFiles = findFiles(adminDir, /\.json$/);
             const allAdminJson5Files = findFiles(adminDir, /\.json5$/);
             // Split JSON files into admin/*.json and admin/i18n/**/*.json
-            const adminDirectJsonFiles = allAdminJsonFiles.filter(file => !file.includes(`${path.sep}i18n${path.sep}`));
+            // Exclude tsconfig.json as it may contain JSON5 syntax (comments, trailing commas)
+            const adminDirectJsonFiles = allAdminJsonFiles.filter(file => !file.includes(`${path.sep}i18n${path.sep}`) && !file.endsWith(`${path.sep}tsconfig.json`));
             const i18nJsonFiles = allAdminJsonFiles.filter(file => file.includes(`${path.sep}i18n${path.sep}`));
             if (adminDirectJsonFiles.length > 0) {
                 describe(`admin/*.json files`, () => {

src/tests/packageFiles/index.ts

@@ -302,13 +302,30 @@ export function validatePackageFiles(adapterDir: string): void {
         });
 
         describe(`Validate JSON files`, () => {
+            // Validate base directory JSON files
+            describe(`Base directory JSON files`, () => {
+                for (const filename of ['package.json', 'io-package.json']) {
+                    const filePath = path.join(adapterDir, filename);
+                    if (fs.existsSync(filePath)) {
+                        it(`${filename} contains valid JSON`, () => {
+                            expect(() => {
+                                JSON.parse(fs.readFileSync(filePath, 'utf8'));
+                            }, `${filename} contains invalid JSON!`).not.to.throw();
+                        });
+                    }
+                }
+            });
+
             // Find all JSON and JSON5 files in admin/ directory (recursively)
             const adminDir = path.join(adapterDir, 'admin');
             const allAdminJsonFiles = findFiles(adminDir, /\.json$/);
             const allAdminJson5Files = findFiles(adminDir, /\.json5$/);
 
             // Split JSON files into admin/*.json and admin/i18n/**/*.json
-            const adminDirectJsonFiles = allAdminJsonFiles.filter(file => !file.includes(`${path.sep}i18n${path.sep}`));
+            // Exclude tsconfig.json as it may contain JSON5 syntax (comments, trailing commas)
+            const adminDirectJsonFiles = allAdminJsonFiles.filter(
+                file => !file.includes(`${path.sep}i18n${path.sep}`) && !file.endsWith(`${path.sep}tsconfig.json`),
+            );
             const i18nJsonFiles = allAdminJsonFiles.filter(file => file.includes(`${path.sep}i18n${path.sep}`));
 
             if (adminDirectJsonFiles.length > 0) {