Skip to content

DO NOT REVIEW: MWPW-199041: Add find-replace IO action#987

Draft
yesil wants to merge 14 commits into
mainfrom
MWPW-199041
Draft

DO NOT REVIEW: MWPW-199041: Add find-replace IO action#987
yesil wants to merge 14 commits into
mainfrom
MWPW-199041

Conversation

@yesil

@yesil yesil commented Jun 18, 2026

Copy link
Copy Markdown
Contributor

Adds the bulk-edit-find Adobe I/O Runtime action — the Find stage of M@S bulk Find & Replace. Given search text, a field scope, and filters, it returns every matching content fragment in a consumer so Studio can list them and the sibling bulk-edit-replace action (MWPW-199042) can act on them.

Two-stage by necessity: the AEM CF Search API cannot scope full-text to one element, so AEM does recall (path + fullText + tags/status, paged by cursor) and the action does precision — a case-(in)sensitive substring match against the scoped field(s) via a data-driven SCOPE_FIELDS map (everywhere, the listed fields, and tags). Backend-only; consumed by the Studio UI in MWPW-191300.

bulk-edit-replace is registered as a 501 shell only; its implementation lands under MWPW-199042.

Resolves https://jira.corp.adobe.com/browse/MWPW-199041
QA Checklist: https://wiki.corp.adobe.com/display/adobedotcom/M@S+Engineering+QA+Use+Cases

  • C1. Cover code with Unit Tests (24 tests; find.js 100% stmt/fn/line + replace.js shell)
  • C2. Add a Nala test (N/A — backend IO action, no rendered surface; confirm with #fishbags)
  • C3. Verify all Checks are green (unit tests)
  • C4. PR description contains working Test Page link (N/A — no UI surface yet; tested via unit suite + action endpoint)
  • C5. Ready to demo from Test Page
  • C.6 Re-read Jira to validate all ACs addressed

Test URLs

Backend IO action — no rendered page. Validated via the studio unit suite (io/studio: npm test).

@yesil yesil added the run nala label Jun 18, 2026
@aem-code-sync

aem-code-sync Bot commented Jun 18, 2026

Copy link
Copy Markdown

Hello, I'm the AEM Code Sync Bot and I will run some actions to deploy your branch.
In case there are problems, just click the checkbox below to rerun the respective action.

  • Re-sync branch
Commits

@github-actions

Copy link
Copy Markdown

SNOW Change Request Transaction ID [IO]: 0000019c1eec85a3-13f8a5e2

@yesil yesil changed the title MWPW-199041: Add find-replace-search IO action DO NOT REVIEW: MWPW-199041: Add find-replace-search IO action Jun 18, 2026
@yesil yesil marked this pull request as draft June 18, 2026 09:37
@codecov

codecov Bot commented Jun 18, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 89.44%. Comparing base (692153c) to head (dc70181).

Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff             @@
##             main     #987      +/-   ##
==========================================
+ Coverage   89.39%   89.44%   +0.04%     
==========================================
  Files         269      271       +2     
  Lines       84202    84399     +197     
==========================================
+ Hits        75275    75491     +216     
+ Misses       8927     8908      -19     
Files with missing lines Coverage Δ
io/studio/src/bulk-edit/find.js 100.00% <100.00%> (ø)
io/studio/src/bulk-edit/replace.js 100.00% <100.00%> (ø)

... and 4 files with indirect coverage changes


Continue to review full report in Codecov by Harness.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 692153c...dc70181. Read the comment docs.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@yesil yesil changed the title DO NOT REVIEW: MWPW-199041: Add find-replace-search IO action DO NOT REVIEW: MWPW-199041: Add find-replace IO action Jun 18, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant