fix(anglicise): unify drifted is_excluded_text_line functions

xaoscience · xaoscience · commit cf8f55a49ac4 · 2026-03-21T20:48:30.000+01:00
The check and fix steps had diverged; each had exclusion patterns the other was missing (file://, Authorization header, standalone --color/--center flags, case-insensitive assignment matching, broader ${} variable detection). Unified to a single superset pattern in all 4 copies (live + template × check + fix).
diff --git a/.github/workflows/anglicise.yml b/.github/workflows/anglicise.yml
@@ -318,7 +318,7 @@ jobs:
 
           is_excluded_text_line() {
             local line="$1"
-            echo "$line" | grep -qE '(https?://|file://|github/|[?&](color|center|gray|licence)=[a-zA-Z0-9]+|"Authorization": "Bearer|types:[[:space:]]*\[[^]]+\]|--[a-z-]+ (center|color|gray)|\$\{?(color|center|gray);\}|\b(color|center|gray|licence)[_A-Z]*=|license/licence|licence/license|license_file|licence_file|LICENSE(_FILES?)?|LICENSE.*LICENCE|LICENCE.*LICENSE|gum |\\$|shields\.io|stop-color|[a-z-]color[a-z-]*:|color[a-z-]*:[[:space:]]*#|align-items:)'
+            echo "$line" | grep -qE "(https?://|file://|github/|[?&](color|center|gray|licence)=|=[\"'](center|color|licence)|\"Authorization\": \"Bearer|types:[[:space:]]*\[[^]]+\]|--[a-z-]+ (center|color|gray)|--color|--center|\$\{?(color|center|gray|licence)|\b(color|center|gray|licence|COLOR|CENTER|GRAY|LICENCE)[_A-Z]*=|license/licence|licence/license|license_file|licence_file|LICENSE(_FILES?)?|LICENSE.*LICENCE|LICENCE.*LICENSE|gum |\\$|shields\.io|stop-color|[a-z-]color[a-z-]*:|color[a-z-]*:[[:space:]]*#|align-items:)"
           }
 
           echo "## British English Spelling Check" >> $GITHUB_STEP_SUMMARY
@@ -427,7 +427,7 @@ jobs:
 
           is_excluded_text_line() {
             local line="$1"
-            echo "$line" | grep -qE "(https?://|github/|[?&](color|center|gray|licence)=|=[\"'](center|color|licence)|types:[[:space:]]*\[[^]]+\]|--[a-z-]+ (center|color|gray)|--color|--center|\$\{?(color|center|gray|licence)|\b(COLOR|CENTER|GRAY|LICENCE)[_A-Z]*=|license/licence|licence/license|license_file|licence_file|LICENSE(_FILES?)?|LICENSE.*LICENCE|LICENCE.*LICENSE|gum |\\$|shields\.io|stop-color|[a-z-]color[a-z-]*:|color[a-z-]*:[[:space:]]*#|align-items:)"
+            echo "$line" | grep -qE "(https?://|file://|github/|[?&](color|center|gray|licence)=|=[\"'](center|color|licence)|\"Authorization\": \"Bearer|types:[[:space:]]*\[[^]]+\]|--[a-z-]+ (center|color|gray)|--color|--center|\$\{?(color|center|gray|licence)|\b(color|center|gray|licence|COLOR|CENTER|GRAY|LICENCE)[_A-Z]*=|license/licence|licence/license|license_file|licence_file|LICENSE(_FILES?)?|LICENSE.*LICENCE|LICENCE.*LICENSE|gum |\\$|shields\.io|stop-color|[a-z-]color[a-z-]*:|color[a-z-]*:[[:space:]]*#|align-items:)"
           }
           
           FIXED_COUNT=0
diff --git a/workflows-templates/anglicise.yml b/workflows-templates/anglicise.yml
@@ -321,7 +321,7 @@ jobs:
 
           is_excluded_text_line() {
             local line="$1"
-            echo "$line" | grep -qE '(https?://|file://|github/|[?&](color|center|gray|licence)=[a-zA-Z0-9]+|"Authorization": "Bearer|types:[[:space:]]*\[[^]]+\]|--[a-z-]+ (center|color|gray)|\$\{?(color|center|gray);\}|\b(color|center|gray|licence)[_A-Z]*=|license/licence|licence/license|license_file|licence_file|LICENSE(_FILES?)?|LICENSE.*LICENCE|LICENCE.*LICENSE|gum |\\$|shields\.io|stop-color|[a-z-]color[a-z-]*:|color[a-z-]*:[[:space:]]*#|align-items:)'
+            echo "$line" | grep -qE "(https?://|file://|github/|[?&](color|center|gray|licence)=|=[\"'](center|color|licence)|\"Authorization\": \"Bearer|types:[[:space:]]*\[[^]]+\]|--[a-z-]+ (center|color|gray)|--color|--center|\$\{?(color|center|gray|licence)|\b(color|center|gray|licence|COLOR|CENTER|GRAY|LICENCE)[_A-Z]*=|license/licence|licence/license|license_file|licence_file|LICENSE(_FILES?)?|LICENSE.*LICENCE|LICENCE.*LICENSE|gum |\\$|shields\.io|stop-color|[a-z-]color[a-z-]*:|color[a-z-]*:[[:space:]]*#|align-items:)"
           }
 
           echo "## British English Spelling Check" >> $GITHUB_STEP_SUMMARY
@@ -430,7 +430,7 @@ jobs:
 
           is_excluded_text_line() {
             local line="$1"
-            echo "$line" | grep -qE "(https?://|github/|[?&](color|center|gray|licence)=|=[\"'](center|color|licence)|types:[[:space:]]*\[[^]]+\]|--[a-z-]+ (center|color|gray)|--color|--center|\$\{?(color|center|gray|licence)|\b(COLOR|CENTER|GRAY|LICENCE)[_A-Z]*=|license/licence|licence/license|license_file|licence_file|LICENSE(_FILES?)?|LICENSE.*LICENCE|LICENCE.*LICENSE|gum |\\$|shields\.io|stop-color|[a-z-]color[a-z-]*:|color[a-z-]*:[[:space:]]*#|align-items:)"
+            echo "$line" | grep -qE "(https?://|file://|github/|[?&](color|center|gray|licence)=|=[\"'](center|color|licence)|\"Authorization\": \"Bearer|types:[[:space:]]*\[[^]]+\]|--[a-z-]+ (center|color|gray)|--color|--center|\$\{?(color|center|gray|licence)|\b(color|center|gray|licence|COLOR|CENTER|GRAY|LICENCE)[_A-Z]*=|license/licence|licence/license|license_file|licence_file|LICENSE(_FILES?)?|LICENSE.*LICENCE|LICENCE.*LICENSE|gum |\\$|shields\.io|stop-color|[a-z-]color[a-z-]*:|color[a-z-]*:[[:space:]]*#|align-items:)"
           }
           
           FIXED_COUNT=0

Original file line number	Diff line number	Diff line change
`@@ -318,7 +318,7 @@ jobs:`
`318`	`318`
`319`	`319`	`is_excluded_text_line() {`
`320`	`320`	`local line="$1"`
`321`		`- echo "$line" \| grep -qE '(https?://\|file://\|github/\|[?&](color\|center\|gray\|licence)=[a-zA-Z0-9]+\|"Authorization": "Bearer\|types:[[:space:]]\[[^]]+\]\|--[a-z-]+ (center\|color\|gray)\|\$\{?(color\|center\|gray);\}\|\b(color\|center\|gray\|licence)[_A-Z]=\|license/licence\|licence/license\|license_file\|licence_file\|LICENSE(_FILES?)?\|LICENSE.LICENCE\|LICENCE.LICENSE\|gum \|\\$\|shields\.io\|stop-color\|[a-z-]color[a-z-]:\|color[a-z-]:[[:space:]]*#\|align-items:)'`
	`321`	+ echo "$line" \| grep -qE "(https?://\|file://\|github/\|[?&](color\|center\|gray\|licence)=\|=[\"'](center\|color\|licence)\|\"Authorization\": \"Bearer\|types:[[:space:]]\[[^]]+\]\|--[a-z-]+ (center\|color\|gray)\|--color\|--center\|\$\{?(color\|center\|gray\|licence)\|\b(color\|center\|gray\|licence\|COLOR\|CENTER\|GRAY\|LICENCE)[_A-Z]=\|license/licence\|licence/license\|license_file\|licence_file\|LICENSE(_FILES?)?\|LICENSE.LICENCE\|LICENCE.LICENSE\|gum \|\\$\|shields\.io\|stop-color\|[a-z-]color[a-z-]:\|color[a-z-]:[[:space:]]*#\|align-items:)"
`322`	`322`	`}`
`323`	`323`
`324`	`324`	`echo "## British English Spelling Check" >> $GITHUB_STEP_SUMMARY`
`@@ -427,7 +427,7 @@ jobs:`
`427`	`427`
`428`	`428`	`is_excluded_text_line() {`
`429`	`429`	`local line="$1"`
`430`		`- echo "$line" \| grep -qE "(https?://\|github/\|[?&](color\|center\|gray\|licence)=\|=[\"'](center\|color\|licence)\|types:[[:space:]]\[[^]]+\]\|--[a-z-]+ (center\|color\|gray)\|--color\|--center\|\$\{?(color\|center\|gray\|licence)\|\b(COLOR\|CENTER\|GRAY\|LICENCE)[_A-Z]=\|license/licence\|licence/license\|license_file\|licence_file\|LICENSE(_FILES?)?\|LICENSE.LICENCE\|LICENCE.LICENSE\|gum \|\\$\|shields\.io\|stop-color\|[a-z-]color[a-z-]:\|color[a-z-]:[[:space:]]*#\|align-items:)"`
	`430`	+ echo "$line" \| grep -qE "(https?://\|file://\|github/\|[?&](color\|center\|gray\|licence)=\|=[\"'](center\|color\|licence)\|\"Authorization\": \"Bearer\|types:[[:space:]]\[[^]]+\]\|--[a-z-]+ (center\|color\|gray)\|--color\|--center\|\$\{?(color\|center\|gray\|licence)\|\b(color\|center\|gray\|licence\|COLOR\|CENTER\|GRAY\|LICENCE)[_A-Z]=\|license/licence\|licence/license\|license_file\|licence_file\|LICENSE(_FILES?)?\|LICENSE.LICENCE\|LICENCE.LICENSE\|gum \|\\$\|shields\.io\|stop-color\|[a-z-]color[a-z-]:\|color[a-z-]:[[:space:]]*#\|align-items:)"
`431`	`431`	`}`
`432`	`432`
`433`	`433`	`FIXED_COUNT=0`
Original file line number	Diff line number	Diff line change
`@@ -321,7 +321,7 @@ jobs:`
`321`	`321`
`322`	`322`	`is_excluded_text_line() {`
`323`	`323`	`local line="$1"`
`324`		`- echo "$line" \| grep -qE '(https?://\|file://\|github/\|[?&](color\|center\|gray\|licence)=[a-zA-Z0-9]+\|"Authorization": "Bearer\|types:[[:space:]]\[[^]]+\]\|--[a-z-]+ (center\|color\|gray)\|\$\{?(color\|center\|gray);\}\|\b(color\|center\|gray\|licence)[_A-Z]=\|license/licence\|licence/license\|license_file\|licence_file\|LICENSE(_FILES?)?\|LICENSE.LICENCE\|LICENCE.LICENSE\|gum \|\\$\|shields\.io\|stop-color\|[a-z-]color[a-z-]:\|color[a-z-]:[[:space:]]*#\|align-items:)'`
	`324`	+ echo "$line" \| grep -qE "(https?://\|file://\|github/\|[?&](color\|center\|gray\|licence)=\|=[\"'](center\|color\|licence)\|\"Authorization\": \"Bearer\|types:[[:space:]]\[[^]]+\]\|--[a-z-]+ (center\|color\|gray)\|--color\|--center\|\$\{?(color\|center\|gray\|licence)\|\b(color\|center\|gray\|licence\|COLOR\|CENTER\|GRAY\|LICENCE)[_A-Z]=\|license/licence\|licence/license\|license_file\|licence_file\|LICENSE(_FILES?)?\|LICENSE.LICENCE\|LICENCE.LICENSE\|gum \|\\$\|shields\.io\|stop-color\|[a-z-]color[a-z-]:\|color[a-z-]:[[:space:]]*#\|align-items:)"
`325`	`325`	`}`
`326`	`326`
`327`	`327`	`echo "## British English Spelling Check" >> $GITHUB_STEP_SUMMARY`
`@@ -430,7 +430,7 @@ jobs:`
`430`	`430`
`431`	`431`	`is_excluded_text_line() {`
`432`	`432`	`local line="$1"`
`433`		`- echo "$line" \| grep -qE "(https?://\|github/\|[?&](color\|center\|gray\|licence)=\|=[\"'](center\|color\|licence)\|types:[[:space:]]\[[^]]+\]\|--[a-z-]+ (center\|color\|gray)\|--color\|--center\|\$\{?(color\|center\|gray\|licence)\|\b(COLOR\|CENTER\|GRAY\|LICENCE)[_A-Z]=\|license/licence\|licence/license\|license_file\|licence_file\|LICENSE(_FILES?)?\|LICENSE.LICENCE\|LICENCE.LICENSE\|gum \|\\$\|shields\.io\|stop-color\|[a-z-]color[a-z-]:\|color[a-z-]:[[:space:]]*#\|align-items:)"`
	`433`	+ echo "$line" \| grep -qE "(https?://\|file://\|github/\|[?&](color\|center\|gray\|licence)=\|=[\"'](center\|color\|licence)\|\"Authorization\": \"Bearer\|types:[[:space:]]\[[^]]+\]\|--[a-z-]+ (center\|color\|gray)\|--color\|--center\|\$\{?(color\|center\|gray\|licence)\|\b(color\|center\|gray\|licence\|COLOR\|CENTER\|GRAY\|LICENCE)[_A-Z]=\|license/licence\|licence/license\|license_file\|licence_file\|LICENSE(_FILES?)?\|LICENSE.LICENCE\|LICENCE.LICENSE\|gum \|\\$\|shields\.io\|stop-color\|[a-z-]color[a-z-]:\|color[a-z-]:[[:space:]]*#\|align-items:)"
`434`	`434`	`}`
`435`	`435`
`436`	`436`	`FIXED_COUNT=0`