feat(codecommit): throw ValidationErrors instead of untyped Errors (#33854)

### Issue # (if applicable)

Relates to #32569

### Reason for this change

untyped Errors are not recommended

### Description of changes

ValidationErrors everywhere

### Describe any new or updated permissions being added

None

### Description of how you validated changes

Existing tests. Exemptions granted as this is a refactor of existing code.

### Checklist
- [x] My code adheres to the [CONTRIBUTING GUIDE](https://github.com/aws/aws-cdk/blob/main/CONTRIBUTING.md) and [DESIGN GUIDELINES](https://github.com/aws/aws-cdk/blob/main/docs/DESIGN_GUIDELINES.md)

----

*By submitting this pull request, I confirm that my contribution is made under the terms of the Apache-2.0 license*

Author: kaizencc

packages/aws-cdk-lib/.eslintrc.js

@@ -41,8 +41,9 @@ const enableNoThrowDefaultErrorIn = [
   'aws-cloudwatch-actions',
   'aws-codeartifact',
   'aws-codebuild',
-  'aws-cognito',
+  'aws-codecommit',
   'aws-codedeploy',
+  'aws-cognito',
   'aws-ecr',
   'aws-elasticloadbalancing',
   'aws-elasticloadbalancingv2',

packages/aws-cdk-lib/aws-codecommit/lib/code.ts

@@ -3,6 +3,7 @@ import * as path from 'path';
 import { Construct } from 'constructs';
 import { CfnRepository } from './codecommit.generated';
 import * as assets from '../../aws-s3-assets';
+import { UnscopedValidationError, ValidationError } from '../../core';
 
 /**
  * Represents the structure to pass into the underlying CfnRepository class.
@@ -28,7 +29,7 @@ export abstract class Code {
 
     const statResult = fs.statSync(resolvedPath);
     if (!statResult || !statResult.isDirectory()) {
-      throw new Error(`'${directoryPath}' needs to be a path to a directory (resolved to: '${resolvedPath }')`);
+      throw new UnscopedValidationError(`'${directoryPath}' needs to be a path to a directory (resolved to: '${resolvedPath }')`);
     }
 
     return new PathResolvedCode(resolvedPath, branch);
@@ -44,7 +45,7 @@ export abstract class Code {
 
     const statResult = fs.statSync(resolvedPath);
     if (!statResult || !statResult.isFile()) {
-      throw new Error(`'${filePath}' needs to be a path to a ZIP file (resolved to: '${resolvedPath }')`);
+      throw new UnscopedValidationError(`'${filePath}' needs to be a path to a ZIP file (resolved to: '${resolvedPath }')`);
     }
 
     return new PathResolvedCode(resolvedPath, branch);
@@ -86,9 +87,9 @@ class AssetCode extends Code {
     super();
   }
 
-  public bind(_scope: Construct): CodeConfig {
+  public bind(scope: Construct): CodeConfig {
     if (!this.asset.isZipArchive) {
-      throw new Error('Asset must be a .zip file or a directory (resolved to: ' + this.asset.assetPath + ' )');
+      throw new ValidationError('Asset must be a .zip file or a directory (resolved to: ' + this.asset.assetPath + ' )', scope);
     }
 
     return {

packages/aws-cdk-lib/aws-codecommit/lib/repository.ts

@@ -5,7 +5,7 @@ import * as notifications from '../../aws-codestarnotifications';
 import * as events from '../../aws-events';
 import * as iam from '../../aws-iam';
 import * as kms from '../../aws-kms';
-import { ArnFormat, IResource, Lazy, Resource, Stack } from '../../core';
+import { ArnFormat, IResource, Lazy, Resource, Stack, ValidationError } from '../../core';
 import { addConstructMetadata, MethodMetadata } from '../../core/lib/metadata-resource';
 
 /**
@@ -605,7 +605,7 @@ export class Repository extends RepositoryBase {
     }
 
     if (this.triggers.find(prop => prop.name === name)) {
-      throw new Error(`Unable to set repository trigger named ${name} because trigger names must be unique`);
+      throw new ValidationError(`Unable to set repository trigger named ${name} because trigger names must be unique`, this);
     }
 
     this.triggers.push({