Remove escapeExpression from @ember/template

[NullVoxPopuli]

Co-Authored-By: @chriskrycho

Rebase of: #20360

This also gets a mention here: emberjs/rfcs#1003 😅 (for migration docs)

[kategengler]

If it was private, shouldn't need to be in the RFC, I'd think

[kategengler]

May have been private but Handlebars.Utils.escapeExpression has uses in addons...

[NullVoxPopuli]

May have been private but Handlebars.Utils.escapeExpression has uses in addons...

people using private apis should be used to what happens? 😅 🙃

idk, what's the process here? do we need a deprecation in this case?

[kategengler]

Do they know it is private if it is hanging out on Ember.Handlebars.Utils.escapeExpression?

If it is truly private, we need an intimate API deprecation (1 LTS cycle), but if it is public, we need an RFC + full deprecation.

[bertdeblock]

I know ember-intl uses it as well.

[anandfresh]

Why escapeExpression was removed ?

[NullVoxPopuli]

it wasn't public API, as stated by

#20360

background: #16817

(which also lists some alternatives you can use)

[NullVoxPopuli]

I've now rebased this PR, as I forgot about it 😅

[kategengler]

Some test failures

[github-actions]

Development Assets

Diff

--- main/out.txt	2025-05-26 18:18:28.000000000 +0000
+++ pr/./pr-15477160240/out.txt	2025-06-05 21:00:28.000000000 +0000
@@ -1,5 +1,5 @@
  2.2M └─┬ .
-1015K   ├─┬ @ember
+1014K   ├─┬ @ember
  205K   │ ├─┬ -internals
   69K   │ │ ├─┬ views
   64K   │ │ │ └─┬ lib
@@ -17,7 +17,7 @@
  9.4K   │ │ ├── deprecations
  7.4K   │ │ ├── metal
  7.0K   │ │ ├── string
- 5.1K   │ │ ├── glimmer
+ 5.0K   │ │ ├── glimmer
  4.9K   │ │ ├── utils
  4.9K   │ │ ├── routing
  4.5K   │ │ ├── error-handling
@@ -63,7 +63,7 @@
  4.2K   │ ├── deprecated-features
  4.1K   │ ├── template-factory
  4.1K   │ └── version
- 709K   ├── shared-chunks
+ 708K   ├── shared-chunks
  384K   ├─┬ @glimmer
  166K   │ ├── runtime
   60K   │ ├── opcode-compiler

Details

	This PR	main
