fix: onEvent function to pass all the options to rule resource (#35829)

### Issue # (if applicable)

N/A

### Reason for this change

`onEvent()` has a lot of options other than description e.g. `ruleName` `detail` etc..
The old code didn't pass those values to rule, so it will never be propagated properly, so the consumer of this method will think that we're respecting their options, but in real we're discarding it.

### Description of changes

Creating the rule first with all the options so all the options from the consumer get propagated properly to the rule, then add the specific things that you want.

### Describe any new or updated permissions being added

N/A


### Description of how you validated changes

Add unit tests

### 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: gasolima

packages/@aws-cdk/aws-bedrock-alpha/bedrock/agents/agent.ts

@@ -133,21 +133,15 @@ export abstract class AgentBase extends Resource implements IAgent {
    * @returns An EventBridge Rule configured for agent events
    */
   public onEvent(id: string, options: events.OnEventOptions = {}): events.Rule {
-    // Create rule with minimal props and event pattern
-    const rule = new events.Rule(this, id, {
-      description: options.description,
-      eventPattern: {
-        source: ['aws.bedrock'],
-        detail: {
-          'agent-id': [this.agentId],
-        },
+    const rule = new events.Rule(this, id, options);
+    rule.addEventPattern({
+      source: ['aws.bedrock'],
+      detail: {
+        'agent-id': [this.agentId],
       },
     });
 
-    // Add target if provided
-    if (options.target) {
-      rule.addTarget(options.target);
-    }
+    rule.addTarget(options.target);
     return rule;
   }
 

packages/@aws-cdk/aws-bedrock-alpha/test/bedrock/agents/agent.test.ts

@@ -293,16 +293,27 @@ describe('Agent', () => {
 
       const rule = agent.onEvent('TestRule', {
         description: 'Custom rule description',
+        ruleName: 'MyCustomEventRuleName',
+        eventPattern: {
+          account: ['123456789012'],
+          region: ['us-east-1'],
+          detail: {
+            test: 'value',
+          },
+        },
       });
 
       expect(rule).toBeDefined();
       Template.fromStack(stack).hasResourceProperties('AWS::Events::Rule', {
         Description: 'Custom rule description',
+        Name: 'MyCustomEventRuleName',
         EventPattern: {
           source: ['aws.bedrock'],
           detail: {
             'agent-id': [{ 'Fn::GetAtt': [Match.stringLikeRegexp('TestAgent[A-Z0-9]+'), 'AgentId'] }],
           },
+          account: ['123456789012'],
+          region: ['us-east-1'],
         },
       });
     });

packages/@aws-cdk/example-construct-library/lib/example-resource.ts

@@ -201,11 +201,7 @@ abstract class ExampleResourceBase extends Resource implements IExampleResource
    * as it simplifies the implementation code (less branching).
    */
   public onEvent(id: string, options: events.OnEventOptions = {}): events.Rule {
-    const rule = new events.Rule(this, id, {
-      description: options.description,
-      ruleName: options.ruleName,
-      crossStackScope: options.crossStackScope,
-    });
+    const rule = new events.Rule(this, id, options);
     rule.addTarget(options.target);
     rule.addEventPattern({
       // obviously, you would put your resource-specific values here

packages/@aws-cdk/example-construct-library/test/example-resource.test.ts

@@ -121,13 +121,28 @@ describe('Example Resource', () => {
     });
 
     test('onEvent adds an Event Rule', () => {
-      exampleResource.onEvent('MyEvent');
+      exampleResource.onEvent('MyEvent', {
+        description: 'Custom rule description',
+        ruleName: 'MyCustomEventRuleName',
+        eventPattern: {
+          account: ['123456789012'],
+          region: ['us-east-1'],
+          detail: {
+            test: 'value',
+          },
+        },
+      });
 
       Template.fromStack(stack).hasResourceProperties('AWS::Events::Rule', {
+        Description: 'Custom rule description',
+        Name: 'MyCustomEventRuleName',
         EventPattern: {
           detail: {
             'example-resource-name': [EXAMPLE_RESOURCE_NAME],
+            'test': 'value',
           },
+          account: ['123456789012'],
+          region: ['us-east-1'],
         },
       });
     });