Skip to content

Commit 29727e6

Browse files
authored
fix: Make RemoveDuplicateNugetPackage failure a non-blocker for transformation
fix : adding a try catch block for error handling
1 parent f2ea9ac commit 29727e6

File tree

1 file changed

+14
-10
lines changed

1 file changed

+14
-10
lines changed

server/aws-lsp-codewhisperer/src/language-server/netTransform/artifactManager.ts

Lines changed: 14 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -61,17 +61,21 @@ export class ArtifactManager {
6161
}
6262

6363
async removeDuplicateNugetPackagesFolder(request: StartTransformRequest) {
64-
const packagesFolder = path.join(
65-
this.workspacePath,
66-
artifactFolderName,
67-
sourceCodeFolderName,
68-
packagesFolderName
69-
)
70-
if (fs.existsSync(packagesFolder)) {
71-
fs.rmSync(packagesFolder, { recursive: true, force: true })
72-
this.logging.log(
73-
`Removed packages folder ${packagesFolder} from source code directory to be uploaded because it is a duplicate of references folder from artifacts`
64+
try {
65+
const packagesFolder = path.join(
66+
this.workspacePath,
67+
artifactFolderName,
68+
sourceCodeFolderName,
69+
packagesFolderName
7470
)
71+
if (fs.existsSync(packagesFolder)) {
72+
fs.rmSync(packagesFolder, { recursive: true, force: true })
73+
this.logging.log(
74+
`Removed packages folder ${packagesFolder} from source code directory to be uploaded because it is a duplicate of references folder from artifacts`
75+
)
76+
}
77+
} catch (error) {
78+
this.logging.log('Failed to remove packages folder: ' + error)
7579
}
7680
}
7781

0 commit comments

Comments
 (0)