Skip to content

Conversation

@pelikhan
Copy link
Contributor

No keys, so getting 401s right now.

@pelikhan pelikhan merged commit abb544c into main Aug 13, 2025
6 checks passed
@pelikhan pelikhan deleted the pelikhan/codex2 branch August 13, 2025 00:00
pelikhan added a commit that referenced this pull request Aug 22, 2025
* Refactor sanitize_output.cjs: move neutralization functions inside sanitizeContent and update allowed domains logic

* Add documentation for sanitizeContent and neutralization functions

* Rename output "comment" field to "issue_comment" (#14)

* Initial plan

* Rename output comment field to issue_comment - implementation complete

Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>

* Fix existing workflow to use issue_comment instead of comment

Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>

---------

Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
Co-authored-by: Peli de Halleux <pelikhan@users.noreply.github.com>

* Fix formatting of IssueComment field in OutputConfig struct

* Refactor URL sanitization in multiple workflows: extract domain and protocol filtering into separate functions

---------

Co-authored-by: Copilot <198982749+Copilot@users.noreply.github.com>
Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
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.

4 participants