fix(agent): run execute hook on initial render - #347
Conversation
📝 Contributor License Agreement requiredBefore this PR can merge, every contributor must sign the Infisical CLA. Still needs to sign: Once everyone has signed, the check updates automatically — no need to close and reopen the PR. |
|
| Filename | Overview |
|---|---|
| packages/cmd/agent.go | Extends the existing post-render execute-hook path to the initial successful template render; no actionable defect was identified. |
| packages/cmd/agent_execute_test.go | Adds focused coverage confirming the initial render writes its template and invokes the configured command. |
Reviews (1): Last reviewed commit: "fix(agent): run execute hook on initial ..." | Re-trigger Greptile
Description 📣
Fixes #340.
Run the configured post-render execute hook after every successful template render, including the first render after an agent restart. This ensures applications reload when a rotated secret is written while the agent is offline.
No new dependencies.
Type ✨
Tests 🛠️
go test -vet=off ./packages/cmd go build ./...The focused regression test verifies that an initial literal-template render executes the configured command. Standard
go test ./packages/cmdis currently blocked by pre-existing vet findings inpackages/cmd/run.go.