Fix double 100% progress on CompileResponse

[alessio-perugini]

Please check if the PR fulfills these requirements

See how to contribute

• The PR has no duplicates (please search among the Pull Requests
  before creating one)
• The PR follows
  our contributing guidelines
• Tests for the changes have been added (for bug fixes / features)
• Docs have been added / updated (for bug fixes / features)
• UPGRADING.md has been updated with a migration guide (for breaking changes)
• configuration.schema.json updated if new parameters are added.

What kind of change does this PR introduce?

Fixes double 100% status on task upgrade when compiling

What is the current behavior?

Right now the builder command calls the runCommands 2 times which leads to having 2 100% values.

What is the new behavior?

The new behavior only sends 1 100% task status and completed property at true when done. I intentionally didn't try to refactor this legacy component.

Does this PR introduce a breaking change, and is titled accordingly?

Other information

[codecov]

Codecov Report

Patch coverage: 68.51% and project coverage change: -0.02% ⚠️

Comparison is base (b678f6f) 62.97% compared to head (d1a7a6a) 62.95%.
Report is 2 commits behind head on master.

Additional details and impacted files

@@            Coverage Diff             @@
##           master    #2225      +/-   ##
==========================================
- Coverage   62.97%   62.95%   -0.02%     
==========================================
  Files         220      220              
  Lines       19501    19538      +37     
==========================================
+ Hits        12280    12300      +20     
- Misses       6137     6149      +12     
- Partials     1084     1089       +5     

Flag	Coverage Δ
unit	62.95% <68.51%> (-0.02%)	⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

Files Changed	Coverage Δ
arduino/cores/packagemanager/package_manager.go	76.61% <0.00%> (-0.44%)	⬇️
arduino/resources/index.go	46.29% <33.33%> (-2.73%)	⬇️
commands/instances.go	64.43% <64.28%> (-0.27%)	⬇️
legacy/builder/builder.go	84.34% <86.36%> (-0.66%)	⬇️
legacy/builder/container_setup.go	82.35% <100.00%> (-0.99%)	⬇️
legacy/builder/types/context.go	97.61% <100.00%> (+0.08%)	⬆️

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.