Skip to content

ci: migrate workflows to shared reusable workflows - #1829

Merged
JunyangHe merged 6 commits into
mainfrom
feat/caller-workflows
Jul 27, 2026
Merged

ci: migrate workflows to shared reusable workflows#1829
JunyangHe merged 6 commits into
mainfrom
feat/caller-workflows

Conversation

@JunyangHe

@JunyangHe JunyangHe commented Jul 24, 2026

Copy link
Copy Markdown
Contributor

Description

Replace repository-specific GitHub Actions implementations with callers for reusable workflows
in aws/agentcore-devx-devtools.

  • Migrate Slack issue and open-PR notifications.
  • Migrate PR security review, CI failure reporting, Strands command, restricted agent, and
    public sync workflows.
  • Combine PR size and title checks into pr-size-title.
  • Preserve existing triggers and workflow inputs.
  • Use CodeBuild runners where required.
  • Forward secrets using secrets: inherit.
  • Pin reusables to commit SHA 458c0a684af0f9e3a013ec05cd23851def4f9cab.
  • Remove temporary feature-branch test triggers.

Type of Change

  • Bug fix
  • New feature
  • Breaking change
  • Documentation update
  • Other: CI workflow migration to shared reusable workflows

Testing

Validated all eight callers against their reusable workflow contracts, including YAML syntax,
required inputs, permissions, CodeBuild runner selection, secrets: inherit, triggers, and
commit-SHA references.

No application source code changed, so application tests were not run.

  • 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 (not applicable)

Checklist

  • I have read the CONTRIBUTING document
  • I have added any necessary tests that prove my fix is effective or my feature works
  • I have updated the documentation accordingly (no documentation changes required)
  • I have added an appropriate example to the documentation to outline the feature, or no
    new docs are needed
  • My changes generate no new warnings
  • Any dependent changes have been merged and published

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

@JunyangHe
JunyangHe requested a review from a team July 24, 2026 19:59
@github-actions github-actions Bot added size/xl PR size: XL agentcore-harness-reviewing AgentCore Harness review in progress labels Jul 24, 2026
@agentcore-devx-automation agentcore-devx-automation Bot added the claude-security-reviewing Claude Code /security-review in progress label Jul 24, 2026
@github-actions github-actions Bot removed the agentcore-harness-reviewing AgentCore Harness review in progress label Jul 24, 2026
@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 Jul 24, 2026
@github-actions

github-actions Bot commented Jul 24, 2026

Copy link
Copy Markdown
Contributor

Coverage Report

Status Category Percentage Covered / Total
🔵 Lines 40.4% 15089 / 37343
🔵 Statements 39.66% 16084 / 40546
🔵 Functions 34.51% 2575 / 7461
🔵 Branches 33.8% 10054 / 29742
Generated in workflow #4226 for commit 17eb11e by the Vitest Coverage Report Action

@JunyangHe JunyangHe changed the title Feat/caller workflows ci: migrate workflows to shared reusable workflows Jul 25, 2026
@github-actions github-actions Bot added size/xl PR size: XL and removed size/xl PR size: XL labels Jul 25, 2026
@agentcore-devx-automation agentcore-devx-automation Bot added the claude-security-reviewing Claude Code /security-review in progress label Jul 25, 2026
@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 Jul 25, 2026
notgitika
notgitika previously approved these changes Jul 27, 2026
@@ -1,23 +1,20 @@
name: Restricted Agent

# RESTRICTED ACCESS VERSION

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

can we add this comment back?

workflow_dispatch:
inputs:
prompt:
description: 'Prompt for the agent'

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

why have we removed these '?

@JunyangHe JunyangHe Jul 27, 2026

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think the ' are not needed for functionality. It is the convention not to use '' for strings in yaml

Comment thread .github/workflows/sync-from-public.yml Outdated

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

can we remove this workflow from this repo? it is not relevant to this one (only for the staging repo)

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Oh yea it landed here by accident. Will remove.

@github-actions github-actions Bot added size/xl PR size: XL and removed size/xl PR size: XL labels Jul 27, 2026
@agentcore-devx-automation agentcore-devx-automation Bot added the claude-security-reviewing Claude Code /security-review in progress label Jul 27, 2026
@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 Jul 27, 2026
@github-actions github-actions Bot added size/xl PR size: XL and removed size/xl PR size: XL labels Jul 27, 2026
@agentcore-devx-automation agentcore-devx-automation Bot added the claude-security-reviewing Claude Code /security-review in progress label Jul 27, 2026
@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 Jul 27, 2026
@JunyangHe
JunyangHe merged commit ac8dec6 into main Jul 27, 2026
67 of 70 checks passed
@JunyangHe
JunyangHe deleted the feat/caller-workflows branch July 27, 2026 18:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size/xl PR size: XL

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants