Skip to content

refactor(formatter): centralize word scanning helpers#22

Merged
MayerTim merged 1 commit into
mainfrom
refactor/formatter-word-scanning-helpers
Jun 10, 2026
Merged

refactor(formatter): centralize word scanning helpers#22
MayerTim merged 1 commit into
mainfrom
refactor/formatter-word-scanning-helpers

Conversation

@MayerTim

Copy link
Copy Markdown
Owner

Summary

  • Add shared formatter scanner helpers for SQL word-start, word-part, next-word, previous-word, and outside-segment word collection.
  • Add parenthesis-depth-aware word collection for structural formatter passes that already track top-level SQL words.
  • Replace duplicated local word-scanning loops in Watcom structural formatter passes.
  • Preserve the existing scanner state model, protected-text boundaries, formatter behavior, and formatter pass order.

Validation

  • npm run validate
  • npm run package:vsix
  • [x ] npm run test:extension

Notes

This PR is behavior-preserving. It does not change formatter output, formatter pass order, formatter defaults, metadata behavior, diagnostics behavior, extension behavior, packaging behavior, tokenizer architecture, or public documentation.

@MayerTim MayerTim merged commit aeea4bb into main Jun 10, 2026
1 check passed
@MayerTim MayerTim deleted the refactor/formatter-word-scanning-helpers branch June 10, 2026 22:09
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