Skip to content

fix(ci): restore refactor PR automation - #2002

Closed
aidandaly24 wants to merge 1 commit into
mainfrom
fix/refactor-workflow-default-main
Closed

fix(ci): restore refactor PR automation#2002
aidandaly24 wants to merge 1 commit into
mainfrom
fix/refactor-workflow-default-main

Conversation

@aidandaly24

@aidandaly24 aidandaly24 commented Aug 14, 2026

Copy link
Copy Markdown
Contributor

Description

Restore PR automation for pull requests targeting refactor and make AgentCore Harness review publishing reliable.

  • Include refactor in CodeQL, PR Size and Title, and Claude Security Review branch filters.
  • Fetch existing PR discussion with the workflow's short-lived GITHUB_TOKEN.
  • Require Harness to return a delimited final review and publish it from the workflow runner.
  • Fail the workflow when Harness omits a final review or GitHub rejects the review.
  • Remove the baked GitHub API credential from future Harness images.

The workflow token remains on the GitHub Actions runner and is not sent to the Harness runtime.

Related Issue

N/A - workflow repair for the unreleased refactor branch.

Documentation PR

N/A

Type of Change

  • Bug fix
  • New feature
  • Breaking change
  • Documentation update
  • Other (please describe):

Testing

How have you tested the change?

  • I ran npm run test:unit and npm run test:integ
  • I ran npm run typecheck
  • I ran npm run lint
  • If I modified src/assets/, I ran npm run test:update-snapshots and committed the updated snapshots

Additional verification:

  • python -m unittest test_harness_review.py
  • python -m py_compile harness_review.py test_harness_review.py
  • npm run format:check
  • Parsed every workflow YAML file
  • Live dispatch 31832968154 reached the secret-fetch step but the OIDC trust policy rejects non-main refs; rerun after this PR merges to exercise Harness and GitHub review publishing end to end

Checklist

  • I have read the CONTRIBUTING document
  • I have added necessary tests that prove the fix is effective
  • I have updated the documentation accordingly
  • No new example is needed for this CI-only change
  • My changes generate no new warnings
  • No dependent package changes are required

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the
terms of your choice.

@aidandaly24
aidandaly24 requested a review from a team August 14, 2026 19:22
@github-actions github-actions Bot added the size/m PR size: M label Aug 14, 2026
@agentcore-devx-automation agentcore-devx-automation Bot added the claude-security-reviewing Claude Code /security-review in progress label Aug 14, 2026
@github-actions github-actions Bot added the agentcore-harness-reviewing AgentCore Harness review in progress label Aug 14, 2026
@github-actions

Copy link
Copy Markdown
Contributor

Package Tarball

aws-agentcore-0.27.0.tgz

How to install

gh release download pr-2002-tarball --repo aws/agentcore-cli --pattern "*.tgz" --dir /tmp/pr-tarball
npm install -g /tmp/pr-tarball/aws-agentcore-0.27.0.tgz

@agentcore-devx-automation

Copy link
Copy Markdown
Contributor

Claude Security Review: no high-confidence findings. (run)

@agentcore-devx-automation agentcore-devx-automation Bot removed the claude-security-reviewing Claude Code /security-review in progress label Aug 14, 2026
@github-actions github-actions Bot added size/m PR size: M and removed size/m PR size: M agentcore-harness-reviewing AgentCore Harness review in progress labels Aug 14, 2026
@github-actions

Copy link
Copy Markdown
Contributor

Coverage Report

Status Category Percentage Covered / Total
🔵 Lines 40.65% 15274 / 37572
🔵 Statements 39.92% 16287 / 40799
🔵 Functions 34.85% 2614 / 7499
🔵 Branches 33.96% 10185 / 29984
Generated in workflow #4339 for commit ea4e78b by the Vitest Coverage Report Action

@aidandaly24

Copy link
Copy Markdown
Contributor Author

Closing as superseded. The security branch policy and Harness reviewer implementation need to move into aws/agentcore-devx-devtools; the replacement CLI PR will keep only thin caller triggers pinned to the shared implementation.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size/m PR size: M

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant