add aggregate reporting for rules add command
This commit is contained in:
@@ -2740,6 +2740,7 @@ Examples:
|
|||||||
}
|
}
|
||||||
|
|
||||||
const removalResults = [];
|
const removalResults = [];
|
||||||
|
const addResults = [];
|
||||||
|
|
||||||
for (const profile of expandedProfiles) {
|
for (const profile of expandedProfiles) {
|
||||||
if (!isValidProfile(profile)) {
|
if (!isValidProfile(profile)) {
|
||||||
@@ -2762,6 +2763,14 @@ Examples:
|
|||||||
console.log(
|
console.log(
|
||||||
chalk.blue(`Completed adding rules for profile: ${profile}`)
|
chalk.blue(`Completed adding rules for profile: ${profile}`)
|
||||||
);
|
);
|
||||||
|
|
||||||
|
// Store result with profile name for summary
|
||||||
|
addResults.push({
|
||||||
|
profileName: profile,
|
||||||
|
success: addResult.success,
|
||||||
|
failed: addResult.failed
|
||||||
|
});
|
||||||
|
|
||||||
console.log(
|
console.log(
|
||||||
chalk.green(
|
chalk.green(
|
||||||
`Summary for ${profile}: ${addResult.success} rules added, ${addResult.failed} failed.`
|
`Summary for ${profile}: ${addResult.success} rules added, ${addResult.failed} failed.`
|
||||||
@@ -2780,6 +2789,28 @@ Examples:
|
|||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
|
// Print summary for additions
|
||||||
|
if (action === RULES_ACTIONS.ADD && addResults.length > 0) {
|
||||||
|
const totalSuccess = addResults.reduce((sum, r) => sum + r.success, 0);
|
||||||
|
const totalFailed = addResults.reduce((sum, r) => sum + r.failed, 0);
|
||||||
|
const successfulProfiles = addResults
|
||||||
|
.filter((r) => r.success > 0)
|
||||||
|
.map((r) => r.profileName);
|
||||||
|
|
||||||
|
if (successfulProfiles.length > 0) {
|
||||||
|
console.log(
|
||||||
|
chalk.green(
|
||||||
|
`\nSuccessfully added rules for: ${successfulProfiles.join(', ')}`
|
||||||
|
)
|
||||||
|
);
|
||||||
|
console.log(
|
||||||
|
chalk.green(
|
||||||
|
`Total: ${totalSuccess} rules added, ${totalFailed} failed.`
|
||||||
|
)
|
||||||
|
);
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
// Print summary for removals
|
// Print summary for removals
|
||||||
if (action === RULES_ACTIONS.REMOVE) {
|
if (action === RULES_ACTIONS.REMOVE) {
|
||||||
const successes = removalResults
|
const successes = removalResults
|
||||||
|
|||||||
Reference in New Issue
Block a user