Skip to content

fix(ci): pin strands command runs to the commit the command was issued against - #608

Merged
zastrowm merged 1 commit into
strands-agents:mainfrom
zastrowm:agent-tasks/toctou-pin-headsha
Sep 18, 2026
Merged

zastrowm merged 1 commit into
strands-agents:mainfrom
zastrowm:agent-tasks/toctou-pin-headsha

Conversation

@zastrowm

Copy link
Copy Markdown
Member

Same change as the other consumers, comment path only (this repo has no auto-review): strands-command.yml consumes the authorization job's resolved head and passes it to the input parser instead of resolving live HEAD. No-op against the current shared action until strands-agents/devtools#82 lands, which must merge after this.

…d against

Wire the authorization job's resolved head into the input parser so /strands
runs act on the commit the command was issued against instead of live HEAD.
@zastrowm
zastrowm requested a review from a team as a code owner September 18, 2026 16:37
@zastrowm
zastrowm deployed to auto-approve September 18, 2026 17:35 — with GitHub Actions Active
@zastrowm
zastrowm merged commit 29ffbeb into strands-agents:main Sep 18, 2026
21 of 22 checks passed

This branch was successfully deployed

1 active deployment
auto-approve — 91196881 Deployed Sep 18, 2026 by zastrowm via check-access-and-checkout #609
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