diff --git a/.Jules/palette.md b/.Jules/palette.md index f8c1fbd..9296c26 100644 --- a/.Jules/palette.md +++ b/.Jules/palette.md @@ -5,3 +5,4 @@ ## 2026-06-30 - Added Emojis to CLI Output Messages **Learning:** Adding subtle emojis to informative CLI output headers (like "Created/updated files" and "Next steps") provides clearer visual cues for developers scanning long CLI output. **Action:** Always include relevant emojis in summary output text to make success and informational messages more visually distinguishable from routine command logs. +## 2026-07-06 - Added Emojis to CLI Warning Messages\n**Learning:** Just like errors, warnings in CLIs benefit from visual cues like emojis to draw attention without being as harsh as an error.\n**Action:** When adding or modifying warning messages in the CLI, use '⚠️ Warning: [Message]' format. diff --git a/appguardrail_core/org_bundle.py b/appguardrail_core/org_bundle.py index 3634d07..6022518 100644 --- a/appguardrail_core/org_bundle.py +++ b/appguardrail_core/org_bundle.py @@ -282,7 +282,7 @@ def gh_pr_list( except subprocess.CalledProcessError as exc: warning = f"Skipped PR collection for {full_name}: {gh_error_message(exc)}" warnings.append(warning) - print(f"warning: {warning}", file=sys.stderr) + print(f"⚠️ Warning: {warning}", file=sys.stderr) continue for pull in repo_pulls: pull["repository"] = {"nameWithOwner": full_name} diff --git a/tests/test_org_readiness_report_script.py b/tests/test_org_readiness_report_script.py index 506b51c..2262bcc 100644 --- a/tests/test_org_readiness_report_script.py +++ b/tests/test_org_readiness_report_script.py @@ -120,6 +120,6 @@ def fake_gh_json(args): "Skipped PR collection for ContextualWisdomLab/bad: HTTP 502: Bad Gateway" ] assert ( - "warning: Skipped PR collection for ContextualWisdomLab/bad" + "⚠️ Warning: Skipped PR collection for ContextualWisdomLab/bad" in capsys.readouterr().err )