Skip to content

Fix SPSS VLS compatible names - #3

Merged
TonisOrmisson merged 2 commits into
mainfrom
agent/spss-vls-compatible-names
Jul 30, 2026
Merged

Fix SPSS VLS compatible names#3
TonisOrmisson merged 2 commits into
mainfrom
agent/spss-vls-compatible-names

Conversation

@TonisOrmisson

@TonisOrmisson TonisOrmisson commented Jul 30, 2026

Copy link
Copy Markdown
Contributor

What changed

  • parse and rebuild SPSS subtype-14 compatible-name records strictly
  • replace compatible names atomically and fail closed on malformed or duplicate metadata
  • keep ZSAV offsets stable and expose a shared var_compat_names comparator
  • add shared fixtures, regression tests, and documentation
  • verify the fix together with the merged SQL transformation workflow

Why

Subtype-14 long-variable-name metadata must stay consistent when variables are normalized or rewritten. Stale or malformed records can otherwise make SAV and ZSAV files disagree about variable identity.

Validation

  • focused subtype-14 regression tests
  • full combined non-service suite: 129 passed
  • GitHub Actions matrix rerun on the current main integration
  • independent blocker-only review completed with no remaining findings

Closes #2

@TonisOrmisson
TonisOrmisson marked this pull request as ready for review July 30, 2026 09:01
@TonisOrmisson
TonisOrmisson merged commit a899eb7 into main Jul 30, 2026
24 checks passed
@TonisOrmisson
TonisOrmisson deleted the agent/spss-vls-compatible-names branch July 31, 2026 10:35
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.

bug(spss): rewrite subtype-14 VLS keys with compatible names

1 participant