Remove all try-catch blocks from CalculaBaseCalculoIbsCbs

[Copilot]

Addresses code review feedback on PR #42 to remove all exception handling from the CalculaBaseCalculoIbsCbs class.

Changes

• Removed all try-catch blocks from 4 calculation methods:
  • CalcularValorPis()
  • CalcularValorCofins()
  • CalcularValorIcms()
  • CalcularValorIssqn()
• Removed the using System; directive (no longer needed)

// Before
private decimal CalcularValorPis()
{
    try
    {
        var tributacaoPis = new TributacaoPis(_tributavel, _tipoDesconto);
        return tributacaoPis.Calcula().Valor;
    }
    catch
    {
        // Em caso de erro no cálculo, retorna 0 para não impactar o cálculo do IBS/CBS
        return 0m;
    }
}

// After  
private decimal CalcularValorPis()
{
    var tributacaoPis = new TributacaoPis(_tributavel, _tipoDesconto);
    return tributacaoPis.Calcula().Valor;
}

⚠️ Breaking Change

This is a breaking behavioral change. Previously, these methods would silently return 0 on any exception. Now, any exceptions from the calculation methods will propagate to calling code, allowing proper error handling at a higher level in the application.

Testing

• ✅ Build successful for all target frameworks (net45, net48, net6.0, net8.0)
• ✅ All 143 unit tests pass on .NET 8.0
• ✅ No security vulnerabilities detected

---

✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.

[robertorp]

@copilot remova todos try catch CalculaBaseCalculoIbsCbs

[Copilot]

@copilot remova todos try catch CalculaBaseCalculoIbsCbs

Concluído em bec1482. Removi todos os blocos try-catch de CalculaBaseCalculoIbsCbs (4 métodos: CalcularValorPis, CalcularValorCofins, CalcularValorIcms e CalcularValorIssqn). Todos os 143 testes continuam passando.