chore: sweep deprecated needs.activation.outputs.* references from authoring examples - #47968
Merged
Merged
Conversation
…s.text form Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
Copilot
AI
changed the title
[WIP] Sweep workflow markdown and docs for deprecated needs.activation.outputs references
chore: sweep deprecated needs.activation.outputs.* references from authoring examples
Jul 25, 2026
pelikhan
approved these changes
Jul 25, 2026
Contributor
There was a problem hiding this comment.
Pull request overview
Updates an authoring example to use the modern activation-output expression.
Changes:
- Replaces the deprecated expression in a runtime-import example.
- Revises compatibility guidance, though the deprecated runtime-import claim is inaccurate.
Show a summary per file
| File | Description |
|---|---|
scratchpad/activation-output-transformations.md |
Modernizes the activation-output example and compatibility note. |
Review details
Tip
Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
- Files reviewed: 1/1 changed files
- Comments generated: 1
- Review effort level: Medium
| ``` | ||
|
|
||
| Even if `instructions.md` is edited to include new references to `needs.activation.outputs.text`, the transformation ensures these work correctly. | ||
| Even if `instructions.md` is edited at runtime to include new references (using either the modern `steps.sanitized.outputs.text` or the deprecated `needs.activation.outputs.text` form), the compiler-generated substitution table handles both correctly. |
Contributor
|
🎉 This pull request is included in a new release. Release: |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Hippo flagged
needs.activation.outputs.{text,title,body}as a maintenance tail — the compiler still accepts and auto-rewrites these deprecated expressions tosteps.sanitized.outputs.*(with a deprecation warning), but new authoring should use the modern form directly.Findings
.github/workflows/— already clean; no deprecated references in any compiled workflow sourcedocs/src/— already clean;templating.mdalready documentssteps.sanitized.outputs.*exclusively.github/aw/— already clean.skill-optimizer/— deprecated form retained intentionally: tests exercise the transformation and backward-compat layer; skill-optimizer cases simulate a user asking how to migrateChange
scratchpad/activation-output-transformations.md— the "Runtime-Import Compatibility" section contained an example workflow using the deprecated form. Updated tosteps.sanitized.outputs.textand revised the surrounding note to clarify both forms are handled at runtime (for backward compat) but the modern form is preferred:needs.activation.outputs.text/title/bodyreferences #47743