fix(diagnostics): add diagnostic logging around compilation and unpack to aid troubleshooting
This commit is contained in:
@@ -1,5 +1,13 @@
|
|||||||
# Changelog
|
# Changelog
|
||||||
|
|
||||||
|
## 2026-03-05 - 4.1.5 - fix(diagnostics)
|
||||||
|
add diagnostic logging around compilation and unpack to aid troubleshooting
|
||||||
|
|
||||||
|
- Enable TypeScript CompilerOptions.listEmittedFiles to surface emitted file info
|
||||||
|
- Log destination directory contents and emitted file/error counts after compile and after unpack (only when not quiet and not json)
|
||||||
|
- Add unpack-specific diagnostic logs: shouldUnpack skip, detectNesting result and nestedPath, and nested/destination directory listings before removing sibling directories
|
||||||
|
- All logs use console.log and are wrapped in try/catch to avoid throwing; changes are informational and non-breaking
|
||||||
|
|
||||||
## 2026-03-05 - 4.1.4 - fix(deps)
|
## 2026-03-05 - 4.1.4 - fix(deps)
|
||||||
bump @git.zone/tspublish dependency to ^1.11.2
|
bump @git.zone/tspublish dependency to ^1.11.2
|
||||||
|
|
||||||
|
|||||||
@@ -3,6 +3,6 @@
|
|||||||
*/
|
*/
|
||||||
export const commitinfo = {
|
export const commitinfo = {
|
||||||
name: '@git.zone/tsbuild',
|
name: '@git.zone/tsbuild',
|
||||||
version: '4.1.4',
|
version: '4.1.5',
|
||||||
description: 'A tool for compiling TypeScript files using the latest nightly features, offering flexible APIs and a CLI for streamlined development.'
|
description: 'A tool for compiling TypeScript files using the latest nightly features, offering flexible APIs and a CLI for streamlined development.'
|
||||||
}
|
}
|
||||||
|
|||||||
@@ -349,6 +349,7 @@ export class TsCompiler {
|
|||||||
const options: CompilerOptions = {
|
const options: CompilerOptions = {
|
||||||
...customOptions,
|
...customOptions,
|
||||||
outDir: destDir,
|
outDir: destDir,
|
||||||
|
listEmittedFiles: true,
|
||||||
};
|
};
|
||||||
|
|
||||||
currentTask++;
|
currentTask++;
|
||||||
@@ -364,9 +365,33 @@ export class TsCompiler {
|
|||||||
emittedFiles.push(...result.emittedFiles);
|
emittedFiles.push(...result.emittedFiles);
|
||||||
errorSummaries.push(result.errorSummary);
|
errorSummaries.push(result.errorSummary);
|
||||||
|
|
||||||
|
// DIAGNOSTIC: Log directory contents after compilation
|
||||||
|
if (!isQuiet && !isJson) {
|
||||||
|
try {
|
||||||
|
const fs = await import('fs');
|
||||||
|
const dirEntries = fs.readdirSync(destDir, { withFileTypes: true });
|
||||||
|
const dirs = dirEntries.filter((e: any) => e.isDirectory()).map((e: any) => e.name).sort();
|
||||||
|
const fileCount = dirEntries.filter((e: any) => e.isFile()).length;
|
||||||
|
console.log(` 📋 [DIAG] After emit: ${dirs.length} dirs [${dirs.join(', ')}], ${fileCount} top-level files`);
|
||||||
|
console.log(` 📋 [DIAG] emittedFiles.length=${result.emittedFiles.length}, totalErrors=${result.errorSummary.totalErrors}`);
|
||||||
|
} catch (e: any) { console.log(` 📋 [DIAG] Error reading destDir: ${e.message}`); }
|
||||||
|
}
|
||||||
|
|
||||||
// Perform unpack if compilation succeeded
|
// Perform unpack if compilation succeeded
|
||||||
if (result.errorSummary.totalErrors === 0) {
|
if (result.errorSummary.totalErrors === 0) {
|
||||||
await performUnpack(pattern, destDir, this.cwd);
|
await performUnpack(pattern, destDir, this.cwd);
|
||||||
|
|
||||||
|
// DIAGNOSTIC: Log directory contents after unpack
|
||||||
|
if (!isQuiet && !isJson) {
|
||||||
|
try {
|
||||||
|
const fs = await import('fs');
|
||||||
|
const dirEntries = fs.readdirSync(destDir, { withFileTypes: true });
|
||||||
|
const dirs = dirEntries.filter((e: any) => e.isDirectory()).map((e: any) => e.name).sort();
|
||||||
|
const fileCount = dirEntries.filter((e: any) => e.isFile()).length;
|
||||||
|
console.log(` 📋 [DIAG] After unpack: ${dirs.length} dirs [${dirs.join(', ')}], ${fileCount} top-level files`);
|
||||||
|
} catch (e: any) { console.log(` 📋 [DIAG] Error reading destDir after unpack: ${e.message}`); }
|
||||||
|
}
|
||||||
|
|
||||||
successfulOutputDirs.push(destDir);
|
successfulOutputDirs.push(destDir);
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|||||||
@@ -87,16 +87,30 @@ export class TsUnpacker {
|
|||||||
public async unpack(): Promise<boolean> {
|
public async unpack(): Promise<boolean> {
|
||||||
// Check if we should unpack based on config
|
// Check if we should unpack based on config
|
||||||
if (!(await this.shouldUnpack())) {
|
if (!(await this.shouldUnpack())) {
|
||||||
|
console.log(` 📋 [DIAG-UNPACK] Skipping unpack: shouldUnpack=false`);
|
||||||
return false;
|
return false;
|
||||||
}
|
}
|
||||||
|
|
||||||
// Check if nested structure exists
|
// Check if nested structure exists
|
||||||
if (!(await this.detectNesting())) {
|
const hasNesting = await this.detectNesting();
|
||||||
|
console.log(` 📋 [DIAG-UNPACK] detectNesting=${hasNesting}, nestedPath=${this.getNestedPath()}`);
|
||||||
|
if (!hasNesting) {
|
||||||
return false;
|
return false;
|
||||||
}
|
}
|
||||||
|
|
||||||
const nestedPath = this.getNestedPath();
|
const nestedPath = this.getNestedPath();
|
||||||
|
|
||||||
|
// DIAGNOSTIC: Log contents before unpack
|
||||||
|
try {
|
||||||
|
const destEntries = fs.readdirSync(this.destDir, { withFileTypes: true });
|
||||||
|
const destDirs = destEntries.filter(e => e.isDirectory()).map(e => e.name).sort();
|
||||||
|
console.log(` 📋 [DIAG-UNPACK] Before removeSibling: destDir dirs=[${destDirs.join(', ')}]`);
|
||||||
|
const nestedEntries = fs.readdirSync(nestedPath, { withFileTypes: true });
|
||||||
|
const nestedDirs = nestedEntries.filter(e => e.isDirectory()).map(e => e.name).sort();
|
||||||
|
const nestedFiles = nestedEntries.filter(e => e.isFile()).length;
|
||||||
|
console.log(` 📋 [DIAG-UNPACK] Nested dir (${this.sourceFolderName}/): ${nestedDirs.length} dirs [${nestedDirs.join(', ')}], ${nestedFiles} files`);
|
||||||
|
} catch (e: any) { console.log(` 📋 [DIAG-UNPACK] Error reading: ${e.message}`); }
|
||||||
|
|
||||||
// Delete sibling folders (not the source folder)
|
// Delete sibling folders (not the source folder)
|
||||||
await this.removeSiblingDirectories();
|
await this.removeSiblingDirectories();
|
||||||
|
|
||||||
|
|||||||
Reference in New Issue
Block a user