Skip to content

Conversation

@dsyme
Copy link
Contributor

@dsyme dsyme commented Sep 17, 2025

Summary

  • Improved content sanitization logic in collect_ndjson_output.cjs
  • Added more robust handling of URLs, protocols, and command-line flags
  • Enhanced safe-outputs workflow instructions in compiler

Key Changes

  • Refined URL sanitization to preserve command-line flags with colons
  • Improved protocol redaction while keeping specific patterns intact
  • Updated workflow compiler to provide more detailed safe-outputs instructions
  • Added comprehensive test cases for various sanitization scenarios

@pelikhan
Copy link
Contributor

We should not allow xml comments which can hide things.

@dsyme dsyme merged commit 27f39d8 into main Sep 17, 2025
8 of 10 checks passed
@dsyme dsyme deleted the san branch September 17, 2025 12:08
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.

3 participants