Skip to content

MWPW-199042: Bulk-edit CSV replace worker#999

Open
yesil wants to merge 16 commits into
MWPW-199041from
MWPW-199042
Open

MWPW-199042: Bulk-edit CSV replace worker#999
yesil wants to merge 16 commits into
MWPW-199041from
MWPW-199042

Conversation

@yesil

@yesil yesil commented Jun 22, 2026

Copy link
Copy Markdown
Contributor

To be merged after: #987

Adds CSV-driven bulk-edit to M@S Studio's bulk-edit IO action: export find results to CSV, re-upload an edited CSV, and run an async replace worker that applies the changes (with dry-run + report). Builds on the bulk-edit find action (MWPW-199041, #987).

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

Please do the steps below before submitting your PR for a code review or QA

  • C1. Cover code with Unit Tests
  • C2. Add a Nala test (double check with #fishbags if nala test is needed)
  • C3. Verify all Checks are green (unit tests, nala tests)
  • C4. PR description contains working Test Page link where the feature can be tested
  • C5: you are ready to do a demo from Test Page in PR (bonus: write a working demo script that you'll use on Thursday, you can eventually put in your PR)
  • C.6 read your Jira one more time to validate that you've addressed all AC's and nothing is missing

Test URLs:

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

aem-code-sync Bot commented Jun 22, 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-148fca31

@yesil yesil mentioned this pull request Jun 22, 2026
13 tasks
@yesil yesil changed the base branch from main to MWPW-199041 June 22, 2026 14:30
Keep find+replace on 199042; 199041 find-only subset already incorporated.

Co-authored-by: Cursor <cursoragent@cursor.com>
@codecov

codecov Bot commented Jun 22, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 91.62500% with 67 lines in your changes missing coverage. Please review.
✅ Project coverage is 89.65%. Comparing base (dda4e3e) to head (e228fa5).

Files with missing lines Patch % Lines
io/studio/src/bulk-edit/bulk-edit.js 89.10% 33 Missing ⚠️
io/studio/src/bulk-edit/replace-worker.js 93.18% 21 Missing ⚠️
io/studio/src/bulk-edit/replace.js 92.50% 12 Missing ⚠️
io/studio/src/common.js 93.33% 1 Missing ⚠️
Additional details and impacted files

Impacted file tree graph

@@               Coverage Diff               @@
##           MWPW-199041     #999      +/-   ##
===============================================
- Coverage        89.95%   89.65%   -0.30%     
===============================================
  Files              282      280       -2     
  Lines            88978    86416    -2562     
===============================================
- Hits             80042    77479    -2563     
- Misses            8936     8937       +1     
Files with missing lines Coverage Δ
io/studio/src/bulk-edit/csv.js 97.30% <100.00%> (+0.13%) ⬆️
io/studio/src/common.js 91.63% <93.33%> (+0.05%) ⬆️
io/studio/src/bulk-edit/replace.js 92.50% <92.50%> (ø)
io/studio/src/bulk-edit/replace-worker.js 93.18% <93.18%> (ø)
io/studio/src/bulk-edit/bulk-edit.js 93.46% <89.10%> (-2.07%) ⬇️

... and 27 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 dda4e3e...e228fa5. 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 MWPW-199042: Bulk-edit CSV export/upload and replace worker MWPW-199042: Bulk-edit CSV replace worker Jun 22, 2026
Bring 199041 history; resolve conflicts keeping replace stack and
find+replace API on 199042.

Co-authored-by: Cursor <cursoragent@cursor.com>
Consolidate export and find-results helpers into bulk-edit.js; update
replace-worker imports and tests for the merged module layout.

Co-authored-by: Cursor <cursoragent@cursor.com>
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