Skip to content

Support repos with sub-actions - #6

Merged
akx merged 2 commits into
masterfrom
many-actions
Oct 27, 2025
Merged

Support repos with sub-actions#6
akx merged 2 commits into
masterfrom
many-actions

Conversation

@akx

@akx akx commented Oct 27, 2025

Copy link
Copy Markdown
Owner

No description provided.

@akx
akx requested a review from Copilot October 27, 2025 18:46

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull Request Overview

This PR adds support for GitHub Actions that use subdirectories within their repositories (e.g., github/codeql-action/init, github/codeql-action/analyze). The key change extracts the repository name (first two path segments) when fetching version information, allowing the tool to correctly handle actions that reference specific sub-actions within a repository.

Key Changes:

  • Added a repository_name property to ActionSpec that extracts the repo name from actions with subdirectories
  • Modified version fetching logic to use the repository name instead of the full action path
  • Added test coverage for subdirectory action handling

Reviewed Changes

Copilot reviewed 4 out of 4 changed files in this pull request and generated 1 comment.

File Description
gha_tools/action_updater.py Added repository_name cached property and updated version fetching to use it for subdirectory actions
tests/test_autoupdate.py Added test case validating subdirectory action updates (e.g., codeql-action variants)
tests/autoupdate_victim/codeql.yml Created test fixture with codeql-action subdirectory references
.github/workflows/test.yml Updated workflow dependencies and enabled mypy job (unrelated to core PR purpose)

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread gha_tools/action_updater.py
@akx
akx merged commit 7dc402b into master Oct 27, 2025
5 checks passed
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.

2 participants