Skip to content

Conversation

Copy link
Contributor

Copilot AI commented Oct 6, 2025

Plan to fix ForCreation type parameter order

  • Analyze codebase and understand current state
  • Identify all files that need changes
  • Update IPolicyConfiguration interface in Policies.cs
  • Update PolicyConfiguration implementation in PoliciesApiInternal.cs
  • Update BuildPolicyBuilder and ApplyToPolicyBuilder class definitions
  • Update all IBuildPolicyBuilder and IApplyToPolicyBuilder interface usages
  • Run tests to verify changes
  • Verify MapperGenerator uses type parameters correctly
  • Verify CodeBuilder uses type parameters correctly
  • Final verification

Changes made:

  1. Fixed type parameter order in interface and implementation:

    • Updated ForCreation<TDestination, TSource>() to ForCreation<TSource, TDestination>()
    • Updated ForModification<TDestination, TSource>() to ForModification<TSource, TDestination>()
  2. Improved MapperGenerator.cs clarity:

    • Renamed sourceType/destinationType to entityType/dtoType to avoid confusion
    • Added clarifying comments showing the mapping direction for each method
    • Project: Entity → DTO
    • Build: DTO → Entity
    • ApplyTo: DTO → Entity
  3. Added clarifying comments to CodeBuilder.cs:

    • Documented what SourceType and DestinationType represent for each method
    • Made it clear what method signature gets generated

Verification:

  • ✅ All 95 tests pass
  • ✅ Sample project builds successfully
  • ✅ Type parameters are used correctly throughout the codebase
  • ✅ No breaking changes to existing usage
Original prompt

Modifica la firma di ForCreation in ProjectR per usare parametri di tipo più chiari e corretti: TSource come primo parametro e TDestination come secondo, sia nell'interfaccia IPolicyConfiguration che nelle implementazioni e negli extension method. Aggiorna anche eventuali riferimenti e test di conseguenza.

*This pull request was created as a result of the following prompt from Copilot chat.* > Modifica la firma di ForCreation in ProjectR per usare parametri di tipo più chiari e corretti: TSource come primo parametro e TDestination come secondo, sia nell'interfaccia IPolicyConfiguration che nelle implementazioni e negli extension method. Aggiorna anche eventuali riferimenti e test di conseguenza.

💬 Share your feedback on Copilot coding agent for the chance to win a $200 gift card! Click here to start the survey.

@lucafabbri
Copy link
Owner

@copilot devi verificare che dentro al MapperGenerator e dentro al CodeBuilder i parametri siano utilizzati correttamente

Copilot AI changed the title [WIP] Rinomina type parameter e parametri di ForCreation per chiarezza (TSource, TDestination) Correggi l'ordine dei parametri di tipo in ForCreation e ForModification per usare TSource e TDestination in modo coerente Oct 6, 2025
Copilot AI requested a review from lucafabbri October 6, 2025 10:40
Copy link
Contributor Author

Copilot AI commented Oct 6, 2025

@copilot devi verificare che dentro al MapperGenerator e dentro al CodeBuilder i parametri siano utilizzati correttamente

Ho verificato e corretto l'utilizzo dei parametri in MapperGenerator e CodeBuilder.

Modifiche effettuate (commit d15f835):

  1. MapperGenerator.cs: Rinominato le variabili sourceType/destinationType in entityType/dtoType per chiarezza, dato che i nomi precedenti erano fuorvianti nel contesto del metodo Build. Aggiunto commenti per chiarire la direzione del mapping:

    • Project: Entity → DTO
    • Build: DTO → Entity
    • ApplyTo: DTO → Entity
  2. CodeBuilder.cs: Aggiunto commenti per documentare cosa rappresentano SourceType e DestinationType in ogni metodo generato.

I parametri sono ora utilizzati correttamente e in modo coerente con la nuova firma dell'interfaccia. Tutti i test passano (95/95).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants