Skip to content

codex naming#14

Merged
pelikhan merged 4 commits intomainfrom
pelikhan/codex2
Aug 13, 2025
Merged

codex naming#14
pelikhan merged 4 commits intomainfrom
pelikhan/codex2

Conversation

@pelikhan
Copy link
Copy Markdown
Collaborator

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>
Copilot AI added a commit that referenced this pull request Apr 30, 2026
…ds from current run

The smoke test was failing at safe_outputs when resolve_pull_request_review_thread
targeted a pre-existing thread that was already resolved from a previous run.

Update test #15 to capture thread IDs from review comments created in the current run.
Update test #16 to only target unresolved threads (preferring those from test #14),
and skip gracefully with ⚠️ if no unresolved threads exist instead of failing with an API error.

Fixes: #29336"

Agent-Logs-Url: https://github.com/github/gh-aw/sessions/e28a2eef-c003-4eb0-98fd-1175fee61f52

Co-authored-by: gh-aw-bot <259018956+gh-aw-bot@users.noreply.github.com>
pelikhan pushed a commit that referenced this pull request Apr 30, 2026
…#29373)

* Initial plan

* fix: smoke claude resolve review thread targets only unresolved threads from current run

The smoke test was failing at safe_outputs when resolve_pull_request_review_thread
targeted a pre-existing thread that was already resolved from a previous run.

Update test #15 to capture thread IDs from review comments created in the current run.
Update test #16 to only target unresolved threads (preferring those from test #14),
and skip gracefully with ⚠️ if no unresolved threads exist instead of failing with an API error.

Fixes: #29336"

Agent-Logs-Url: https://github.com/github/gh-aw/sessions/e28a2eef-c003-4eb0-98fd-1175fee61f52

Co-authored-by: gh-aw-bot <259018956+gh-aw-bot@users.noreply.github.com>

---------

Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
Co-authored-by: gh-aw-bot <259018956+gh-aw-bot@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.

1 participant