DEVX-15: Fix pair encoding to handle spaces in platform commands #28

Merged
emil merged 0 commits from DEVX-15-fix-pair-space-encoding into master 2026-06-23 19:42:57 +00:00
Owner

Summary

Pair encoding uses pipe-delimited fields expanded unquoted via $TEST_PAIRS in CI. Commands with spaces (e.g. 'sleep infinity') get split by the shell, causing molecule to receive 'sleep' without 'infinity'. Fix by encoding spaces as SPACE in encode() and decoding in parse_pair()/decode().

Closes DEVX-15

## Summary Pair encoding uses pipe-delimited fields expanded unquoted via $TEST_PAIRS in CI. Commands with spaces (e.g. 'sleep infinity') get split by the shell, causing molecule to receive 'sleep' without 'infinity'. Fix by encoding spaces as __SPACE__ in encode() and decoding in parse_pair()/decode(). Closes DEVX-15
emil reviewed 2026-06-23 19:42:15 +00:00
emil left a comment
Author
Owner

Automated PR Review

  • Architecture compliance: OK
  • Best practices: OK
  • Security: OK
  • i18n: OK
  • Resource management: OK
  • Documentation: WARNING — source files changed but no docs updated
  • Tests: WARNING — source files changed but no test files updated
  • Commit conventions: OK

No issues found by automated checks.


Auto-merge: If all CI checks pass, this PR will be merged automatically.

## Automated PR Review - Architecture compliance: OK - Best practices: OK - Security: OK - i18n: OK - Resource management: OK - Documentation: WARNING — source files changed but no docs updated - Tests: WARNING — source files changed but no test files updated - Commit conventions: OK No issues found by automated checks. --- **Auto-merge:** If all CI checks pass, this PR will be merged automatically.
emil merged commit b3d0dd8ca7 into master 2026-06-23 19:42:57 +00:00
emil deleted branch DEVX-15-fix-pair-space-encoding 2026-06-23 19:42:58 +00:00
Sign in to join this conversation.
No Reviewers
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: oblachno-oss/devx#28