Conversation
Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com> Agent-Logs-Url: https://github.com/github/gh-aw/sessions/8d17358b-a92c-45df-8cbd-f0f45c966c06
There was a problem hiding this comment.
Pull request overview
Updates the user-facing description for the aw_context workflow_dispatch input and regenerates compiled workflow lockfiles to reflect that change.
Changes:
- Rephrased
awContextInputDescriptionin the workflow compiler. - Regenerated workflow
.lock.ymlfiles viamake recompileto apply the updated description across compiled workflows.
Reviewed changes
Copilot reviewed 164 out of 164 changed files in this pull request and generated 2 comments.
| File | Description |
|---|---|
| pkg/workflow/compiler_aw_context.go | Updates the source-of-truth description string used when injecting the aw_context input into workflow_dispatch. |
| .github/workflows/*.lock.yml | Regenerated compiled workflow lockfiles to carry the updated aw_context input description into workflow YAML. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| // awContextInputDescription is the description for the aw_context workflow_dispatch input. | ||
| // It signals to users that this input is managed internally by the agentic workflow system. | ||
| const awContextInputDescription = "(Internal) JSON context injected by the calling agentic workflow. Not intended for direct user input." | ||
| const awContextInputDescription = "Agent caller context (used internally by Agentic Workflows)." |
There was a problem hiding this comment.
The new aw_context input description is less explicit than the doc comment above it: it no longer indicates that the value is JSON and that users should not set it manually. Consider rephrasing to retain an explicit “internal / not intended for manual input” warning (and mentioning JSON if that’s still accurate) so the workflow_dispatch UI doesn’t encourage misuse.
| inputs: | ||
| aw_context: | ||
| default: "" | ||
| description: (Internal) JSON context injected by the calling agentic workflow. Not intended for direct user input. | ||
| description: Agent caller context (used internally by Agentic Workflows). | ||
| required: false |
There was a problem hiding this comment.
This lock file updates the workflow_dispatch input description, but the embedded Safe Outputs tool schema later in the same file still contains the old aw_context description (in the tools_meta.json content around line ~356). That leaves inconsistent user-facing metadata within one compiled artifact; please ensure the lock regeneration updates that embedded schema too (or that it sources the description from the same constant) and re-run the generator.
Rephrases the
awContextInputDescriptionconstant inpkg/workflow/compiler_aw_context.gofrom:to:
All 177 workflow lock files were regenerated via
make recompile.