CI pipeline smoke test#2
Open
RidhaOracle wants to merge 51 commits into
Open
Conversation
Update workflows and contributor docs, and remove the obsolete PR template consistency check.
Add the required 2026 Oracle copyright notice to the GitHub contribution automation and CI helper files so server-side validation accepts the change. Keep shell shebangs as the first line.
Add the github action
Corrects grammar in a
No high-confidence issues found.
No tests needed for this comment-only change. |
Replace the blocked third-party Codex GitHub Action with a repository-owned, standard-library Responses API client. Load the client from the PR base SHA, verify merge parents and current write permission, treat the PR checkout only as bounded diff data, and expose the API key only to the trusted client. Pin the remaining GitHub-owned actions, validate and base64-transport review output, restrict the posting job to issues:write, and fail the final status unless both review generation and comment delivery succeed. Add unit coverage for event validation, merge verification, bounded Git isolation, request authentication, response parsing, Unicode and size limits, and output encoding.
Replace full upstream commit SHA pins with the requested actions/github-script@v7 and actions/checkout@v4 references.
Keep the pull request focused on the production workflow and review client by removing the standalone unit test file as requested.
Replace Codex action with repository-owned API reviewer
RidhaOracle
force-pushed
the
codex/ci-pipeline-smoke
branch
from
c85a680 to
5814190
Compare
Grant pull-requests: write to the separately permissioned post_feedback job. The repository-owned reviewer generated feedback successfully, but GitHub rejected the issue-comment request with HTTP 403 when the job had only issues: write.
…ssion Allow the reviewer to comment on pull requests
Make a behavior-neutral comment correction under mysql-test to exercise the PR build, MTR, formatting, labeling, review, and ownership automation merged into trunk.
Temporarily mask the SafeProcess child exit status in the smoke-test PR so the synchronized Codex review has a concrete, high-confidence defect to identify. This change is only for validating review behavior.
Create a fresh synchronize event without changing the controlled review defect.
Correct a possessive typo while preserving the controlled Codex review probe in the pull request.
Trigger a fresh pull_request_target synchronization after rebasing the smoke-test branch onto the trunk version of the repository-owned reviewer.
Trigger a fresh pull_request_target run after merging the pull-requests: write permission for the post_feedback job.
RidhaOracle
force-pushed
the
codex/ci-pipeline-smoke
branch
from
5814190 to
64938fd
Compare
1. Change summaryCorrects two comments in 2. Review findings
3. Test gaps or risksThe exit-status change would require explicit tests proving that a child’s nonzero exit code is propagated. For a comment-only smoke-test PR, the safer correction is to remove the behavioral change entirely. |
github-actions
Bot
added
MTR Failed
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
1 participant
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.
Purpose: exercise the pull-request automation newly merged into trunk with a behavior-neutral comment correction.\n\nExpected coverage:\n- PR Build\n- MTR\n- Format Check\n- Auto Label (tests)\n- Codex PR Review for a trusted actor\n- Assign Code Owners after OCA Verified is applied\n\nThis is a temporary smoke-test PR and should be closed after the checks are verified.