Skip to content

Fix DAFOCD officer ladder regex for six missed AFSCs - #56

Merged
saturnflyer merged 1 commit into
mainfrom
officer-ladder-regex-fix
Jul 16, 2026
Merged

Fix DAFOCD officer ladder regex for six missed AFSCs#56
saturnflyer merged 1 commit into
mainfrom
officer-ladder-regex-fix

Conversation

@saturnflyer

Copy link
Copy Markdown
Member

Summary

  • The DAFOCD qualification-ladder anchor required a comma between the concrete code and its title and couldn't absorb a glued shredout letter or a leading decorative glyph, so 6 live officer specialties were silently missed by extraction: 17DX, 44BX, 45AX, 47BX, 47PX, 48GX.
  • The anchor now accepts a comma-or-whitespace separator (a bare optional comma would admit glued-shred prose elsewhere in the document), absorbs a trailing shred letter without folding it into the concrete code, and tolerates a leading Private Use Area glyph some cards carry before AFSC.
  • Regenerated officer.yml adds the 6 specialties (130 -> 136 entries) with every existing entry byte-identical. The shred letter flows through as a shredout via each card's own shredout table (e.g. 17DXW -> Warfighter Communications).

Notes

  • Verified against the source PDF directly, not from memory of the career fields.
  • Negative tests confirm the loosened separator still rejects glued-shred prose lines that aren't real ladder entries.

The DAFOCD qualification-ladder anchor required a comma between the
concrete code and its title and could neither absorb a glued shredout
letter nor a leading decorative glyph, so six live officer specialties
never extracted. The anchor now accepts a comma-OR-whitespace separator
(a bare optional comma would admit glued-shred prose such as
"AFSC 42B3Z requires ..."), absorbs a trailing shred letter without
baking it into the concrete code, and tolerates a leading Private Use
Area glyph. The regenerated officer.yml adds the six specialties with
every existing entry byte-identical; the shred letter flows through as
a shredout via each card's shredout table (17DXW -> Warfighter
Communications).

Added: Officer specialties 17DX, 44BX, 45AX, 47BX, 47PX, 48GX from the DAFOCD
Fixed: officer ladder missed no-comma, glued-shred, and glyph-prefixed cards
@saturnflyer
saturnflyer requested a review from a team as a code owner July 16, 2026 10:30
@saturnflyer
saturnflyer merged commit 09c5350 into main Jul 16, 2026
3 checks passed
@saturnflyer
saturnflyer deleted the officer-ladder-regex-fix branch July 16, 2026 11:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant