aws-samples / aws-samples/aws-serverless-developer-experience-workshop-dotnet

Cross-runtime consistency fixes

Open
#82 0 comments 0 reactions 0 assignees View on GitHub
pending-release
Dominant language
C#
Stars
19
Forks
13
PR merge metrics
No merged PRs in 30d

Description

## Summary

A set of fixes to align the .NET runtime with the other runtimes (Java, Python, TypeScript) for functional equivalence and cross-runtime consistency.

---

## Contracts Service

### Metric name correction (`ContractEventHandler.cs`)
- `CreateContractAsync`: renamed metric from `"NewContracts"` to `"ContractCreated"`
- Aligns with the canonical metric name used by all other runtimes

### Missing metric (`ContractEventHandler.cs`)
- `UpdateContractAsync`: added `Metrics.AddMetric("ContractUpdated", 1, MetricUnit.Count, MetricResolution.Standard)` after a successful `UpdateItemAsync` call
- The update path previously emitted no custom metric; this brings it in line with all other runtimes

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.