Conversation
Collaborator
pelikhan
commented
Sep 12, 2025
pelikhan
commented
- properly wired typechecking
- fix usage of github actions apis
…pe annotations, and improve best practices documentation for GitHub Actions.
…ce on avoiding 'any' type in documentation
…es, noImplicitThis, and noImplicitReturns to true in tsconfig.json
- Improved error handling in Claude and Codex workflows by ensuring error messages are properly formatted and checked for instance type. - Added JSDoc comments to various functions to enhance code documentation and maintainability, including: - `parseClaudeLog` - `formatToolUse` - `formatMcpName` - `formatMcpParameters` - `formatBashCommand` - `truncateString` - `parseCodexLog` - `validateErrors` - `extractLevel` - `extractMessage` - Updated logging statements to use `core.info` for consistency and clarity in output.
…flow scripts - Updated all instances of console.log to core.info for consistent logging across create_code_scanning_alert, create_comment, create_discussion, create_issue, create_pr_review_comment, create_pull_request, push_to_pr_branch, update_issue, and other workflow scripts. - Replaced console.error with core.error for error logging in parse_claude_log and other scripts. - Changed console.warn to core.warning where applicable to provide clearer logging semantics. - Enhanced log messages to include more context and structured information for better traceability.
… JSON parsing across workflow scripts
…s workflow scripts
- Replaced console.log statements with core.info, core.debug, core.warning, and core.setFailed in multiple workflow files for improved logging consistency and clarity. - Updated workflows related to safe outputs, pull requests, issue updates, and agent logs to enhance error handling and output messaging. - Ensured that all log messages are appropriately categorized to facilitate better debugging and monitoring of workflow executions.
Collaborator
Author
|
@copilot fix tests |
Contributor
dsyme
added a commit
that referenced
this pull request
Sep 25, 2025
…PR inclusion (#787) * Initial plan * Move aw_info.json generation to /tmp directory and add cleanup step Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com> * Remove cleanup step for /tmp/aw_info.json as requested Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com> * update lock files --------- 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: Don Syme <dsyme@users.noreply.github.com> Co-authored-by: Don Syme <dsyme@github.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.