Rename GetActionPinWithData to ResolveActionPin in pkg/actionpins

[Copilot]

This updates pkg/actionpins naming to better reflect behavior: the function doesn’t just “get with data,” it resolves a pin via dynamic lookup first, then embedded fallback. The change improves API discoverability without changing resolution logic.

• API rename (pkg/actionpins)

  • Renamed exported function:
    • GetActionPinWithData(actionRepo, version, ctx) → ResolveActionPin(actionRepo, version, ctx)
  • Updated the function doc comment to match the new intent-focused name.

• Call-site updates

  • Updated internal usage in pkg/actionpins/GetCachedActionPin.
  • Updated pkg/workflow delegation in getActionPinWithData(...) to call actionpins.ResolveActionPin(...).

• Package docs sync

  • Updated pkg/actionpins/README.md public API table to reference ResolveActionPin.

// before
pinnedRef, err := actionpins.GetActionPinWithData(actionRepo, version, data.PinContext())

// after
pinnedRef, err := actionpins.ResolveActionPin(actionRepo, version, data.PinContext())

[Copilot]

Pull request overview

Renames the exported action pin resolution helper in pkg/actionpins to better reflect that it performs resolution (dynamic lookup + embedded fallback), and updates internal call sites/docs accordingly.

Changes:

• Renamed exported API GetActionPinWithData → ResolveActionPin in pkg/actionpins.
• Updated internal callers (pkg/actionpins cache path and pkg/workflow delegation) to use the new name.
• Updated pkg/actionpins/README.md public API table to reflect the new function name.

Show a summary per file

File	Description
pkg/workflow/action_pins.go	Updates workflow delegation helper to call actionpins.ResolveActionPin.
pkg/actionpins/actionpins.go	Renames the exported resolver function and updates internal cache helper call site.
pkg/actionpins/README.md	Syncs public API documentation to the renamed function.

Copilot's findings

Tip

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

• Files reviewed: 3/3 changed files
• Comments generated: 1

[Copilot]

Renaming this exported function is a breaking API change for any downstream consumers of pkg/actionpins. To preserve backward compatibility (consistent with other deprecated APIs in this repo), consider keeping GetActionPinWithData as a thin wrapper that calls ResolveActionPin, and mark it with a // Deprecated: Use ResolveActionPin. doc comment. If the intention is to break API, this should at least be accompanied by an explicit deprecation/removal plan in release notes.