Skip to content

Fix telemetry escaping reversibility#34

Merged
orban merged 1 commit intonightshift/bus-factorfrom
nightshift/logging-audit-fix3
Apr 15, 2026
Merged

Fix telemetry escaping reversibility#34
orban merged 1 commit intonightshift/bus-factorfrom
nightshift/logging-audit-fix3

Conversation

@orban
Copy link
Copy Markdown
Owner

@orban orban commented Apr 4, 2026

Summary

  • make hook telemetry TSV-safe by escaping backslashes, tabs, newlines, and carriage returns before writing
  • decode escaped telemetry only at dashboard display time so literal backslash sequences remain visible to operators
  • add regression coverage for literal escape sequences and control-character round trips, and document the logging contract

Testing

  • bash tests/test_telemetry.sh
  • bash tests/test_hooks.sh
  • bash tests/test_feedback_loop.sh
  • bash tests/test_stop_hook.sh
  • python -m pytest eval-harness/tests/test_task_runner.py

Nightshift-Task: logging-audit

Nightshift-Ref: https://github.com/marcus/nightshift
@orban orban merged commit 79374d0 into nightshift/bus-factor Apr 15, 2026
@orban orban deleted the nightshift/logging-audit-fix3 branch April 15, 2026 20:16
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