Accept both %-wrapped and non-wrapped variable arguments in act#311
Merged
miguelg719 merged 1 commit intov0from Feb 24, 2026
Merged
Accept both %-wrapped and non-wrapped variable arguments in act#311miguelg719 merged 1 commit intov0from
miguelg719 merged 1 commit intov0from
Conversation
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.
Note
Low Risk
Small, localized string-substitution and prompt-text changes; main risk is accidental over-replacement of argument strings if variable keys overlap with normal text.
Overview
actargument variable substitution now supports both%name%placeholders (preferred) and legacy unwrappednamereplacement, avoiding the previous nested-comprehension behavior that could duplicate/over-replace arguments.The LLM
build_act_observe_promptinstructions are updated to ask for variables to be wrapped in%…%, and the package version is bumped to0.5.11.Written by Cursor Bugbot for commit 9ce4efd. This will update automatically on new commits. Configure here.
Summary by cubic
Accept both %-wrapped and plain variable placeholders in act arguments for reliable substitution and fewer mismatches. Updated the act prompt to ask for %-wrapped variables, and bumped the package to 0.5.11.
Written for commit 9ce4efd. Summary will update on new commits. Review in cubic