Update Versions - #71
Merged
Merged
Conversation
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
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.
Changepacks
devup-mcp@0.9.0 → 0.10.0 - crates/devup-mcp/Cargo.toml
Minor
devup_ui_validatereportedfile-placementagainst Next.js files whose default export the framework requires -app/layout.tsxwithout one is a build error, so the advice could not be taken, and because the finding is a warning it failedstrict: trueon a correct application;not-found.tsxcollected two findings for the same reason. The page-name rule judged every component in a route file rather than the one the route renders, so a correctexport default function NoticePagewas reported for its ownPageContenthelper, andone-component-per-filethen asked for that helper to be moved somewhere the file-placement rule forbids - leavingsrc/components/as the only destination for a component used by one route. At a monorepo roottheme_for_validationtook the firstdevup.jsonthe sorted scan found, soapps/frontcode was validated againstapps/admin's theme and its real tokens came back asunknown-tokenerrors suggesting the other package's token as the fix; it now refuses to choose and returnscandidateProjectRootsnaming theprojectRootthat selects each. A call with neithertsxnorfilesvalidated the empty string and answeredok: true, okReason: "clean"about code never sent, and is now refused;cleanitself is split soclean-unverified-tokenssays a$tokenreference existed but no theme was resolved to check it against. Tokens insidecss()/globalCss()/keyframes()were invisible to the token check, so an unknown one there passed in silence. On the Figma side a Section whose candidate menu exceeded the plugin transport was refused outright, returning nothing for a menu that was merely long and reproducing the same refusal on every retry because the ceiling follows the Section's size; it now surrenders text previews first, then shortens from the end under the sameprojectionTruncatedthe traversal and candidate caps already raise, andallScreensstill refuses a truncated index rather than pretending to be whole. Of the twenty-five codes the plugin scripts throw only two carry a JSON payload, so the rest arrived aspluginCode: nullwith generic advice; the code is now read from the message for all of them, and the two that name a caller-fixable condition are mapped - a missing node or page reached callers as-32603 INTERNAL_ERRORwhen the README promises the JSON-RPC code alone separates "fix the arguments" from "stop and report", which is the most common mistake there is. The placement contract now carries the collected parent's fill, because learning that the desktop frame behind an app screen was#E6EDE6otherwise took a second Figma acquisition for a colour the capture already held. Finally, a newself_updatemodule keeps the binary current without anyone tending it: the existing release check now fetches the published asset for the platform, proves it by running--self-checkon it, and renames it into place - never touching the running image, so the stdio pipe the host already holds is never at risk, and the next start is the one that differs. It stands down where it is not the owner of its install, and reports which mechanism is in force underserver.updateAvailable.autoUpdate.