Skip to content

Comments

Accept both %-wrapped and non-wrapped variable arguments in act#311

Merged
miguelg719 merged 1 commit intov0from
patch_regex_in_variables
Feb 24, 2026
Merged

Accept both %-wrapped and non-wrapped variable arguments in act#311
miguelg719 merged 1 commit intov0from
patch_regex_in_variables

Conversation

@miguelg719
Copy link
Collaborator

@miguelg719 miguelg719 commented Feb 24, 2026

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
act argument variable substitution now supports both %name% placeholders (preferred) and legacy unwrapped name replacement, avoiding the previous nested-comprehension behavior that could duplicate/over-replace arguments.

The LLM build_act_observe_prompt instructions are updated to ask for variables to be wrapped in %…%, and the package version is bumped to 0.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

@miguelg719 miguelg719 merged commit 3d0dfa7 into v0 Feb 24, 2026
4 checks passed
Copy link

@cubic-dev-ai cubic-dev-ai bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No issues found across 3 files

Confidence score: 5/5

  • Automated review surfaced no issues in the provided summaries.
  • No files require special attention.

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.

1 participant