#๐พ-core-development
1 messages ยท Page 280 of 1
there are two pretty much identical components
one is compact the other isnt
you can differentiate them by the text size
which is what I did
vee do you just ban them from making issues cus never have i ever seen anyone complain in comments after you close their issues
locked?
it can match
// random thing
variant: "text-xs/normal",
// bunch of random code here
variant: "not the one we want",
// matches code here
yeah I know but
well
you can limit the wild card
but the text size is the only way I found to differentiate them
besides the class names on the past
well rn I enabled this
i usually lock dumb issues and sometimes block them
yeah that
nice to know
what will we do about this
trying to find something
limit length of this wildcard (50 might not be enough)
match: /reverse:!0,.{0,50}?barClassName:.+?\}\)\]/,
this is kind of crazy
yeah, i dont like it that much
like change it to
_expanded: n = !!arguments[0]?.isBetterFolders && arguments[0]._expanded
I mean yeah if that works
maybe add Children.map to this
but like
the let is still a bit needed because there's a component that exports the same stuff
it cant match that object instead
this app is so laggy oh my god
unknown
this is likely to break
I think .MUTED there is a class
I dont wanna fix that right now tho
that one is a painful patch
let it be
someone figure out what it is
yeah, iirc they have them in an object at the top of the file
oh ur right
then it's fine
phew
didn't test but it looks right
$intl xb78ZZ
Nothing found :(
bleh?
match: /(?<=let ?(?:\i,)*?{folderNode:\i,setNodeRef:\i,.+?expanded:(\i),.+?;)(?=let\[\{dragging:)/,
this seems like a good anchor
also seems like it might break frequently
A11Y_CHECKBOX_CHECKED
:DDDD
@elder plinth didnt you make a script to do this
someone guess this for me
STOP BOOMING @limber skiff
nookies committing 9/11 in vencord repo
BOOM BOOM
fuck u giothub
T NEVER ENDS

i sound like a broken record
vee should plugins without tags have some relevant tags added?
i looked and like 7 plugins have tags
๐
LOL
I'm gonna stay here until tomorrow limiting the length ๐
but not all of them need iirc
match: /\.Fragment,\{children:.{0,200}?\?null:(?:\i\(\),){3}(?<=nowPlaying:(\i)=!1,.+?overlay:\i,[^}]+?\}=(\i).+?)(?=\1&&)/,
bit less generic, not sure how much better it can get
ill get it eventually
definition of insanity
???
371741
GUILD_FOLDER_TOOLTIP_A11Y_LABEL
he did, did he not push?
no ๐ญ
vintl SZeUdR
Nothing found :(
much better
vesktop has this and .STREAMER_MODE_ENABLE, is broken what is it actually supposed to do?
is the find or patch broken
why was ths one removed?
whats the syntax for co-author
this is better i think
oh god no ๐ญ
vtest sadanslargehole-pr
Now testing! 
same
I just need to see if I didnt cause a regression
vintl LgCPMt
Nothing found :(
RoleColorEverywhere (had no effect):
ID: 112753
Match: ```
/tag:"strong",variant:"text-md/medium"(?<=onContextMenu:.{0,15}((\i),(\i),\i).+?)/
**__SortFriendRequests (had no effect):__**
ID: `594968`
Match: ```
/(?<=\.\i,children:\[)\(0,.{0,100}user:\i,hovered:\i+?(?=,\(0)(?<=user:(\i).+?)/
ShowHiddenChannels (had no effect):
ID: 379768
Match: ```
/return(0,\i.jsxs?)(\i.\i,{channelId:(\i).id(?=.+?((0,\i.jsxs?)("div",{className:\i.\i,children:[.{0,100}\i.length.>0.+?]})),)/
**__FriendsSince (found no module):__**
ID: `-`
Match: ```
.connections,userId:
findByCode(".actionBarIconaaaaaaaaaaa")
ShowHiddenChannels (had no effect):
ID: 379768
Match: ```
/return(0,\i.jsxs?)(\i.\i,{channelId:(\i).id(?=.+?((0,\i.jsxs?)("div",{className:\i.\i,children:[.{0,100}\i.length.>0.+?]})),)/
**__SortFriendRequests (had no effect):__**
ID: `594968`
Match: ```
/(?<=\.\i,children:\[)\(0,.{0,100}user:\i,hovered:\i+?(?=,\(0)(?<=user:(\i).+?)/
FriendsSince (found no module):
ID: -
Match: ```
.connections,userId:
VencordToolbox (took 5.7ms):
ID: 500831
Match: ```
/(?<=trailing:.{0,50})\i.Fragment,{(?=.+?className:(\i))/
findByCode(".actionBarIconaaaaaaaaaaa")
uuuuuuuuuuh
๐
Iconaaaaaaaaaaaaaaaaaa
broke two patches
it wasa on 1 bad patch how did it regress
my bad
๐
@austere talon add vhash, reverse of vintl
doing that rn
i think its better
going to try a few guess at the key before i just use ::raw
also huh
maybe i might try hooking up gemma with hash tool and have it guess the key
idk if context window is big enough to feed the entire discord json
vtest sadanslargehole-pr canary
Now testing! 
Now testing! 
what's the state of things right now, settings work but plugins are broken?
or do settings not work too
got it first try
SortFriendRequests (had no effect):
ID: 594968
Match: ```
/(?<=.\i,children:[)(0,.{0,150}user:\i,hovered:\i+?(?=,(0)(?<=user:(\i).+?)/
**__FriendsSince (found no module):__**
ID: `-`
Match: ```
.connections,userId:
findByCode(".actionBarIconaaaaaaaaaaa")
it doesnt fail on my machine?
SADANS LARGE HOLE
?remind 1w add GAME_FRIEND to intl list
Alright @fossil inlet, in 1 week: add GAME_FRIEND to intl list
how dead is vencord
not that bad
forever. it's never coming back
yay
Now testing! 
getting there
cursed?
SortFriendRequests (had no effect):
ID: 594968
Match: ```
/(?<=.\i,children:[)(0,.{0,150}user:\i,hovered:\i+?(?=,(0)(?<=user:(\i).+?)/
**__FriendsSince (found no module):__**
ID: `-`
Match: ```
.connections,userId:
findByCode(".actionBarIconaaaaaaaaaaa")
theres 2-3 broken patches thats all
yea
actionBarIconaaaaaaaaaaa
SADANS LARGE HOLE
actionBarIconaaaaaaaaaaa
i think shc died
unless its fixed on some other branch
vencord devs actually so fast wth
dont
its not out
not yet
its just on other branch
someone else will figure why that only fails on reporter
we still need to push main
it just tried brute forcing by using a list of terms and going through all permutations. extremely slow the longer the key and suseptible to discord adding a random fuck ass word to the key that you never in a million years would have thought to add to the word bank and so it never finds the hash. long since deleted it lol
const ActionBarIcon = findByCodeLazy("Children.map", "isValidElement", "dangerous:");
cute emoticon
+0+
its fine
not really
IgnoreActivities is patching this right
it matches against a use of the prop
or the one below
LOVE
do you want to see the old one?
oh down there
insane how useful the class names were
unfathomable I could guess the a11y one but the rest are somehow different enough to elude me
why not do smth like
match: /\i&&!\i||\i\?null/,
replace: "$self.renderToggleButton(),$&
whats the a11y one
I just added collectibles cause ik thats what they call profile stuff
love
nameplates were the first things in the collectibles key on profile data
good job
aha this is the 2nd one
eh
try this
in general, i think intl is better than prop order
vtest sadanslargehole-pr canary
Now testing! 
intl can easily get duplicated
they re-use strings
SortFriendRequests (had no effect):
ID: 594968
Match: ```
/(?<=.\i,children:[)(0,.{0,150}user:\i,hovered:\i+?(?=,(0)(?<=user:(\i).+?)/
@limber skiff guhhhhhh
that's on you
how
I didnt touch the replacement
i had the bug in my original fix
noticed and fixed it when i replaced it with yours
i think you copy-pasted and re-added it
guh
ah
how is it at the start
oh
have to escape | lmfao
way better
wait im confused
it matches smth else
here it's here
this is correct
HURRY UP DUDE
what is wrong with you
oh my god shithub cant handle our pr
SHIIIITHUUUUUUBBBB HURRY
you really want to go to sleep
bro so bad it applied twice
i'm not sure if this is right please verify this is right @Nuckyz
was just looking at that, its 100% wrong
vtest sadanslargehole-pr
Now testing! 
ill fix that one
what is nookies cooking 
seems fine?
^
SortFriendRequests (had no effect):
ID: 4
Match: ```
/(?<=.\i,children:[)(0,.{0,100}user:\i,hovered:\i.+?(?=,(0)(?<=user:(\i).+?)/
SortFriendRequests (had no effect):
ID: 4
Match: ```
/(?<=.\i,children:[)(0,.{0,100}user:\i,hovered:\i.+?(?=,(0)(?<=user:(\i).+?)/
**__RoleColorEverywhere (had no effect):__**
ID: `112753`
Match: ```
/tag:"strong",variant:"text-md\/medium"(?<=onContextMenu:.{0,15}\((\i),(\i),\i\).+?)/
it should be here
what happened to rce on staple
wait im stupid
okay the fuck u mean still broken
i thought that was one the one below it
how
4
is it normal that that monitor thing is missing
installermessage is my fav game icl
oh not a thing on linux
okay makes sense
is it tho
my finger getting sore from scrolling throught code for hours
where should it be
is this a different module
dont think so
no
vtest sadanslargehole-pr canary
Now testing! 
{
find: "#{intl::EMBEDDED_ACTIVITIES_DEVELOPER_ACTIVITY}",
replacement: {
match: /,rendersPlaceholder:.{0,200}?lineClamp:1.{0,50}?(?=!\i&&\i\?)(?<=application:(\i).+?)/,
replace: "$&$self.renderToggleActivityButton($1),"
}
}
@fossil inlet
{
find: "#{intl::EMBEDDED_ACTIVITIES_DEVELOPER_ACTIVITY}",
replacement: {
match: /,(?=!(\i)&&\i\?)(?<=application:(\i).+?\1="large_banner"===.+?)/,
replace: (_, _1, application) => `,$self.renderToggleActivityButton(${application}),`
}
}
Seems like everything is working fine (for now) 
your patch works fine @austere talon
FINALLY
{
find: "#{intl::EMBEDDED_ACTIVITIES_DEVELOPER_ACTIVITY}",
replacement: {
match: /lineClamp:1.{0,50}?(?=!\i&&\i\?.+?application:(\i))/,
replace: "$&$self.renderToggleActivityButton($1),"
}
}
what are yall fixing
looks good
ignoreactivities
this is how it should look tho
this looks so bad
well that requires more work cuz the component is shared rn
the color here is wrong too
for some reason
vtest sadanslargehole-pr
Now testing! 
eh?
oh
someone should try and guess
Seems like everything is working fine (for now) 
RoleColorEverywhere (had no effect):
ID: 112753
Match: ```
/tag:"strong",variant:"text-md/medium"(?<=onContextMenu:.{0,15}((\i),(\i),\i).+?)/
(fixes this borked component)
yeah that's the span
@fossil inlet
why is it not all: true anymore
wrong branch
i didn't do that one, but there is only one module for it to match, both before and after
(that the find matches against)
seems wrong
shouldnt it be className={classes(RoleBorderClasses.roleCircle, RoleBorderClasses.roleCircle)}
@alpine ridge
wait
oh
they removed one
one of those doesnt exist anymore
why was the match changed anyway
old onme should work
just revert to the old one
ye
is that everything
ive found another issue with showhiddenthings, looks like an easy fix
@limber skiff 
bruh they just reverted
joking?
yeah sorry
what
where did it g o
๐ญ
vesktop also needs fixing 
nobody uses vesktop anyway 
I USE IT
same
have you considered using vencord instead
@alpine ridge can pull and try now
should be good
vtest sadanslargehole-pr
Now testing! 

I just forced pushed into sadan large hole
OnePingPerDM (took 10.9ms):
ID: 845618
Match: ```
/(\i.\i.getDesktopType()===\i.\i.NEVER))/
RoleColorEverywhere (had no effect):
ID: 112753
Match: ```
/tag:"strong",variant:"text-md/medium"(?<=onContextMenu:.{0,15}((\i),(\i),\i).+?)/
wait
i think i know why
asd
@limber skiff what's better, text-md\/(?:medium|normal) or text-md\/\w+?
stable compat
@austere talon anything more?
did this get fixed
lgtm

is patches not exposed on window anymore
what's wrong
oh Vencord.WebpackPatcher.patches
should look like this, but for some reason they dont render for V
minimal issue
oh, existing issue?
can get fixed later if it's our fault
someone tell discord to remove them
please
it will fix performance instantly i think
somehow vesktop only had 2 patches break
I already know something will going to broken after we push
so let the users find it
yes
I didnt test every plugin
push to main
u do it
oki
normal
vtest dev
Now testing! 
๐
๐
now discord will revert
i think i have a stable running on my laptop from like early december
I dont think a lot of things will break if they revert
there were only some minimal bundler changes
besides the css mangling
Seems like everything is working fine (for now) 
Seems like everything is working fine (for now) 
they revert to the one with fully hashed class names
that's probably going to happen
i wont release ext yet
ye
?remind tomorrow check if this is still an issue
Alright @fossil inlet, in 1 day: check if this is still an issue
is it just me or is the role dot clipping the text
I love seeing users happy when things get fixed
i was here o7
thanks for the help everyone
THE CULPRIT
SPREAD HATE
I KNEW IT
kill
(dont complain on that pr)
genuinely so funny that everything broke because of this teeny tiny pr
discord doesnt use the default
he just wanted to sabotage us!!
they use an _ not a -
yes but they had made it be full hash
and reverted
lets mention that PR in our PR
oh my god
hopefully it stays that way
they need to know what they did.
so discord updated, then manually changed it back
wait yeah
my guess for why is because source maps cant help with that
so it's a massive pain
even if you have them
i doubt it, but it's not out of the question
i wonder if them changing it back to sane was to be nice to modders or if they care about class format
also i really hate this change tbh
it will ruin sites in general
so many sites will become less themeable thanks to this default change
should i complain on the PR
yes
vev dont forget about extension 
The Vencord Chrome Extension is up to date! (v1.13.12)
how the hell
it's very likely that some things might still be broken and went unnoticed
i'm blind
give users some time to report issues
specially cuz I touched 50 patches and didnt double check
I trust my regex
๐
always has been, think it started in webpack with css-loader
@austere talon do you know the perf impact of *=? would it be worth to go back to ^=
help
should i dm advaith
yes
yes
yes
yes
destroying the evidence
restore that
because you had done already
it still needed
not merged?
no
alr
i reverted that because doesnt work
i think vencord needs more dev branches
well rn works well
dev4 and dev5 when
dev4 soon
we can delete these right
yeah
these dont show for me
npm notice Access token expired or revoked. Please try logging in again.
who
code scanning
npm error 403 403 Forbidden - PUT https://registry.npmjs.org/@vencord%2Ftypes - Two-factor authentication or granular access token with bypass 2fa enabled is required to publish packages.
please shut
npm so annoying man
you love npm revoking all tokens
uh oh
someone stream on vc
try watching my stream
I dont have all plugins enabled tho
can you run reporter build and watch my stream?
I closed my vscode rn
oh huh
maybe
my messages love taking years to send, but i have the same issue on stock
what discord mods are still maintained nowdays
i tried in dms tho not vc
for desktop
if theres a difference
BetterDiscord, Vencord
how is bd doing with this change anyway
Replugged a little bit
did it affect them at all
moonlight?
yeah moonlight seems to be
goooood
the mod itself broke too

works for me
our patcher was lasting so long without problems
I never really realized the module definition could change format
out of curiosity did shelter have issues
this seems like a vesktop issue
crash whenever i try to start stream
also did the Card import get moved somewhere else all my plugins using it fail build
person with the issue in support was on stable btw @fossil inlet
i think
like
stock
cant repro
no
this tho...
its a react error
lol
trying to render something thats not a component
Ay
Ay ay ay ay
well debug it
oh god that's going to fuck with my memory
nope but I mean shelter is designed for that
i can repro 100%
decor got wacky
the previewer shows this now
(not related to this update)
even on old stable it's like that
It doesnt even work ๐
fix everything (but decor)
why is this not caught by our reporter
it's inside LazyComponentWebpack
seems to have broke
weird
findComponentByCode("#{intl::PREMIUM_UPSELL_PROFILE_AVATAR_DECO_INLINE_UPSELL_DESCRIPTION}")
it seems like my stable client has died
vintl hmyYK8
Nothing found :(
not duplicate right
nope
leading .
its fine
yeah what is this message send issue
you cant even ctrl+r if it happens
ive had it a few times by now too
this happens to me on stock stable, discord fucked something
this happens to me on stock stable, discord fucked something
is that the crash?
spammer
i think it might be
let me try to fix this too
prob just a new default boolean prob if i had to guess
I would appreacite having my react devtools
i cant connect to voice either
no lol
id try it out if i could connect to a channel ๐ญ
helps when debugging the cause of crashes
why is_dev check
just search VesktopRenderer
idk why it doesn't give clickable link
make sure u have sourcemaps on
vesktop should have
nah
ah yeah this is wrong
you do need a protocol but file doesn't work
need to make a custom protocol
then why is file used for the webpack modules
I meant for the source maps
oh
build.mjs: Line 59
const sourceMapFooter = s => watch ? "" : `//# sourceMappingURL=vencord://${s}.js.map`;
whats latest
.3 i think
oh good

RCE broken
how
cuz users were complaining in support
should i just forward genuine "this doesn't work" messages here?
yes
im wondering too
cuz i have it on too
๐ญ
yu get used to it being gone so quick
we did but just wasn't fixed
there were so many other destructured variables in between that it was matching a destructuring statement a thousand kilometers away
and not erroring purely because of lucky since the variablas had other values in the scope
๐
updated to 1.6.3 and no longer crash when trying to stream
okay I can eat now
vee did this and i told him but he didnt care
I was going to fix but I forgot
anyway you have to be mentally disturbed to use coloured chat messages anyway
so awful
I was lowkey saving you by breaking that feature
Insane
rip I restarted my client
went to support and clicked update now for funnies
restarted my whole client
It's fine tho I think

wait more
we can do it within like 10 hours or something
won't change much
also need to push dev to main before it
the lag seems to have gotten better btw
funny
like a few days ago discord was lagging hard for me (even stock)
now it's smooth again

I wonder what actually caused it
anyone ever got this before
i think it means the renderer crashed
not sure
i can consistently cause this by telling companion to download all modules
oom?
dont think so
i have 64gb
and it's never oomed before
is load lazy chunks meant to fetch the same one three times
wonder what the flamegraph looks like
i mean reporter works on github
I didn't really review it because too lazy and I trust nookie ๐
guh?
im not sure if related, or if even a new thing or not, I just assumed it was cause of the new webpack things or w/e but anytime I press test patch it takes ages to get me a result and sometimes times out. dont think it's ever done that to me before. only seems to happen on multi replacements. single patches are fine
yeah thats discord being really show
nvm happens to single patches to. random if it works instantly or takes ages or times out
yeah so this means its fetching too much
have to throttle requests a bit
add a sleep


