Skip to content

Add support for multiple pipes in placeholder modifiers#17

Merged
henriquemoody merged 1 commit intomainfrom
copilot/add-multiple-pipes-support
Feb 8, 2026
Merged

Add support for multiple pipes in placeholder modifiers#17
henriquemoody merged 1 commit intomainfrom
copilot/add-multiple-pipes-support

Conversation

Copy link
Contributor

Copilot AI commented Feb 7, 2026

  • Explore codebase structure and understand current implementation
  • Modify PlaceholderFormatter regex to support multiple pipes separated by |
  • Update processPlaceholder to split pipes and apply them sequentially
  • Create comprehensive tests for multiple pipes functionality
  • Test escaped pipes (\|) are treated as arguments not separators
  • Add documentation for the new feature
  • Verify implementation with manual integration tests
  • Run code review (1 comment - verified regex pattern is correct)
  • Run security checks (passed - no issues detected)
  • Update docs/modifiers/Modifiers.md with multiple pipes documentation
Original prompt

This section details on the original issue you should resolve

<issue_title>Add support for multiple pipes in placeholder modifiers</issue_title>
<issue_description>At the moment, we can only have 1 pipe modifying the value of a placeholder (e.g. {{placeholder|trans}}). But we could enable support for adding multiple pipes to a placeholder.

{{placeholder|trans|quote}}

It will probably not be possible to combine all placeholders, but it's interesting feature for us to have. If we consider the possibility of #8, we should make sure that we can still use | as an argument of a modifier, as long as it's escaped (\|),</issue_description>

Comments on the Issue (you are @copilot in this section)


✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.

Copilot AI changed the title [WIP] Add support for multiple pipes in placeholder modifiers Add support for multiple pipes in placeholder modifiers Feb 7, 2026
Copilot AI requested a review from henriquemoody February 7, 2026 09:09
Copy link
Member

@henriquemoody henriquemoody left a comment

Choose a reason for hiding this comment

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

Great changes!

Now make sure docs/modifiers/Modifiers.md is up-to-date as well

Copy link
Contributor Author

Copilot AI commented Feb 7, 2026

Great changes!

Now make sure docs/modifiers/Modifiers.md is up-to-date as well

Updated! Added a "Chaining Multiple Modifiers" section to docs/modifiers/Modifiers.md with examples and explanation of the sequential application behavior. (d8fb54c)

Copilot AI requested a review from henriquemoody February 7, 2026 09:20
@henriquemoody henriquemoody force-pushed the copilot/add-multiple-pipes-support branch from d8fb54c to 9f4d9d8 Compare February 8, 2026 23:23
@codecov-commenter
Copy link

codecov-commenter commented Feb 8, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 99.48%. Comparing base (1a07f89) to head (faeaead).
⚠️ Report is 1 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff            @@
##               main      #17   +/-   ##
=========================================
  Coverage     99.47%   99.48%           
- Complexity      163      166    +3     
=========================================
  Files            24       24           
  Lines           382      388    +6     
=========================================
+ Hits            380      386    +6     
  Misses            2        2           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@henriquemoody henriquemoody marked this pull request as ready for review February 8, 2026 23:26
@henriquemoody henriquemoody force-pushed the copilot/add-multiple-pipes-support branch from 9f4d9d8 to 7d5a76d Compare February 8, 2026 23:42
Placeholders can now chain multiple modifiers sequentially using the
pipe separator, e.g. {{value|date:Y/m/d|mask:5-8}}. Each modifier
receives the output of the previous one, applied left to right.

Escaped pipes (\|) within modifier arguments are preserved as literal
characters rather than treated as separators.

Assisted-by: Copilot
Assisted-by: OpenCode (ollama-cloud/glm-4.7)
Assisted-by: Claude Code (Claude Opus 4.6)
@henriquemoody henriquemoody force-pushed the copilot/add-multiple-pipes-support branch from 7d5a76d to faeaead Compare February 8, 2026 23:43
@henriquemoody henriquemoody merged commit faeaead into main Feb 8, 2026
8 checks passed
@henriquemoody henriquemoody deleted the copilot/add-multiple-pipes-support branch February 8, 2026 23:44
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.

Add support for multiple pipes in placeholder modifiers

3 participants