chore: bump deslop-js to ^0.0.19#679
Merged
Merged
Conversation
Co-Authored-By: Aiden Bai <aiden.bai05@gmail.com>
Contributor
Author
🤖 Devin AI EngineerI'll be helping with this pull request! Here's what you should know: ✅ I will automatically:
Note: I can only respond to comments from users who have write access to this repository. ⚙️ Control Options:
|
commit: |
Contributor
|
React Doctor found 3 files changed in this pull request, but none matched the files covered by its enabled checks. Scope: 3 files changed on Generated by React Doctor. Questions? Contact founders@million.dev. |
Co-Authored-By: Aiden Bai <aiden.bai05@gmail.com>
devin-ai-integration
Bot
changed the title
gcharang
added a commit
to gcharang/react-doctor
that referenced
this pull request
Jun 4, 2026
Sync upstream (PR #2): brings millionco#658 "Add to CI" handoff + default CI setup, millionco#644 rule demotions, millionco#677 retire rn-animate-layout-property, millionco#678/millionco#679/millionco#676, millionco#646 version-packages, and the sync-upstream workflow onto the hardened fork. Conflicts resolved keeping fork identity (FORK_* constants, github:gcharang/ react-doctor#pinned spec, local score, no phone-home) while taking upstream behavior: - core/constants.ts: keep FORK_* block; new CI_URL points at the fork, not react.doctor/ci. - build-handoff-payload / install-doctor-script / install-git-hook-file / install-react-doctor / render-diagnostics: take upstream's refactors (async install, namespace fs/path imports, extracted workflow module, renderer rework), re-applying the fork npx spec on top. - tests + terminal-visuals snapshot: upstream layout/asserts with the fork spec. Silent (non-conflict) fork patches: - install-github-workflow.ts / open-workflow-pull-request.ts: repoint the generated CI workflow + PR body to gcharang/react-doctor@pinned and the fork repo (upstream shipped millionco/react-doctor@v1 + react.doctor/ci). - root package.json: deslop-js ^0.0.17 -> ^0.0.19 to match millionco#679 and the per-package manifests (auto-merge kept the stale root pin). - CI_TRUST_COMPANIES: drop the named-company endorsement for neutral wording. Regenerated and re-staged into this merge: prebuilt dist/ (cli.js + engine, no source maps), vendored prompts/ (328 recipes, unchanged rule set), and the config schema. Grep gate clean across shipped surfaces. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Bumps
deslop-jsfrom^0.0.17→0.0.18-dev.17136abin bothpackages/react-doctorandpackages/core.Link to Devin session: https://app.devin.ai/sessions/7769a2e335df43ca82715073fb7d81c5
Requested by: @aidenybai
Note
Low Risk
Dependency-only bump with no source changes; main risk is regressions in the bundled dead-code/deslop analysis behavior shipped with react-doctor.
Overview
Bumps the
deslop-jsdependency from^0.0.17to^0.0.19inpackages/coreandpackages/react-doctor, withpnpm-lock.yamlupdated to resolvedeslop-js@0.0.19for both workspaces.The lockfile diff also drops many
libcfields on existing optional native packages (e.g. sharp, Next SWC, oxc bindings)—typical lockfile churn from re-resolution, not a separate code change.Reviewed by Cursor Bugbot for commit 81fa777. Bugbot is set up for automated code reviews on this repo. Configure here.