Skip to content

[DevTools] chore: extract pure functions from fiber/renderer.js#35924

Open
hoxyq wants to merge 1 commit intomainfrom
devtools/chore
Open

[DevTools] chore: extract pure functions from fiber/renderer.js#35924
hoxyq wants to merge 1 commit intomainfrom
devtools/chore

Conversation

@hoxyq
Copy link
Contributor

@hoxyq hoxyq commented Feb 27, 2026

I am in a process of splitting down the renderer implementation into smaller units of logic that can be reused. This change is about extracting pure functions only.

@hoxyq hoxyq requested a review from eps1lon February 27, 2026 16:39
@meta-cla meta-cla bot added the CLA Signed label Feb 27, 2026
@hoxyq hoxyq changed the title chore: extract pure functions from fiber/renderer.js [DevTools] chore: extract pure functions from fiber/renderer.js Feb 27, 2026
isErrorBoundary,
getSecondaryEnvironmentName,
areEqualRects,
} from './shared/DevToolsFiberUtils';
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'd rather we separate things by feature and not introduce yet another utils dumping ground in DevTools.

ioExistsInSuspenseAncestor,
getAwaitInSuspendedByFromIO,
getVirtualEndTime,
} from './shared/DevToolsFiberSuspenseUtils';
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
} from './shared/DevToolsFiberSuspenseUtils';
} from './shared/DevToolsFiberSuspense';

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.

2 participants