Dev	2.2M └─┬ . 1014K ├─┬ @ember 205K │ ├─┬ -internals 69K │ │ ├─┬ views 64K │ │ │ └─┬ lib 23K │ │ │ ├── mixins 22K │ │ │ ├── system 10K │ │ │ ├── views 4.3K │ │ │ └── compat 35K │ │ ├─┬ runtime 30K │ │ │ └─┬ lib 21K │ │ │ ├── mixins 5.7K │ │ │ └── ext 26K │ │ ├─┬ meta 21K │ │ │ └── lib 11K │ │ ├── owner 9.4K │ │ ├── deprecations 7.4K │ │ ├── metal 7.0K │ │ ├── string 5.0K │ │ ├── glimmer 4.9K │ │ ├── utils 4.9K │ │ ├── routing 4.5K │ │ ├── error-handling 4.5K │ │ ├── utility-types 4.2K │ │ ├── container 4.2K │ │ ├── browser-environment 4.1K │ │ └── environment 183K │ ├─┬ routing 28K │ │ └── lib 149K │ ├─┬ object 66K │ │ └─┬ lib 62K │ │ └── computed 114K │ ├─┬ template-compiler 109K │ │ └─┬ lib 20K │ │ ├── plugins 4.6K │ │ ├── system 4.1K │ │ └── -internal 66K │ ├─┬ application 5.6K │ │ └── lib 52K │ ├─┬ debug 21K │ │ └── lib 38K │ ├─┬ array 4.9K │ │ └── lib 31K │ ├─┬ engine 4.7K │ │ └── lib 27K │ ├── runloop 22K │ ├─┬ utils 18K │ │ └── lib 20K │ ├── helper 11K │ ├── destroyable 9.8K │ ├── instrumentation 9.4K │ ├── controller 7.4K │ ├── service 7.2K │ ├── owner 6.2K │ ├── component 5.6K │ ├── canary-features 5.5K │ ├── modifier 5.1K │ ├── template-compilation 5.0K │ ├── enumerable 5.0K │ ├── test 4.4K │ ├── template 4.4K │ ├── renderer 4.2K │ ├── deprecated-features 4.1K │ ├── template-factory 4.1K │ └── version 708K ├── shared-chunks 384K ├─┬ @glimmer 166K │ ├── runtime 60K │ ├── opcode-compiler 30K │ ├── manager 22K │ ├── validator 14K │ ├── program 12K │ ├── reference 11K │ ├── destroyable 10K │ ├─┬ tracking 4.4K │ │ └── primitives 10K │ ├── util 8.1K │ ├── node 7.3K │ ├── global-context 6.4K │ ├── wire-format 5.0K │ ├── vm 4.9K │ ├── encoder 4.6K │ ├── owner 4.1K │ └── env 60K ├─┬ ember-testing 56K │ └─┬ lib 14K │ ├── test 14K │ ├── helpers 10K │ ├── ext 6.5K │ └── adapters 31K ├── backburner.js 25K ├── ember 24K ├── route-recognizer 18K ├─┬ @simple-dom 14K │ └── document 9.2K ├── dag-map 4.3K ├── rsvp 4.3K └── router_js	2.2M └─┬ . 1015K ├─┬ @ember 205K │ ├─┬ -internals 69K │ │ ├─┬ views 64K │ │ │ └─┬ lib 23K │ │ │ ├── mixins 22K │ │ │ ├── system 10K │ │ │ ├── views 4.3K │ │ │ └── compat 35K │ │ ├─┬ runtime 30K │ │ │ └─┬ lib 21K │ │ │ ├── mixins 5.7K │ │ │ └── ext 26K │ │ ├─┬ meta 21K │ │ │ └── lib 11K │ │ ├── owner 9.4K │ │ ├── deprecations 7.4K │ │ ├── metal 7.0K │ │ ├── string 5.1K │ │ ├── glimmer 4.9K │ │ ├── utils 4.9K │ │ ├── routing 4.5K │ │ ├── error-handling 4.5K │ │ ├── utility-types 4.2K │ │ ├── container 4.2K │ │ ├── browser-environment 4.1K │ │ └── environment 183K │ ├─┬ routing 28K │ │ └── lib 149K │ ├─┬ object 66K │ │ └─┬ lib 62K │ │ └── computed 114K │ ├─┬ template-compiler 109K │ │ └─┬ lib 20K │ │ ├── plugins 4.6K │ │ ├── system 4.1K │ │ └── -internal 66K │ ├─┬ application 5.6K │ │ └── lib 52K │ ├─┬ debug 21K │ │ └── lib 38K │ ├─┬ array 4.9K │ │ └── lib 31K │ ├─┬ engine 4.7K │ │ └── lib 27K │ ├── runloop 22K │ ├─┬ utils 18K │ │ └── lib 20K │ ├── helper 11K │ ├── destroyable 9.8K │ ├── instrumentation 9.4K │ ├── controller 7.4K │ ├── service 7.2K │ ├── owner 6.2K │ ├── component 5.6K │ ├── canary-features 5.5K │ ├── modifier 5.1K │ ├── template-compilation 5.0K │ ├── enumerable 5.0K │ ├── test 4.4K │ ├── template 4.4K │ ├── renderer 4.2K │ ├── deprecated-features 4.1K │ ├── template-factory 4.1K │ └── version 709K ├── shared-chunks 384K ├─┬ @glimmer 166K │ ├── runtime 60K │ ├── opcode-compiler 30K │ ├── manager 22K │ ├── validator 14K │ ├── program 12K │ ├── reference 11K │ ├── destroyable 10K │ ├─┬ tracking 4.4K │ │ └── primitives 10K │ ├── util 8.1K │ ├── node 7.3K │ ├── global-context 6.4K │ ├── wire-format 5.0K │ ├── vm 4.9K │ ├── encoder 4.6K │ ├── owner 4.1K │ └── env 60K ├─┬ ember-testing 56K │ └─┬ lib 14K │ ├── test 14K │ ├── helpers 10K │ ├── ext 6.5K │ └── adapters 31K ├── backburner.js 25K ├── ember 24K ├── route-recognizer 18K ├─┬ @simple-dom 14K │ └── document 9.2K ├── dag-map 4.3K ├── rsvp 4.3K └── router_js