Tweak the PR comment for consistency when there are no new mappings generated

[chesterbot01]

Minor adjustment to keep the comment style consistent if there are no new mappings generated.

Previously, if new mappings were generated, the comment appeared as:

Found 0 unmapped Shopify category.

With the update, it will now read:

## 🤖 Automatic Taxonomy Mapping Update
No unmapped Shopify categories were found. As a result, **no changes have been made to this PR.**

[jeanrick]

Suggest renaming disagree_messages to something like low_confidence_mappings

Suggested change

	all_generated_mappings: [], disagree_messages: [], total_count: 0, current_iteration_count: 0
	all_generated_mappings: [], low_confidence_mappings: [], total_count: 0, num_mappings_generated: 0

[jeanrick]

Is it possible for the system to have 0 recommendation for a missing mapping? i.e. can we have a scenario where all_generated_mappings.size is less than current_iteration_count? If yes, then we should include a third section where we tell the user that we were unable to generate mappings for the categories that are still missing (and list the ones that are missing).

Looking at how we use this, a better name for current_iteration_count would be something like starting_num_of_unmapped_categories

product-taxonomy/app/commands/generate_missing_mappings_command.rb

Line 233 in 4d65576

current_iteration_count: unmapped_categories[:id_name_map].size,

[chesterbot01]

can we have a scenario where all_generated_mappings.size is less than current_iteration_count

No, semantic search will consistently provide some results based on proximity. And this question is thought-provoking, we can get rid of current_iteration_count entirely because all_generated_mappings.length always == current_iteration_count

[chesterbot01]

Renamed parameters and removed redundant one.