WA-VERIFY-086: Fix Mongoid::QueryCache → Mongo::QueryCache API migration (Mongoid 8 Phase 1)

[kitcommerce]

Summary

WA-VERIFY-083 (PR #1075) identified Mongoid::QueryCache as a HIGH severity breaking change in the Mongoid 8 upgrade path. This issue implements the migration.

Affected Call Sites (from #1069 assessment)

grep -r 'Mongoid::QueryCache' app/ lib/ config/ --include='*.rb'

Known affected files:

• core/app/middleware/workarea/admin_search_query_wrapper.rb
• core/app/middleware/workarea/releasable.rb
• Rack middleware initializer

Approach

1. Replace Mongoid::QueryCache with Mongo::QueryCache at all call sites
2. Verify Mongo::QueryCache is available in the mongo gem version we use
3. Add a test to confirm cache works correctly after change
4. Run bundle exec rake test to confirm no regressions

Acceptance Criteria

• No Mongoid::QueryCache references remain in production code
• Mongo::QueryCache used consistently
• Tests pass under default appraisal
• Notes file documents the migration

Client Impact

None (internal query optimization layer; behavior unchanged).

Related

• Blocks: WA-RAILS7-004: Mongoid 8.x upgrade — implement changes from research doc #690 (WA-RAILS7-004 Mongoid 8 upgrade)
• Research: PR WA-VERIFY-083: Mongoid 8 upgrade readiness assessment (#1069) #1075 (WA-VERIFY-083 assessment)

[kitcommerce]

Changes requested (dispatcher)

Quick correctness catch before Wave 2/3 proceeds:

• On mongo driver 2.23.0, responds to , not .
• PR WA-VERIFY-086: migrate QueryCache to Mongo::QueryCache #1086 currently changes → (and tests/docs mention ). That will raise at runtime.

Fix suggestion:

• Replace calls with .

• If we need a compat wrapper, use:

  but for 2.23.0, prefer .

After pushing the fix, move this back to and we'll rerun the pipeline.

[kitcommerce]

Changes requested (dispatcher) — corrected

(Previous comment got mangled by shell interpolation; reposting with code formatting.)

Quick correctness catch before Wave 2/3 proceeds:

• On mongo driver 2.23.0, Mongo::QueryCache responds to clear, not clear_cache.
• PR WA-VERIFY-086: migrate QueryCache to Mongo::QueryCache #1086 currently changes Mongoid::QueryCache.clear_cache → Mongo::QueryCache.clear_cache (and tests/docs mention clear_cache). That will raise NoMethodError at runtime.

Fix suggestion:

• Replace calls with Mongo::QueryCache.clear.
• If we need a compat wrapper, use:

  if Mongo::QueryCache.respond_to?(:clear_cache)
    Mongo::QueryCache.clear_cache
  else
    Mongo::QueryCache.clear
  end

  but for 2.23.0, prefer clear.

After pushing the fix, move this back to status:ready-for-review and we'll rerun the pipeline.