Skip to content

v0.3.22 - Fix Whitespace Violation Reporting

Choose a tag to compare

View all tags
@mmcky mmcky released this 11 Oct 00:30
· 38 commits to main since this release
v0.3.22
da76e27

Bug Fix 🐛

Fixes issue where whitespace violations (qe-writing-008) were being detected but not applied.

Problem

  • LLM detected 6 whitespace violations
  • All 6 were skipped: "Applied 0/6 fixes"
  • Warning: "Fix quality warnings: 6"
  • Root cause: Prompt instruction "Current text and Suggested fix MUST be different" was interpreted too strictly
  • LLM avoided reporting whitespace fixes because they looked visually "the same"

Solution

  • ✅ Clarified CRITICAL instruction: "must change something" instead of "MUST be different"
  • ✅ Added explicit note: "Even if the visual difference is subtle (like whitespace changes), ensure the suggested fix actually corrects the violation"
  • ✅ LLM now understands that subtle differences (whitespace) are valid and important

Impact

  • Fixes qe-writing-008 (Remove excessive whitespace) rule application
  • Should improve fix application rate for all subtle text changes
  • Updated writing prompt version to 0.3.22

Full Changelog: https://github.com/QuantEcon/action-style-guide/blob/main/CHANGELOG.md

Assets 2
Loading