Skip to content

Remove legacy Paper renderer shim cleanup from artifact commit workflow#36297

Merged
rubennorte merged 1 commit intomainfrom
fix-artifact-commit-workflow
Apr 17, 2026
Merged

Remove legacy Paper renderer shim cleanup from artifact commit workflow#36297
rubennorte merged 1 commit intomainfrom
fix-artifact-commit-workflow

Conversation

@zeyap
Copy link
Copy Markdown
Contributor

@zeyap zeyap commented Apr 17, 2026

Summary

PR #36285 deleted the Paper (legacy) renderer, including the shim file
scripts/rollup/shims/react-native/ReactNative.js. However, the
runtime_commit_artifacts workflow still tries to rm this file after
moving build artifacts into compiled-rn/. Since the file no longer
exists in the build output, rm (without -f) fails and kills the
entire step.

This has caused every run of the Commit Artifacts workflow to fail since #36285 landed on April 16, blocking both builds/facebook-www and builds/facebook-fbsource branches from receiving new build artifacts. This in turn blocks DiffTrain from syncing React changes into Meta's internal monorepo.

PR #36285 deleted the Paper (legacy) renderer, including the shim file
ReactNative.js. The artifact commit workflow still tries to rm this file,
which fails and blocks the entire pipeline since April 16.
@zeyap zeyap requested a review from rubennorte April 17, 2026 01:48
@meta-cla meta-cla bot added the CLA Signed label Apr 17, 2026
@react-sizebot
Copy link
Copy Markdown

Comparing: 77319e2...ee82a06

Critical size changes

Includes critical production bundles, as well as any change greater than 2%:

Name +/- Base Current +/- gzip Base gzip Current gzip
oss-stable/react-dom/cjs/react-dom.production.js = 6.84 kB 6.84 kB = 1.88 kB 1.88 kB
oss-stable/react-dom/cjs/react-dom-client.production.js = 612.91 kB 612.91 kB = 108.30 kB 108.30 kB
oss-experimental/react-dom/cjs/react-dom.production.js = 6.84 kB 6.84 kB = 1.88 kB 1.88 kB
oss-experimental/react-dom/cjs/react-dom-client.production.js = 678.85 kB 678.85 kB = 119.27 kB 119.27 kB
facebook-www/ReactDOM-prod.classic.js = 698.58 kB 698.58 kB = 122.74 kB 122.74 kB
facebook-www/ReactDOM-prod.modern.js = 688.90 kB 688.90 kB = 121.12 kB 121.12 kB

Significant size changes

Includes any change greater than 0.2%:

(No significant changes)

Generated by 🚫 dangerJS against ee82a06

@rubennorte rubennorte merged commit bf45a68 into main Apr 17, 2026
244 checks passed
github-actions bot pushed a commit that referenced this pull request Apr 17, 2026
…ow (#36297)

## Summary

PR #36285 deleted the Paper (legacy) renderer, including the shim file
`scripts/rollup/shims/react-native/ReactNative.js`. However, the
`runtime_commit_artifacts` workflow still tries to `rm` this file after
moving build artifacts into `compiled-rn/`. Since the file no longer
exists in the build output, `rm` (without `-f`) fails and kills the
entire step.

This has caused **every run of the Commit Artifacts workflow to fail
since #36285 landed on April 16**, blocking both `builds/facebook-www`
and `builds/facebook-fbsource` branches from receiving new build
artifacts. This in turn blocks DiffTrain from syncing React changes into
Meta's internal monorepo.

DiffTrain build for [bf45a68](bf45a68)
github-actions bot pushed a commit that referenced this pull request Apr 17, 2026
…ow (#36297)

## Summary

PR #36285 deleted the Paper (legacy) renderer, including the shim file
`scripts/rollup/shims/react-native/ReactNative.js`. However, the
`runtime_commit_artifacts` workflow still tries to `rm` this file after
moving build artifacts into `compiled-rn/`. Since the file no longer
exists in the build output, `rm` (without `-f`) fails and kills the
entire step.

This has caused **every run of the Commit Artifacts workflow to fail
since #36285 landed on April 16**, blocking both `builds/facebook-www`
and `builds/facebook-fbsource` branches from receiving new build
artifacts. This in turn blocks DiffTrain from syncing React changes into
Meta's internal monorepo.

DiffTrain build for [bf45a68](bf45a68)
@hoxyq hoxyq deleted the fix-artifact-commit-workflow branch April 17, 2026 11:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants