[function-namer] Go function rename plan: pkg/actionpins #28431

• getActionPins() — ✅ Clear internal cache-loader, no change needed
• countPinKeyMismatches() — ✅ Clear, no change needed
• buildByRepoIndex() — ✅ Clear, no change needed
• GetActionPinsByRepo() — ✅ Clear (returns all pins for repo), no change needed
• GetActionPinByRepo() — ⚠️ Rename suggested (see table above)
• GetContainerPin() — ✅ Clear, no change needed
• getLatestActionPinReference() — ✅ Clear internal helper, no change needed
• FormatPinnedActionReference() — ✅ Clear, no change needed
• FormatCacheKey() — ✅ Clear, no change needed
• ExtractRepo() — ✅ Clear, no change needed
• ExtractVersion() — ✅ Clear, no change needed
• isValidFullSHA() — ✅ Clear thin wrapper, no change needed
• findCompatiblePin() — ✅ Clear, no change needed
• initWarnings() — ✅ Clear internal initializer, no change needed
• recordPinResolutionFailure() — ✅ Clear, no change needed
• ResolveActionPin() — ✅ Clear, no change needed
• ResolveLatestActionPin() — ✅ Clear, no change needed

Agentic Implementation Plan

This issue is designed to be assigned to a coding agent. The agent should implement
all rename suggestions below in a single pull request.

Prerequisites

• Read the function body of GetActionPinByRepo to confirm it returns the latest (highest version) pin
• Check that no Go interface requires the name GetActionPinByRepo to be preserved

Implementation Steps

1. Rename GetActionPinByRepo in pkg/actionpins/actionpins.go

// Old
// GetActionPinByRepo returns the latest ActionPin for a given repository, if any.
func GetActionPinByRepo(repo string) (ActionPin, bool) {

// New
// GetLatestActionPinByRepo returns the latest ActionPin for a given repository, if any.
func GetLatestActionPinByRepo(repo string) (ActionPin, bool) {

2. Update all call sites

grep -rn "GetActionPinByRepo" pkg/ --include="*.go"

Known call sites to update:

• pkg/workflow/action_pins.go:77 — actionpins.GetActionPinByRepo(repo) → actionpins.GetLatestActionPinByRepo(repo)
• The unexported wrapper getActionPinByRepo in pkg/workflow/action_pins.go should also be renamed to getLatestActionPinByRepo for consistency

Also update test files:

grep -rn "GetActionPinByRepo" pkg/ --include="*_test.go"

Known test files to update:

• pkg/workflow/action_pins_test.go — TestGetActionPinByRepo → TestGetLatestActionPinByRepo
• pkg/actionpins/spec_test.go — TestSpec_PublicAPI_GetActionPinByRepo and call sites

3. Verify compilation after rename

make build

4. Run tests after all renames are complete

make test-unit
make lint

Commit Convention

refactor: rename GetActionPinByRepo to GetLatestActionPinByRepo for clarity

Validation Checklist

• GetActionPinByRepo renamed to GetLatestActionPinByRepo in pkg/actionpins/actionpins.go
• Wrapper getActionPinByRepo renamed to getLatestActionPinByRepo in pkg/workflow/action_pins.go
• All call sites updated (Go files and test files)
• make build passes with no errors
• make test-unit passes
• make lint passes
• PR description explains the agent-discoverability rationale

Notes for the Agent

• This is a pure rename refactor — behavior must not change, only names
• If a rename causes unexpected complexity (e.g., name conflicts, interface constraints), skip it and leave a comment in the PR explaining why
• Follow existing naming conventions documented in AGENTS.md